Skip to content
This repository was archived by the owner on Dec 9, 2020. It is now read-only.

[WIP] Provision lb node when multiple infra nodes#704

Closed
bogdando wants to merge 1 commit intoopenshift:masterfrom
bogdando:multi-infra-lb
Closed

[WIP] Provision lb node when multiple infra nodes#704
bogdando wants to merge 1 commit intoopenshift:masterfrom
bogdando:multi-infra-lb

Conversation

@bogdando
Copy link
Contributor

What does this PR do?

Provision a load balancer node, when having a multi-infra nodes setup.
The expected configuration for the LB node and the use case has to be elaborated and documented.
Currently, it gets no special configuration for infra nodes.

How should this be manually tested?

TBD

Is there a relevant Issue open for this?

Provide a link to any open issues that describe the problem you are solving.

Who would you like to review this?

cc: @tomassedovic PTAL

Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
@bogdando bogdando added the osp label Aug 29, 2017
@bogdando
Copy link
Contributor Author

What is the use case for a lb node and multi infra nodes? I can't find any docs or details on that. Which configuration the lb node is expected to get for multi infra nodes? What is the test case for it?

Shall the lb node's haproxy config provide a load balance for a docker registry, when deploying a multiple infra nodes?
I can see only a single logical frontend/backend definition stanza for master lb:

frontend  atomic-openshift-api
    bind *:8443
    default_backend atomic-openshift-api
    mode tcp
    option tcplog

backend atomic-openshift-api
    balance source
    mode tcp
    server      master0 192.168.99.12:8443 check

so in order to continue this PR, I need to know more for supported use cases of lb for registry and routers, and what else hosted at the infra nodes?..

@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

2 similar comments
@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

2 similar comments
@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@bogdando
Copy link
Contributor Author

It's time to resubmit on top of openshift/openshift-ansible#6039

@bogdando bogdando closed this Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants