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

Feat: add tools required for infra-reports #66

Merged
merged 4 commits into from
Mar 3, 2022

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Feb 24, 2022

As part of jenkins-infra/helpdesk#2789, this PR ensures that the required tools to allow running the scripts of https://github.com/jenkins-infra/infra-reports are present.

It adds the following elements:

  • jq and yq in their latest version available in the package manager of the base image (Alpine 3.15 when writing these lines)
  • ruby in the latest 3.0.x version available in the package manager of the base image (Alpine 3.15 when writing these lines).
    • No updatecli for this as it's a bit hard to retrieve the Alpine package version for this version. No worries though: when the base image will be upgraded to a more recent version of Alpine that would bump Ruby's minor (or major) version, then the build would fail and would require attention.
  • Latest versions of the rubygems for http and graphql are installed.

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal dduportal added the enhancement New feature or request label Feb 24, 2022
@dduportal
Copy link
Contributor Author

dduportal commented Feb 24, 2022

We are facing a "PermissionDenied" error: #8 1.548 ERROR: While executing gem ... (Gem::FilePermissionError)

This error is caused by https://wiki.alpinelinux.org/wiki/Release_Notes_for_Alpine_3.14.0#faccessat2 (e.g. Alpine Linux >=3.14) with the fact that we are building Docker images using https://github.com/genuinetools/img/releases/tag/v0.5.11 (which implies an older container runtime than the required one in Alpine release notes).

There are multiple solutions:

  • Stop using img and replace it by something else (Docker VM wink wink, implies updating the shared library)
  • Add the specified seccomp profile (subject to NOT work as we start the img pod unconfined for seccomp)
  • Switch base image from Alpine to Debian

@dduportal
Copy link
Contributor Author

I'm going to the "update shared library to allow using Docker Engine" :)

@lemeurherve
Copy link
Member

Do we really have to?

@dduportal
Copy link
Contributor Author

Do we really have to?

have to what ? :)

@dduportal dduportal force-pushed the feat/infra-report-ruby branch from 23c7fae to 802cab0 Compare March 2, 2022 11:53
Dockerfile Outdated Show resolved Hide resolved
lemeurherve
lemeurherve previously approved these changes Mar 2, 2022
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
lemeurherve
lemeurherve previously approved these changes Mar 2, 2022
@dduportal dduportal marked this pull request as draft March 2, 2022 15:49
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
lemeurherve
lemeurherve previously approved these changes Mar 3, 2022
Jenkinsfile_k8s Outdated Show resolved Hide resolved
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal dduportal enabled auto-merge (squash) March 3, 2022 18:41
@dduportal dduportal merged commit 83758df into jenkins-infra:main Mar 3, 2022
@dduportal dduportal deleted the feat/infra-report-ruby branch March 3, 2022 18:43
dduportal added a commit that referenced this pull request Jul 20, 2022
dduportal added a commit that referenced this pull request Jul 20, 2022
* breaking: remove ruby installation

- Introduced in #66
- Not needed since jenkins-infra/infra-reports#43

* Update cst.yml

* Update Dockerfile
dduportal added a commit that referenced this pull request Jul 20, 2022
* breaking: remove ruby installation

- Introduced in #66
- Not needed since jenkins-infra/infra-reports#43

* Update cst.yml

* Update Dockerfile
dduportal added a commit that referenced this pull request Jul 20, 2022
* chore: Update key "metadataTest.labels[11].value" from file "/tmp/updat...

... ecli/jenkins-infra/docker-helmfile/cst.yml"

Made with ❤️️ by updatecli

* chore: Update key "metadataTest.labels[11].value" from file "/tmp/updat...

... ecli/jenkins-infra/docker-helmfile/cst.yml"

Made with ❤️️ by updatecli

* breaking: remove ruby installation (#134)

* breaking: remove ruby installation

- Introduced in #66
- Not needed since jenkins-infra/infra-reports#43

* Update cst.yml

* Update Dockerfile

* chore: fix build failue introduced with latest Alpine which has content in the /tmp directory

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>

Co-authored-by: Jenkins Infra Bot (updatecli) <60776566+jenkins-infra-bot@users.noreply.github.com>
Co-authored-by: Damien Duportal <damien.duportal@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants