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

oc cluster up master branch broken #16627

Closed
mjudeikis opened this issue Sep 30, 2017 · 2 comments · Fixed by #16649
Closed

oc cluster up master branch broken #16627

mjudeikis opened this issue Sep 30, 2017 · 2 comments · Fixed by #16649
Assignees
Labels
component/networking kind/bug Categorizes issue or PR as related to a bug. priority/P1
Milestone

Comments

@mjudeikis
Copy link
Contributor

oc cluster up fails due move to array-based clusterNetworkCIDR:

oc v3.7.0-alpha.1+55a3968-807-dirty                                                                    
kubernetes v1.7.0+80709908fd                                                                         
features: Basic-Auth  

new generated config:

networkConfig:               
  clusterNetworks:           
  - cidr: 10.128.0.0/14      
    hostSubnetLength: 9      

Old config:

networkConfig:
  clusterNetworkCIDR: 10.128.0.0/14
  hostSubnetLength: 9

error:

I0930 16:20:47.520068   17336 plugins.go:101] No cloud provider specified.                               
E0930 16:20:47.526582   17336 controllermanager.go:357] Server isn't healthy yet.  Waiting a little while                                                                     
F0930 16:20:47.532135   17336 start_allinone.go:100] Server could not start: invalid CIDR address:       

rollback config works, but openshift binary already generates new style config

 _output/local/bin/linux/amd64/oc cluster up --use-existing-config --host-config-dir=/var/lib/origin/openshift.local.config/

@csrwng @jim-minter

@smarterclayton smarterclayton added the kind/bug Categorizes issue or PR as related to a bug. label Oct 1, 2017
@smarterclayton
Copy link
Contributor

@openshift/networking

We may have to update this change by defaulting the new list to the singleton value and then during validation allowing it to continue if they're equal.

@karmab
Copy link
Contributor

karmab commented Oct 2, 2017

is there a workaround for new installs ( willing to test latest asb)

openshift-merge-robot added a commit that referenced this issue Oct 4, 2017
Automatic merge from submit-queue (batch tested with PRs 16644, 16649, 16656, 16651, 16663).

Switch from defaulting to converting clusterNetworkCIDR

Fixes break. On disk, deprecated fields override the first value in
the clusterNetworks array if specified. When writing out, the deprecated
fields are set to the first value of the network CIDR.

Fixes #16627

This fixes oc cluster up. @csrwng @JacobTanenbaum
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/networking kind/bug Categorizes issue or PR as related to a bug. priority/P1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants