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

添加node发现的bug #541

Closed
jacky-neo opened this issue May 5, 2019 · 7 comments
Closed

添加node发现的bug #541

jacky-neo opened this issue May 5, 2019 · 7 comments

Comments

@jacky-neo
Copy link

版本是kubeasz 1.0.1, 通过aszctl添加节点后,/etc/ansible/hosts里[kube-node]会有新的节点带“NEW_NODE=yes”, 但把已有的节点删除掉一个了。感觉是加新节点是用第一个替换的。 请楼主看看

@jacky-neo
Copy link
Author

再补充一下,如被替换的节点是etcd节点,在hosts文件里也没了

@gjmzj
Copy link
Collaborator

gjmzj commented May 6, 2019

新增节点不是替换操作

function add-node() {
    # check new node's address regexp
    [[ $1 =~ ^(2(5[0-5]{1}|[0-4][0-9]{1})|[0-1]?[0-9]{1,2})(\.(2(5[0-5]{1}|[0-4][0-9]{1})|[0-1]?[0-9]{1,2})){3}$ ]] || { echo "[ERROR] Invalid ip address!"; return 1; }

    # check if the new node already exsited
    sed -n '/^\[kube-master/,/^\[harbor/p' $BASEPATH/hosts|grep "^$1" && { echo "[ERROR] node $1 already existed!"; return 2; }

    # add a node into 'kube-node' group
    sed -i "/\[kube-node/a $1 NEW_NODE=yes" $BASEPATH/hosts

    # check if playbook runs successfully
    ansible-playbook $BASEPATH/tools/20.addnode.yml -e NODE_TO_ADD=$1 || { sed -i "/$1 NEW_NODE=yes/d" $BASEPATH/hosts; return 2; }

    # save current cluster context if needed
    [ -f "$BASEPATH/.cluster/current_cluster" ] && save_context
}

@gjmzj
Copy link
Collaborator

gjmzj commented May 6, 2019

方便的话,提供下新增节点ip和原有/etc/ansible/hosts文件,查找下原因

@jacky-neo
Copy link
Author

jacky-neo commented May 6, 2019

抱歉只能贴局部。 双主的hosts. 下面斜线的内容。
操作流程是这样的,先通过aszctl删除20节点,再用它加入20节点。结果208没了。估计是带20的,就误操作了。208是我后自己加的

_[deploy]
172.16.0.128 NTP_ENABLED=yes

[etcd]
172.16.0.235 NODE_NAME=etcd1
172.16.0.208 NODE_NAME=etcd2
172.16.0.163 NODE_NAME=etcd3

[kube-master]
172.16.0.128
172.16.0.122

[kube-node]
172.16.0.20 NEW_NODE=yes
172.16.0.235
172.16.0.163
172.16.0.48
172.16.0.33
172.16.0.116
172.16.0.208_

@gjmzj
Copy link
Collaborator

gjmzj commented May 6, 2019

确认了,不是 easzctl add-node的问题,是 easzctl clean-node的问题,感谢反馈,后续会提交fix

@gjmzj
Copy link
Collaborator

gjmzj commented May 6, 2019

已提交修复 fbaab41

@jacky-neo
Copy link
Author

不客气,希望kubeasz越来越健壮

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants