Skip to content

Commit d79dac7

Browse files
committed
feat: restrict jhack to dev preset only
1 parent 43a2ed7 commit d79dac7

File tree

4 files changed

+4
-13
lines changed

4 files changed

+4
-13
lines changed

internal/config/presets.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ var defaultPackages []string = []string{
3535
// defaultSnaps is the set of snaps installed for all presets.
3636
var defaultSnaps map[string]SnapConfig = map[string]SnapConfig{
3737
"charmcraft": {Channel: "latest/stable"},
38-
"jhack": {Channel: "latest/stable", Connections: []string{"jhack:dot-local-share-juju"}},
3938
"jq": {Channel: "latest/stable"},
4039
"yq": {Channel: "latest/stable"},
4140
}
@@ -134,6 +133,7 @@ var devPreset *Config = &Config{
134133
Snaps: MergeMaps(defaultSnaps, map[string]SnapConfig{
135134
"rockcraft": {Channel: "latest/stable"},
136135
"snapcraft": {Channel: "latest/stable"},
136+
"jhack": {Channel: "latest/stable", Connections: []string{"jhack:dot-local-share-juju"}},
137137
}),
138138
},
139139
}

tests/preset-k8s/task.yaml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,10 @@ execute: |
88
"$SPREAD_PATH"/concierge --trace prepare -p k8s
99
1010
# Check that relevant snaps are installed
11-
for s in juju k8s kubectl jq yq charmcraft rockcraft jhack; do
11+
for s in juju k8s kubectl jq yq charmcraft rockcraft; do
1212
snap list "$s" | MATCH $s
1313
done
1414
15-
# Check jhack got it's interface connected
16-
snap connections jhack | MATCH "personal-files jhack:dot-local-share-juju :personal-files manual"
17-
1815
# Check the relevant debs are installed
1916
command -v pip | MATCH /usr/bin/pip
2017
python3 -m venv -h | head -n1 | grep -q -e "usage: venv"

tests/preset-machine/task.yaml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,10 @@ execute: |
88
"$SPREAD_PATH"/concierge --trace prepare -p machine
99
1010
# Check that relevant snaps are installed
11-
for s in juju lxd jq yq charmcraft snapcraft jhack; do
11+
for s in juju lxd jq yq charmcraft snapcraft; do
1212
snap list "$s" | MATCH $s
1313
done
1414
15-
# Check jhack got it's interface connected
16-
snap connections jhack | MATCH "personal-files jhack:dot-local-share-juju :personal-files manual"
17-
1815
# Check the relevant debs are installed
1916
command -v pip | MATCH /usr/bin/pip
2017
python3 -m venv -h | head -n1 | grep -q -e "usage: venv"

tests/preset-microk8s/task.yaml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,10 @@ execute: |
88
"$SPREAD_PATH"/concierge --trace prepare -p microk8s
99
1010
# Check that relevant snaps are installed
11-
for s in juju microk8s kubectl jq yq charmcraft rockcraft jhack; do
11+
for s in juju microk8s kubectl jq yq charmcraft rockcraft; do
1212
snap list "$s" | MATCH $s
1313
done
1414
15-
# Check jhack got it's interface connected
16-
snap connections jhack | MATCH "personal-files jhack:dot-local-share-juju :personal-files manual"
17-
1815
# Check the relevant debs are installed
1916
command -v pip | MATCH /usr/bin/pip
2017
python3 -m venv -h | head -n1 | grep -q -e "usage: venv"

0 commit comments

Comments
 (0)