Skip to content

Add dockerfile to build image for red hat certification #393

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

Merged
merged 5 commits into from
Jun 16, 2022

Conversation

FushuWang
Copy link
Contributor

Motivation

The red hat container certification requires the container:

  • To use the base image registry.access.redhat.com/ubi8/ubi from red hat.
  • Have labels: name, vendor, maintainer, version, release, summary, description.
  • Use a non-root user, but User nonroot:nonroot will cause pod error no users found.
  • Have a directory licenses with licenses file under the root path.

Modifications

  • Add a new dockerfile redhat.Dockerfile.
  • Add an Apache 2.0 license file.
  • Add a new docker build command in Makefile.

Verifying this change

I have tested the new image, it can pass the preflight check of the red hat container certification.

  • Make sure that the change passes the CI checks.

Documentation

Check the box below.

Need to update docs?

  • no-need-doc

@FushuWang FushuWang requested review from nlu90 and freeznet as code owners June 15, 2022 09:30
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Jun 15, 2022
@freeznet freeznet added type/enhancement Indicates an improvement to an existing feature m/2022-12 labels Jun 16, 2022
@freeznet
Copy link
Member

@nlu90 is it acceptable to use Apache 2.0 license?

@nlu90
Copy link
Contributor

nlu90 commented Jun 16, 2022

@nlu90 is it acceptable to use Apache 2.0 license?

It's okay to use it

@freeznet freeznet merged commit 48884cc into master Jun 16, 2022
@freeznet freeznet deleted the fswang/redhat-certification-image branch June 16, 2022 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
m/2022-12 no-need-doc This pr does not need any document type/enhancement Indicates an improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants