Skip to content

Conversation

mattiaswal
Copy link
Contributor

@mattiaswal mattiaswal commented Sep 3, 2025

Tests if the correct boot partition and version is used.

Also add a script to add new releases to existing gns3 marketplace appliance, this fix #1107

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

@mattiaswal mattiaswal requested a review from wkz September 3, 2025 11:26
@mattiaswal mattiaswal force-pushed the test-sanity branch 5 times, most recently from e059409 to e250a65 Compare September 3, 2025 15:04
@mattiaswal
Copy link
Contributor Author

mattiaswal commented Sep 4, 2025

What do you think @wkz i think this is the best aproach, to add the version in output/images since this will allow us to maybe not have a squashfs image? Or am i overthinking.

And when running tests, we does not use the squash image as default, we use qcow2 so we need read it out before we start the testsystem anyway.

Copy link
Contributor

@wkz wkz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this! 🙏

Will be so nice to not have this failure mode go undetected again. 💪

You need first to clone gns3-registry from https://github.com/GNS3/gns3-registry

Follow this steps:
1. ./utils/bump-gns3.py 25.08.0 ../../gns3/gns3-registry/appliances/infix.gns3a
2. TEST in gns3, just add the new version, start it.
3. run python3 check.py inside gns3-registry, verify infix is ok.
4. Create a pull request to gns3-registry

This fix #1107
@mattiaswal mattiaswal added the ci:main Build default defconfig, not minimal label Sep 4, 2025
@mattiaswal mattiaswal requested a review from wkz September 4, 2025 18:06
Copy link
Contributor

@wkz wkz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test/env Outdated
@@ -154,7 +153,7 @@ containerize=yes
files=
basedir="$ixdir"

while getopts "b:cCDf:hiKp:q:rt:" opt; do
while getopts "b:cCDf:hiKp:q:rt:v:" opt; do
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
while getopts "b:cCDf:hiKp:q:rt:v:" opt; do
while getopts "b:cCDf:hiKp:q:rt:" opt; do

Not used anymore. Maybe you could also get rid of the whitespace change above.

One to verify it is the correct version on the duts and one
that check that it is the correct bootorder.
@mattiaswal mattiaswal merged commit 497990a into main Sep 5, 2025
6 checks passed
@mattiaswal mattiaswal deleted the test-sanity branch September 5, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:main Build default defconfig, not minimal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add script to simplify GNS3 marketplace updates
2 participants