Skip to content
This repository has been archived by the owner on Jan 21, 2024. It is now read-only.

fix: keep the parent image label with the proper remoting version (fixes hadolint violations) #308

Merged
merged 1 commit into from
Sep 2, 2023

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Nov 23, 2022

Self-describing title :)

1 commit per rule.

  • LABEL is removed (along with associated arguments) because it was not the same value between images, and it was overriding the parent's image, causing confusion. Now the "version" mentioned in the labels is the remoting version.

@dduportal dduportal requested a review from a team as a code owner November 23, 2022 16:26
@dduportal
Copy link
Contributor Author

Putting back as "Draft": the Windows SHELL should be the cmd.exe, not powershell.exe. When I tested locally, I tried the wrong Dockerfiles.

@dduportal dduportal marked this pull request as draft November 23, 2022 16:51
…xes hadolint violations)

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal dduportal marked this pull request as ready for review September 2, 2023 09:42
@dduportal dduportal added bug and removed chore labels Sep 2, 2023
@dduportal dduportal changed the title chore(hadolint) fix violations on Dockerfiles fix: keep the parent image label with the proper remoting version (fixes hadolint violations) Sep 2, 2023
@dduportal
Copy link
Contributor Author

Rebased against the (simplified) master branch, simplified the change and advertise as a bugfix instead of a chore

@dduportal dduportal merged commit 27976ca into jenkinsci:master Sep 2, 2023
5 checks passed
@dduportal dduportal deleted the chore/hadolint branch September 2, 2023 10:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants