Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(spread): core22 provider setup cleanup #4668

Merged
merged 1 commit into from
Mar 15, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions spread.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,8 @@ suites:
- ubuntu-22.04-armhf
- ubuntu-22.04-s390x
- ubuntu-22.04-ppc64el
environment:
SNAPCRAFT_BUILD_ENVIRONMENT: ""

tests/spread/core22/environment/:
summary: core22 environment tests
Expand Down
3 changes: 0 additions & 3 deletions tests/spread/core22/chisel-base/task.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
summary: Build a simple snap

environment:
SNAPCRAFT_BUILD_ENVIRONMENT: ""

restore: |
snapcraft clean
rm -f ./*.snap
Expand Down
3 changes: 0 additions & 3 deletions tests/spread/core22/clean/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,6 @@ restore: |
execute: |
cd "./snaps/$SNAP"

# Unset SNAPCRAFT_BUILD_ENVIRONMENT=host.
unset SNAPCRAFT_BUILD_ENVIRONMENT

snapcraft pack
snapcraft clean part1
lxc --project=snapcraft list | grep snapcraft-clean
Expand Down
1 change: 1 addition & 0 deletions tests/spread/core22/craftctl/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ restore: |
cd "$SNAP"
rm -f ./*.snap
rm -Rf work
snapcraft clean --destructive-mode

#shellcheck source=tests/spread/tools/snapcraft-yaml.sh
. "$TOOLS_DIR/snapcraft-yaml.sh"
Expand Down
4 changes: 2 additions & 2 deletions tests/spread/core22/invalid-utf8/task.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
summary: Handle invalid utf-8 text during the build

restore: |
snapcraft clean
snapcraft clean --destructive-mode
rm -f ./*.snap

execute: |
snapcraft pack -v 2>&1 | MATCH ":: hi � bye"
snapcraft pack --destructive-mode -v 2>&1 | MATCH ":: hi � bye"
3 changes: 1 addition & 2 deletions tests/spread/core22/package-repo-archs/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ summary: Test using package-repositories with different architectures on core22
environment:
SNAP_ARCH/i386: i386
SNAP_ARCH/armhf: armhf
SNAPCRAFT_BUILD_ENVIRONMENT: ""

restore: |
snapcraft clean
Expand All @@ -16,4 +15,4 @@ execute: |
cp "$SNAP_ARCH".yaml snap/snapcraft.yaml

# The part's build script does the checking.
snapcraft build --verbose --use-lxd
snapcraft build --verbose
1 change: 0 additions & 1 deletion tests/spread/core22/package-repositories/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ environment:
SNAP/test_apt_ppa: test-apt-ppa
SNAP/test_pin: test-pin
SNAP/test_multi_keys: test-multi-keys
SNAPCRAFT_BUILD_ENVIRONMENT: ""

restore: |
cd "$SNAP"
Expand Down
4 changes: 2 additions & 2 deletions tests/spread/core22/packing/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ prepare: |
# set_base "$SNAP_DIR/snap/snapcraft.yaml"

restore: |
snapcraft clean
snapcraft clean --destructive-mode
rm -Rf subdir ./*.snap

#shellcheck source=tests/spread/tools/snapcraft-yaml.sh
Expand All @@ -26,7 +26,7 @@ restore: |

execute: |
# shellcheck disable=SC2086
snapcraft $CMD
snapcraft $CMD --destructive-mode

if echo "$CMD" | grep subdir/output; then
test -f subdir/output.snap
Expand Down
2 changes: 0 additions & 2 deletions tests/spread/core22/remove-hook/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,6 @@ restore: |
install_snapcraft

execute: |
unset SNAPCRAFT_BUILD_ENVIRONMENT

# create a base instance by running snapcraft
snapcraft pull --use-lxd --verbosity=trace

Expand Down
8 changes: 4 additions & 4 deletions tests/spread/core22/set-version-twice/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ prepare: |
. "$TOOLS_DIR/snapcraft-yaml.sh"

restore: |
snapcraft clean
snapcraft clean --destructive-mode
rm -Rf subdir ./*.snap
rm -f snap/*.yaml

Expand All @@ -16,14 +16,14 @@ execute: |
mkdir -p snap
cp original-snapcraft.yaml snap/snapcraft.yaml

snapcraft prime
snapcraft prime --destructive-mode

cp modified-snapcraft.yaml snap/snapcraft.yaml

snapcraft build part2
snapcraft prime
snapcraft prime --destructive-mode

cp original-snapcraft.yaml snap/snapcraft.yaml

snapcraft build part2
snapcraft prime
snapcraft prime --destructive-mode
3 changes: 2 additions & 1 deletion tests/spread/core22/snap-creation/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ prepare: |

restore: |
cd "$SNAP"
snapcraft clean --destructive-mode
rm -f ./*.snap
rm -Rf work

Expand All @@ -19,5 +20,5 @@ restore: |

execute: |
cd "$SNAP"
snapcraft 2>&1 | tee progress.txt
snapcraft --destructive-mode 2>&1 | tee progress.txt
grep "Created snap package ${SNAP}_1.0_amd64.snap" progress.txt
5 changes: 2 additions & 3 deletions tests/spread/core22/try/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,10 @@ execute: |
mkdir prime
chmod a+w prime

unset SNAPCRAFT_BUILD_ENVIRONMENT
# Prime first to regression test snapcore/snapcraft#4219
snapcraft prime --use-lxd
snapcraft prime
# Followed by the actual try
snapcraft try --use-lxd
snapcraft try

find prime/meta/snap.yaml
find prime/usr/bin/hello
Expand Down
Loading