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.

Sign up for GitHub

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

Jump to bottom

Need to enable the aggregator by default. #5056

Closed
deads2k opened this issue Aug 10, 2017 · 2 comments
Closed

Need to enable the aggregator by default. #5056

deads2k opened this issue Aug 10, 2017 · 2 comments
Assignees
@sdodson @ewolinetz
Labels
affects_3.7

Comments

@deads2k
Copy link
Contributor

deads2k commented Aug 10, 2017

In 3.7, the aggregator is on by default. The front proxy keys and certs used to enable it should be on by default in 3.7
This was referenced Aug 10, 2017
Merged
Merged
@ewolinetz
Copy link
Contributor

@deads2k is this simply a flipping the default from false to true for installing? Or do we also need to split up the installation of the aggregator, api server, and service brokers as part of this (and only set up the aggregator as part of a normal install)?

@deads2k
Copy link
Contributor Author

deads2k commented Sep 6, 2017

@deads2k is this simply a flipping the default from false to true for installing? Or do we also need to split up the installation of the aggregator, api server, and service brokers as part of this (and only set up the aggregator as part of a normal install)?

You should separate the enablement of the aggregator (its certs and keys) and have it unconditionally on for 3.7.

This was referenced Sep 8, 2017
Merged
Merged
openshift-merge-robot added a commit that referenced this issue Sep 21, 2017
@openshift-merge-robot
Merge pull request #5363 from ewolinetz/default_aggregation
a390624 
Automatic merge from submit-queue

Updating to always configure api aggregation with installation

This moves the wiring of the aggregator up into the config playbook as we want to enable this by default with an installation.

Resolves #5056
openshift-merge-robot added a commit to openshift/origin that referenced this issue Jan 11, 2018
@openshift-merge-robot
Merge pull request #15739 from deads2k/tsb-15-revert-authn
8043bc0 
Automatic merge from submit-queue (batch tested with PRs 15739, 18060).

Revert back to the "normal" apiserver authentication

Blocked on openshift/openshift-ansible#5056
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sdodson sdodson

@ewolinetz ewolinetz

Labels
affects_3.7
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@sdodson @ewolinetz @deads2k