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: fix hadolint/shellcheck violations #320

Merged
merged 9 commits into from
Nov 12, 2022

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Nov 12, 2022

This PR fixes a set of hadolint/shellcheck violations on the images.

Tried to proceed 1 commit per rule, so we can easily rollback only one.

I'm not sure yet how to efficiently load hadolint recursively on ALL Dockerfile though.

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
… tag (tracked by dependabot)

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal dduportal marked this pull request as ready for review November 12, 2022 17:34
@dduportal dduportal requested a review from a team as a code owner November 12, 2022 17:34
@dduportal dduportal changed the title Chore/hadolint chore: fix hadolint/shellcheck violations Nov 12, 2022
@dduportal dduportal mentioned this pull request Nov 12, 2022
6 tasks
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
…l chars

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal dduportal merged commit a096c55 into jenkinsci:master Nov 12, 2022
@dduportal dduportal deleted the chore/hadolint branch November 12, 2022 19:56
@basil
Copy link
Member

basil commented Nov 30, 2022

Possible cause of JENKINS-70157? Of all the commits I looked through between 3071.v7e9b_0dc08466-1 and 3071.v7e9b_0dc08466-5 this seemed like the most likely candidate. CC @dduportal

@dduportal
Copy link
Contributor Author

Possible cause of JENKINS-70157? Of all the commits I looked through between 3071.v7e9b_0dc08466-1 and 3071.v7e9b_0dc08466-5 this seemed like the most likely candidate. CC @dduportal

That could also be impacting ci.jenkins.io. Currently checking, thanks for the notice @basil

@dduportal
Copy link
Contributor Author

I can't reproduce the issue on my Windows setup any help is welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants