-
Notifications
You must be signed in to change notification settings - Fork 77
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
noble: add noble to tests (SC-1647) #2858
Conversation
Jira: SC-1647 GitHub Issues: Launchpad Bugs: No Launchpad bugs are fixed by this PR. (No commits have LP: #### references) Documentation: The changes in this PR do not require documentation changes. 👍 this comment to confirm that this is correct. |
689622c
to
d79d1ce
Compare
a650c78
to
64d2188
Compare
60a6647
to
7215319
Compare
73a35c9
to
b1ec5fa
Compare
b1ec5fa
to
22cec43
Compare
22cec43
to
ac243df
Compare
58b9483
to
1b8e5b0
Compare
a9cf3e5
to
315cf87
Compare
a40eac5
to
3ff325a
Compare
6ef38a5
to
c087243
Compare
without these directories, apt prints warnings on noble Fixes: #3060
c087243
to
a98867a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @orndorffgrant , LGTM
else | ||
SKIP_PROPOSED="--skip-proposed" | ||
fi | ||
sg sbuild -c "mk-sbuild $SKIP_PROPOSED ${{ matrix.release }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious, what does the skip-proposed
flag do here?
@@ -33,3 +33,6 @@ Feature: Full Auto-Attach Endpoint | |||
| jammy | aws.pro | | |||
| jammy | azure.pro | | |||
| jammy | gcp.pro | | |||
| noble | aws.pro | | |||
| noble | azure.pro | | |||
| noble | gcp.pro | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason we do not test this feature on mantic here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saw mantic
missing on a few other tests which test across all releases too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mantc
has no Pro images - Those are only for LTS
Why is this needed?
This PR adds noble to all the relevant tests
Test Steps
CI for noble should pass
Checklist
Does this PR require extra reviews?