k8s heat templates are confusing master with minion

Bug #1608923 reported by Piotr Parczewski
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Low
Drago

Bug Description

k8s master is the first instance being spawned by bay creation - however, it's configured with 'minion' user. This is highly confusing.

Steps to reproduce:

1. Use devstack with magnum.
2. Create bay with coe = kubernetes

rajiv (rajiv-kumar)
Changed in magnum:
assignee: nobody → rajiv (rajiv-kumar)
Revision history for this message
rajiv (rajiv-kumar) wrote :

There are two possibilities for solution:

1) Change the user name from the minion to some generic name, not dependant on the role of the instance. Name can be something describing driver name or used base image. So choices might be
   a) kubernetes
   b) fedora/coreos

2) Change the user name of the instance hosting master, from the "minion" to "master" and keep minion user name as it is.

No. 2 option introduce little complexity into the code. For just user name, we have to add extra code and maintain it. Now it has to be decided whether it is worth implementing or not.

No. 1.b suffer from the same disadvantage of No 2.

No. 1.a is easy to implement and does not require addition of extra code, just name from "minion" to generic name(TBD) will be changed.

Revision history for this message
Spyros Trigazis (strigazi) wrote :

This choice was made when these templates were suppose to work with different distros [1].

We should the name change and left as it is the image. In our case will be fedora.

[1] https://github.com/larsks/heat-kubernetes/commit/747d06c8797ca6f14629683a3fb3376d8aae3782

Changed in magnum:
importance: Undecided → Wishlist
status: New → Opinion
importance: Wishlist → Low
status: Opinion → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

Fix proposed to branch: master
Review: https://review.openstack.org/351926

Changed in magnum:
assignee: rajiv (rajiv-kumar) → Drago (dragorosson)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/351926
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=83d154607e626ad09cee75521d6a3b51f48b2c6d
Submitter: Jenkins
Branch: master

commit 83d154607e626ad09cee75521d6a3b51f48b2c6d
Author: Drago Rosson <email address hidden>
Date: Fri Aug 5 20:39:42 2016 +0000

    Remove kube-user.yaml

    Since the fedora atomic already gets set up with the user "fedora",
    we can get rid of the "minion" user since it does not add any value
    and it is somewhat confusing logging into master nodes as "minion".

    Change-Id: I8c842d08780a786a1568f2d50219e580334ef487
    Closes-bug: #1608923

Changed in magnum:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/magnum 3.0.0

This issue was fixed in the openstack/magnum 3.0.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.