Skip to content

Create script to build datastream images from source #5153

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 1 commit into from
Jan 31, 2020

Conversation

JAORMX
Copy link
Contributor

@JAORMX JAORMX commented Jan 29, 2020

This script runs the build_product script (taking the product as a
parameter), and it pushes the ssg-$PRODUCT-ds.xml file to a container.

The container is created by using a BuildConfig and an ImageStream,
these are updated by pushing the local XML file to the container build.

This is just one step of the build process which helps with local
development. In further iterations, I'll create a BuildConfig that builds
inside a container as opposed to requiring a local file.

Example:

./utils/build_ds_container.sh ocp4

@JAORMX
Copy link
Contributor Author

JAORMX commented Jan 29, 2020

Note that this makes some assumptions (e.g. the naming scheme of the ds file is ssg--ds.xml). This works for ocp4 and that's what I tested. I haven't tested this for rhel yet.

@JAORMX JAORMX force-pushed the build-ds-container branch from e42447a to 611f46e Compare January 29, 2020 14:57
@JAORMX
Copy link
Contributor Author

JAORMX commented Jan 29, 2020

Just tested it with RHEL and did some fixes.

Copy link
Collaborator

@jhrozek jhrozek left a comment

Choose a reason for hiding this comment

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

LGTM from the point of view of the openshift commands and the Dockerfile etc. I'll let someone from the CaC team chime in whether the root of the dir is the best location for the script. But it looks good to me, thank you!

Also, if there's any developer documentation maybe it should be amended to point to this new script. But that can be done in a separate PR, let's get this one merged so that people can experiment with the content creation easily

@jan-cerny
Copy link
Collaborator

@JAORMX Great! Thanks. Please move it to utils directory or elsewhere.

This script runs the build_product script (taking the product as a
parameter), and it pushes the ssg-$PRODUCT-ds.xml file to a container.

The container is created by using a BuildConfig and an ImageStream,
these are updated by pushing the local xml file to the container build.

This is just one step of the build process which helps with local
development. In further iterations I'll create a BuildConfig that builds
inside a container as opposed to requiring a local file.

Example:

    ./utils/build_ds_container.sh ocp4
@JAORMX JAORMX force-pushed the build-ds-container branch from 611f46e to 076c0b1 Compare January 30, 2020 15:13
@JAORMX
Copy link
Contributor Author

JAORMX commented Jan 30, 2020

@jan-cerny done! thanks for the feedback

@jan-cerny jan-cerny self-assigned this Jan 31, 2020
@jan-cerny jan-cerny added this to the 0.1.49 milestone Jan 31, 2020
@jan-cerny jan-cerny merged commit a7d889a into ComplianceAsCode:master Jan 31, 2020
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