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

UBI image for 'configure' container? #5912

Open
planetf1 opened this issue Nov 4, 2021 · 8 comments
Open

UBI image for 'configure' container? #5912

planetf1 opened this issue Nov 4, 2021 · 8 comments
Labels
containers Docker, docker-compose, Kubernetes, operators enhancement New feature or request pinned Keep open (do not time out) security Security related (high priority)

Comments

@planetf1
Copy link
Member

planetf1 commented Nov 4, 2021

The egeria container image is based on the RedHat UBI 8 openjdk base image. See https://www.redhat.com/en/blog/introducing-red-hat-universal-base-image . This is a supported, secure , OCI compliant image that passes container security checks on quay.io

However we also use some other containers to support demos and UI, which are based on lighter images such as alpine, and these may not be as secure.

Full data is available on quay.io by selecting the image under https://quay.io/organization/odpi, for example the egeria base image at https://quay.io/repository/odpi/egeria?tab=tags

We should consider using secure base images for all the container images we build (not just in egeria, but in other projects such as our connectors)

The downside is that newer images may increase the memory footprint - this may in particular be more painful on arm platforms. (the mitigation there may be needing alternate image definitions)

cc: @lpalashevski

@planetf1 planetf1 added the security Security related (high priority) label Nov 4, 2021
@planetf1 planetf1 self-assigned this Nov 4, 2021
@planetf1 planetf1 changed the title Secure container base images Secure & OCI compliant container base images Nov 4, 2021
@planetf1
Copy link
Member Author

planetf1 commented Dec 21, 2021

We still have

Still needed:

  • supported base image
  • tested/updated for security vulnarabilities
  • consistent with image used for base egeria
  • hosted on quay.io so will avoid dockerhub pull request limitations (but note that either side could have outages)

@cmgrote
Copy link
Member

cmgrote commented Dec 21, 2021

The small configure image is used in various places where Jupyter is not needed, in particular for parts of setting up the PTS and CTS charts -- so keen we keep that image as its own image (nice and small).

@planetf1 planetf1 added triage New bug/issue which needs checking & assigning containers Docker, docker-compose, Kubernetes, operators enhancement New feature or request and removed triage New bug/issue which needs checking & assigning labels Jan 25, 2022
@planetf1
Copy link
Member Author

#289 and #311 address updating our UI containers to UBI
Focus here on 'configure'. There is a UBI new micro image. Will review security scans / docs and figure out next step for this container

@planetf1 planetf1 changed the title Secure & OCI compliant container base images UBI image for 'configure' container? Mar 25, 2022
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the no-issue-activity Issues automatically marked as stale because they have not had recent activity. label May 25, 2022
@planetf1 planetf1 removed the no-issue-activity Issues automatically marked as stale because they have not had recent activity. label May 27, 2022
@planetf1
Copy link
Member Author

I'm thinking that perhaps we can remove this container entirely, and use a regular image (the lightest 'UBI' image ideally as it provides a better security stance).

We only use this image for simple scripts, which can be injected through a volume mount/config map or similar techniques.

This would mean one less image to maintain, scan & allow our process to focus on the egeria specifics

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the no-issue-activity Issues automatically marked as stale because they have not had recent activity. label Aug 23, 2022
@planetf1 planetf1 removed the no-issue-activity Issues automatically marked as stale because they have not had recent activity. label Aug 24, 2022
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the no-issue-activity Issues automatically marked as stale because they have not had recent activity. label Oct 25, 2022
@planetf1 planetf1 added pinned Keep open (do not time out) and removed no-issue-activity Issues automatically marked as stale because they have not had recent activity. labels Oct 31, 2022
@planetf1 planetf1 removed their assignment May 15, 2023
@planetf1
Copy link
Member Author

This image should be harmless, but can likely be replaced with use of a standard, simple image, without any added tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
containers Docker, docker-compose, Kubernetes, operators enhancement New feature or request pinned Keep open (do not time out) security Security related (high priority)
Projects
No open projects
Status: Todo
Development

No branches or pull requests

2 participants