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: add azcopy updatecli manifest #205

Conversation

lemeurherve
Copy link
Member

Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

2 deprecated syntax in the manifest

updatecli/updatecli.d/azcopy.yml Outdated Show resolved Hide resolved
updatecli/updatecli.d/azcopy.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

Other than deprecated syntax:

2024-04-02T11:42:25.759Z] TARGETS
[2024-04-02T11:42:25.759Z] ========
[2024-04-02T11:42:25.759Z] 
[2024-04-02T11:42:25.759Z] updateDockerfileArgVersion
[2024-04-02T11:42:25.759Z] --------------------------
[2024-04-02T11:42:25.759Z] 
[2024-04-02T11:42:25.759Z] **Dry Run enabled**
[2024-04-02T11:42:25.759Z] 
[2024-04-02T11:42:26.023Z] ⚠ The line #70, matched by the keyword "ARG" and the matcher "AZCOPY_VERSION", is changed from "ARG AZCOPY_VERSION=10.21.2-20231106" to "ARG AZCOPY_VERSION=10.24.0-20240326".
[2024-04-02T11:42:26.023Z] ⚠ - changed lines [70] of file "/tmp/updatecli/github/jenkins-infra/docker-builder/Dockerfile"
[2024-04-02T11:42:26.023Z] 
[2024-04-02T11:42:26.023Z] updateCstVersion
[2024-04-02T11:42:26.023Z] ----------------
[2024-04-02T11:42:26.023Z] 
[2024-04-02T11:42:26.023Z] **Dry Run enabled**
[2024-04-02T11:42:26.023Z] 
[2024-04-02T11:42:26.023Z] WARNING: current yaml key is "metadataTest.labels[6].value" and should be updated to "$.metadataTest.labels[6].value"
[2024-04-02T11:42:26.280Z] ⚠ - change detected:
[2024-04-02T11:42:26.280Z] 	* key "$.metadataTest.labels[6].value" should be updated from "10.21.2-20231106" to "10.24.0-20240326", in file "cst.yml"

LGTM

Co-authored-by: Damien Duportal <damien.duportal@gmail.com>
@lemeurherve lemeurherve merged commit 8b9be51 into jenkins-infra:main Apr 2, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants