Skip to content

Change keepalived VIP subnet mask to host mask#100

Merged
yboaron merged 1 commit intoopenshift:masterfrom
yboaron:set_keepalived_mask_tohost
Sep 18, 2020
Merged

Change keepalived VIP subnet mask to host mask#100
yboaron merged 1 commit intoopenshift:masterfrom
yboaron:set_keepalived_mask_tohost

Conversation

@yboaron
Copy link
Contributor

@yboaron yboaron commented Sep 17, 2020

To work around [1] bug in network manager the subnet mask of Keepalived VIP address was set to the value of control plane IP subnet mask.
So, after the network manager bug was resolved we can safely change VIP subnet mask to a host subnet mask.

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1700415

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 17, 2020
@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 17, 2020
@cybertron
Copy link
Member

/lgtm

@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cybertron, yboaron

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 17, 2020
@yboaron
Copy link
Contributor Author

yboaron commented Sep 18, 2020

/retitle Change keepalived VIP subnet mask to host mask

@openshift-ci-robot openshift-ci-robot changed the title [WIP] Change keepalived VIP subnet mask to host Change keepalived VIP subnet mask to host mask Sep 18, 2020
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 18, 2020
@yboaron yboaron merged commit 9a51036 into openshift:master Sep 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants