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

MCO399: Added entitled host requirement for libreswan layering example #60105

Merged
merged 1 commit into from
May 18, 2023

Conversation

mburke5678
Copy link
Contributor

@mburke5678 mburke5678 added this to the Planned for 4.13 GA milestone May 16, 2023
@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 16, 2023
@kalexand-rh
Copy link
Contributor

The branch/enterprise-4.14 label has been added to this PR.

This is because your PR targets the main branch and is labeled for enterprise-4.13. And any PR going into main must also target the latest version branch (enterprise-4.14).

If the update in your PR does NOT apply to version 4.14 onward, please re-target this PR to go directly into the appropriate version branch or branches (enterprise-4.x) instead of main.

@sinnykumari
Copy link

/lgtm

Note that we also have mentioned this in example comment https://github.com/openshift/rhcos-image-layering-examples/blob/master/libreswan/Containerfile#L8 , so user would still be aware of the requirement of RHEL entitled system.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 16, 2023
@openshift openshift deleted a comment from openshift-ci bot May 16, 2023
@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label May 16, 2023
@mburke5678 mburke5678 removed the peer-review-needed Signifies that the peer review team needs to review this PR label May 16, 2023
@mburke5678
Copy link
Contributor Author

@sergiordlr PTAL

@sergiordlr
Copy link

The fish and the libreswan examples worked fine when we used an entitled machine to create the images and our own base images (the base image in the Containerfile does not work).

We specify in the documentation to use the cluster's base image, so I think it is ok.

The example works now, but we can't guarantee that this example will continue working, since we don't test this github repo for every commit, though. I don't know if it is possible, but wouldn't it be better to embed the examples instead of linking them?

cc: @rioliu-rh

@sinnykumari
Copy link

The fish and the libreswan examples worked fine when we used an entitled machine to create the images and our own base images (the base image in the Containerfile does not work).

We specify in the documentation to use the cluster's base image, so I think it is ok.

The example works now, but we can't guarantee that this example will continue working, since we don't test this github repo for every commit, though. I don't know if it is possible, but wouldn't it be better to embed the examples instead of linking them?

Yeah agree. In future, we maybe able to add some test cases in repo. But testing each example for each PR won't be feasible. Mostly what we would to test is what goes into our product doc repo.

@sergiordlr
Copy link

LGTM

@mburke5678 mburke5678 merged commit 3a3582d into openshift:main May 18, 2023
1 check passed
@mburke5678 mburke5678 deleted the mco-layering-add-entitlement branch May 18, 2023 19:53
@mburke5678
Copy link
Contributor Author

/cherrypick enterprise-4.13

@openshift-cherrypick-robot

@mburke5678: new pull request created: #60210

In response to this:

/cherrypick enterprise-4.13

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mburke5678
Copy link
Contributor Author

/cherrypick enterprise-4.14

@openshift-cherrypick-robot

@mburke5678: new pull request created: #60211

In response to this:

/cherrypick enterprise-4.14

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.13 branch/enterprise-4.14 lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants