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

fix(remotebuild): use legacy launchpad credentials if they exist #4892

Merged
merged 4 commits into from
Jul 6, 2024

Conversation

mr-cal
Copy link
Collaborator

@mr-cal mr-cal commented Jul 3, 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)

Changes:

  • Load legacy launchpad credentials if they exist and emit a deprecation warning
  • Allow build-for: [all] in the new remote builder
  • Overhaul remote-build spread tests:
    • Add tests per-base
    • Test with new and old credential filepaths
    • Test with and without platforms and architecture definitions
    • Test with new and legacy remote builders

CI job for remote build spread tests: https://github.com/canonical/snapcraft/actions/runs/9784405987

Depends on canonical/craft-application#373
Fixes #4889
(CRAFT-3077)

@mr-cal
Copy link
Collaborator Author

mr-cal commented Jul 5, 2024

Tests will fail until craft-application 3.1.0 gets out the door

If the legacy remote-build credentials exist and the new credentials do
not, emit a deprecation warning and use the legacy credentials.

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
@mr-cal mr-cal force-pushed the work/CRAFT-3077/launchpad-credentials branch from debe5b4 to 0c5cf46 Compare July 5, 2024 21:04
@mr-cal mr-cal added the rebase label Jul 5, 2024
@mr-cal
Copy link
Collaborator Author

mr-cal commented Jul 5, 2024

Force-pushed in prep for a rebase

Copy link
Contributor

@lengau lengau left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

.github/workflows/spread-scheduled.yaml Show resolved Hide resolved
@mr-cal mr-cal merged commit 124c932 into main Jul 6, 2024
9 of 10 checks passed
@mr-cal mr-cal deleted the work/CRAFT-3077/launchpad-credentials branch July 6, 2024 01:52
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.

Use launchpad credentials from the new location and fallback to the legacy location
3 participants