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

ibmcloud: PeerPod demo instructions for OpenShift clusters #2180

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

frankbu
Copy link
Contributor

@frankbu frankbu commented Dec 3, 2024

Added doc with streamlined instructions to set up the peer pod demo environment using an OpenShift cluster.

Also added references and made small corrections to the README.

@frankbu frankbu marked this pull request as ready for review December 4, 2024 14:06
@frankbu frankbu requested a review from a team as a code owner December 4, 2024 14:06
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

Some comments so far, the last of which is a blocker as I can't deploy to the cluster

src/cloud-api-adaptor/ibmcloud/ROKS_SETUP.md Show resolved Hide resolved
src/cloud-api-adaptor/ibmcloud/ROKS_SETUP.md Show resolved Hide resolved
src/cloud-api-adaptor/ibmcloud/ROKS_SETUP.md Show resolved Hide resolved
src/cloud-api-adaptor/ibmcloud/ROKS_SETUP.md Outdated Show resolved Hide resolved
src/cloud-api-adaptor/ibmcloud/ROKS_SETUP.md Show resolved Hide resolved
src/cloud-api-adaptor/ibmcloud/ROKS_SETUP.md Show resolved Hide resolved
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

Apart from the comments I left I think there are minor improvement about having a more consistent base directory (it might just be me assuming things but I seemed to have to skip between the root project directory and src/cloud-api-adaptor a few times. But overall it looks great and I was able to get peer pods running on ROKS after your assistance. Once the comments are incorporated I will re-review and approve. Thanks @frankbu!


```bash
kubectl get pods -n default
kubectl get peerpod -n default
Copy link
Member

Choose a reason for hiding this comment

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

Sorry, it's probably embarrassing user error, but there isn't a type peerpod:

kubectl get peerpod -n default
error: the server doesn't have a resource type "peerpod"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, I think the operator adds the CRD for peerpod. It works for me:

$ kubectl get peerpod -n default
NAME                        AGE
helloworld-resource-qtlr4   44h

src/cloud-api-adaptor/ibmcloud/ROKS_SETUP.md Show resolved Hide resolved
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

Looks good for now. Thanks very much!

Added doc with streamlined instructions to set up the peer pod demo
environment using an OpenShift cluster.

Also added references and made small corrections to the README.

Signed-off-by: Frank Budinsky <[email protected]>
Remove unneeded advertise the extended resource step from README.

Signed-off-by: Frank Budinsky <[email protected]>
Copy link
Member

@bpradipt bpradipt left a comment

Choose a reason for hiding this comment

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

/lgtm

@stevenhorsman stevenhorsman merged commit dff72ea into confidential-containers:main Dec 9, 2024
21 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.

3 participants