• Kubernetes Submit Queue's avatar
    Merge pull request #36008 from MrHohn/addon-rc-migrate · c98fc701
    Kubernetes Submit Queue authored
    Automatic merge from submit-queue
    
    Migrates addons from RCs to Deployments
    
    Fixes #33698.
    
    Below addons are being migrated:
    - kube-dns
    - GLBC default backend
    - Dashboard UI
    - Kibana
    
    For the new deployments, the version suffixes are removed from their names. Version related labels are also removed because they are confusing and not needed any more with regard to how Deployment and the new Addon Manager works.
    
    The `replica` field in `kube-dns` Deployment manifest is removed for the incoming DNS horizontal autoscaling feature #33239.
    
    The `replica` field in `Dashboard` Deployment manifest is also removed because the rescheduler e2e test is manually scaling it.
    
    Some resource limit related fields in `heapster-controller.yaml` are removed, as they will be set up by the `addon resizer` containers. Detailed reasons in #34513.
    
    Three e2e tests are modified:
    - `rescheduler.go`: Changed to resize Dashboard UI Deployment instead of ReplicationController.
    - `addon_update.go`: Some namespace related changes in order to make it compatible with the new Addon Manager.
    - `dns_autoscaling.go`: Changed to examine kube-dns Deployment instead of ReplicationController.
    
    Both of above two tests passed on my own cluster. The upgrade process --- from old Addons with RCs to new Addons with Deployments --- was also tested and worked as expected.
    
    The last commit upgrades Addon Manager to v6.0. It is still a work in process and currently waiting for #35220 to be finished. (The Addon Manager image in used comes from a non-official registry but it mostly works except some corner cases.)
    
    @piosz @gmarek could you please review the heapster part and the rescheduler test?
    
    @mikedanese @thockin 
    
    cc @kubernetes/sig-cluster-lifecycle 
    
    ---
    
    Notes:
    - Kube-dns manifest still uses *-rc.yaml for the new Deployment. The stale file names are preserved here for receiving faster review. May send out PR to re-organize kube-dns's file names after this.
    - Heapster Deployment's name remains in the old fashion(with `-v1.2.0` suffix) for avoiding describe this upgrade transition explicitly. In this way we don't need to attach fake apply labels to the old Deployments.
    c98fc701
Name
Last commit
Last update
..
e2e Loading commit data...
e2e_node Loading commit data...
fixtures Loading commit data...
images Loading commit data...
integration Loading commit data...
kubemark Loading commit data...
list Loading commit data...
soak Loading commit data...
utils Loading commit data...
OWNERS Loading commit data...
test_owners.csv Loading commit data...