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
Update Ironic Keepalived container documentation #559
Update Ironic Keepalived container documentation #559
Conversation
c1e8b67
to
2eb2d76
Compare
/assign @maelk |
/test-integration |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kashifest, maelk 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.
lgtm
URL is used with `ironic-keepalived-config/image_patch.yaml` to replace the | ||
default image URL with the correct URL through kustomization. | ||
URL is used with `ironic-deployment/keepalived/keepalived_patch.yaml` to replace | ||
the default image URL with the correct URL through kustomization. | ||
|
||
**Important Note** |
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 forgot to add, Important Note part is already included in deploying.md, do we need to keep this here as well?
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 don't see any harm. I would rather keep it here specially since the variables are expected for this container.
/lgtm |
The kustomization workflow has changed significantly and the different deployment scenarios have also been updated. This PR removes the obsolete info from Ironic Keepalived container documentation and also updates the documentation with the way it should be deployed now.