-
Notifications
You must be signed in to change notification settings - Fork 125
fix: replace git reference injection with wheel artifacts in charm test workflows #2252
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
Draft
Copilot
wants to merge
14
commits into
main
Choose a base branch
from
copilot/fix-charm-test-git-reference
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
71e3cd2
Initial plan
Copilot 0b9a43c
Replace git reference injection with wheel artifacts in charm test wo…
Copilot 2738a52
Fix wheel path resolution in workflows using echo command expansion
Copilot 205a864
Make path resolution consistent across all workflows
Copilot d012d43
Revert charmcraft-pack.yaml changes - pack runs in sandboxed environment
Copilot 9d95fe4
Rename build-wheels.yaml to _build-wheels.yaml to indicate it's internal
Copilot 6d4a687
Use more explicit pattern to match only ops wheel in hello-charm-tests
Copilot d184690
Fix wheel glob expansion - use ls instead of echo to expand patterns
Copilot e4d195b
Fix quote placement in glob patterns for published-charms-tests
Copilot 5347bff
feat: add debugging steps to show installed package versions in charm…
Copilot 08749c2
feat: show all installed dependencies in test environments to identif…
Copilot d1701db
fix: use uv pip freeze to show installed dependencies in uv-based tes…
Copilot 2bbca32
fix: use uv pip freeze without specifying Python to use uv-managed en…
Copilot c5f97df
fix: force upgrade pytest-interface-tester to >=3.4 in observability …
Copilot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,38 @@ | ||
| name: Build Operator Wheels | ||
|
|
||
| on: | ||
| workflow_call: | ||
| outputs: | ||
| artifact-name: | ||
| description: "Name of the artifact containing the wheels" | ||
| value: ${{ jobs.build-wheels.outputs.artifact-name }} | ||
|
|
||
| permissions: {} | ||
|
|
||
| jobs: | ||
| build-wheels: | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| artifact-name: ${{ steps.artifact-name.outputs.name }} | ||
| steps: | ||
| - name: Checkout the operator repository | ||
| uses: actions/checkout@v6 | ||
| with: | ||
| persist-credentials: false | ||
|
|
||
| - name: Set up uv | ||
| uses: astral-sh/setup-uv@1e862dfacbd1d6d858c55d9b792c756523627244 # v7.1.4 | ||
|
|
||
| - name: Build wheels | ||
| run: uv build --all | ||
|
|
||
| - name: Set artifact name | ||
| id: artifact-name | ||
| run: echo "name=operator-wheels-${{ github.sha }}" >> "$GITHUB_OUTPUT" | ||
|
|
||
| - name: Upload wheels as artifact | ||
| uses: actions/upload-artifact@v5 | ||
| with: | ||
| name: ${{ steps.artifact-name.outputs.name }} | ||
| path: dist/*.whl | ||
| retention-days: 1 | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.