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

deletion of configuration when netplugin is not up, leaves stale/inconsistent configuration #46

Closed
jainvipin opened this issue Apr 9, 2015 · 1 comment

Comments

@jainvipin
Copy link

Deletion of the config looks for oper state and wouldn't clean up the intermediate state when netplugin is not up.

examples# netdcli -cfg one_host_multiple_nets.json 
2015/04/09 03:08:59 updating the global config to new state &{{0xc208038030 } 0.01 tenant-one {11.1.0.0 16 24 11-28 } {vlan}} 

Later take out the configuration using:

examples# netdcli -cfg none.json 
2015/04/09 03:09:07 error getting host id from hostIp  Subnet /0 err 'subnet length 0 not supported 
'
2015/04/09 03:09:07 error 'subnet length 0 not supported 
' deleting epid host1-native-intf 
2015/04/09 03:09:08 network not configured 
2015/04/09 03:09:08 error '100: Key not found (/contiv/master/config/nets/infra) [212771]' deleting net infra 
2015/04/09 03:09:09 freeing subnet 11.1.0.0/%!s(uint=24) 
2015/04/09 03:09:09 freeing subnet 11.1.1.0/%!s(uint=24) 

One could see that all config state and some oper state still remains intact, which should have gone away:

examples# etcdctl ls --recursive /contiv/contiv/config
/contiv/config/nets
/contiv/config/nets/infra
/contiv/config/eps
/contiv/config/global
/contiv/config/auto-subnet
/contiv/config/auto-subnet/tenant-one
/contiv/config/auto-vlan
/contiv/config/auto-vlan/tenant-one
/contiv/master
/contiv/master/config
/contiv/master/config/nets
/contiv/master/config/nets/orange
/contiv/master/config/nets/purple
/contiv/master/config/hosts
/contiv/oper
/contiv/oper/eps
/contiv/oper/auto-subnet
/contiv/oper/auto-subnet/tenant-one
/contiv/oper/auto-vlan
/contiv/oper/auto-vlan/tenant-one
/contiv/oper/global
@shaleman
Copy link
Contributor

Closing this issue since netdcli is deprecated

dseevr pushed a commit to dseevr-dev/netplugin that referenced this issue Nov 28, 2017
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