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

feat: use platform name in build instance name #601

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

mr-cal
Copy link
Contributor

@mr-cal mr-cal commented Jan 8, 2025

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox?

Changes the build instance name to use the platform rather than the build-on and build-for architectures.

This prevents the same build environment being used for multiple platforms. See canonical/charmcraft#2069 for examples.

Fixes #600
Unblocks canonical/charmcraft#2069
(CRAFT-3859)

@mr-cal mr-cal force-pushed the work/CRAFT-3859-platform-in-name branch from ed273d0 to 2d776a3 Compare January 8, 2025 22:56
@mr-cal
Copy link
Contributor Author

mr-cal commented Jan 9, 2025

Integration test failure looks unrelated, something is going on with launchpad. https://api.staging.launchpad.net/devel/people/+me isn't responding

pyproject.toml Outdated Show resolved Hide resolved
@mr-cal mr-cal marked this pull request as ready for review January 9, 2025 15:00
@mr-cal mr-cal requested review from lengau and mattculler January 9, 2025 15:00
Copy link
Contributor

@mattculler mattculler left a comment

Choose a reason for hiding this comment

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

Excellent

craft_application/services/provider.py Show resolved Hide resolved
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.

Nice, thanks!

@mr-cal mr-cal force-pushed the work/CRAFT-3859-platform-in-name branch 2 times, most recently from 4d20752 to d93c378 Compare January 10, 2025 19:36
@mr-cal mr-cal added the rebase label Jan 10, 2025
@mr-cal
Copy link
Contributor Author

mr-cal commented Jan 10, 2025

forced push to bump craft-providers and prep for a rebase

@mr-cal mr-cal force-pushed the work/CRAFT-3859-platform-in-name branch from d93c378 to 7e165b1 Compare January 10, 2025 19:38
@mr-cal
Copy link
Contributor Author

mr-cal commented Jan 10, 2025

blocked by #603

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
This ensures each platform is built in its own build environment.

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
@mr-cal mr-cal force-pushed the work/CRAFT-3859-platform-in-name branch from 7e165b1 to ca8a159 Compare January 13, 2025 17:23
@mr-cal
Copy link
Contributor Author

mr-cal commented Jan 13, 2025

force-pushing to update date in changelog and prep for rebase

@mr-cal mr-cal merged commit 5ae7f87 into main Jan 13, 2025
10 checks passed
@mr-cal mr-cal deleted the work/CRAFT-3859-platform-in-name branch January 13, 2025 17:37
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 platform for build environment names
3 participants