-
Notifications
You must be signed in to change notification settings - Fork 22
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
add custom capo readme #88
add custom capo readme #88
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Gal-Zaidman The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for sharing your work in progress doc Gal. I have few suggestions.
docs/custom-capo.md
Outdated
When we develop the CAPO its usually helpful to test it against a running cluster. | ||
The following doc explains how to do it. | ||
|
||
## Replacing the docker image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would replace "docker" as container.
docs/custom-capo.md
Outdated
@@ -0,0 +1,58 @@ | |||
# Creating a custom cluster api provider ovirt and adding it to the cluster | |||
|
|||
When we develop the CAPO its usually helpful to test it against a running cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's CAPO? :) I am assuming is: Cluster API Provider OVirt (CAPO)
I would than use: When we develop the Cluster API Provider oVirt (CAPO) its ....
|
||
1.Lunch a cluster: | ||
|
||
Start an Openshift cluster on top of oVirt as usual with the required release image. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like a note here like:
For more info about installing and launching a new cluster using oVirt check:
docs/custom-capo.md
Outdated
|
||
## Replacing the docker image | ||
|
||
1.Lunch a cluster: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Launch
docs/custom-capo.md
Outdated
|
||
Start an Openshift cluster on top of oVirt as usual with the required release image. | ||
|
||
2. Create the docker image: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:s/docker/container/g
docs/custom-capo.md
Outdated
4. Replace the CAPO image | ||
|
||
```bash | ||
oc -n openshift-machine-api edit configmap/machine-api-operator-images |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$ oc
```bash | ||
oc -n openshift-machine-api edit configmap/machine-api-operator-images | ||
``` | ||
and replace clusterAPIControllerOvirt with the pull (by digest) of your image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Users and devel will ask: How can I find the digest from the my image? Can we add an example here?
docs/custom-capo.md
Outdated
5. verify that the image has been replaced: | ||
|
||
```bash | ||
oc -n openshift-machine-api get pods |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$ oc
docs/custom-capo.md
Outdated
5. verify that the image has been replaced: | ||
|
||
```bash | ||
oc -n openshift-machine-api get pods |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would copy/past a real example of the command
134159d
to
3853b10
Compare
@Gal-Zaidman: The following test failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
documentation only |
No description provided.