Skip to content

fix(infra): change artifact path to include scope #37

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.

Sign up for GitHub

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

Merged
netanelC merged 12 commits into from
Feb 6, 2025
Merged

fix(infra): change artifact path to include scope #37

Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
161ea68
fix(infra): change artifact path
Feb 4, 2025
cc975b9
fix(infra): dispatch
Feb 4, 2025
d39e91a
fix(infra): dispatch values
Feb 4, 2025
1d60cd9
fix(infra): dispatch values
Feb 4, 2025
8d3d9d0
fix(infra): restore
Feb 4, 2025
b0653f1
fix(infra): debug-ci
Feb 4, 2025
256d6bf
fix(infra): debug-ci
Feb 4, 2025
abb4aa0
fix(infra): no json
Feb 4, 2025
b313d25
fix(infra): //
Feb 4, 2025
e0fd9c8
fix(infra): remove debugging
Feb 4, 2025
1ea7abb
fix(infra): remove debugging
Feb 4, 2025
69e49ae
fix(infra): remove comment
Feb 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions .github/workflows/update-artifacts-file.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,11 @@ jobs:
if (fs.existsSync(path)) {
data = JSON.parse(fs.readFileSync(path, 'utf8'));
}
// Ensure nested structure: docker -> registry -> artifact:tag
if (!data['${{ inputs.type }}']) data['${{ inputs.type }}'] = {};
// Ensure nested structure: docker -> registry -> scope -> artifact:tag
if (!data['${{ inputs.type }}']['${{ secrets.ACR_URL }}']) data['${{ inputs.type }}']['${{ secrets.ACR_URL }}'] = {};
data['${{ inputs.type }}']['${{ secrets.ACR_URL }}']['${{ inputs.artifact-name }}'] = '${{ inputs.artifact-tag }}';
const compositeKey = '${element}/${{ inputs.artifact-name }}';
// Assign the artifact tag to the composite key
data['${{ inputs.type }}']['${{ secrets.ACR_URL }}'][compositeKey] = '${{ inputs.artifact-tag }}';
// Writing new file contents
fs.writeFileSync(path, JSON.stringify(data, null, 2));
"
Expand Down