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

muliti-arch-builders/splunk: only build the splunk sidecar container #1045

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

marmijo
Copy link
Member

@marmijo marmijo commented Oct 8, 2024

The repo we are using to build the ITPAAS splunkforwarder container on the RHCOS multi-arch builders is out of date and references a splunkforwarder rpm that no longer exists in the internal repos. By omitting the build step for this container, we can pull in the already built image from ITPAAS that includes a more recent RPM.
We'll still use --arch=x86_64 for the sidecar container to ensure proper logging on non x86_64 builders.

See: #1044

The repo we are using to build the ITPAAS splunkforwarder container
on the RHCOS multi-arch builders is out of date and references a
splunkforwarder rpm that no longer exists in the internal repos.
By omitting the build step for this container, we can pull in the
already built image from ITPAAS that includes a more recent RPM.
We'll still use `--arch=x86_64` for the sidecar container to ensure
proper logging on non x86_64 builders.

See: coreos#1044
@dustymabe
Copy link
Member

previously we weren't pulling any containers from $internal, we were accessing internal git repos but we weren't pulling any containers from internal registries. (IIUC)

are we sure we will be able to pull from those registries? one way to check is to log in to the various multi-arch builders and podman pull it.

@marmijo
Copy link
Member Author

marmijo commented Oct 8, 2024

are we sure we will be able to pull from those registries? one way to check is to log in to the various multi-arch builders and podman pull it.

yes, we can pull from that registry. I've already restarted the splunk container on the RHCOS aarch64 and s390x builders and it could podman pull the container just fine 👍

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@marmijo

This comment was marked as outdated.

@marmijo marmijo merged commit 7ff32b9 into coreos:main Oct 8, 2024
2 checks passed
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.

2 participants