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: cherry-pick fixes to hotfix/8.3 #4949

Merged
merged 4 commits into from
Aug 1, 2024
Merged

Conversation

mr-cal
Copy link
Collaborator

@mr-cal mr-cal commented Jul 31, 2024

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox run -m lint?
  • Have you successfully run tox run -e test-py310? (supported versions: py39, py310, py311, py312)

Snapcraft 8.4 is delayed due to #4884 and https://forum.snapcraft.io/t/kde-global-auto-connect-qt-common-themes/40878/15.

In the interim, we need an 8.3.2 release.

This PR cherry-picks 2 hotfixes and 2 test updates from main. The test updates will fix the failing CI in #4946.

mr-cal and others added 4 commits July 31, 2024 15:38
Disable `build-base: devel` spread tests until we have the resources to
debug the underlying issue with 24.10 buildd images and
`systemd-resolved` (#4921).

We should be able to address this issue in 2024-Nov.

Fixes #4910
(CRAFT-3105)

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Adds a spread test for `edit-validation-sets` and
`list-validation-sets`.

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Co-authored-by: Alex Lowe <alex.lowe@canonical.com>
- Use the correct data types for API calls
- Switch to pydantic models
- Catch model errors locally rather than relying on the store
  to validate the data
- Make the boilerplate validation set valid 

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
)

Signed-off-by: Aristo Chen <aristo.chen@canonical.com>
@mr-cal mr-cal added the rebase label Jul 31, 2024
Copy link
Contributor

@tigarmo tigarmo 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 this!

@mr-cal mr-cal merged commit 6425f1c into hotfix/8.3 Aug 1, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants