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

Update the L3 HA document in the Howtos section #273

Merged
merged 3 commits into from
Apr 12, 2024
Merged

Conversation

PioMis1984
Copy link
Contributor

No description provided.

@PioMis1984
Copy link
Contributor Author

I've added tabs: one for legacy regions and the second for new regions. There are only a minor differences between the two.
Also added a warning message at the top with our recommendation to use LBaaS instead.

Copy link
Contributor

@matthiasbernhardt matthiasbernhardt left a comment

Choose a reason for hiding this comment

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

Only two small remarks on the distinction between our regions that may also apply to other places with wordings like "in cloud environments" which may need to be reworded in both variants to apply to certain cloud environments but not all.

I think we do not cover the usage of floating IPs in conjunction with such virtual IPs yet. Perhaps we can add that or at least create a new ticket to do it separately.

user/pages/03.Howtos/10.l3-high-availability/docs.en.md Outdated Show resolved Hide resolved
user/pages/03.Howtos/10.l3-high-availability/docs.en.md Outdated Show resolved Hide resolved
Copy link
Contributor

@matthiasbernhardt matthiasbernhardt left a comment

Choose a reason for hiding this comment

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

I'm fine with this MR now.

I think we do not cover the usage of floating IPs in conjunction with such virtual IPs yet. If we cannot add that now, we should at least create a new ticket to do it separately.

@PioMis1984 PioMis1984 merged commit 10fec38 into master Apr 12, 2024
1 check 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