| *** sdake has joined #openstack-containers | 00:03 | |
| Drago | I think it is 1.3 https://github.com/openstack/magnum/blob/master/magnum/api/rest_api_version_history.rst | 00:06 |
|---|---|---|
| *** hongbin has quit IRC | 00:10 | |
| *** tonanhngo has quit IRC | 00:10 | |
| *** jwcroppe has quit IRC | 00:13 | |
| *** zul has quit IRC | 00:13 | |
| *** absubram has quit IRC | 00:13 | |
| *** vijendar has joined #openstack-containers | 00:13 | |
| *** tonanhngo has joined #openstack-containers | 00:20 | |
| *** vijendar1 has joined #openstack-containers | 00:22 | |
| *** tonanhngo has quit IRC | 00:25 | |
| *** wangbo has quit IRC | 00:25 | |
| chetna | Hi Drago, I was going through the patch https://review.openstack.org/#/c/352358/ and I could see that 3 sec groups to be created. Would you advise keeping the 2 sec groups for master for k8s and CoreOS for now? | 00:29 |
| Drago | chetna: I haven't looked into it, but I'll bet that even having configurable rules can be done in 1 security group for the master | 00:32 |
| Drago | Even if that's not true, until there are actually configurable rules, I think it should be 1 security group for the master | 00:33 |
| chetna | ok, Thanks Drago :) | 00:33 |
| *** shu-mutou-AWAY is now known as shu-mutou | 00:33 | |
| *** vijendar has quit IRC | 00:36 | |
| *** vijendar has joined #openstack-containers | 00:37 | |
| Drago | Welcome | 00:38 |
| *** vijendar has quit IRC | 00:43 | |
| *** vijendar1 has quit IRC | 00:43 | |
| *** zul has joined #openstack-containers | 00:43 | |
| *** jwcroppe has joined #openstack-containers | 00:59 | |
| *** chetna has quit IRC | 01:02 | |
| *** tkarke has quit IRC | 01:06 | |
| *** tonanhngo has joined #openstack-containers | 01:12 | |
| *** tonanhngo has quit IRC | 01:12 | |
| *** tovin07_ has joined #openstack-containers | 01:14 | |
| *** vijendar has joined #openstack-containers | 02:04 | |
| *** vijendar_ has joined #openstack-containers | 02:05 | |
| *** chetna has joined #openstack-containers | 02:08 | |
| *** chetna has quit IRC | 02:09 | |
| *** wangbo has joined #openstack-containers | 02:09 | |
| *** vijendar_ has quit IRC | 02:09 | |
| *** wangbo has quit IRC | 02:10 | |
| *** vijendar_ has joined #openstack-containers | 02:10 | |
| *** EricGonc_ has quit IRC | 02:24 | |
| *** tkarke has joined #openstack-containers | 02:25 | |
| *** tkarke has quit IRC | 02:26 | |
| *** EricGonczer_ has joined #openstack-containers | 02:27 | |
| *** vijendar_ has quit IRC | 02:34 | |
| *** vijendar has quit IRC | 02:34 | |
| *** vijendar has joined #openstack-containers | 02:35 | |
| *** syed__ has quit IRC | 02:35 | |
| *** vijendar has quit IRC | 02:39 | |
| *** wangbo has joined #openstack-containers | 02:43 | |
| *** yuanying has quit IRC | 02:52 | |
| *** wangbo has quit IRC | 02:56 | |
| *** wangbo has joined #openstack-containers | 03:08 | |
| *** chetna has joined #openstack-containers | 03:09 | |
| *** chetna has quit IRC | 03:10 | |
| yatin | rochaporto: Thanks for the info about your plan. Yes, please give it a try(<<if you have something to try for the etcd tls we can give it a go) | 03:11 |
| *** vijendar has joined #openstack-containers | 03:15 | |
| *** EricGonczer_ has quit IRC | 03:20 | |
| *** hongbin has joined #openstack-containers | 03:29 | |
| *** vijendar1 has joined #openstack-containers | 03:33 | |
| *** sdake_ has joined #openstack-containers | 03:33 | |
| *** tonanhngo has joined #openstack-containers | 03:34 | |
| *** vijendar2 has joined #openstack-containers | 03:35 | |
| *** vijendar1 has quit IRC | 03:35 | |
| *** vijendar1 has joined #openstack-containers | 03:36 | |
| *** vijendar2 has quit IRC | 03:36 | |
| *** vijendar has quit IRC | 03:37 | |
| *** sdake has quit IRC | 03:37 | |
| *** vijendar1 has quit IRC | 03:38 | |
| *** tonanhngo has quit IRC | 03:39 | |
| *** Drago has quit IRC | 03:40 | |
| *** fragatin_ has joined #openstack-containers | 03:47 | |
| openstackgerrit | yatin proposed openstack/python-magnumclient: Use assert_called_*/assert_not_called to verify mock calls https://review.openstack.org/398069 | 03:47 |
| *** adrian_otto has joined #openstack-containers | 03:50 | |
| *** yuanying has joined #openstack-containers | 03:50 | |
| *** fragatina has quit IRC | 03:51 | |
| *** fragatin_ has quit IRC | 03:52 | |
| openstackgerrit | yatin proposed openstack/python-magnumclient: Use assert_called_*/assert_not_called to verify mock calls https://review.openstack.org/398069 | 03:53 |
| *** chetna has joined #openstack-containers | 03:55 | |
| *** chetna has quit IRC | 03:56 | |
| *** Jeffrey4l has joined #openstack-containers | 04:03 | |
| *** tkarke has joined #openstack-containers | 04:05 | |
| *** tkarke has quit IRC | 04:06 | |
| *** GB21 has joined #openstack-containers | 04:12 | |
| *** adrian_otto has quit IRC | 04:22 | |
| *** hongbin has quit IRC | 04:23 | |
| *** bks has joined #openstack-containers | 04:24 | |
| *** adrian_otto has joined #openstack-containers | 04:25 | |
| *** adrian_otto has quit IRC | 04:26 | |
| *** adrian_otto has joined #openstack-containers | 04:27 | |
| *** vmud213 has joined #openstack-containers | 04:30 | |
| *** mkrai has joined #openstack-containers | 04:39 | |
| *** khamtamtun has joined #openstack-containers | 04:40 | |
| *** tonanhngo has joined #openstack-containers | 04:44 | |
| *** khamtamtun has quit IRC | 04:44 | |
| *** adrian_otto has quit IRC | 04:47 | |
| *** chetna has joined #openstack-containers | 04:48 | |
| *** tonanhngo has quit IRC | 04:48 | |
| *** adrian_otto has joined #openstack-containers | 04:49 | |
| *** chetna has quit IRC | 04:49 | |
| *** sdake_ is now known as sdake | 04:54 | |
| *** sdake_ has joined #openstack-containers | 05:10 | |
| *** sdake has quit IRC | 05:14 | |
| *** chhavi has joined #openstack-containers | 05:17 | |
| *** dims has quit IRC | 05:20 | |
| *** chhavi has quit IRC | 05:21 | |
| *** wangbo has quit IRC | 05:23 | |
| *** sdake_ has quit IRC | 05:24 | |
| *** tonanhngo has joined #openstack-containers | 05:28 | |
| *** tonanhngo has quit IRC | 05:32 | |
| *** chetna has joined #openstack-containers | 05:36 | |
| *** chetna has quit IRC | 05:37 | |
| *** wangbo has joined #openstack-containers | 05:37 | |
| *** adrian_otto has quit IRC | 05:46 | |
| *** adrian_otto has joined #openstack-containers | 05:50 | |
| *** vimal has joined #openstack-containers | 05:50 | |
| *** chetna has joined #openstack-containers | 05:53 | |
| *** chetna has quit IRC | 05:55 | |
| *** harlowja has quit IRC | 05:55 | |
| *** adrian_otto has quit IRC | 05:58 | |
| *** dims has joined #openstack-containers | 05:59 | |
| *** vmud213 has quit IRC | 05:59 | |
| *** vmud213 has joined #openstack-containers | 06:00 | |
| *** tkarke has joined #openstack-containers | 06:17 | |
| *** tkarke has quit IRC | 06:18 | |
| *** vijendar has joined #openstack-containers | 06:39 | |
| *** vijendar has quit IRC | 06:40 | |
| *** chetna has joined #openstack-containers | 06:41 | |
| *** chetna has quit IRC | 06:42 | |
| *** snarwade has quit IRC | 06:57 | |
| *** snarwade has joined #openstack-containers | 07:00 | |
| *** manikanta_tadi has quit IRC | 07:05 | |
| *** mjura has joined #openstack-containers | 07:11 | |
| *** chetna has joined #openstack-containers | 07:14 | |
| *** chetna has quit IRC | 07:15 | |
| *** GB21 has quit IRC | 07:16 | |
| *** mjura has quit IRC | 07:19 | |
| *** mjura has joined #openstack-containers | 07:19 | |
| *** manikanta_tadi has joined #openstack-containers | 07:21 | |
| *** chetna has joined #openstack-containers | 07:23 | |
| *** chetna has quit IRC | 07:24 | |
| *** GB21 has joined #openstack-containers | 07:29 | |
| openstackgerrit | Hieu LE proposed openstack/magnum: [docs]Update quickstart guide to use bay-config command https://review.openstack.org/398153 | 07:30 |
| openstackgerrit | Hieu LE proposed openstack/magnum: [docs]Update quickstart guide to use cluster-config command https://review.openstack.org/398153 | 07:31 |
| *** houming has joined #openstack-containers | 07:36 | |
| *** pcaruana has joined #openstack-containers | 07:41 | |
| *** tkarke has joined #openstack-containers | 07:41 | |
| *** tkarke has quit IRC | 07:42 | |
| *** manikanta_tadi has quit IRC | 07:45 | |
| *** chetna has joined #openstack-containers | 07:55 | |
| *** chetna has quit IRC | 07:56 | |
| *** ricolin has joined #openstack-containers | 08:00 | |
| *** houming has quit IRC | 08:02 | |
| *** wangbo has quit IRC | 08:07 | |
| *** wangbo has joined #openstack-containers | 08:08 | |
| *** chhavi has joined #openstack-containers | 08:09 | |
| *** houming has joined #openstack-containers | 08:12 | |
| *** vmud213 has quit IRC | 08:20 | |
| *** wangbo has quit IRC | 08:22 | |
| *** vmud213 has joined #openstack-containers | 08:23 | |
| *** tovin07 has quit IRC | 08:24 | |
| *** mkrai has quit IRC | 08:26 | |
| *** chetna has joined #openstack-containers | 08:28 | |
| *** chetna has quit IRC | 08:29 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Drop id suffix in launch-an-instance guide https://review.openstack.org/398183 | 08:33 |
| *** wangbo has joined #openstack-containers | 08:34 | |
| *** GB21 has quit IRC | 08:38 | |
| *** chhavi has quit IRC | 08:50 | |
| *** GB21 has joined #openstack-containers | 08:51 | |
| openstackgerrit | Merged openstack/magnum: [docs]Update quickstart guide to use cluster-config command https://review.openstack.org/398153 | 08:52 |
| *** tovin07 has joined #openstack-containers | 08:57 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Drop id suffix in launch-an-instance guide https://review.openstack.org/398183 | 09:09 |
| *** chhavi has joined #openstack-containers | 09:12 | |
| *** chetna has joined #openstack-containers | 09:17 | |
| *** chetna has quit IRC | 09:18 | |
| *** tovin07 has quit IRC | 09:23 | |
| *** takashi_ has joined #openstack-containers | 09:38 | |
| *** chhavi has quit IRC | 09:51 | |
| *** tovin07_ has quit IRC | 09:57 | |
| *** tkarke has joined #openstack-containers | 10:08 | |
| *** tkarke has quit IRC | 10:09 | |
| *** wangbo has quit IRC | 10:13 | |
| openstackgerrit | yatin proposed openstack/python-magnumclient: Fix: typo fixed-network --> fixed-subnet https://review.openstack.org/398248 | 10:14 |
| *** chhavi has joined #openstack-containers | 10:15 | |
| *** GB21 has quit IRC | 10:22 | |
| openstackgerrit | zhangyanxian proposed openstack/magnum: Fix typo in cover.sh https://review.openstack.org/398258 | 10:28 |
| openstackgerrit | zhangyanxian proposed openstack/magnum: Fix typo in cover.sh https://review.openstack.org/398258 | 10:30 |
| *** wangbo has joined #openstack-containers | 10:32 | |
| *** GB21 has joined #openstack-containers | 10:34 | |
| *** omnipresent has joined #openstack-containers | 10:52 | |
| openstackgerrit | yatin proposed openstack/python-magnumclient: Fix: some typos in unit test https://review.openstack.org/398248 | 10:57 |
| *** omnipresent has quit IRC | 10:57 | |
| *** zenoway has joined #openstack-containers | 11:02 | |
| *** bks has quit IRC | 11:04 | |
| *** houming has left #openstack-containers | 11:04 | |
| *** GB21 has quit IRC | 11:18 | |
| *** adisky has joined #openstack-containers | 11:20 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 11:23 | |
| *** GB21 has joined #openstack-containers | 11:31 | |
| *** chhavi has quit IRC | 11:33 | |
| openstackgerrit | yatin proposed openstack/python-magnumclient: Fix: some typos in unit test https://review.openstack.org/398248 | 11:36 |
| *** vmud213 has quit IRC | 11:45 | |
| *** chhavi has joined #openstack-containers | 11:48 | |
| *** zenoway has quit IRC | 11:59 | |
| *** zenoway has joined #openstack-containers | 11:59 | |
| *** zenoway has quit IRC | 12:04 | |
| *** zenoway has joined #openstack-containers | 12:07 | |
| *** pc_m has quit IRC | 12:14 | |
| *** pc_m has joined #openstack-containers | 12:16 | |
| *** GB21 has quit IRC | 12:22 | |
| *** catintheroof has joined #openstack-containers | 12:23 | |
| *** EricGonczer_ has joined #openstack-containers | 12:35 | |
| *** tkarke has joined #openstack-containers | 12:35 | |
| *** tkarke has quit IRC | 12:36 | |
| *** EricGonczer_ has quit IRC | 12:37 | |
| *** sdake has joined #openstack-containers | 12:53 | |
| *** chhavi has quit IRC | 12:56 | |
| *** tkarke has joined #openstack-containers | 12:58 | |
| *** tkarke has quit IRC | 12:59 | |
| *** sdake_ has joined #openstack-containers | 13:05 | |
| *** coreyob has quit IRC | 13:06 | |
| *** EricGonczer_ has joined #openstack-containers | 13:08 | |
| *** EricGonczer_ has quit IRC | 13:08 | |
| *** sdake has quit IRC | 13:08 | |
| *** takashi_ has quit IRC | 13:10 | |
| *** chhavi has joined #openstack-containers | 13:11 | |
| *** dave-mccowan has joined #openstack-containers | 13:26 | |
| *** coreyob has joined #openstack-containers | 13:26 | |
| *** GB21 has joined #openstack-containers | 13:27 | |
| *** chhavi has quit IRC | 13:33 | |
| *** coreyob has quit IRC | 13:48 | |
| *** Administrator_ has quit IRC | 13:51 | |
| *** chhavi has joined #openstack-containers | 13:51 | |
| *** Administrator_ has joined #openstack-containers | 13:52 | |
| *** eliqiao has quit IRC | 13:56 | |
| *** eliqiao has joined #openstack-containers | 13:57 | |
| *** coreyob has joined #openstack-containers | 13:59 | |
| *** coreyob has quit IRC | 14:03 | |
| *** chhavi has quit IRC | 14:07 | |
| *** Administrator_ has quit IRC | 14:13 | |
| *** Administrator_ has joined #openstack-containers | 14:14 | |
| openstackgerrit | yatin proposed openstack/python-magnumclient: Fix: some typos in unit test https://review.openstack.org/398248 | 14:14 |
| *** coreyob has joined #openstack-containers | 14:15 | |
| *** jperry has joined #openstack-containers | 14:16 | |
| *** Drago1 has joined #openstack-containers | 14:28 | |
| *** dave-mccowan has quit IRC | 14:30 | |
| *** Drago1 has quit IRC | 14:33 | |
| openstackgerrit | Drago proposed openstack/magnum: Add Flatten Attributes Specification https://review.openstack.org/393094 | 14:43 |
| *** coreyob has quit IRC | 14:44 | |
| *** Drago1 has joined #openstack-containers | 14:45 | |
| *** chris_hultin|AWA is now known as chris_hultin | 14:45 | |
| strigazi | Drago, I have a question about workItem 1 in flatten attrs. We will leave the fields in the db empty in cluster and cluster_templates if it's cluster_attributes? | 14:48 |
| strigazi | Drago1 ^^ | 14:48 |
| *** vmud213 has joined #openstack-containers | 14:49 | |
| strigazi | I | 14:50 |
| strigazi | I'll post in the review | 14:50 |
| Drago1 | strigazi: No, I think you're supposed to write to both the old and the new fields | 14:50 |
| strigazi | Drago1. we want this for safety? | 14:51 |
| Drago1 | strigazi: Yes, imagine having two conductors running different versions of magnum | 14:51 |
| *** tonanhngo has joined #openstack-containers | 14:51 | |
| strigazi | but we'll keep the rpcs as it is I guess? | 14:52 |
| strigazi | nova does this, AFAIK | 14:53 |
| Drago1 | Let the old version be N. N+1 writes to old and new, reads from old. N+2 writes to old and new, reads from new. N+3 can write and read to only new. | 14:54 |
| *** GB21 has quit IRC | 14:54 | |
| *** coreyob has joined #openstack-containers | 14:55 | |
| Drago1 | There was a talk at the Austin summit that went over this. I'm not sure what we need to do to enable this | 14:55 |
| *** adrian_otto has joined #openstack-containers | 14:57 | |
| *** vimal has quit IRC | 14:57 | |
| Drago1 | strigazi: And the idea is to not let any two conductors be more than 1 version apart | 14:58 |
| *** ricolin has quit IRC | 14:58 | |
| *** hongbin has joined #openstack-containers | 14:58 | |
| *** sdake has joined #openstack-containers | 14:58 | |
| *** sdake_ has quit IRC | 15:02 | |
| *** coreyob has quit IRC | 15:02 | |
| *** coreyob has joined #openstack-containers | 15:05 | |
| strigazi | Drago1: I don't know if currently the a newton conductor can talk to a mitaka db | 15:05 |
| strigazi | I think it can't, but i'm not sure | 15:05 |
| *** adrian_otto has quit IRC | 15:06 | |
| strigazi | Maybe it can't, mitaka has bays and newton has clusters | 15:06 |
| strigazi | we need to version the way the conductor talk to the db too | 15:06 |
| strigazi | similarly to the api versioning | 15:07 |
| Drago1 | strigazi: Well, if magnum doesn't support rolling upgrades yet, we should look into it. | 15:08 |
| strigazi | we must versions the db api for this | 15:10 |
| strigazi | *version | 15:10 |
| strigazi | I think we need to create a version starting from adding the keypair to cluster. To allow newton conductor to talk to an ocata db | 15:13 |
| *** coreyob has quit IRC | 15:14 | |
| *** jasond has joined #openstack-containers | 15:14 | |
| Drago1 | strigazi: I'm currently talking to the Heat team about this | 15:15 |
| Drago1 | strigazi: On #heat if you want to join | 15:15 |
| *** hieulq_ has joined #openstack-containers | 15:17 | |
| *** absubram has joined #openstack-containers | 15:25 | |
| *** coreyob has joined #openstack-containers | 15:25 | |
| Drago1 | strigazi: So since we have ovo, do we need two migrations? | 15:26 |
| *** EricGonczer_ has joined #openstack-containers | 15:28 | |
| *** hieulq_ has quit IRC | 15:30 | |
| *** EricGonc_ has joined #openstack-containers | 15:30 | |
| *** syed__ has joined #openstack-containers | 15:31 | |
| *** EricGonczer_ has quit IRC | 15:32 | |
| *** vmud213 has quit IRC | 15:34 | |
| Drago1 | strigazi: I'm going to completely take out the migrations part of the flatten attributes patch | 15:35 |
| Drago1 | strigazi: I thought it mattered. Seems like it doesn't | 15:35 |
| strigazi | +1 | 15:35 |
| strigazi | all chnages will happen at once, we can afford service downtime in magnum. The resources created by magnum won't have downtime anyway | 15:36 |
| Drago1 | strigazi: Yeah | 15:36 |
| Drago1 | strigazi: It would be *neat* though :) | 15:37 |
| strigazi | it's an overkill | 15:37 |
| *** jwcroppe has quit IRC | 15:37 | |
| openstackgerrit | Drago proposed openstack/magnum: Add Flatten Attributes Specification https://review.openstack.org/393094 | 15:37 |
| strigazi | IMO, when I asked our nova service manager, he told me if you have a lot of free time you can do it :) | 15:37 |
| strigazi | "IMO" goes to "it's an overkill" :) | 15:38 |
| Drago1 | strigazi: Agreed, unless it's pretty easy | 15:39 |
| Drago1 | Which it's not | 15:39 |
| Drago1 | At least right now | 15:40 |
| strigazi | Drago1, maybe magnum becomes such a critical service and this feature will be a must :) | 15:41 |
| strigazi | kidding, if our upgrades are bug free an upgrade in magnum is very fast | 15:41 |
| *** vmud213 has joined #openstack-containers | 15:43 | |
| Drago1 | True | 15:44 |
| *** jwcroppe has joined #openstack-containers | 15:55 | |
| *** dave-mccowan has joined #openstack-containers | 16:04 | |
| *** mjura has quit IRC | 16:06 | |
| *** jmckind has joined #openstack-containers | 16:07 | |
| Drago1 | Dang yatin | 16:07 |
| Drago1 | Catching all my mistakes | 16:07 |
| Drago1 | :) | 16:07 |
| openstackgerrit | Drago proposed openstack/magnum: Add Flatten Attributes Specification https://review.openstack.org/393094 | 16:10 |
| Drago1 | Sorry strigazi | 16:10 |
| *** ramishra has quit IRC | 16:19 | |
| *** EricGonc_ has quit IRC | 16:20 | |
| *** ramishra has joined #openstack-containers | 16:20 | |
| *** EricGonczer_ has joined #openstack-containers | 16:23 | |
| *** adrian_otto has joined #openstack-containers | 16:26 | |
| jvgrant_ | Drago1, strigazi: we decided that clusters and nodegroups will have a reference to the templates used to create them, correct? | 16:29 |
| jvgrant_ | or just Clusters for Cluster-upgrades? | 16:30 |
| Drago1 | jvgrant_: If the template versioning spec passes, yes | 16:34 |
| jvgrant_ | Drago1: ok, updating the NodeGroup spec to reflect that | 16:36 |
| *** strigazi is now known as strigazi_AFK | 16:46 | |
| openstackgerrit | Merged openstack/magnum: Fix typo in cover.sh https://review.openstack.org/398258 | 16:46 |
| openstackgerrit | Merged openstack/magnum: Drop id suffix in launch-an-instance guide https://review.openstack.org/398183 | 16:46 |
| *** zenoway has quit IRC | 16:47 | |
| openstackgerrit | Merged openstack/magnum: [install] Update rabbitmq configuration https://review.openstack.org/396941 | 16:47 |
| *** zenoway has joined #openstack-containers | 16:47 | |
| *** zenoway has quit IRC | 16:51 | |
| *** absubram has quit IRC | 16:53 | |
| *** vmud213 has quit IRC | 16:59 | |
| *** vmud213 has joined #openstack-containers | 16:59 | |
| *** chetna has joined #openstack-containers | 16:59 | |
| *** absubram has joined #openstack-containers | 17:01 | |
| jvgrant_ | Drago1: pint | 17:01 |
| jvgrant_ | Drago1: ping | 17:02 |
| Drago1 | jvgrant_: quart | 17:02 |
| jvgrant_ | lol | 17:02 |
| jvgrant_ | Drago1: question on your TODO in nodegroup around multiple deletes | 17:02 |
| jvgrant_ | Drago1: Do you think that is something we need to add now or something that could be added in the future if found necessary? | 17:03 |
| Drago1 | jvgrant_: That would be something I think we should consider now because we could pick a command syntax that prevents us from adding it elegantly in the future | 17:04 |
| *** chetna has quit IRC | 17:04 | |
| *** chetna has joined #openstack-containers | 17:04 | |
| jvgrant_ | Drago1: what if we limit it to multiple deletes only when you specify the cluster? It seems deleting from the same cluster is when you will want to use multiple deletes in the same command | 17:05 |
| Drago1 | jvgrant_: Still ambiguous | 17:06 |
| *** sdake has quit IRC | 17:06 | |
| Drago1 | jvgrant_: Consider `magnum nodegroup-delete foo bar` | 17:07 |
| jvgrant_ | Drago1: i lean towards not allowing multiple deletes with the nodegroup command. To me nodegroup commands feel targeted at one nodegroup | 17:07 |
| Drago1 | What if there is a foo cluster and foo nodegroup | 17:07 |
| jvgrant_ | for example should you allow multiple cluster deletes with the cluster-delete command? | 17:07 |
| Drago1 | jvgrant_: Yes, and we do | 17:08 |
| *** sdake has joined #openstack-containers | 17:08 | |
| Drago1 | already | 17:09 |
| jvgrant_ | wasn't aware of that | 17:09 |
| Drago1 | It's handy | 17:09 |
| jvgrant_ | for cluster you just keep adding the cluster names to the delete? | 17:11 |
| Drago1 | jvgrant_: Yes | 17:11 |
| jvgrant_ | yeah our positional optional cluster name messes that up :( | 17:11 |
| Drago1 | jvgrant_: It's even positional in cluster-delete right now! | 17:12 |
| Drago1 | jvgrant_: We could make it not multiple so it's not ambiguous | 17:13 |
| Drago1 | jvgrant_: Like you said | 17:13 |
| Drago1 | It's not too much less convenient... | 17:13 |
| jvgrant_ | if it is handy i'd like to keep it... | 17:13 |
| jvgrant_ | trying to think of a clean way to represent that when each nodegroup could be represented by 1 or 2 parameters | 17:14 |
| jvgrant_ | cluster only ever has 1 per cluster | 17:14 |
| Drago1 | The minimum for it to be multiple is 2, so it would look like `magnum ng-delete cluster ng ng` | 17:15 |
| Drago1 | while 1 can look like either `magnum ng-delete cluster ng` or `magnum ng-delete ng` | 17:16 |
| Drago1 | So 1 or 2 args means single, while 3+ means multiple | 17:16 |
| jvgrant_ | we could add the --multiple when used it is always just nodegroup without cluster | 17:16 |
| Drago1 | IF we make cluster mandatory | 17:16 |
| Drago1 | *IF we make cluster mandatory for multiple ngs | 17:16 |
| swatson | Drago 2 args meaning single is a bit confusing | 17:16 |
| jvgrant_ | that would work if we limit multiple delete to within the same cluster | 17:17 |
| Drago1 | swatson: I'm thinking about this from a programmatic angle | 17:17 |
| swatson | ah | 17:17 |
| Drago1 | A user might thing "oh this is obvious to me" but could be ambiguous to the parser | 17:17 |
| Drago1 | *think | 17:18 |
| Drago1 | jvgrant_: Or add a flag that could change the behavior if you want multiple in different clusters | 17:18 |
| jvgrant_ | i think limiting it to the cluster covers the main use case and is a good start. We could then add the "--multiple" flag for separate nodegroups in the future if needed | 17:18 |
| Drago1 | magnum nodegroup-delete c1ng1 | 17:19 |
| Drago1 | magnum nodegroup-delete c1 c1ng1 | 17:19 |
| Drago1 | magnum nodegroup-delete c1 c1ng1 c1ng2 | 17:19 |
| Drago1 | magnum nodegroup-delete —multiple c1ng1 c2ng2 | 17:19 |
| jvgrant_ | that would work | 17:19 |
| Drago1 | magnum nodegroup-delete c1ng1 c1ng2 (INVALID) | 17:20 |
| Drago1 | magnum nodegroup-delete c1ng1 c2ng1 (INVALID) | 17:20 |
| Drago1 | magnum nodegroup-delete c1 c1ng1 c1ng2 c1ng3 | 17:20 |
| swatson | I'm leaning towards the first one not even being valid | 17:20 |
| Drago1 | magnum nodegroup-delete c1 c1ng1 c1ng2 c1ng3 c1ng4 | 17:21 |
| swatson | I would prefer requiring a cluster name in every case | 17:21 |
| Drago1 | swatson: I think some would prefer to not have to specify the ng name | 17:21 |
| jvgrant_ | it really isn't needed though as a nodegroup can be found on its own | 17:21 |
| swatson | Drago1: Could we add a flag to specify the cluster? | 17:22 |
| Drago1 | swatson: The question that would convince me is if not requiring the cluster argument is overly dangerous | 17:22 |
| Drago1 | swatson: Why? | 17:22 |
| *** vmud213 has quit IRC | 17:22 | |
| Drago1 | What would the purpose be? | 17:22 |
| swatson | I just like the consistency of each command looking and behaving similarly | 17:23 |
| Drago1 | swatson: None of the other commands have a cluster flag | 17:23 |
| swatson | I meant for ng-delete alone | 17:23 |
| Drago1 | Okay, but how does that help consistency? | 17:23 |
| jvgrant_ | we could just remove cluster all together. It really isn't needed other than to resolve naming conflicts which we already do using UUID | 17:24 |
| Drago1 | jvgrant_: Interesting idea | 17:24 |
| swatson | I'm looking at all the examples you just posted and only one doesn't have the cluster listed apart from the --multiple | 17:24 |
| jvgrant_ | Drago1: that would match the behavior of all the other resources | 17:25 |
| jvgrant_ | Drago1: and we have it optional in all anyway | 17:25 |
| Drago1 | Okay, let's play with this idea | 17:26 |
| Drago1 | magnum nodegroup-create mycluster mynodegroupname … | 17:26 |
| Drago1 | Or magnum nodegroup-create —cluster mycluster mynodegroupname …? | 17:26 |
| Drago1 | I find the second one ugly | 17:27 |
| swatson | yeah I see what you're saying | 17:27 |
| jvgrant_ | yeah, the create is why we added it... | 17:28 |
| Drago1 | I do think that it would be reasonable for a user to expect commands of the form `magnum nodegroup-DoAThing foo bar baz` to have a similar structure | 17:28 |
| Drago1 | so I don't know that dropping cluster quite makes sense | 17:28 |
| jvgrant_ | ok, i'm back to what we suggested before with the --multiple flag | 17:29 |
| jvgrant_ | i think that one makes the most sense if we want to keep it clean for create and other places | 17:29 |
| Drago1 | magnum nodegroup-create mycluster myng (valid) | 17:29 |
| Drago1 | magnum nodegroup-delete mycluster myng (should also be valid) | 17:29 |
| jvgrant_ | agreed | 17:30 |
| swatson | +1 | 17:30 |
| jvgrant_ | i don't think the multiple delete options is worth making the other commands make less sense | 17:31 |
| Drago1 | jvgrant_, swatson: I'm trying to keep the job of the magnumclient in mind too. If the syntax for `magnum nodegroup-delete arg1 arg2` can be confined to arg1==cluster, arg2==nodegroup, then the client can blindly send a request to magnum and if that errors because it was really two nodegroups, tell the user to use —multiple instead | 17:31 |
| Drago1 | jvgrant_: So do you think we should introduce the idea of multiple delete in the nodegroup spec at all? | 17:32 |
| Drago1 | If so, I'm fine with that | 17:32 |
| jvgrant_ | Drago1: i'm ok either way. I don't think it is really essential, but i'm ok with the --multiple flag schema we worked out above | 17:33 |
| Drago1 | Followup question: are you still fine with cluster being optional in `magnum nodegroup-{everything besides create}`? | 17:33 |
| jvgrant_ | yeah, i think that makes sense as it is not really needed. I'm ok with making it mandatory too if we really want | 17:34 |
| Drago1 | Same to you swatson ^ | 17:34 |
| swatson | Drago1 Yeah I think it's fine | 17:34 |
| Drago1 | jvgrant_: If we don't have to, I'm definitely on the non-mandatory side | 17:34 |
| jvgrant_ | same | 17:34 |
| *** tkarke has joined #openstack-containers | 17:34 | |
| Drago1 | jvgrant_: Okay, so take out the "…" on line 1006 | 17:35 |
| jvgrant_ | ok, i think for now i'm going to remove the multiple delete from the spec to keep it simple. We have resolved that there is a way to add it to the command latter with little issue so we are not blocking it with this command layout | 17:35 |
| Drago1 | jvgrant_: +1 | 17:36 |
| *** absubram has quit IRC | 17:37 | |
| *** EricGonczer_ has quit IRC | 17:37 | |
| Drago1 | jvgrant_: Can you fix the signature of the nodegroup-template-update command? | 17:38 |
| Drago1 | It's missing the cluster arg | 17:38 |
| Drago1 | That's the only one I see that it's missing | 17:39 |
| jvgrant_ | Drago1: will do | 17:39 |
| Drago1 | jvgrant_, swatson: I like the new command syntax. Seems pretty clean while also allowing for complex things to be specified | 17:40 |
| jvgrant_ | Drago1: +1 | 17:40 |
| *** fragatina has joined #openstack-containers | 17:41 | |
| Drago1 | And consistent | 17:41 |
| swatson | I'm into it | 17:42 |
| jvgrant_ | Drago1: wait, why would nodegroup-template-update need the cluster arg? template is not connected to any cluster | 17:42 |
| Drago1 | jvgrant_: *ClusterTemplate arg | 17:43 |
| Drago1 | jvgrant_: It CAN be associated with a CT | 17:43 |
| *** tkarke has quit IRC | 17:43 | |
| jvgrant_ | ok, got it | 17:44 |
| *** david-lyle has quit IRC | 17:48 | |
| *** david-lyle has joined #openstack-containers | 17:48 | |
| *** tonanhngo has quit IRC | 17:51 | |
| *** chris_hultin is now known as chris_hultin|AWA | 17:51 | |
| openstackgerrit | Jaycen Grant proposed openstack/magnum: Add NodeGroup specification https://review.openstack.org/352734 | 17:52 |
| *** adisky has quit IRC | 17:55 | |
| *** coreyob has quit IRC | 17:57 | |
| *** randallburt has joined #openstack-containers | 17:58 | |
| *** randallburt has quit IRC | 18:02 | |
| *** fragatin_ has joined #openstack-containers | 18:04 | |
| *** coreyob has joined #openstack-containers | 18:05 | |
| *** fragatina has quit IRC | 18:08 | |
| *** coreyob has quit IRC | 18:10 | |
| *** jperry has quit IRC | 18:10 | |
| *** randallburt has joined #openstack-containers | 18:12 | |
| *** catinthe_ has joined #openstack-containers | 18:13 | |
| *** absubram has joined #openstack-containers | 18:14 | |
| *** harlowja has joined #openstack-containers | 18:14 | |
| *** catintheroof has quit IRC | 18:15 | |
| openstackgerrit | Jaycen Grant proposed openstack/magnum: Add NodeGroup specification https://review.openstack.org/352734 | 18:15 |
| *** catintheroof has joined #openstack-containers | 18:15 | |
| *** randallburt has quit IRC | 18:17 | |
| *** catinthe_ has quit IRC | 18:18 | |
| *** coreyob has joined #openstack-containers | 18:20 | |
| *** jperry has joined #openstack-containers | 18:24 | |
| *** coreyob has quit IRC | 18:25 | |
| *** randallburt has joined #openstack-containers | 18:29 | |
| *** tkarke has joined #openstack-containers | 18:30 | |
| *** EricGonczer_ has joined #openstack-containers | 18:31 | |
| *** Administrator_ has quit IRC | 18:32 | |
| *** tonanhngo has joined #openstack-containers | 18:32 | |
| *** Administrator_ has joined #openstack-containers | 18:32 | |
| *** coreyob has joined #openstack-containers | 18:36 | |
| *** tonanhngo has quit IRC | 18:36 | |
| *** tonanhngo has joined #openstack-containers | 18:38 | |
| *** tonanhngo has quit IRC | 18:39 | |
| *** tonanhngo has joined #openstack-containers | 18:39 | |
| *** coreyob has quit IRC | 18:40 | |
| *** pcastets has quit IRC | 18:44 | |
| *** vijendar has joined #openstack-containers | 18:50 | |
| *** vijendar1 has joined #openstack-containers | 18:51 | |
| *** coreyob has joined #openstack-containers | 18:51 | |
| *** coreyob has quit IRC | 18:56 | |
| *** vijendar has quit IRC | 18:59 | |
| *** vijendar has joined #openstack-containers | 19:00 | |
| *** jwcroppe has quit IRC | 19:00 | |
| *** coreyob has joined #openstack-containers | 19:06 | |
| *** chris_hultin|AWA is now known as chris_hultin | 19:10 | |
| *** jwcroppe has joined #openstack-containers | 19:15 | |
| *** chris_hultin is now known as chris_hultin|AWA | 19:16 | |
| *** chris_hultin|AWA is now known as chris_hultin | 19:17 | |
| *** absubram has quit IRC | 19:30 | |
| *** fragatin_ has quit IRC | 19:41 | |
| *** fragatina has joined #openstack-containers | 19:41 | |
| *** absubram has joined #openstack-containers | 19:42 | |
| *** coreyob has quit IRC | 19:46 | |
| *** EricGonczer_ has quit IRC | 19:48 | |
| *** pcaruana has quit IRC | 19:51 | |
| *** EricGonczer_ has joined #openstack-containers | 19:54 | |
| *** coreyob has joined #openstack-containers | 19:58 | |
| openstackgerrit | Randall Burt proposed openstack/magnum: [WiP] Move cluster sync into the Heat driver https://review.openstack.org/397961 | 20:01 |
| *** vijendar has quit IRC | 20:02 | |
| *** coreyob has quit IRC | 20:02 | |
| randallburt | whoops, too quick on that one | 20:04 |
| *** vijendar has joined #openstack-containers | 20:11 | |
| *** coreyob has joined #openstack-containers | 20:15 | |
| *** vijendar has quit IRC | 20:28 | |
| *** vijendar1 has quit IRC | 20:28 | |
| *** jwcroppe has quit IRC | 20:30 | |
| *** jwcroppe has joined #openstack-containers | 20:32 | |
| *** sdake has quit IRC | 20:35 | |
| *** sdake has joined #openstack-containers | 20:37 | |
| *** chetna has quit IRC | 20:37 | |
| *** coreyob has quit IRC | 20:38 | |
| *** chetna has joined #openstack-containers | 20:50 | |
| *** coreyob has joined #openstack-containers | 20:51 | |
| *** vijendar has joined #openstack-containers | 20:54 | |
| *** vijendar1 has joined #openstack-containers | 20:57 | |
| *** rcernin has joined #openstack-containers | 20:58 | |
| *** vijendar has quit IRC | 21:00 | |
| *** coreyob has quit IRC | 21:00 | |
| *** coreyob has joined #openstack-containers | 21:05 | |
| *** vijendar has joined #openstack-containers | 21:05 | |
| *** coreyob has quit IRC | 21:08 | |
| *** coreyob has joined #openstack-containers | 21:20 | |
| *** coreyob has quit IRC | 21:23 | |
| *** vijendar1 has quit IRC | 21:26 | |
| *** vijendar has quit IRC | 21:26 | |
| *** jberkus has joined #openstack-containers | 21:26 | |
| *** EricGonczer_ has quit IRC | 21:31 | |
| *** jberkus has quit IRC | 21:31 | |
| *** catinthe_ has joined #openstack-containers | 21:32 | |
| *** Jeffrey4l has quit IRC | 21:33 | |
| Drago1 | Hey hongbin | 21:34 |
| *** Drago1 is now known as Drago | 21:34 | |
| hongbin | Drago: sup | 21:35 |
| *** coreyob has joined #openstack-containers | 21:35 | |
| Drago | hongbin: The "arbitrary" parameters are only to make the spec more concise. In the implementation, they will be flags for all of the real attributes | 21:35 |
| *** catintheroof has quit IRC | 21:36 | |
| Drago | hongbin: We did not want to list the ~27 possible flags you could pass | 21:36 |
| hongbin | Drago: ok. get that. | 21:36 |
| hongbin | Drago: maybe you guys want to clarify it in the spec (when i read it, it seems to suggest all flags can be free-form). | 21:38 |
| Drago | hongbin: Does that cover most of the comments on the NodeGroup spec? | 21:38 |
| *** coreyob has quit IRC | 21:38 | |
| hongbin | Drago: i will go back to the nodegroup spec after a revision | 21:39 |
| pc_m | Hi. I wanted to try to play with magnum using devstack and had some questions for folks. I'm using the getting started web page, but having some issues. Anyone able to field some general questions? | 21:39 |
| hongbin | Drago: i need to give it more thoughts about the design, which is not trivial | 21:41 |
| Drago | hongbin: I see there is another patchset now. On patch set 23, it looks like 4/6 of your comments were about the arbitrary flags | 21:41 |
| pc_m | Q1: Can I stack for magnum, in a VM on my Mac? | 21:41 |
| Drago | pc_m: Yes, magnum should not be special | 21:42 |
| Drago | So it should work if you can get other services working. | 21:42 |
| pc_m | Q2: What OS should I use? I've been trying Ubuntu 16.04, 14.04, and RHEL 7, and hitting issues with each (using server in lab behind FW). | 21:42 |
| Drago | hongbin: I was asking if those 4 comments were about the same issue | 21:42 |
| hongbin | Drago: yes, i still have issues of the flags, but maybe it is not so important | 21:43 |
| Drago | pc_m: I have a devstack working on bare metal with 14.04 | 21:43 |
| pc_m | Drago: OK. I'll have to retry on Mac with VM. | 21:43 |
| hongbin | Drago: i will give it more thoughts on the high level design | 21:43 |
| *** sdake_ has joined #openstack-containers | 21:44 | |
| Drago | hongbin: Cool. I was not sure what you meant by "inconsistent with OpenStack CLI style" until you clarified it was about the arbitrary flags (which I agree, would be bad) | 21:44 |
| pc_m | Drago: Hmm. With Ubuntu 16.04, I was able to stack, but when I created a cluster, it failed. Found that the VM creates are failing. Created a vanilla Cirros VM and it failed in Nova. :( | 21:44 |
| openstackgerrit | Randall Burt proposed openstack/magnum: [WiP] Move cluster sync into the Heat driver https://review.openstack.org/397961 | 21:45 |
| hongbin | Drago: the style of indentation | 21:45 |
| pc_m | With Ubuntu 14.04, it is failing to stack. Gets to the end, and says python-magnumclient did not install, but there is a egg file in /usr/local/lib.... | 21:45 |
| Drago | pc_m: A common issue is that your cloud vms don't have internet connectivity | 21:45 |
| Drago | hongbin: The indentation is purely a visual aid | 21:46 |
| pc_m | With RHEL, it failed in Octavia saying the qemu-kvm required but not installed. Found there is an open bug to fix that. | 21:46 |
| Drago | hongbin: It has been a bit controversial with other people than you too | 21:46 |
| hongbin | Drago: i know, but the indentation suggests that this flag only valid in this position | 21:46 |
| pc_m | Drago: Hmm. I'm behind a FW so that is an issue (been fighting with proxy setup). | 21:47 |
| *** sdake has quit IRC | 21:47 | |
| Drago | hongbin: Well that is probably true, because you must specify it after —nodegroup | 21:47 |
| hongbin | Drago: i never see an OpenStack CLI that has a flag (i.e. --image) that override the previous flag (e.g. --nodegroup ) | 21:47 |
| Drago | hongbin: It is not an override, it establishes a new context for the flags that come after it | 21:48 |
| hongbin | Drago: if you have to do that, maybe set it in a single flag (i.e. --nodegroup XXX:image=XXX,flavor=XXX,...) | 21:48 |
| Drago | hongbin: We have considered a few different styles | 21:49 |
| Drago | hongbin: I find specifying everything with commas to be awkward | 21:49 |
| Drago | hongbin: It is worth more discussion though, definitely | 21:49 |
| hongbin | Drago: but that is what most of the OpenStack CLI is doing | 21:50 |
| hongbin | Drago: yes, just let others to comment on it. | 21:50 |
| *** jwcroppe has quit IRC | 21:50 | |
| Drago | hongbin: Consider this though: | 21:50 |
| jvgrant_ | hongbin: originally it was closer to what you are suggesting but feedback from team was they didn't like it. We have come a long way since then so let see if that has changed | 21:50 |
| *** catintheroof has joined #openstack-containers | 21:51 | |
| Drago | hongbin: You have `magnum nodegroup-create my_nodegroup —image some_image` | 21:51 |
| Drago | hongbin: With what we have right now, the syntax is exactly the same in cluster-create | 21:51 |
| *** catinthe_ has quit IRC | 21:51 | |
| pc_m | Drago: Do you have any experience with using the setup, behind a FW (with proxy)? Any tips on how to get things going? I've used DevStack on these systems before (with just Nova/Neutron/VPNaaS) and have been able to spin up VMs. Anything different, when using Magnum? | 21:51 |
| Drago | magnum cluster-create my_cluster —nodegroup my_nodegroup —image some_image | 21:51 |
| *** coreyob has joined #openstack-containers | 21:51 | |
| hongbin | jvgrant_: i see. if others like it, i don't have isue | 21:52 |
| hongbin | issue | 21:52 |
| Drago | pc_m: Magnum currently requires internet in the vms. Nova can create vms just fine, but the user data on those vms needs internet for the cluster to fully complete | 21:53 |
| jvgrant_ | hongbin: let's see what feedback we get now after all the updates. I think it is still a good discussion to have and something we need to get right. | 21:54 |
| pc_m | Drago: Do you know if there is a way to do that, when behind a FW? | 21:55 |
| hongbin | jvgrant_: maybe you also want to ping the openstackclient team for feedback, eventually, the CLI will move to there. | 21:55 |
| Drago | pc_m: Or it at least needs to be able to resolve the URLs it needs. I think it depends on discovery.etcd.io and docker hub (for pulling the k8s images) | 21:55 |
| Drago | pc_m: Maybe gcr.io instead of docker hub | 21:56 |
| jvgrant_ | hongbin: good idea, we should consider what the CLI would look like with their CLI standards as well | 21:57 |
| *** EricGonczer_ has joined #openstack-containers | 21:57 | |
| Drago | jvgrant_, hongbin: They might even think what we've come up with makes sense :) | 21:57 |
| Drago | ;) | 21:57 |
| *** tonanhngo_ has joined #openstack-containers | 21:58 | |
| Drago | pc_m: You may get more useful help from tonanhngo or randallburt | 21:58 |
| pc_m | Drago: Thanks for the info. I did see this http://docs.openstack.org/developer/magnum/magnum-proxy.html | 21:59 |
| pc_m | not sure if that will help. | 21:59 |
| randallburt | I'm probably not more helpful. If we're pulling images from public sources, you need to either poke holes for those things or proxy/use something local | 22:00 |
| *** EricGonczer_ has quit IRC | 22:01 | |
| Drago | pc_m: That looks pretty promising! I learn something new every day | 22:01 |
| jvgrant_ | pc_m: i know that swatson was working on getting a setup working behind proxy. I think he found a bug in it but he might have some answers | 22:01 |
| *** tonanhngo has quit IRC | 22:01 | |
| pc_m | Drago: Have you ever seen an issue with Devstack, where at the end of the process it fails saying that python-magnumclient did not get installed? | 22:01 |
| swatson | pc_m: jvgrant is referring to this bug: https://bugs.launchpad.net/magnum/+bug/1623788 | 22:01 |
| openstack | Launchpad bug 1623788 in Magnum "Cluster creation fails on discovery when behind a proxy even after providing proxy information" [Medium,Triaged] - Assigned to rajiv (rajiv-kumar) | 22:01 |
| pc_m | jvgrant_: That would be great | 22:01 |
| swatson | If I recall correctly, the proxy information that was set for the client wasn't getting used | 22:02 |
| *** tonanhngo_ has quit IRC | 22:03 | |
| Drago | pc_m: Sorry, no :/ | 22:04 |
| *** coreyob has quit IRC | 22:04 | |
| pc_m | Drago: I think that was the failure I was seeing, when trying from VM under MacOS. | 22:04 |
| pc_m | 2016-11-15 19:28:32.583 | [ERROR] /home/pcm/devstack/inc/python:221 The following LIBS_FROM_GIT were not installed correct: python-magnumclient | 22:05 |
| Drago | pc_m: I have not seen that error, but in case someone else may have or will in the future, could you file a bug report? http://bugs.launchpad.net/magnum | 22:06 |
| pc_m | Sure. | 22:06 |
| Drago | Thanks! | 22:07 |
| *** jwcroppe has joined #openstack-containers | 22:07 | |
| *** adrian_otto has quit IRC | 22:08 | |
| pc_m | #link https://bugs.launchpad.net/magnum/+bug/1642409 | 22:09 |
| openstack | Launchpad bug 1642409 in Magnum "python-magnumclient not installing under devstack" [Undecided,New] | 22:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/magnum: Updated from global requirements https://review.openstack.org/398608 | 22:09 |
| *** vijendar has joined #openstack-containers | 22:16 | |
| *** vijendar1 has joined #openstack-containers | 22:20 | |
| *** vijendar has quit IRC | 22:21 | |
| *** tonanhngo has joined #openstack-containers | 22:26 | |
| *** absubram has quit IRC | 22:26 | |
| *** vijendar has joined #openstack-containers | 22:28 | |
| *** tonanhngo has quit IRC | 22:30 | |
| *** coreyob has joined #openstack-containers | 22:35 | |
| *** adrian_otto has joined #openstack-containers | 22:35 | |
| *** absubram has joined #openstack-containers | 22:37 | |
| *** coreyob has quit IRC | 22:41 | |
| *** jperry has quit IRC | 22:42 | |
| *** vijendar has quit IRC | 22:43 | |
| *** coreyob has joined #openstack-containers | 22:50 | |
| *** vijendar has joined #openstack-containers | 22:50 | |
| *** catintheroof has quit IRC | 22:54 | |
| *** jasond has quit IRC | 22:55 | |
| *** vijendar1 has quit IRC | 22:55 | |
| *** vijendar has quit IRC | 22:55 | |
| openstackgerrit | Merged openstack/magnum: Add insecure option in functional tests for SSL endpoints https://review.openstack.org/395893 | 22:58 |
| *** chetna has quit IRC | 23:04 | |
| *** chetna has joined #openstack-containers | 23:04 | |
| Drago | hongbin: I've made a couple of comments on the upgrade spec | 23:05 |
| hongbin | reading it | 23:06 |
| *** sdake has joined #openstack-containers | 23:06 | |
| *** dave-mccowan has quit IRC | 23:07 | |
| *** jwcroppe has quit IRC | 23:09 | |
| *** sdake_ has quit IRC | 23:10 | |
| *** jwcroppe has joined #openstack-containers | 23:10 | |
| *** jwcroppe has quit IRC | 23:14 | |
| *** randallburt has quit IRC | 23:16 | |
| *** sdake has quit IRC | 23:16 | |
| tkarke | Hi, I'm getting the following error while running cluster-config command "ERROR: 'EntryPoint' object has no attribute 'resolve'". Traceback: http://paste.openstack.org/show/589531/ | 23:18 |
| Drago | tkarke: That's weird | 23:23 |
| Drago | Never seen that before | 23:23 |
| Drago | tkarke: Looks to be in your magnumclient though. What version are you running? | 23:23 |
| tkarke | Im running 2.3.1 | 23:23 |
| Drago | tkarke: Straight from pip, or from source? | 23:24 |
| Drago | *pypi | 23:24 |
| *** jmckind has quit IRC | 23:24 | |
| tkarke | I installed it using zypper | 23:25 |
| *** dfflanders has joined #openstack-containers | 23:25 | |
| Drago | Oh, OpenSUSE? | 23:26 |
| tkarke | that's right | 23:26 |
| *** jwcroppe has joined #openstack-containers | 23:26 | |
| Drago | tkarke: I have no experience with either, but if you can, I'd try installing it via pip | 23:26 |
| Drago | Maybe there's something wrong with the crypto package | 23:27 |
| hongbin | Drago: see my reply? | 23:28 |
| Drago | hongbin: Why does it matter what is in the cluster's ClusterTemplate? | 23:29 |
| *** chris_hultin is now known as chris_hultin|AWA | 23:29 | |
| Drago | The CT could have None for the coe_version. Doesn't matter as long as the Cluster has the attribute | 23:30 |
| tkarke | I have Magnum running inside a container, within the container the command runs fine but running this from outside the container I get this error. The cryptography package version is the same in both the places. Other magnumclient commands work just fine | 23:30 |
| Drago | I don't see the same complexity problem | 23:30 |
| Drago | tkarke: Supposedly it's the same. I have seen, once, a package that got installed incorrectly and it looked like it had files from two different versions. It was crazy | 23:31 |
| *** jwcroppe has quit IRC | 23:31 | |
| *** Jeffrey4l has joined #openstack-containers | 23:31 | |
| Drago | And it reported the correct version too | 23:31 |
| tkarke | I see, I'll try reinstalling the client and try it again | 23:31 |
| Drago | hongbin: Upgrades go from A to B. A == whatever the cluster is, B == the attributes in the new ClusterTemplate. I don't see why the old ClusterTemplate has any part in this | 23:32 |
| Drago | tkarke: Yeah, until it keeps happening, I'd try the classic turn it off and back on again | 23:33 |
| tkarke | Thanks Drago! | 23:33 |
| Drago | tkarke: Only thank me if it's fixed :) | 23:34 |
| tkarke | :) | 23:35 |
| Drago | hongbin: Also, there's the attribute-locking spec that we have not discussed. In your scenario, a user probably would not be able to update coe_version because that would be something an operator would lock on that CT. But that's just something to think about and is tangent to this conversation | 23:36 |
| *** zhugaoxiao has joined #openstack-containers | 23:38 | |
| *** khamtamtun has joined #openstack-containers | 23:39 | |
| *** Administrator_ has quit IRC | 23:40 | |
| *** sdake has joined #openstack-containers | 23:40 | |
| *** khamtamtun has quit IRC | 23:42 | |
| hongbin | Drago: then, you don't need to do upgrade with cluster template | 23:43 |
| hongbin | Drago: you can do upgrade by directly modifying the cluster | 23:43 |
| hongbin | Drago: then, the reference still doesn't make sense | 23:44 |
| Drago | hongbin: Yes, that is true, but the purpose of specifying a new cluster template is to update the association for operators to be able to track | 23:44 |
| Drago | hongbin: Operators like CERN want to be able to see which clusters are on which ClusterTemplate version, and upgrade them to a new one | 23:45 |
| Drago | hongbin: Everything you said is true, but this is about tracking | 23:45 |
| hongbin | Drago: maybe we could track it in a different way without duplicating the attribute | 23:45 |
| Drago | hongbin: That sounds like a tough thing to do, if not impossible | 23:47 |
| Drago | hongbin: The attributes will be on the new clustertemplate because users need to be able to create brand-new clusters from the new version | 23:47 |
| hongbin | Drago: ok, that makes sense | 23:48 |
| hongbin | Drago: however, let me think how to model it better | 23:49 |
| Drago | hongbin: Okay | 23:49 |
| *** david-lyle_ has joined #openstack-containers | 23:50 | |
| *** david-lyle has quit IRC | 23:51 | |
| *** rcernin has quit IRC | 23:52 | |
| *** syed__ has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!