*** vilobhmm111 has quit IRC | 00:00 | |
*** tsekiyam_ has joined #openstack-nova | 00:04 | |
*** jerrygb_ has joined #openstack-nova | 00:05 | |
*** mtanino has quit IRC | 00:07 | |
*** tsekiyama has quit IRC | 00:08 | |
*** jerrygb_ has quit IRC | 00:08 | |
*** tsekiyam_ has quit IRC | 00:09 | |
*** jerrygb has quit IRC | 00:09 | |
*** rushil has quit IRC | 00:10 | |
*** su_zhang has joined #openstack-nova | 00:13 | |
*** Guest18166 is now known as dims_ | 00:17 | |
*** su_zhang has quit IRC | 00:18 | |
*** thorst has joined #openstack-nova | 00:18 | |
*** RA has joined #openstack-nova | 00:19 | |
*** RA is now known as Guest35886 | 00:19 | |
*** su_zhang has joined #openstack-nova | 00:22 | |
*** leeantho has quit IRC | 00:26 | |
*** ijw has quit IRC | 00:28 | |
*** lucas-dinner has quit IRC | 00:28 | |
*** stevemar has joined #openstack-nova | 00:33 | |
*** exploreshaifali has quit IRC | 00:33 | |
*** eliqiao has joined #openstack-nova | 00:34 | |
eliqiao | hi sdague, around? | 00:34 |
---|---|---|
*** stevemar has quit IRC | 00:36 | |
*** takedakn has joined #openstack-nova | 00:36 | |
*** zzzeek has quit IRC | 00:36 | |
*** davideagnello has quit IRC | 00:37 | |
*** davideagnello has joined #openstack-nova | 00:37 | |
*** hitalia has quit IRC | 00:37 | |
*** ssurana has quit IRC | 00:39 | |
*** markvoelker has joined #openstack-nova | 00:40 | |
*** thorst has quit IRC | 00:40 | |
*** achanda has quit IRC | 00:41 | |
*** xyang1 has quit IRC | 00:42 | |
*** vilobhmm11 has joined #openstack-nova | 00:47 | |
*** ijw has joined #openstack-nova | 00:48 | |
*** gyee has quit IRC | 00:49 | |
*** davideagnello has quit IRC | 00:49 | |
*** davideagnello has joined #openstack-nova | 00:49 | |
*** asettle has joined #openstack-nova | 00:50 | |
*** whenry_ has joined #openstack-nova | 00:55 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix deleted non-public flavor lookups for non-admin users https://review.openstack.org/211809 | 00:55 |
*** asettle has quit IRC | 00:56 | |
*** Apoorva_ has joined #openstack-nova | 00:56 | |
*** asettle has joined #openstack-nova | 00:56 | |
*** EinstCrazy has joined #openstack-nova | 00:57 | |
*** nic has quit IRC | 00:59 | |
*** Apoorva has quit IRC | 00:59 | |
*** Apoorva_ has quit IRC | 00:59 | |
*** takedakn has quit IRC | 01:00 | |
*** Apoorva has joined #openstack-nova | 01:00 | |
*** Apoorva has quit IRC | 01:03 | |
*** ijw has quit IRC | 01:03 | |
*** Apoorva_ has joined #openstack-nova | 01:03 | |
*** angdraug has quit IRC | 01:04 | |
*** Apoorva_ has quit IRC | 01:05 | |
*** Apoorva has joined #openstack-nova | 01:05 | |
*** asettle has quit IRC | 01:06 | |
*** asettle has joined #openstack-nova | 01:07 | |
*** Apoorva has quit IRC | 01:09 | |
*** Apoorva has joined #openstack-nova | 01:09 | |
*** achanda has joined #openstack-nova | 01:11 | |
*** stevemar has joined #openstack-nova | 01:11 | |
*** tonytan4ever has quit IRC | 01:13 | |
*** whenry_ has quit IRC | 01:19 | |
*** su_zhang has quit IRC | 01:24 | |
*** whenry_ has joined #openstack-nova | 01:25 | |
*** nelsnels_ has joined #openstack-nova | 01:30 | |
*** nelsnelson has quit IRC | 01:33 | |
*** rushil has joined #openstack-nova | 01:38 | |
*** shuquan has joined #openstack-nova | 01:39 | |
*** jerrygb has joined #openstack-nova | 01:44 | |
*** jerrygb has quit IRC | 01:45 | |
*** jerrygb has joined #openstack-nova | 01:45 | |
*** Apoorva has quit IRC | 01:46 | |
alex_xu | morning nova | 01:46 |
*** salv-orlando has joined #openstack-nova | 01:46 | |
*** Daisy has joined #openstack-nova | 01:47 | |
*** rushil has quit IRC | 01:48 | |
*** whenry_ has quit IRC | 01:51 | |
*** shuquan has quit IRC | 01:54 | |
*** shuquan has joined #openstack-nova | 01:56 | |
*** salv-orlando has quit IRC | 01:56 | |
*** shuquan has quit IRC | 01:57 | |
*** dims_ has quit IRC | 01:57 | |
*** baoli has joined #openstack-nova | 01:57 | |
*** dims_ has joined #openstack-nova | 01:57 | |
*** haomaiwang has joined #openstack-nova | 01:58 | |
*** rushil has joined #openstack-nova | 01:58 | |
*** shuquan has joined #openstack-nova | 01:58 | |
*** rushil_ has joined #openstack-nova | 01:59 | |
*** haomaiwang has quit IRC | 02:01 | |
eliqiao | morning. | 02:01 |
*** haomaiwang has joined #openstack-nova | 02:01 | |
*** kiranr has joined #openstack-nova | 02:01 | |
*** yuntongjin has joined #openstack-nova | 02:02 | |
*** rushil has quit IRC | 02:03 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 02:04 |
*** shyu has joined #openstack-nova | 02:05 | |
*** yamahata has joined #openstack-nova | 02:08 | |
*** dims_ has quit IRC | 02:10 | |
*** asettle has quit IRC | 02:10 | |
*** dims_ has joined #openstack-nova | 02:13 | |
*** dims__ has joined #openstack-nova | 02:18 | |
*** dims_ has quit IRC | 02:18 | |
*** bswartz has joined #openstack-nova | 02:19 | |
*** mwagner_lap has joined #openstack-nova | 02:21 | |
*** darrenc is now known as darrenc_afk | 02:21 | |
*** dims__ has quit IRC | 02:23 | |
*** dave-mccowan has quit IRC | 02:26 | |
openstackgerrit | Jeffrey Zhang proposed openstack/nova: Remove useless code in _poll_volume_usage function https://review.openstack.org/223947 | 02:29 |
*** yamahata has quit IRC | 02:29 | |
*** unicell has quit IRC | 02:33 | |
*** baoli has quit IRC | 02:37 | |
*** ferest has joined #openstack-nova | 02:38 | |
*** baoli has joined #openstack-nova | 02:38 | |
*** ferest has quit IRC | 02:46 | |
*** asettle has joined #openstack-nova | 02:49 | |
*** darrenc_afk is now known as darrenc | 02:49 | |
*** whenry_ has joined #openstack-nova | 02:50 | |
*** eliqiao has left #openstack-nova | 02:56 | |
*** eliqiao has joined #openstack-nova | 02:56 | |
*** haomaiwang has quit IRC | 03:01 | |
*** markvoelker has quit IRC | 03:01 | |
*** haomaiwa_ has joined #openstack-nova | 03:01 | |
openstackgerrit | Rafael Folco proposed openstack/nova: libvirt: check min required qemu/libvirt versions on ppc64/ppc64le https://review.openstack.org/226555 | 03:07 |
*** nelsnels_ has quit IRC | 03:08 | |
*** boris-42 has quit IRC | 03:09 | |
openstackgerrit | Rafael Folco proposed openstack/nova: libvirt: check min required qemu/kvm versions on ppc64/ppc64le https://review.openstack.org/226555 | 03:09 |
*** shuquan has quit IRC | 03:10 | |
*** shuquan has joined #openstack-nova | 03:11 | |
*** david-lyle has joined #openstack-nova | 03:13 | |
*** Daisy has quit IRC | 03:14 | |
*** nelsnelson has joined #openstack-nova | 03:14 | |
*** Daisy has joined #openstack-nova | 03:14 | |
*** markvoelker has joined #openstack-nova | 03:16 | |
*** ssurana has joined #openstack-nova | 03:18 | |
*** zhangjn_ has joined #openstack-nova | 03:18 | |
*** su_zhang has joined #openstack-nova | 03:18 | |
*** EinstCrazy has quit IRC | 03:19 | |
*** zhangjn has quit IRC | 03:20 | |
*** otter768 has quit IRC | 03:21 | |
*** shuquan has quit IRC | 03:25 | |
*** shuquan has joined #openstack-nova | 03:28 | |
*** ssurana has quit IRC | 03:28 | |
*** ssurana has joined #openstack-nova | 03:28 | |
*** fawadkhaliq has joined #openstack-nova | 03:32 | |
*** ssurana has quit IRC | 03:32 | |
*** baoli has quit IRC | 03:32 | |
*** Daisy has quit IRC | 03:33 | |
*** Daisy has joined #openstack-nova | 03:33 | |
*** Daisy has quit IRC | 03:38 | |
*** ijw has joined #openstack-nova | 03:44 | |
*** armax has quit IRC | 03:44 | |
*** shuquan has quit IRC | 03:45 | |
*** Marga__ has quit IRC | 03:46 | |
*** Marga_ has joined #openstack-nova | 03:46 | |
*** dims_ has joined #openstack-nova | 03:47 | |
*** diogogmt has quit IRC | 03:48 | |
*** ijw has quit IRC | 03:49 | |
*** Marga_ has quit IRC | 03:50 | |
*** shuquan has joined #openstack-nova | 03:53 | |
*** rushil_ has quit IRC | 03:53 | |
*** dims_ has quit IRC | 03:54 | |
*** jerrygb has quit IRC | 03:56 | |
*** mdrabe has joined #openstack-nova | 03:56 | |
*** shuquan has quit IRC | 03:58 | |
*** haomaiwa_ has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-nova | 04:01 | |
*** Marga_ has joined #openstack-nova | 04:02 | |
*** changbl has quit IRC | 04:04 | |
*** yuntongjin has quit IRC | 04:05 | |
*** changbl has joined #openstack-nova | 04:09 | |
*** ijw has joined #openstack-nova | 04:09 | |
*** ijw has quit IRC | 04:09 | |
*** ijw has joined #openstack-nova | 04:09 | |
openstackgerrit | Yingxin proposed openstack/nova: Fix MetricWeigher to use MonitorMetricList https://review.openstack.org/221620 | 04:10 |
*** mdrabe has quit IRC | 04:12 | |
*** pixelb has joined #openstack-nova | 04:13 | |
*** jwcroppe has joined #openstack-nova | 04:14 | |
*** whenry_ has quit IRC | 04:16 | |
*** achanda has quit IRC | 04:17 | |
*** salv-orlando has joined #openstack-nova | 04:17 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova: Set migration status to 'error' on live-migration failure https://review.openstack.org/215483 | 04:18 |
*** pixelb has quit IRC | 04:19 | |
*** hitalia has joined #openstack-nova | 04:19 | |
*** zhangjn_ has quit IRC | 04:24 | |
*** zhangjn has joined #openstack-nova | 04:25 | |
*** zhangjn_ has joined #openstack-nova | 04:28 | |
*** zhangjn has quit IRC | 04:29 | |
*** kiranr has quit IRC | 04:32 | |
*** zhangjn_ has quit IRC | 04:32 | |
*** zhangjn has joined #openstack-nova | 04:33 | |
*** fawadk has joined #openstack-nova | 04:33 | |
*** salv-orlando has quit IRC | 04:35 | |
*** fawadkhaliq has quit IRC | 04:35 | |
*** Marga_ has quit IRC | 04:36 | |
*** Marga_ has joined #openstack-nova | 04:37 | |
*** zhangjn_ has joined #openstack-nova | 04:39 | |
*** yuntongjin has joined #openstack-nova | 04:40 | |
*** deepthi has joined #openstack-nova | 04:40 | |
*** zhangjn has quit IRC | 04:42 | |
*** zhangjn_ has quit IRC | 04:42 | |
*** zhangjn has joined #openstack-nova | 04:43 | |
*** Sree has joined #openstack-nova | 04:44 | |
*** Sree_ has joined #openstack-nova | 04:46 | |
*** Sree_ is now known as Guest14662 | 04:46 | |
*** achanda has joined #openstack-nova | 04:48 | |
*** yuntongjin2 has joined #openstack-nova | 04:48 | |
*** Sree has quit IRC | 04:49 | |
*** yuntongjin has quit IRC | 04:49 | |
*** asettle has quit IRC | 04:55 | |
*** deepakcs has joined #openstack-nova | 04:56 | |
*** zhangjn has quit IRC | 04:57 | |
*** Sree has joined #openstack-nova | 04:59 | |
*** zhangjn has joined #openstack-nova | 04:59 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-nova | 05:01 | |
*** Guest14662 has quit IRC | 05:02 | |
*** ildikov has quit IRC | 05:21 | |
*** otter768 has joined #openstack-nova | 05:22 | |
*** otter768 has quit IRC | 05:26 | |
*** sahid has joined #openstack-nova | 05:29 | |
*** sahid_ has joined #openstack-nova | 05:35 | |
*** salv-orlando has joined #openstack-nova | 05:45 | |
*** boris-42 has joined #openstack-nova | 05:48 | |
*** dims_ has joined #openstack-nova | 05:50 | |
*** salv-orlando has quit IRC | 05:52 | |
*** su_zhang has quit IRC | 05:54 | |
*** kiran-r has joined #openstack-nova | 05:54 | |
*** dims_ has quit IRC | 05:55 | |
*** kiranr has joined #openstack-nova | 05:58 | |
*** ildikov has joined #openstack-nova | 06:00 | |
*** haomaiwang has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-nova | 06:01 | |
*** kiran-r has quit IRC | 06:02 | |
*** kiranr has quit IRC | 06:02 | |
openstackgerrit | Hiroyuki Eguchi proposed openstack/nova: Cannot rebuild a instance booted from volume https://review.openstack.org/201458 | 06:06 |
*** Sree has quit IRC | 06:07 | |
*** jerrygb has joined #openstack-nova | 06:08 | |
*** Sree has joined #openstack-nova | 06:08 | |
*** jerrygb has quit IRC | 06:12 | |
*** fawadk has quit IRC | 06:14 | |
*** fawadkhaliq has joined #openstack-nova | 06:17 | |
*** scheuran has joined #openstack-nova | 06:17 | |
*** bswartz has quit IRC | 06:18 | |
*** Guest35886 has quit IRC | 06:18 | |
eliqiao | alex_xu ping | 06:20 |
alex_xu | eliqiao: pong | 06:20 |
eliqiao | re https://review.openstack.org/#/c/224438/6/nova/utils.py | 06:20 |
eliqiao | what cases you expecting test. | 06:20 |
*** bswartz has joined #openstack-nova | 06:21 | |
alex_xu | eliqiao: dnsmasq in linux and windows the max length of hostname supported | 06:21 |
*** nkrinner has joined #openstack-nova | 06:21 | |
alex_xu | eliqiao: do you have windows virtual matchine? | 06:22 |
*** stevemar has quit IRC | 06:23 | |
eliqiao | alex_xu: I don't | 06:23 |
*** jwcroppe has quit IRC | 06:23 | |
alex_xu | eliqiao: ok, let me try to find someone have one | 06:23 |
*** jwcroppe has joined #openstack-nova | 06:23 | |
eliqiao | alex_xu: I am think we are over-kill on this case. | 06:25 |
alex_xu | eliqiao: why? | 06:25 |
*** jwcroppe has quit IRC | 06:28 | |
*** yuntongjin2 has quit IRC | 06:30 | |
eliqiao | it is hacking we do this in nova, (I mean for the truncation stuff), metadata service /config-drive will grab hostname from nova's database and will set it to dnsmasq service or Operatation System (/etc/hostname) | 06:31 |
*** eliqiao has left #openstack-nova | 06:31 | |
*** vilobhmm11 has quit IRC | 06:31 | |
*** eliqiao has joined #openstack-nova | 06:31 | |
*** fawadk has joined #openstack-nova | 06:32 | |
*** EinstCrazy has joined #openstack-nova | 06:32 | |
alex_xu | eliqiao: if we don't truncate it, then the dns name for host won't works | 06:33 |
eliqiao | maybe I think it too simple. but as sean side, what if it's a very long host name like "aaa.bbb.ccc.ddd" , after truncation , we get "aaa.b" which can not be resolved? | 06:33 |
*** Murali has joined #openstack-nova | 06:33 | |
*** fawadkhaliq has quit IRC | 06:33 | |
eliqiao | for this case, how can we do the truncation? | 06:33 |
alex_xu | eliqiao: Actually Sean suggest warning log for truncation | 06:33 |
eliqiao | sean's words : "Also, truncation shouldn't be silent, because it could produce things like foo.c which won't resolve." | 06:33 |
eliqiao | alex_xu: give a log warnning makes some sense | 06:34 |
eliqiao | alex_xu: would you please advise how to testing the maximum length of an vm's hostname could be? | 06:37 |
alex_xu | eliqiao: actually I'm try to test now | 06:37 |
*** romainh has joined #openstack-nova | 06:40 | |
*** abhishekk has joined #openstack-nova | 06:46 | |
*** rdopiera has joined #openstack-nova | 06:47 | |
*** fawadkhaliq has joined #openstack-nova | 06:51 | |
*** kiranr has joined #openstack-nova | 06:51 | |
*** fawadk has quit IRC | 06:52 | |
*** karimb has joined #openstack-nova | 06:54 | |
*** sahid_ has quit IRC | 06:54 | |
*** matrohon has joined #openstack-nova | 06:55 | |
*** alex_klimov has joined #openstack-nova | 06:55 | |
*** alex_klimov has quit IRC | 06:56 | |
*** alex_klimov has joined #openstack-nova | 06:56 | |
*** sahid_ has joined #openstack-nova | 06:56 | |
eliqiao | alex_xu not sure if this is what you want http://www.freesoft.org/CIE/RFC/1035/9.htm | 06:57 |
*** browne has quit IRC | 06:57 | |
*** fawadk has joined #openstack-nova | 06:58 | |
alex_xu | eliqiao: rfc said 255, but actually situation may not | 06:58 |
*** snikitin has joined #openstack-nova | 06:59 | |
*** fawadkhaliq has quit IRC | 07:00 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-nova | 07:01 | |
eliqiao | alex_xu: free to come to my cube? | 07:02 |
alex_xu | eliqiao: yea | 07:02 |
*** lpetrut has joined #openstack-nova | 07:04 | |
*** fawadkhaliq has joined #openstack-nova | 07:09 | |
*** fawadk has quit IRC | 07:10 | |
*** abhishekk_ has joined #openstack-nova | 07:10 | |
*** rajesht__ has joined #openstack-nova | 07:10 | |
*** kodokuu has joined #openstack-nova | 07:11 | |
*** gszasz has joined #openstack-nova | 07:11 | |
*** abhishekk has quit IRC | 07:12 | |
*** rajesht_ has quit IRC | 07:13 | |
*** sahid_ has quit IRC | 07:14 | |
*** sahid has quit IRC | 07:14 | |
*** sahid has joined #openstack-nova | 07:14 | |
*** achanda has quit IRC | 07:14 | |
*** karimb has quit IRC | 07:15 | |
*** markvoelker has quit IRC | 07:15 | |
*** karimb has joined #openstack-nova | 07:16 | |
ndipanov | alex_xu, found a bug in that last patch you already reviewed. Will push the change if you could take a look that would be awesome | 07:20 |
*** otter768 has joined #openstack-nova | 07:23 | |
*** stevemar has joined #openstack-nova | 07:23 | |
*** jwcroppe has joined #openstack-nova | 07:23 | |
*** jichen has joined #openstack-nova | 07:25 | |
*** ptm_ has joined #openstack-nova | 07:25 | |
*** karimb has quit IRC | 07:27 | |
*** stevemar has quit IRC | 07:27 | |
*** otter768 has quit IRC | 07:28 | |
*** ptm__ has joined #openstack-nova | 07:28 | |
*** cfriesen__ has quit IRC | 07:28 | |
alex_xu | ndipanov: sure, will do after few minutes | 07:29 |
*** ptm_ has quit IRC | 07:31 | |
*** acc_ has quit IRC | 07:31 | |
*** karimb has joined #openstack-nova | 07:32 | |
*** sayali has quit IRC | 07:34 | |
*** lpetrut has quit IRC | 07:36 | |
*** sayali has joined #openstack-nova | 07:38 | |
ndipanov | alex_xu, still not posted :( | 07:40 |
*** salv-orlando has joined #openstack-nova | 07:40 | |
alex_xu | ndipanov: yea...I just read the comment | 07:40 |
alex_xu | ndipanov: basically, 'resize' in 'resize' works, but yes, it is wrong. Maybe it is ok for now? | 07:41 |
* alex_xu didn't get the meaning of ' this should not work for old flavors' yet, thinking | 07:42 | |
*** Sree_ has joined #openstack-nova | 07:42 | |
*** Sree_ is now known as Guest89063 | 07:43 | |
ndipanov | alex_xu, trying to figure that out | 07:45 |
*** Sree has quit IRC | 07:46 | |
*** jlanoux has joined #openstack-nova | 07:46 | |
*** salv-orl_ has joined #openstack-nova | 07:47 | |
*** salv-orlando has quit IRC | 07:48 | |
ndipanov | alex_xu, so it should actually be new flavors | 07:48 |
ndipanov | we will never get the new flavor this way | 07:49 |
*** fawadkhaliq has quit IRC | 07:49 | |
*** fawadkhaliq has joined #openstack-nova | 07:50 | |
alex_xu | ndipanov: sorry...I didn't get why that related to new/old flavor.. | 07:52 |
*** dims_ has joined #openstack-nova | 07:52 | |
alex_xu | ndipanov: maybe we just need migration.migration_type == 'resize', we needn't 'in' | 07:53 |
ndipanov | alex_xu, ah you are right resize in resize works | 07:53 |
ndipanov | ok never mind | 07:53 |
alex_xu | ndipanov: yea, never mind :) | 07:53 |
ndipanov | alex_xu, only one cup of coffee | 07:54 |
openstackgerrit | Eli Qiao proposed openstack/nova: Give instance default hostname if hostname is empty https://review.openstack.org/224438 | 07:56 |
*** danpb has joined #openstack-nova | 07:57 | |
*** salv-orl_ has quit IRC | 07:59 | |
*** salv-orlando has joined #openstack-nova | 07:59 | |
*** dims_ has quit IRC | 07:59 | |
bauzas | morning Nova | 08:00 |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-nova | 08:01 | |
*** kodokuu_ has joined #openstack-nova | 08:01 | |
*** kodokuu has quit IRC | 08:04 | |
*** BobBall_AWOL is now known as BobBall | 08:04 | |
alex_xu | eliqiao: https://review.openstack.org/224438 there are some typo problem on the user-face message | 08:05 |
alex_xu | eliqiao: and some suggestion about NOTE | 08:05 |
eliqiao | fare enough alex_xu, thx | 08:06 |
alex_xu | eliqiao: np | 08:06 |
*** markus_z has joined #openstack-nova | 08:06 | |
alex_xu | eliqiao: and missing one use-case for truncate_hostname on default hostname | 08:07 |
alex_xu | eliqiao: sorry, s/use-case/test-case/ | 08:08 |
*** haomaiwang has quit IRC | 08:10 | |
*** haomaiwa_ has joined #openstack-nova | 08:12 | |
*** derekh has joined #openstack-nova | 08:13 | |
eliqiao | alex_xu: ya, I get you , and we though it on same case. | 08:15 |
*** aix has joined #openstack-nova | 08:15 | |
alex_xu | eliqiao: cool | 08:15 |
*** yassine__ has joined #openstack-nova | 08:15 | |
*** markvoelker has joined #openstack-nova | 08:15 | |
*** jwcroppe has quit IRC | 08:19 | |
*** lucas-dinner has joined #openstack-nova | 08:19 | |
*** lucasagomes has joined #openstack-nova | 08:19 | |
openstackgerrit | Eli Qiao proposed openstack/nova: Give instance default hostname if hostname is empty https://review.openstack.org/224438 | 08:20 |
eliqiao | alex_xu: ^^^ | 08:20 |
*** markvoelker has quit IRC | 08:20 | |
openstackgerrit | jichenjc proposed openstack/nova: Remove duplicate VALID_NAME_REGEX https://review.openstack.org/218489 | 08:20 |
alex_xu | eliqiao: cool, thanks | 08:20 |
*** ptm__ has quit IRC | 08:23 | |
*** Kevin_Zheng has joined #openstack-nova | 08:24 | |
*** kiran-r has joined #openstack-nova | 08:29 | |
*** kiran-r has quit IRC | 08:34 | |
*** atuvenie has joined #openstack-nova | 08:35 | |
*** alex_klimov has quit IRC | 08:36 | |
*** alex_klimov has joined #openstack-nova | 08:36 | |
*** Daisy has joined #openstack-nova | 08:39 | |
*** kiran-r has joined #openstack-nova | 08:41 | |
ndipanov | johnthetubaguy, https://review.openstack.org/#/c/226411/ | 08:44 |
ndipanov | although that one can be backported - but it has 2x +2s | 08:45 |
*** sudipto has joined #openstack-nova | 08:46 | |
openstackgerrit | Alex Xu proposed openstack/nova: Give instance default hostname if hostname is empty https://review.openstack.org/224438 | 08:54 |
*** Sree has joined #openstack-nova | 08:57 | |
*** Guest89063 has quit IRC | 09:00 | |
*** haomaiwa_ has quit IRC | 09:01 | |
*** _RuiChen has quit IRC | 09:01 | |
*** 16WAAB8UT has joined #openstack-nova | 09:01 | |
*** _RuiChen has joined #openstack-nova | 09:02 | |
*** kiranr has quit IRC | 09:02 | |
*** lucas-dinner has quit IRC | 09:07 | |
johnthetubaguy | ndipanov: sweet, so I how do you want to deal with this bug, do you want to create a new one for the remaining bits, and close out this one for liberty? | 09:07 |
johnthetubaguy | alex_xu: how are we doing with that bug patch? | 09:08 |
*** jerrygb has joined #openstack-nova | 09:08 | |
johnthetubaguy | ndipanov: we can just leave it open, but that seems less satisfying some how | 09:08 |
*** lpetrut has joined #openstack-nova | 09:09 | |
*** dims_ has joined #openstack-nova | 09:11 | |
*** jerrygb has quit IRC | 09:13 | |
*** atuvenie has quit IRC | 09:14 | |
*** kiranr has joined #openstack-nova | 09:15 | |
*** e0ne has joined #openstack-nova | 09:15 | |
*** jwcroppe has joined #openstack-nova | 09:16 | |
*** otter768 has joined #openstack-nova | 09:24 | |
*** stevemar has joined #openstack-nova | 09:25 | |
*** RuiChen has joined #openstack-nova | 09:28 | |
openstackgerrit | John Garbutt proposed openstack/nova: Filter leading/trailing spaces for name field in v2.1 compat mode https://review.openstack.org/222032 | 09:28 |
*** stevemar has quit IRC | 09:28 | |
*** otter768 has quit IRC | 09:28 | |
*** _RuiChen has quit IRC | 09:29 | |
*** alexpilotti has joined #openstack-nova | 09:32 | |
* danpb wonders when master will open for mitaka merges | 09:34 | |
*** ociuhandu has quit IRC | 09:36 | |
johnthetubaguy | danpb: once we get all the RC1 bugs closed | 09:37 |
* danpb untags all bugs from RC1 ;-P | 09:37 | |
johnthetubaguy | I hoped it would have been yesterday, currently gunning for thursday, unless we get done sooner | 09:37 |
johnthetubaguy | danpb: honestly, I am tempted to do that at this point | 09:37 |
*** Daisy has quit IRC | 09:39 | |
*** Daisy has joined #openstack-nova | 09:40 | |
alex_xu | johnthetubaguy: I think this is ready, today I and eliqiao test windows,linxu and dnsmasq https://review.openstack.org/224438 | 09:42 |
*** redbeard has joined #openstack-nova | 09:42 | |
*** kodokuu_ has quit IRC | 09:43 | |
*** Daisy has quit IRC | 09:44 | |
*** Daisy has joined #openstack-nova | 09:44 | |
johnthetubaguy | alex_xu: wow, limit of 63, sounds like the best choice though | 09:44 |
alex_xu | johnthetubaguy: yea | 09:44 |
johnthetubaguy | alex_xu: is our default name larger than that? | 09:44 |
*** dims_ has quit IRC | 09:44 | |
alex_xu | johnthetubaguy: no, actually not, but I saw eliqiao add that, I think it's fine, if someday we make default name longer and the thing won't be broken | 09:45 |
johnthetubaguy | yeah, just thinking it through, I think we are good | 09:45 |
alex_xu | cool | 09:45 |
johnthetubaguy | alex_xu: so thinking about to the "difficult" patch | 09:46 |
johnthetubaguy | I restored your patchset 5 | 09:47 |
johnthetubaguy | I am tempted to say we go for that | 09:47 |
*** jwcroppe_ has joined #openstack-nova | 09:47 | |
alex_xu | johnthetubaguy: yea... | 09:47 |
johnthetubaguy | so I have one comment so far, let me send it | 09:47 |
johnthetubaguy | alex_xu: got a question here: https://review.openstack.org/#/c/222032/7/nova/api/openstack/compute/flavor_manage.py,cm | 09:47 |
alex_xu | johnthetubaguy: actually I commented also, about another tradeoff, anyway I don't think we match legacy_v2 api behaviour totally is right | 09:47 |
* alex_xu is checking | 09:48 | |
alex_xu | johnthetubaguy: actually it's done at underlayer https://github.com/openstack/nova/blob/master/nova/compute/flavors.py#L109 | 09:49 |
*** e0ne has quit IRC | 09:49 | |
*** hitalia has quit IRC | 09:49 | |
*** jwcroppe has quit IRC | 09:49 | |
alex_xu | maybe I should use normailize_name in underlayer also | 09:49 |
johnthetubaguy | alex_xu: ah, OK, its tempting to drop that one, and add it in the API for consistency | 09:50 |
johnthetubaguy | alex_xu: but honestly, that could be a follow up patch | 09:50 |
*** e0ne has joined #openstack-nova | 09:50 | |
alex_xu | johnthetubaguy: yea, that is good way, but need check ec2, legacy v2 and v2.1 | 09:50 |
alex_xu | johnthetubaguy: yea, thanks | 09:50 |
johnthetubaguy | alex_xu: yeah, totally | 09:50 |
*** Sree has quit IRC | 09:52 | |
bauzas | johnthetubaguy: how can I communicate that one RC1 bug is okay to be merged ? | 09:53 |
bauzas | ie. https://review.openstack.org/#/c/221620/5 | 09:53 |
johnthetubaguy | bauzas: not sure what you are asking | 09:54 |
johnthetubaguy | bauzas: I guess its RC1-potential | 09:54 |
johnthetubaguy | oh, wait, thats the really bad regression right? | 09:54 |
bauzas | johnthetubaguy: I mean, there are 21 bugs in https://bugs.launchpad.net/nova/+bugs?field.tag=liberty-rc-potential | 09:54 |
johnthetubaguy | yeah, there are | 09:54 |
bauzas | johnthetubaguy: but I reviewed https://bugs.launchpad.net/nova/+bug/1493680 and I consider the bugfix acceptable | 09:55 |
openstack | Launchpad bug 1493680 in OpenStack Compute (nova) "HostState.metrics is no longer a dict" [High,In progress] - Assigned to Yingxin (cyx1231st) | 09:55 |
bauzas | johnthetubaguy: so I just wonder how I could send the bat-signal that it's worth reviewing it | 09:55 |
bauzas | of course I can ping you guys :) | 09:56 |
bauzas | but are we still using the etherpad of doom ? | 09:56 |
bauzas | that's my wonders | 09:56 |
bauzas | johnthetubaguy: anyhow, I don't want to overthink | 09:58 |
bauzas | johnthetubaguy: pinging should be enough | 09:58 |
johnthetubaguy | yeah, we don't have an etherpad for this right now | 09:59 |
*** claudiub has joined #openstack-nova | 10:00 | |
*** 16WAAB8UT has quit IRC | 10:01 | |
*** lan has joined #openstack-nova | 10:01 | |
*** Daisy has quit IRC | 10:01 | |
*** 6JTAB3MXI has joined #openstack-nova | 10:01 | |
*** Kennan_Vacation has quit IRC | 10:01 | |
*** Daisy has joined #openstack-nova | 10:02 | |
*** Kennan_Vacation has joined #openstack-nova | 10:02 | |
bauzas | fair enough :) | 10:02 |
*** atuvenie has joined #openstack-nova | 10:03 | |
*** Daisy has quit IRC | 10:05 | |
*** Daisy has joined #openstack-nova | 10:05 | |
*** dims_ has joined #openstack-nova | 10:09 | |
*** alex_klimov has quit IRC | 10:09 | |
*** ndipanov has quit IRC | 10:09 | |
*** eglynn has joined #openstack-nova | 10:10 | |
*** kiran-r has quit IRC | 10:10 | |
*** ndipanov has joined #openstack-nova | 10:11 | |
alex_xu | sdague: hope you can take a look at https://review.openstack.org/224438 and https://review.openstack.org/222032 when you wakeup, thanks | 10:13 |
*** IanGovett has joined #openstack-nova | 10:16 | |
*** markvoelker has joined #openstack-nova | 10:17 | |
*** markvoelker has quit IRC | 10:21 | |
openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused refresh_security_group_members() call https://review.openstack.org/182632 | 10:22 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Make secgroup rules refresh with refresh_instance_security_rules() https://review.openstack.org/182633 | 10:22 |
*** alex_klimov has joined #openstack-nova | 10:24 | |
*** Daisy has quit IRC | 10:25 | |
*** Daisy has joined #openstack-nova | 10:26 | |
openstackgerrit | Jian LI proposed openstack/nova: Sync vms az with the host when using aggregate https://review.openstack.org/226683 | 10:27 |
*** ociuhandu has joined #openstack-nova | 10:29 | |
*** Daisy has quit IRC | 10:30 | |
*** Daisy has joined #openstack-nova | 10:30 | |
*** jwcroppe_ has quit IRC | 10:30 | |
*** jwcroppe has joined #openstack-nova | 10:31 | |
*** Daisy has quit IRC | 10:31 | |
*** Daisy has joined #openstack-nova | 10:32 | |
*** jwcroppe has quit IRC | 10:35 | |
*** Daisy has quit IRC | 10:36 | |
sdague | alex_xu: ok, just waking, let me see | 10:36 |
alex_xu | sdague: thanks | 10:37 |
*** alex_klimov has quit IRC | 10:39 | |
*** alex_klimov has joined #openstack-nova | 10:40 | |
*** alex_klimov has quit IRC | 10:40 | |
*** alex_klimov has joined #openstack-nova | 10:40 | |
openstackgerrit | Hans Lindgren proposed openstack/nova: Remove and deprecate conductor provider_fw_rule_get_all() https://review.openstack.org/216241 | 10:43 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused provider firewall rules functionality in nova https://review.openstack.org/184027 | 10:43 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Remove provider_fw_rules tables and associated db api methods https://review.openstack.org/216242 | 10:43 |
openstackgerrit | Ankit Agrawal proposed openstack/nova: Remove unnecessary call to info_cache.delete https://review.openstack.org/226690 | 10:44 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: Use legacy engineFacade from oslo_db.enginefacade https://review.openstack.org/221310 | 10:44 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: use oslo_db.enginefacade https://review.openstack.org/225065 | 10:45 |
*** jichen has quit IRC | 10:46 | |
*** lucasagomes has quit IRC | 10:48 | |
*** exploreshaifali has joined #openstack-nova | 10:51 | |
*** EinstCrazy has quit IRC | 10:51 | |
*** zhangjn has quit IRC | 10:51 | |
*** kiran-r has joined #openstack-nova | 10:53 | |
*** lucasagomes has joined #openstack-nova | 10:53 | |
*** lucasagomes_ has joined #openstack-nova | 11:00 | |
*** lucasagomes_ has left #openstack-nova | 11:00 | |
*** 6JTAB3MXI has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-nova | 11:01 | |
*** shyu has quit IRC | 11:02 | |
*** aix has quit IRC | 11:03 | |
alex_xu | johnthetubaguy: bauzas, looks like 'target' have chance to be 'None' in https://review.openstack.org/#/c/221620/5/nova/scheduler/weights/metrics.py line 103 | 11:04 |
johnthetubaguy | alex_xu: yep, you are right, good spot | 11:05 |
alex_xu | johnthetubaguy: thanks, so looks like that patch missing an testcase also | 11:06 |
*** Daisy has joined #openstack-nova | 11:06 | |
*** salv-orlando has quit IRC | 11:12 | |
*** salv-orlando has joined #openstack-nova | 11:13 | |
*** lan has quit IRC | 11:13 | |
*** Daisy has quit IRC | 11:14 | |
*** Daisy has joined #openstack-nova | 11:15 | |
*** lucasagomes is now known as lucas-hungry | 11:15 | |
*** Daisy has quit IRC | 11:17 | |
*** markvoelker has joined #openstack-nova | 11:17 | |
*** Daisy has joined #openstack-nova | 11:19 | |
*** jwcroppe has joined #openstack-nova | 11:21 | |
*** markvoelker has quit IRC | 11:22 | |
*** haomaiwang has quit IRC | 11:22 | |
*** baoli has joined #openstack-nova | 11:23 | |
*** otter768 has joined #openstack-nova | 11:25 | |
*** shakamunyi has quit IRC | 11:25 | |
rajesht__ | johnthetubaguy: if u hv time, could you please look at this backport https://review.openstack.org/#/c/223968/ | 11:26 |
rajesht__ | johnthetubaguy: thanks :) | 11:26 |
johnthetubaguy | rajesht__: I am not on stable-core I am afraid | 11:26 |
rajesht__ | johnthetubaguy: ok, but you can give your suggestion, if you want :) | 11:27 |
*** otter768 has quit IRC | 11:29 | |
dims_ | rajesht__: stable folks hangout on #openstack-stable | 11:32 |
sudipto | alex_xu, https://review.openstack.org/#/c/221620/5/nova/scheduler/weights/metrics.py - for your comment on this - w.r.t None.value - do you ever see that happening? | 11:32 |
*** Sree has joined #openstack-nova | 11:32 | |
rajesht__ | dims_: thanks for info :) | 11:32 |
*** gongysh has joined #openstack-nova | 11:32 | |
sudipto | alex_xu, since if not target has a if/else inside it - will not always return before ever reaching there? | 11:33 |
*** baoli_ has joined #openstack-nova | 11:33 | |
gongysh | hi, who can tell me where to find the document about nova aggregator and cell document? thanks | 11:33 |
sudipto | alex_xu, the if raises an exception and the else returns. | 11:34 |
*** aix has joined #openstack-nova | 11:34 | |
*** jerrygb has joined #openstack-nova | 11:35 | |
*** baoli has quit IRC | 11:36 | |
*** mwagner_lap has quit IRC | 11:37 | |
*** Daisy has quit IRC | 11:39 | |
*** jerrygb has quit IRC | 11:39 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Locking resources by service token https://review.openstack.org/203880 | 11:42 |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Locking servers by service token https://review.openstack.org/203880 | 11:43 |
openstackgerrit | Hiroyuki Eguchi proposed openstack/nova: Cannot rebuild a instance booted from volume https://review.openstack.org/201458 | 11:43 |
openstackgerrit | Ankit Agrawal proposed openstack/nova: Detach volume after deleting instance with no host https://review.openstack.org/194063 | 11:47 |
*** jerrygb has joined #openstack-nova | 11:48 | |
*** Daisy has joined #openstack-nova | 11:49 | |
*** cristicalin has joined #openstack-nova | 11:49 | |
*** EinstCrazy has joined #openstack-nova | 11:49 | |
alex_xu | sudipto: I didn't see that, but from the code, I guess the code means the scheduler configure want to skip that metric and that metric didn't report from compute node also | 11:50 |
*** sayali has quit IRC | 11:50 | |
alex_xu | s/scheduler configure/scheduler configuration/ | 11:50 |
*** Daisy has quit IRC | 11:51 | |
sudipto | alex_xu, yeah it sounds like that. | 11:51 |
*** dave-mccowan has joined #openstack-nova | 11:51 | |
sudipto | alex_xu, but i feel your comment is valid anyway - since there's a chance that if block won't be hit - and it will eventually come down to None.value. | 11:52 |
alex_xu | sudipto: yea, there is a chance | 11:53 |
*** sayali has joined #openstack-nova | 11:55 | |
*** exploreshaifali has quit IRC | 11:56 | |
*** thorst has joined #openstack-nova | 11:58 | |
*** EinstCrazy has quit IRC | 11:59 | |
*** krtaylor has quit IRC | 12:01 | |
*** Daisy has joined #openstack-nova | 12:02 | |
*** achanda has joined #openstack-nova | 12:03 | |
*** zhangjn has joined #openstack-nova | 12:04 | |
*** EinstCrazy has joined #openstack-nova | 12:04 | |
*** ankit_ag has joined #openstack-nova | 12:04 | |
*** EinstCrazy has quit IRC | 12:06 | |
*** kiran-r has quit IRC | 12:10 | |
*** krtaylor has joined #openstack-nova | 12:13 | |
*** e0ne has quit IRC | 12:13 | |
*** aysyd has joined #openstack-nova | 12:15 | |
*** deepakcs has quit IRC | 12:15 | |
*** pixelb has joined #openstack-nova | 12:17 | |
*** achanda has quit IRC | 12:18 | |
openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Get valid server state https://review.openstack.org/192246 | 12:18 |
*** lucas-hungry is now known as lucasagomes | 12:19 | |
*** Sree has quit IRC | 12:20 | |
*** markvoelker has joined #openstack-nova | 12:20 | |
*** fawadkhaliq has quit IRC | 12:21 | |
*** kiranr has quit IRC | 12:22 | |
*** raildo-afk is now known as raildo | 12:22 | |
*** fawadkhaliq has joined #openstack-nova | 12:22 | |
*** jwcroppe_ has joined #openstack-nova | 12:22 | |
*** mwagner_lap has joined #openstack-nova | 12:23 | |
*** zhenguo has quit IRC | 12:24 | |
bauzas | alex_xu: well, technically, it will never go to L103 if target is None | 12:24 |
bauzas | johnthetubaguy: ^ | 12:24 |
ankit_ag | johnthetubaguy: hi, you around ? | 12:25 |
bauzas | alex_xu: johnthetubaguy: I should have commented that I was agreeing to leave that since we're very close to RC1 | 12:25 |
bauzas | alex_xu: johnthetubaguy: and since I don't know the proposer, I preferred to leave +1 | 12:25 |
*** jwcroppe has quit IRC | 12:25 | |
bauzas | oh sec, yeah there is another if | 12:26 |
bauzas | alex_xu: good point then, providing a new iteration | 12:26 |
*** takedakn has joined #openstack-nova | 12:27 | |
*** zzzeek has joined #openstack-nova | 12:28 | |
*** jwcroppe_ is now known as jwcroppe | 12:29 | |
*** tpeoples has joined #openstack-nova | 12:29 | |
*** Sree has joined #openstack-nova | 12:29 | |
*** abhijeetm has joined #openstack-nova | 12:31 | |
*** bswartz has quit IRC | 12:32 | |
*** edmondsw has joined #openstack-nova | 12:32 | |
*** diogogmt has joined #openstack-nova | 12:32 | |
*** diogogmt has quit IRC | 12:33 | |
alex_xu | bauzas: :) | 12:34 |
alex_xu | johnthetubaguy: bauzas we need merge https://review.openstack.org/#/c/221620/ before firday, right? | 12:35 |
alex_xu | oops, s/firday/friday/ | 12:36 |
*** thorst has left #openstack-nova | 12:38 | |
bauzas | alex_xu: well, it's possible to merge it after and backport it as a candidate for a RC2, but I'd prefer to spin off a new change now | 12:39 |
bauzas | alex_xu: that's why I'm working on a new PS | 12:40 |
bauzas | alex_xu: should be done by the next mins | 12:40 |
alex_xu | bauzas: oh, got it, cool | 12:40 |
alex_xu | bauzas: thanks | 12:40 |
alex_xu | bauzas: I'm just thinking the author is from my team, I can notice him the time, but you will respin the new change, so it's fine now | 12:41 |
bauzas | alex_xu: yeah, it seems he's Chinese, right? | 12:41 |
*** takedakn has quit IRC | 12:41 | |
alex_xu | bauzas: yea, at ShangHai | 12:41 |
bauzas | alex_xu: so I guess he already left | 12:41 |
*** takedakn has joined #openstack-nova | 12:42 | |
alex_xu | bauzas: yes | 12:42 |
*** leitan has joined #openstack-nova | 12:42 | |
bauzas | alex_xu: that's not a problem, I was leaving a few comments that I can fix now :) | 12:42 |
alex_xu | bauzas: yea | 12:43 |
*** takedakn has quit IRC | 12:44 | |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds spec for instance live resize https://review.openstack.org/141219 | 12:44 |
claudiub | johnthetubaguy: hi, if you have some time, can you take a look at the host capabilities spec? https://review.openstack.org/#/c/222200/ | 12:46 |
*** Daisy has quit IRC | 12:46 | |
*** sayali has quit IRC | 12:47 | |
*** Sree has quit IRC | 12:49 | |
*** Sree has joined #openstack-nova | 12:49 | |
*** takedakn has joined #openstack-nova | 12:49 | |
*** sayali has joined #openstack-nova | 12:50 | |
ankit_ag | sdague, dansmith: Hi, I have addressed your review comments on https://review.openstack.org/#/c/217579/ could you please take a look | 12:55 |
*** mdrabe has joined #openstack-nova | 12:58 | |
*** thorst1 has joined #openstack-nova | 13:00 | |
*** kiran-r has joined #openstack-nova | 13:01 | |
*** rlrossit has joined #openstack-nova | 13:02 | |
*** artom has joined #openstack-nova | 13:02 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix MetricWeigher to use MonitorMetricList https://review.openstack.org/221620 | 13:04 |
bauzas | alex_xu: johnthetubaguy: ^ please see the above respined | 13:04 |
*** deepthi has quit IRC | 13:07 | |
*** salv-orlando has quit IRC | 13:07 | |
*** haomaiwang has joined #openstack-nova | 13:08 | |
*** jecarey has joined #openstack-nova | 13:11 | |
bauzas | alex_xu: /me needs some caffeine | 13:13 |
alex_xu | bauzas: :) | 13:13 |
*** ijuwang has joined #openstack-nova | 13:15 | |
sdague | ankit_ag: done | 13:17 |
ankit_ag | sdague: thanks :) | 13:17 |
*** ctrath has joined #openstack-nova | 13:17 | |
*** sudipto has quit IRC | 13:18 | |
*** dims_ has quit IRC | 13:21 | |
*** dims_ has joined #openstack-nova | 13:21 | |
*** alexpilotti has quit IRC | 13:22 | |
*** gongysh has quit IRC | 13:22 | |
*** ankit_ag has quit IRC | 13:23 | |
*** burgerk has joined #openstack-nova | 13:24 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix MetricWeigher to use MonitorMetricList https://review.openstack.org/221620 | 13:24 |
bauzas | alex_xu: johnthetubaguy: ^ that (much better) | 13:24 |
*** otter768 has joined #openstack-nova | 13:25 | |
*** takedakn has quit IRC | 13:25 | |
*** jwcroppe has quit IRC | 13:26 | |
*** stevemar has joined #openstack-nova | 13:26 | |
*** dustins has joined #openstack-nova | 13:28 | |
*** stevemar has quit IRC | 13:29 | |
*** ildikov has quit IRC | 13:30 | |
*** otter768 has quit IRC | 13:30 | |
bauzas | johnthetubaguy: markus_z: so we have 72 new bugs, hard to tell whether we have regressions or not | 13:30 |
openstackgerrit | sahid proposed openstack/nova: libvirt: add cpu schedular priority config https://review.openstack.org/197567 | 13:30 |
openstackgerrit | sahid proposed openstack/nova: libvirt: rework membacking config to support future features https://review.openstack.org/197566 | 13:30 |
openstackgerrit | sahid proposed openstack/nova: virt: add contraint to handle realtime policy https://review.openstack.org/197568 | 13:30 |
openstackgerrit | sahid proposed openstack/nova: libvirt: add realtime support https://review.openstack.org/197569 | 13:30 |
openstackgerrit | sahid proposed openstack/nova: hardware: improve parse_cpu_spec to handle exclusion range https://review.openstack.org/226783 | 13:30 |
bauzas | markus_z: I can see that you made huge work by triaging those and provide a tag | 13:31 |
mriedem | bauzas: many of those 'new' bugs are old bugs | 13:31 |
mriedem | i was going through some yesterday | 13:31 |
bauzas | mriedem: okay, my take is trying to identify over those 72 bugs if we missed something spotted which was crucial | 13:32 |
mriedem | some are actually in progres | 13:32 |
mriedem | can we filter on when they were reported? | 13:33 |
mriedem | like in the last 2 weeks? | 13:33 |
mriedem | i guess not | 13:33 |
mriedem | thanks launchpad | 13:33 |
bauzas | lemme see | 13:33 |
sdague | mriedem: you can sort by it | 13:33 |
sdague | you have to add another field | 13:33 |
bauzas | sdague: right | 13:34 |
sdague | mriedem: it's "age" | 13:34 |
bauzas | yeah, can also see the tags now | 13:34 |
sdague | https://bugs.launchpad.net/nova/+bugs?orderby=-datecreated&start=0 | 13:34 |
mriedem | yeah i see now | 13:35 |
mriedem | thanks | 13:35 |
*** bradjones has quit IRC | 13:35 | |
mriedem | heh https://bugs.launchpad.net/nova/+bug/1484223 | 13:35 |
openstack | Launchpad bug 1484223 in OpenStack Compute (nova) " Revert "Revert "Add VIF_DELETED notification event to Nova""" [Undecided,New] | 13:35 |
*** jaypipes has joined #openstack-nova | 13:36 | |
*** Murali has quit IRC | 13:36 | |
sdague | oh, that's the docimpact auto bug flapping back our way | 13:36 |
mriedem | yeah, i closed it | 13:36 |
mriedem | it's already in the release notes | 13:37 |
*** rdopiera has quit IRC | 13:37 | |
mriedem | we could maybe put something into the nova devref about the event stuff that happens between nova and neutron | 13:37 |
mriedem | but i don't have the energy to write that up | 13:37 |
doffm | sdague: mriedem: I'm happy to help go through them. Should I tag with 'regression' or just post here if I think something is important. | 13:37 |
*** abhishekk_ has quit IRC | 13:37 | |
mriedem | doffm: just post any critical sounding things here | 13:38 |
mriedem | no vmware people around | 13:39 |
mriedem | there are some new vmware bugs | 13:39 |
sdague | https://bugs.launchpad.net/nova/+bug/1494688 - isn't really a vmware bug, this is just the general requests gorp we get all the time | 13:39 |
*** Sree has quit IRC | 13:39 | |
openstack | Launchpad bug 1494688 in oslo.vmware "[VMWARE] Connection pool is full, discarding connection" [Undecided,New] | 13:39 |
*** xyang1 has joined #openstack-nova | 13:39 | |
sdague | who's on irc that works on requests, it would be good to figure out how to handle stuff like that better | 13:40 |
mriedem | sigmavirus? | 13:40 |
mriedem | no idea really | 13:40 |
mriedem | oh works on, idk | 13:41 |
dims_ | sdague: mriedem: sigmavirus24 i think | 13:41 |
openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: use oslo_db.enginefacade https://review.openstack.org/226788 | 13:41 |
*** cfriesen__ has joined #openstack-nova | 13:41 | |
*** lsmola has quit IRC | 13:41 | |
sdague | bauzas: https://bugs.launchpad.net/nova/+bug/1495946 seems very related to your fixes around AZs a couple weeks ago | 13:41 |
openstack | Launchpad bug 1495946 in OpenStack Compute (nova) "Nova boot does not respect zone name if node is specified" [Undecided,New] - Assigned to Raga lahari (raga-lahari) | 13:41 |
sdague | mostly, you might have fixed it by accident, would be interesting to see what your take is on that one | 13:42 |
bauzas | sdague: actually, that's the force_hosts hack | 13:42 |
bauzas | sdague: there is one trick when you want to specify a destination on the boot command | 13:42 |
bauzas | sdague: then, you provide an az hint like 'az:host:node' | 13:43 |
bauzas | sdague: but that's not really an AZ check | 13:43 |
mriedem | i thought most people that used vcenter also used neutron? | 13:43 |
mriedem | https://bugs.launchpad.net/nova/+bug/1498865 | 13:43 |
openstack | Launchpad bug 1498865 in OpenStack Compute (nova) "VMware: nova-network creates a port group only on one ESXi in cluster in case with VlanManager" [Undecided,Confirmed] | 13:43 |
sdague | mriedem: we don't even support esxi any more right? | 13:43 |
bauzas | sdague: so, that's a bit unrelated to the AZ issues we discussed in the past | 13:43 |
sdague | bauzas: sure, I'm curious if given you going through that code you have a pointer on fixing that | 13:44 |
mriedem | sdague: i think they are talking about the backend esxi hosts in the cluster | 13:44 |
bauzas | sdague: see the hack here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L589-L602 | 13:44 |
sdague | because, I agree, it should reject | 13:44 |
bauzas | sdague: I really hate that one, honestly, but that's in the API contract | 13:44 |
mriedem | bauzas: is there anything that you don't hate? :P | 13:44 |
bauzas | mriedem: heh, I'm French, remember ? | 13:45 |
mriedem | oh, right | 13:45 |
bauzas | fortunately, we have a policy for that | 13:45 |
bauzas | sdague: https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L11 | 13:46 |
*** mpaolino has joined #openstack-nova | 13:46 | |
*** burt has joined #openstack-nova | 13:48 | |
doffm | https://bugs.launchpad.net/nova/+bug/1487668 | 13:48 |
openstack | Launchpad bug 1487668 in OpenStack Compute (nova) "nova evacuate doesn't work with neutron dvr architecture" [Undecided,New] | 13:48 |
doffm | https://bugs.launchpad.net/nova/+bug/1492274 | 13:49 |
openstack | Launchpad bug 1492274 in OpenStack Compute (nova) "nova evacuate does not update instance's neutron port location in the DB" [Undecided,New] | 13:49 |
*** jwcroppe has joined #openstack-nova | 13:49 | |
doffm | Two bugs, both recent, could be the same issue. | 13:49 |
*** lsmola has joined #openstack-nova | 13:50 | |
*** e0ne has joined #openstack-nova | 13:51 | |
bauzas | mriedem: valid point on putting the TrustedFilter experimental | 13:51 |
bauzas | mriedem: that's barely the same level of expectation that I was wanting with asking for deprecating the filter | 13:52 |
bauzas | mriedem: so I'm fine with your proposal :) | 13:52 |
bauzas | mriedem: replying on it | 13:52 |
mriedem | bauzas: i think it's too late for deprecation in liberty, but i'd +2 a change that logs a warning saying it's experimental | 13:52 |
mriedem | cool | 13:52 |
bauzas | mriedem: yeah, I just wanted to send a bat-signal that this filter is somehow wrong, and should be rather considered as a POC | 13:53 |
bauzas | rather than something good for production | 13:53 |
mriedem | that reminds me, i was wondering if you posted to the ops list asking if anyone was using it? | 13:53 |
*** xgsa1 has quit IRC | 13:53 | |
bauzas | mriedem: nope | 13:54 |
bauzas | mriedem: but given all the Intel replies I had, I guess it's something which is quite in use | 13:54 |
*** stevemar has joined #openstack-nova | 13:55 | |
mriedem | well, that's not surprising since i think they added the filter and own that external service | 13:55 |
bauzas | yup | 13:55 |
mriedem | just yell at alex_xu whenever that filter angers you | 13:55 |
*** VW has joined #openstack-nova | 13:55 | |
*** su_zhang has joined #openstack-nova | 13:55 | |
bauzas | well, it's more a design problem as I spotted in my email | 13:56 |
*** xgsa has joined #openstack-nova | 13:56 | |
*** haomaiwang has quit IRC | 14:01 | |
sdague | mriedem: https://bugs.launchpad.net/nova/+bug/1491307 that's just a dup right? | 14:01 |
openstack | Launchpad bug 1491307 in OpenStack Security Advisory "secgroup rules doesn't work for instance immediately" [Undecided,Triaged] | 14:01 |
sdague | which you fixed | 14:01 |
*** haomaiwang has joined #openstack-nova | 14:01 | |
mriedem | yes.... | 14:01 |
bauzas | sdague: FWIW, trying to reproduce https://bugs.launchpad.net/nova/+bug/1495946 | 14:01 |
openstack | Launchpad bug 1495946 in OpenStack Compute (nova) "Nova boot does not respect zone name if node is specified" [Undecided,New] - Assigned to Raga lahari (raga-lahari) | 14:01 |
mriedem | sdague: it was fixed under bug 1484738 | 14:02 |
openstack | bug 1484738 in OpenStack Compute (nova) kilo "keyerror when refreshing instance security groups" [High,Fix committed] https://launchpad.net/bugs/1484738 - Assigned to Matt Riedemann (mriedem) | 14:02 |
mriedem | sdague: bug 1491307 is an ossa thing | 14:02 |
openstack | bug 1491307 in OpenStack Security Advisory "secgroup rules doesn't work for instance immediately" [Undecided,Triaged] https://launchpad.net/bugs/1491307 | 14:02 |
mriedem | so i think that's why they left it open? | 14:02 |
mriedem | tristanC: how to we resolve that? ^ | 14:02 |
*** jerrygb has quit IRC | 14:03 | |
*** smatzek has joined #openstack-nova | 14:07 | |
*** jerrygb has joined #openstack-nova | 14:07 | |
mriedem | jroll: can you triage this? https://bugs.launchpad.net/nova/+bug/1498756 | 14:07 |
openstack | Launchpad bug 1498756 in OpenStack Compute (nova) "Power state inconsistence after using Ironic node-set-power-state" [Undecided,New] | 14:07 |
*** takedakn has joined #openstack-nova | 14:08 | |
sdague | bauzas: https://bugs.launchpad.net/nova/+bug/1488986 can you determine what state that should end up in? | 14:08 |
openstack | Launchpad bug 1488986 in OpenStack Compute (nova) "nova scheduler for race condition" [Undecided,New] | 14:08 |
*** exploreshaifali has joined #openstack-nova | 14:08 | |
* bauzas looking | 14:08 | |
mriedem | jroll: nevermind, i know the issue | 14:09 |
*** breitz has quit IRC | 14:10 | |
*** breitz has joined #openstack-nova | 14:10 | |
bauzas | sdague: well, it's rather more a design discussion, because yes, there are some possibly some races | 14:12 |
*** obedmr has joined #openstack-nova | 14:12 | |
*** mtanino has joined #openstack-nova | 14:13 | |
*** abhijeetm has quit IRC | 14:13 | |
bauzas | sdague: I'm tempted to triage it as invalid, as we're following that in https://review.openstack.org/#/c/192260/7/doc/source/scheduler_evolution.rst,cm L141 | 14:13 |
bauzas | (and btw. the scheduler no longer calls the compute node directly) | 14:13 |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds spec for exposing host capabilities https://review.openstack.org/222200 | 14:14 |
bauzas | sdague: to be clear, I'm not seeing a clear actionable item like an usual bugfix and we already have those kind of bugs which are more improvement requests | 14:14 |
mriedem | bauzas: you can always mark as wishlist | 14:15 |
bauzas | sdague: the story could be, yes there could be some races but we do claims on the compute manager that allow retries | 14:15 |
*** fawadkhaliq has quit IRC | 14:15 | |
bauzas | mriedem: yup, I know | 14:15 |
mriedem | 'i wish there weren't races in the code' | 14:15 |
sdague | bauzas: sure, I'm fine with that | 14:15 |
sdague | Opinion / Wishlist is often good for stuff like that | 14:15 |
sdague | like yes, there are issues, this isn't really actionable | 14:16 |
bauzas | oh yeah, good point | 14:16 |
mriedem | markus_z: for things like this https://bugs.launchpad.net/nova/+bug/1498752 | 14:17 |
openstack | Launchpad bug 1498752 in OpenStack Compute (nova) "VMware VM's vmdk files can't be deleted if in different datastore than VM" [Undecided,Incomplete] | 14:17 |
jroll | mriedem: now I'm curious about this lifecycle event stuff | 14:17 |
mriedem | i mark those as incomplete if they don't say what version they are on or provide any kind of logs | 14:17 |
mriedem | jroll: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1222 | 14:18 |
mriedem | https://github.com/openstack/nova/blob/master/nova/virt/driver.py#L1275 | 14:18 |
mriedem | and emit_event below it | 14:18 |
markus_z | mriedem: Yeah, I have to be more rigid when such information is missing. | 14:18 |
mriedem | baoli_: can you triage this? https://bugs.launchpad.net/nova/+bug/1498684 | 14:20 |
openstack | Launchpad bug 1498684 in OpenStack Compute (nova) "Fail to use SR-IOV ports when VFIO is loaded" [Undecided,New] | 14:20 |
mriedem | danpb: ^ or maybe you | 14:20 |
jroll | mriedem: seems like a weird (impossible?) thing for an external service to hook into, unless I'm missing something :) | 14:21 |
mriedem | jroll: it's not impossible if the libvirt and hyper-v drivers are implementing it | 14:21 |
mriedem | the libvirt driver has a thread which listens for events from libvirt itself | 14:21 |
jroll | mmmm | 14:21 |
mriedem | which are power state changes on a guest | 14:21 |
mriedem | jroll: so i guess if i were to implement this in the ironic driver in nova, i'd be listening for events coming out of the ironic service for power state changes on a node | 14:22 |
baoli_ | mriedem: will take a look once I have a chance. | 14:22 |
jroll | could get interesting, could get weird | 14:22 |
mriedem | jroll: could get freaky? | 14:22 |
jroll | mriedem: yeah, I'm thinking rabbit notifications | 14:22 |
jroll | heh | 14:22 |
openstackgerrit | dane-fichter proposed openstack/nova-specs: Nova Support of Glance Image Signing https://review.openstack.org/188874 | 14:22 |
*** browne has joined #openstack-nova | 14:22 | |
*** rushil has joined #openstack-nova | 14:22 | |
jroll | man, I wonder if we could get rid of a bunch of polling this way | 14:23 |
*** Sree has joined #openstack-nova | 14:23 | |
danpb | jroll: bunch of polling for what ? | 14:23 |
danpb | oh i see, you mean in the ironic driver | 14:24 |
jroll | danpb: yeah, the polling for active etc | 14:24 |
*** electrocucaracha has joined #openstack-nova | 14:25 | |
*** burgerk has quit IRC | 14:26 | |
*** eharney has joined #openstack-nova | 14:26 | |
*** tonytan4ever has joined #openstack-nova | 14:28 | |
*** tpeoples has quit IRC | 14:32 | |
mriedem | andrewbogott: did you figure anything out with this? https://bugs.launchpad.net/nova/+bug/1498197 | 14:36 |
openstack | Launchpad bug 1498197 in OpenStack Compute (nova) "No longer able to delete security group rules in kilo" [Undecided,New] | 14:36 |
mriedem | andrewbogott: i don't know what to do with that one | 14:36 |
*** READ10 has joined #openstack-nova | 14:36 | |
andrewbogott | mriedem: I haven’t looked at it much since filing. I was hoping that whoever worked on the db migration would see my stack trace and immediately see the mistake :) | 14:37 |
*** jecarey has left #openstack-nova | 14:37 | |
*** takedakn has quit IRC | 14:37 | |
mriedem | andrewbogott: there isn't a db migration related to this in kilo that i'm aware of | 14:37 |
*** ildikov has joined #openstack-nova | 14:37 | |
*** atuvenie has quit IRC | 14:38 | |
andrewbogott | maybe there should be :) | 14:38 |
andrewbogott | But, I’ll dig into the code later on today | 14:38 |
*** kiran-r has quit IRC | 14:39 | |
*** salv-orlando has joined #openstack-nova | 14:40 | |
*** stevemar has quit IRC | 14:40 | |
bauzas | sdague: FYI, providing force_hosts means that you don't run filters, ie. you don't verify your AZ https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L516-L518 | 14:41 |
bauzas | sdague: because AZFilter isn't called | 14:41 |
bauzas | sdague: so bug 1495946 is rather a Wishlist | 14:42 |
openstack | bug 1495946 in OpenStack Compute (nova) "Nova boot does not respect zone name if node is specified" [Undecided,New] https://launchpad.net/bugs/1495946 - Assigned to Raga lahari (raga-lahari) | 14:42 |
sdague | bauzas: ok, it does seem weird that the use behavior is to let it through | 14:42 |
sdague | yeh, confirmed / wishlist, I'm fine with that | 14:42 |
bauzas | sdague: yup, agreed, that's something I would like to change | 14:42 |
sdague | well, if you want to change it, I'd set it to low instead of wishlist | 14:42 |
bauzas | sdague: following the same patterns as for http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/check-destination-on-migrations.html | 14:42 |
sdague | wishlist is kind of "would be nice, doubt we'll ever get to it" | 14:43 |
bauzas | sdague: well, it would require a spec etc. | 14:43 |
*** r-daneel has joined #openstack-nova | 14:43 | |
bauzas | sdague: and technically, it's a behavioural change, not a bug | 14:43 |
*** aarefiev has quit IRC | 14:43 | |
mriedem | andrewbogott: do you have the full strack trace rather than just the tail? | 14:44 |
andrewbogott | mriedem: yep, it’s here: https://phabricator.wikimedia.org/T112492 | 14:44 |
*** exploreshaifali has quit IRC | 14:44 | |
andrewbogott | I can copy/paste if you think that’s useful | 14:44 |
mriedem | omg pointers | 14:45 |
*** salv-orlando has quit IRC | 14:45 | |
mriedem | nope, i have the trace now, thanks | 14:46 |
*** stevemar has joined #openstack-nova | 14:46 | |
*** haomaiwang has quit IRC | 14:47 | |
*** aarefiev has joined #openstack-nova | 14:49 | |
*** kiranr has joined #openstack-nova | 14:50 | |
mriedem | andrewbogott: can you dump that row from the security_group_rules table? | 14:50 |
andrewbogott | sure, stay tuned | 14:50 |
mriedem | it doesn't really make sense | 14:51 |
mriedem | self.deleted = self.id | 14:51 |
mriedem | both of those columns are integers | 14:51 |
*** emagana has joined #openstack-nova | 14:53 | |
andrewbogott | mriedem: is this the row you’re after? https://dpaste.de/xvff | 14:55 |
*** mahatic has joined #openstack-nova | 14:55 | |
*** claudiub has quit IRC | 14:56 | |
mriedem | looks right | 14:56 |
mriedem | those are the values you provided to the cli | 14:56 |
*** Sree has quit IRC | 14:56 | |
mriedem | id=2769 should be fine | 14:56 |
*** mahatic has left #openstack-nova | 14:56 | |
mriedem | wtf | 14:56 |
*** kiran-r has joined #openstack-nova | 14:56 | |
*** Sree has joined #openstack-nova | 14:56 | |
*** scheuran has quit IRC | 14:58 | |
*** kiran-r has quit IRC | 14:58 | |
andrewbogott | isn’t the code trying to set deleted=2769? And isn’t deleted a bool? | 14:58 |
*** tpeoples has joined #openstack-nova | 14:58 | |
*** Daisy has joined #openstack-nova | 14:58 | |
mriedem | no, deleted is an integer | 14:58 |
mriedem | b/c this is soft delete | 14:58 |
mriedem | can you dump the schema for your security_group_rules table? | 14:59 |
markus_z | AFAIK if the int column "deleted" has a value != 0 it is considered as deleted. And usually the id of the row is used for that value. | 14:59 |
andrewbogott | https://dpaste.de/F7CT | 14:59 |
mriedem | markus_z: right: https://github.com/openstack/oslo.db/blob/master/oslo_db/sqlalchemy/models.py#L144 | 14:59 |
mriedem | andrewbogott: aha | 15:00 |
andrewbogott | Elsewhere in the table I see that value as only 0, 1, and 127 | 15:00 |
mriedem | your id column is int and deleted is tinyint | 15:00 |
andrewbogott | yep | 15:00 |
mriedem | so that's the out of range issue | 15:00 |
andrewbogott | so the old code only set it to 0,1,-1 | 15:00 |
mriedem | old code being? | 15:00 |
mriedem | juno? | 15:00 |
andrewbogott | Um... | 15:00 |
andrewbogott | My install is a continuous upgrade from at least folsom, possibly cactus. | 15:01 |
andrewbogott | :) | 15:01 |
mriedem | but you're currently upgrading from juno to kilo right? | 15:01 |
andrewbogott | I’m sure this worked in icehouse, I suspect that it also worked in juno | 15:01 |
*** Sree has quit IRC | 15:01 | |
andrewbogott | (I was only running Juno for about three weeks, it’s vaguely possible that no one tried to delete a security rule during that time) | 15:02 |
mriedem | i guess i'm not aware of anything changing on the type definition for the deleted column in the last few releases | 15:02 |
*** Daisy has quit IRC | 15:02 | |
mriedem | zzzeek: rpodolyaka: ^ heard of anything like that? | 15:02 |
mriedem | if deleted was a boolean way back in the day then tinyint might make sense, but there would have had to have been a db migratoin in nova that altered that column along the way to make it an int | 15:04 |
bauzas | mriedem: isn't related to the soft-delete effort ? | 15:04 |
andrewbogott | It could be that the type changed way back but the code didn’t actually make use of the larger int type until kilo | 15:04 |
mriedem | andrewbogott: the soft delete thing like this has been around a long time | 15:04 |
andrewbogott | mriedem: meanwhile, I’ll ask my dba to change the type of that field to an int — sounds like that’ll do it. | 15:04 |
*** Daisy has joined #openstack-nova | 15:04 | |
andrewbogott | is tinyint 8 bits or 16? | 15:04 |
mriedem | even back in havana https://github.com/openstack/nova/blob/havana-eol/nova/openstack/common/db/sqlalchemy/models.py#L106 | 15:04 |
mriedem | https://dev.mysql.com/doc/refman/5.0/en/numeric-types.html | 15:04 |
*** zhangjn has quit IRC | 15:05 | |
andrewbogott | huh | 15:05 |
andrewbogott | We would have to be /very/ lucky to have our users only ever try to delete rules with id < 127 | 15:06 |
*** Piet has quit IRC | 15:06 | |
mriedem | andrewbogott: 152 was the db migration that converted the type way back when https://github.com/openstack/nova/blob/havana-eol/nova/db/sqlalchemy/migrate_repo/versions/152_change_type_of_deleted_column.py | 15:07 |
sdague | mriedem: which is folsom, right? | 15:08 |
sdague | sorry, looks like grizzly | 15:08 |
sdague | andrewbogott: honestly, I bet you just got lucky | 15:08 |
andrewbogott | It’s totally possible that my table has been broken since then. I would expect to have noticed, but... | 15:08 |
mriedem | yeah grizzly | 15:09 |
sdague | deleting sec group rules is a thing not a ton of people do | 15:09 |
*** karimb has quit IRC | 15:09 | |
andrewbogott | well, users will add security group rules to solve a failed connection — deleting rules doesn’t solve a problem other than one’s one compulsiveness | 15:09 |
andrewbogott | so, yeah | 15:09 |
*** Daisy has quit IRC | 15:09 | |
*** karimb has joined #openstack-nova | 15:09 | |
*** ZZelle has quit IRC | 15:09 | |
mriedem | alright, marked https://bugs.launchpad.net/nova/+bug/1498197 as invalid | 15:09 |
openstack | Launchpad bug 1498197 in OpenStack Compute (nova) "No longer able to delete security group rules in kilo" [Undecided,Invalid] | 15:09 |
*** Daisy has joined #openstack-nova | 15:10 | |
openstackgerrit | Daniel Berrange proposed openstack/nova-specs: Libvirt driver emulator threads placement policy https://review.openstack.org/225893 | 15:10 |
*** ZZelle has joined #openstack-nova | 15:10 | |
mriedem | andrewbogott: i can provide the address to send consulting fees etc | 15:10 |
mriedem | :) | 15:10 |
andrewbogott | This doesn’t explain why my schema didn’t get upgraded, though, does it? | 15:10 |
markus_z | mriedem: Does that mean the db migration script was missing something? | 15:10 |
*** claudiub has joined #openstack-nova | 15:10 | |
mriedem | possibly 152 was busted | 15:10 |
mriedem | 152 was rolled into 216 in havana | 15:11 |
*** Daisy has quit IRC | 15:11 | |
andrewbogott | It’s definitely moot, since I’m surely the last user with this use case :) | 15:11 |
*** Daisy has joined #openstack-nova | 15:11 | |
openstackgerrit | Daniel Berrange proposed openstack/nova-specs: Libvirt real time instances https://review.openstack.org/139688 | 15:11 |
mriedem | *216 in icehouse | 15:11 |
andrewbogott | But also probably a real bug, if anyone cares | 15:11 |
*** burgerk has joined #openstack-nova | 15:11 | |
mriedem | well, not really a bug we can fix | 15:11 |
mriedem | since this looks right in 216 https://github.com/openstack/nova/blob/icehouse-eol/nova/db/sqlalchemy/migrate_repo/versions/216_havana.py#L883 | 15:12 |
andrewbogott | ah, fair enough. | 15:12 |
andrewbogott | Anyway, the fix for me is straightforward, so I’m happy. Thank you! | 15:12 |
mriedem | \o/ | 15:13 |
*** yamahata has joined #openstack-nova | 15:13 | |
markus_z | cool cool | 15:13 |
*** Apoorva has joined #openstack-nova | 15:13 | |
*** penick has joined #openstack-nova | 15:14 | |
openstackgerrit | Alexis Lee proposed openstack/nova-specs: Add list-server-invalid-option-as-error.rst https://review.openstack.org/219755 | 15:15 |
*** amotoki has quit IRC | 15:17 | |
*** amotoki has joined #openstack-nova | 15:20 | |
johnthetubaguy | mriedem: markus_z: just reaching out to ask about RC1 blockers, have you spotted anything in the bug queue? | 15:21 |
markus_z | johnthetubaguy: Nothing which seems fishy to me | 15:22 |
mriedem | nothing new in the last 2 days that looks like a blocker | 15:22 |
johnthetubaguy | also, anyone else got anything major they spotted, generally | 15:22 |
*** salv-orlando has joined #openstack-nova | 15:22 | |
johnthetubaguy | markus_z: mriedem: cool, thanks | 15:22 |
*** salv-orlando has quit IRC | 15:22 | |
mriedem | i'm just looking at 'new' bugs as of this morning though, not things that have been confirmed or triaged in the last 2 days | 15:23 |
johnthetubaguy | looking close to RC1 then, looking at this: https://launchpad.net/nova/+milestone/liberty-rc1 | 15:23 |
*** salv-orlando has joined #openstack-nova | 15:23 | |
johnthetubaguy | cool, we need an RC2 for translations anyways, so we can always backport things | 15:23 |
*** tsekiyama has joined #openstack-nova | 15:24 | |
johnthetubaguy | So if folks agree about cutting for RC1 now, please do vote on here: https://review.openstack.org/#/c/226340/1 | 15:24 |
openstackgerrit | sahid proposed openstack/nova: libvirt: add cpu schedular priority config https://review.openstack.org/197567 | 15:25 |
openstackgerrit | sahid proposed openstack/nova: libvirt: rework membacking config to support future features https://review.openstack.org/197566 | 15:25 |
openstackgerrit | sahid proposed openstack/nova: virt: add contraint to handle realtime policy https://review.openstack.org/197568 | 15:25 |
openstackgerrit | sahid proposed openstack/nova: libvirt: add realtime support https://review.openstack.org/197569 | 15:25 |
openstackgerrit | sahid proposed openstack/nova: hardware: improve parse_cpu_spec to handle exclusion range https://review.openstack.org/226783 | 15:25 |
johnthetubaguy | once https://review.openstack.org/#/c/226340/1 lands, we hit mitaka! | 15:25 |
mriedem | https://bugs.launchpad.net/nova/+bugs?field.tag=liberty-rc-potential ? | 15:26 |
johnthetubaguy | yeah, there don't seem to be any urgent ones that can't be backported | 15:26 |
*** otter768 has joined #openstack-nova | 15:26 | |
*** tonytan4ever has quit IRC | 15:26 | |
*** n0ano has joined #openstack-nova | 15:26 | |
*** jhesketh has quit IRC | 15:26 | |
johnthetubaguy | but please do help double check that | 15:26 |
dansmith | johnthetubaguy: I'm +2 on the rc1 | 15:27 |
mriedem | i'm going through them now | 15:27 |
dansmith | I'll go through the tagged ones again too | 15:27 |
dansmith | last I checked, nothing stood out to me | 15:27 |
johnthetubaguy | dansmith: cool, thanks | 15:27 |
*** jhesketh has joined #openstack-nova | 15:27 | |
*** step6829 has joined #openstack-nova | 15:28 | |
dansmith | johnthetubaguy: er, we landed ndipanov's patch right? | 15:28 |
ndipanov | yes | 15:28 |
ndipanov | both of them | 15:28 |
dansmith | cool | 15:28 |
johnthetubaguy | cool, I thought so, yey! | 15:28 |
*** tonytan4ever has joined #openstack-nova | 15:28 | |
dansmith | there is this one: https://review.openstack.org/#/c/221620/ | 15:29 |
bauzas | johnthetubaguy: yeah, just spotted a last-minute issue with some RC1 bug, but we can cut RC1 | 15:29 |
dansmith | but it's not even passing unit tests | 15:29 |
dansmith | and it could be backported easily | 15:30 |
bauzas | dansmith: that's the one I was just referring | 15:30 |
dansmith | ah, cool | 15:30 |
*** claudiub has quit IRC | 15:30 | |
bauzas | +1 to cut RC1 now and see how to backport that one to RC2 | 15:30 |
johnthetubaguy | oh, yeah, it failed them | 15:30 |
bauzas | johnthetubaguy: yeah, a NoneType error, classic | 15:30 |
johnthetubaguy | yeah, we can backport | 15:30 |
*** Apoorva has quit IRC | 15:30 | |
johnthetubaguy | so turns out we need an RC2 for the translations anyways | 15:30 |
bauzas | so, +1'ing for mitaka opening | 15:30 |
mriedem | this is still new and marked as liberty rc potential, but there haven't been any updates https://bugs.launchpad.net/nova/+bug/1493512 | 15:31 |
openstack | Launchpad bug 1493512 in OpenStack Compute (nova) "n-cpu misreports shared storage when attempting to block migrate" [Undecided,New] | 15:31 |
*** rushil has quit IRC | 15:31 | |
mriedem | so whatever the fix is could be backported probably, just the comments about regression were troubling | 15:31 |
*** jdurgin1 has joined #openstack-nova | 15:31 | |
*** otter768 has quit IRC | 15:31 | |
johnthetubaguy | ah, yeah, regression sounds bad | 15:31 |
*** alejandrito has joined #openstack-nova | 15:33 | |
*** Piet has joined #openstack-nova | 15:34 | |
mriedem | ndipanov: dansmith: so i guess https://bugs.launchpad.net/nova/+bug/1417667 isn't liberty-rc-potential anymore right? | 15:34 |
openstack | Launchpad bug 1417667 in OpenStack Compute (nova) "migration/evacuation/rebuild/resize of instance with dedicated cpus needs to recalculate cpus on destination" [Medium,In progress] - Assigned to Nikola Đipanov (ndipanov) | 15:34 |
*** browne has quit IRC | 15:35 | |
johnthetubaguy | mriedem: oh, true, I did wonder about splitting the bug in half, and marking one half as merged | 15:35 |
dansmith | mriedem: I thought that was a pretty invasive change, so I'm not sure it was ever really rc potential | 15:36 |
*** matrohon has quit IRC | 15:36 | |
mriedem | sure, i just remove the potential tag so it's not cluterring up https://bugs.launchpad.net/nova/+bugs?field.tag=liberty-rc-potential | 15:36 |
mriedem | what are we doing with https://review.openstack.org/#/c/189544/ ? | 15:37 |
mriedem | backport the schema check once it's fixed? | 15:37 |
dansmith | we should probably mark things that are rc-potential as backport-potential unless we know they're not right? | 15:37 |
mriedem | yeah probably | 15:37 |
dansmith | mriedem: it's -W for a couple weeks, so yeah, backport if and when appropriate | 15:38 |
*** tonytan4ever has quit IRC | 15:38 | |
ndipanov | mriedem, well | 15:38 |
ndipanov | we still have the whole live migration thing | 15:38 |
mriedem | ndipanov: yeah but we arne't holding up rc for it right? | 15:38 |
ndipanov | we got the patches we wanted for rc | 15:39 |
mriedem | i tend to think that the liberty-rc-potential list should be empty when we do rc1 | 15:39 |
dansmith | mriedem: except for the scheduler thing we know we're shooting for rc2 | 15:39 |
dansmith | well, unless we target it or whatever | 15:39 |
mriedem | dansmith: that metrics thing? | 15:39 |
BobBall | mriedem: You just removed rc-potential from https://bugs.launchpad.net/nova/+bug/1496557 - Was that the bug you intended to remove? Don't see it discussed here, so just checking? :) | 15:39 |
openstack | Launchpad bug 1496557 in OpenStack Compute (nova) "xenapi boot from volume is broken after move to ImageMeta object if volume passed in" [High,Fix committed] - Assigned to Andrew Laski (alaski) | 15:39 |
dansmith | mriedem: yeah | 15:39 |
mriedem | BobBall: b/c it's fixed | 15:40 |
mriedem | BobBall: it's not rc potential if it's fixed | 15:40 |
BobBall | Ah fair enough | 15:40 |
* BobBall is clearly not understanding the branching strategy | 15:40 | |
mriedem | markus_z: i'm marking https://bugs.launchpad.net/nova/+bug/1455252 as liberty-backport-potential - since it was reported against juno i don't think it should hold up the liberty rc1 | 15:40 |
openstack | Launchpad bug 1455252 in OpenStack Compute (nova) "enabling serial console breaks live migration" [High,In progress] - Assigned to Alexander Schmidt (alexs-h) | 15:40 |
markus_z | +1 to what dan said about using "backport-potential" and removing "rc-potential". | 15:41 |
openstackgerrit | Daniel Berrange proposed openstack/nova-specs: Virtual guest device role tagging https://review.openstack.org/195662 | 15:41 |
markus_z | mriedem: yeah, john did also discover a possible issue with rolling upgrades in the implementation. | 15:41 |
johnthetubaguy | we have another tag for that, let me go dig that up | 15:41 |
*** browne has joined #openstack-nova | 15:41 | |
markus_z | +1 to what mriedem said about having an empty list for "rc-potential". | 15:42 |
johnthetubaguy | oh, I think it is liberty-backport-potential | 15:42 |
mriedem | yeah, already been marking hose | 15:42 |
markus_z | johnthetubaguy: Yes, that one sounds right ^ | 15:42 |
mriedem | *those | 15:42 |
*** cristicalin has quit IRC | 15:42 | |
johnthetubaguy | so, I think normally we use rc-potential, becuase its stuff that when fixed, might trigger an RC release | 15:42 |
mriedem | it'll be super fun when we don't have 6 month release cycles anymore | 15:42 |
mriedem | rc potential is fine after rc1 | 15:43 |
mriedem | to trigger rc2 | 15:43 |
johnthetubaguy | yeah | 15:43 |
mriedem | but rc potential o na fixed bug before rc1 is not necessary | 15:43 |
*** mdrabe has quit IRC | 15:43 | |
johnthetubaguy | oh, right, totally | 15:43 |
johnthetubaguy | all the bugs fixes before RC1 are just normal bug fixes, nothing extra required | 15:44 |
johnthetubaguy | anyways, if people are happy with cutting RC1 now, please to vote on this patch: https://review.openstack.org/#/c/226340/ | 15:45 |
johnthetubaguy | well, vote on there if you are unhappy is important too I guess | 15:45 |
*** rushil has joined #openstack-nova | 15:45 | |
mriedem | there, -1 | 15:45 |
mriedem | :) | 15:45 |
mriedem | should be done with the rc potentials in a few minutes | 15:46 |
johnthetubaguy | cools | 15:46 |
*** penick has quit IRC | 15:46 | |
openstackgerrit | Christian Berendt proposed openstack/nova: Disable the in tree EC2 API by default https://review.openstack.org/226856 | 15:46 |
*** e0ne has quit IRC | 15:47 | |
*** rushil has quit IRC | 15:47 | |
*** e0ne has joined #openstack-nova | 15:48 | |
zzzeek | mriedem: “that” == deleted used to be a tinyint ? havent seen that.. | 15:49 |
mriedem | zzzeek: pre-grizzly | 15:50 |
zzzeek | mriedem: way before my time | 15:50 |
*** su_zhang has quit IRC | 15:50 | |
*** su_zhang has joined #openstack-nova | 15:51 | |
*** pixelb has quit IRC | 15:52 | |
*** pixelb has joined #openstack-nova | 15:52 | |
dansmith | mriedem: that live migration bug has no patch right? | 15:52 |
dansmith | https://bugs.launchpad.net/nova/+bug/1276214 | 15:52 |
openstack | Launchpad bug 1276214 in OpenStack Compute (nova) "Live migration failure in API doesn't revert task_state to None" [Medium,In progress] - Assigned to Bartosz Fic (bartosz-fic) | 15:52 |
dansmith | and I expect it's not a regression | 15:52 |
mriedem | there is a patch here https://review.openstack.org/#/c/168916/ | 15:53 |
mriedem | dansmith: the bug was reported over 18 months ago, so no, not a regression | 15:53 |
dansmith | ah, I missed that | 15:54 |
dansmith | yeah | 15:54 |
dansmith | so not worth holding rc over | 15:54 |
mriedem | nope, updated the bug | 15:54 |
mriedem | just looking at https://bugs.launchpad.net/nova/+bug/1483645 | 15:54 |
openstack | Launchpad bug 1483645 in OpenStack Compute (nova) "Ephemeral disk size in volume can be bypassed when booting instance" [High,In progress] - Assigned to jichenjc (jichenjc) | 15:54 |
dansmith | okay | 15:54 |
*** sayali has quit IRC | 15:55 | |
*** su_zhang has quit IRC | 15:56 | |
mriedem | there is a patch https://review.openstack.org/#/c/213762/ it has -1 from ndipanov, and there is a workaround i think, so not an rc blocker imo | 15:56 |
*** ijw has quit IRC | 15:56 | |
*** dulek_ has joined #openstack-nova | 15:57 | |
*** dulek_ has quit IRC | 15:57 | |
*** nkrinner has quit IRC | 15:57 | |
mriedem | dansmith: johnthetubaguy: alright, +2 on https://review.openstack.org/#/c/226340/ | 15:58 |
mriedem | should i just approve? | 15:58 |
dansmith | I'd let mr. tuba guy do it, personally :) | 15:58 |
mriedem | he proposed it | 15:58 |
johnthetubaguy | I don't mind doing it | 15:58 |
dansmith | yeah, but two +2s and being the PTL, I think it's safe :) | 15:58 |
*** mdrabe has joined #openstack-nova | 15:58 | |
mriedem | outrageous | 15:59 |
mriedem | there is only one thing more outrageous | 15:59 |
mriedem | https://www.youtube.com/watch?v=YSLhrIuAZak | 15:59 |
markus_z | :D seriously, you have a list on your desktop for each occasion, right? | 16:00 |
johnthetubaguy | markus_z: I was just thinking the same thing | 16:00 |
*** alex_klimov has quit IRC | 16:02 | |
dansmith | heh | 16:03 |
markus_z | Just so that I get it, we tag the bugs again with "liberty-rc-potential" when we want to consider them for rc2? | 16:03 |
mriedem | markus_z: yeah | 16:03 |
*** Apoorva has joined #openstack-nova | 16:03 | |
*** jlanoux has quit IRC | 16:06 | |
markus_z | OK, get it. | 16:06 |
mriedem | btw, why do we have rc2 for translations now? b/c of the soft string freeze they only do translations after rc1 now? | 16:06 |
*** leeantho has joined #openstack-nova | 16:07 | |
*** sayali has joined #openstack-nova | 16:08 | |
*** karimb has quit IRC | 16:10 | |
*** atuvenie has joined #openstack-nova | 16:11 | |
Daisy | stable/liberty is created ? | 16:11 |
mriedem | Daisy: not yet | 16:18 |
mriedem | Daisy: soon https://review.openstack.org/#/c/226340/ | 16:18 |
*** dane-fichter_ has joined #openstack-nova | 16:19 | |
*** jdurgin1 has quit IRC | 16:19 | |
*** romainh has left #openstack-nova | 16:20 | |
*** whenry_ has joined #openstack-nova | 16:21 | |
*** jwcroppe has quit IRC | 16:25 | |
Daisy | mriedem: is this the last patch ? Will you merge other patches ? I wonder if I could create the branch for Liberty translations now. If you don't have string changes, I think I could. | 16:27 |
*** sahid has quit IRC | 16:27 | |
Daisy | I mean, create a branch for Liberty translation in the translation website. | 16:27 |
mriedem | Daisy: there is one bug that still needs fixing which was targeted for rc2 | 16:27 |
mriedem | i have no idea how the branching thing works in zanata | 16:28 |
mriedem | that's a johnthetubaguy or ttx question | 16:28 |
*** yassine__ has quit IRC | 16:28 | |
*** tpeoples has quit IRC | 16:29 | |
Daisy | Will you merge patches for rc1 except the one you mentioned ? Will you have patches with string changes for rc1 ? | 16:29 |
*** nic has joined #openstack-nova | 16:30 | |
Daisy | If no string changes for rc1 now, only waiting for that patch merge, I think I could make a liberty branch in translation website. johnthetubaguy, hope you could see that. | 16:30 |
clarkb | yes I think you can create it then hopefully patches to keep in sync land today | 16:30 |
clarkb | they are very close | 16:30 |
*** kiran-r has joined #openstack-nova | 16:31 | |
*** nic1 has joined #openstack-nova | 16:31 | |
Daisy | ok. then I will create a branch named as "stable-liberty" | 16:31 |
*** armax has joined #openstack-nova | 16:32 | |
johnthetubaguy | Daisy: yes, our open mitaka patch should merge soon | 16:33 |
markus_z | How much release candidate do we usually have until the final release? Just to get a feeling. | 16:34 |
*** nic has quit IRC | 16:34 | |
markus_z | *candidates | 16:35 |
johnthetubaguy | clarkb: once we have stable/liberty for Nova, does the bot start merging translations over in the stable branch, or is that a backport task? | 16:35 |
johnthetubaguy | I remember running a script manually in the past | 16:35 |
clarkb | it will start once my changes to do it merge | 16:35 |
johnthetubaguy | clarkb: sweet | 16:36 |
clarkb | I thibk we are waiting on a fix from me and branches to be cut | 16:36 |
*** tjones has joined #openstack-nova | 16:37 | |
clarkb | I plan to push that fix in a bit currently busy at with family | 16:37 |
bauzas | markus_z: that really depends on the stability, no rule of thumb | 16:38 |
johnthetubaguy | markus_z: two or three is normal-ish | 16:39 |
bauzas | markus_z: if you want to get a taste of what was there for the last releases https://launchpad.net/nova/+series | 16:39 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix MetricWeigher to use MonitorMetricList https://review.openstack.org/221620 | 16:39 |
johnthetubaguy | markus_z: I plan on cutting an RC in about 10 days for translations, it would be something very scary to make me cut an RC2 before then | 16:39 |
markus_z | bauzas: Yessss, graphs, thanks! :) | 16:39 |
bauzas | markus_z: but you won't get the list of bugfixes per RC since all of them are modified to be in the last release GA | 16:39 |
*** diogogmt has joined #openstack-nova | 16:40 | |
bauzas | johnthetubaguy: so I made a last spinoff of https://review.openstack.org/221620 | 16:40 |
bauzas | johnthetubaguy: should fix the last py27 failure | 16:40 |
bauzas | johnthetubaguy: for some reason, the CachingScheduler testcase was not catching that the metrics field was set to none | 16:40 |
markus_z | johnthetubaguy: Yes, makes sense to include the translations in RC2. | 16:42 |
markus_z | bauzas: OK, I don't think that I would need such a list. "Liberty" is still my first "real" cycle and I wanted to get a better feeling how things work. | 16:43 |
mriedem | smatzek: are you working on a fix for this? https://bugs.launchpad.net/nova/+bug/1488111 | 16:44 |
openstack | Launchpad bug 1488111 in OpenStack Compute (nova) "Boot from volumes that fail in initialize_connection are not rescheduled" [Low,Triaged] - Assigned to Samuel Matzek (smatzek) | 16:44 |
smatzek | mriedem: I plan to but have not yet started. | 16:45 |
*** eglynn has quit IRC | 16:47 | |
*** derekh has quit IRC | 16:49 | |
markus_z | mriedem: You are not happy with the additional tags I use on the bugs? | 16:49 |
mriedem | markus_z: like what? | 16:49 |
markus_z | https://bugs.launchpad.net/nova/+bug/1488111 | 16:49 |
openstack | Launchpad bug 1488111 in OpenStack Compute (nova) "Boot from volumes that fail in initialize_connection are not rescheduled" [Low,Triaged] - Assigned to Samuel Matzek (smatzek) | 16:49 |
*** rdopiera has joined #openstack-nova | 16:49 | |
mriedem | markus_z: i removed the spawn tag yeah | 16:50 |
mriedem | i'm not sure what that is meant to convey | 16:50 |
mriedem | it seems too granular | 16:50 |
mriedem | like, a spawn could fail for a ton of reasons | 16:50 |
mriedem | so i wouldn't expect there to be a sub-team that looks at spawn failures | 16:50 |
*** salv-orlando has quit IRC | 16:51 | |
*** salv-orlando has joined #openstack-nova | 16:52 | |
markus_z | It wasn't for the subteams. I wanted to start an analysis of the bugs, to see which features are impaired the most, over all layers. | 16:52 |
mriedem | still | 16:53 |
mriedem | spawn as a tag is way too generic for any useful analysis imo | 16:53 |
mriedem | i can see doing that for live-migrate or resize, things like that being useful | 16:53 |
*** aix has quit IRC | 16:53 | |
*** MarcF has quit IRC | 16:53 | |
*** unicell has joined #openstack-nova | 16:54 | |
johnthetubaguy | I do like the idea of identifying groups of bugs, like "live-migrate" and "quotas" | 16:54 |
markus_z | right, alone it does nothing. My goal was to measure the co-existence with other tags on a bug to see how the spawn in e.g. libvirt is doing. | 16:54 |
*** jwcroppe has joined #openstack-nova | 16:55 | |
*** tonytan4ever has joined #openstack-nova | 16:55 | |
*** Daisy has quit IRC | 16:55 | |
*** tpeoples has joined #openstack-nova | 16:56 | |
*** Daisy has joined #openstack-nova | 16:56 | |
markus_z | mriedem: I'll prepare something in the next weeks to show what my end-goal here is. Then we can discuss if we would benefit from that. deal? | 16:56 |
*** tonytan_brb has joined #openstack-nova | 16:56 | |
johnthetubaguy | markus_z: yeah, not sure spawn is a good place to start, I would look at something like "ceph" or "bdm" maybe? | 16:57 |
*** exploreshaifali has joined #openstack-nova | 16:57 | |
mriedem | well we have a tag 'volumes' that would cover bdms | 16:57 |
mriedem | anyway, bikeshedding | 16:57 |
*** mpaolino has quit IRC | 16:57 | |
mriedem | i wasn't aware these were part of a larger effort | 16:57 |
markus_z | I didn't communicate that good enough. | 16:57 |
johnthetubaguy | markus_z: loving the idea though, maybe a quick ML posting about "hey I am doing this bug thing"? | 16:58 |
*** rushil has joined #openstack-nova | 16:58 | |
johnthetubaguy | I know folks who have done lots of triage in the past might be able to give some pointers (thats not really me, I have to be honest!) | 16:58 |
markus_z | johnthetubaguy: Yeah, I'll do that. | 16:58 |
*** emagana has quit IRC | 16:59 | |
*** armax has quit IRC | 16:59 | |
*** salv-orl_ has joined #openstack-nova | 16:59 | |
*** tonytan4ever has quit IRC | 16:59 | |
*** penick has joined #openstack-nova | 17:00 | |
*** kiran-r has quit IRC | 17:00 | |
*** Daisy has quit IRC | 17:00 | |
*** lucasagomes is now known as lucas-dinner | 17:00 | |
markus_z | johnthetubaguy: Sure, I welcome every input I can get. | 17:00 |
*** tonytan_brb has quit IRC | 17:01 | |
*** salv-orlando has quit IRC | 17:01 | |
*** salv-orlando has joined #openstack-nova | 17:02 | |
*** su_zhang has joined #openstack-nova | 17:03 | |
*** salv-orl_ has quit IRC | 17:04 | |
*** nic1 has quit IRC | 17:06 | |
johnthetubaguy | markus_z: I guess I am thinking sdague might have some good ideas around that | 17:06 |
*** electrocucaracha has quit IRC | 17:06 | |
markus_z | johnthetubaguy: He made a very good post on the ML on that last September: http://lists.openstack.org/pipermail/openstack-dev/2014-September/046517.html | 17:07 |
markus_z | johnthetubaguy: If it is that what you mean. | 17:08 |
johnthetubaguy | ah, yeah, that must be what I was remembering! | 17:08 |
markus_z | I have read it but need to find time to discuss my implementation ideas for that. I have a brain dump text file which contains some things but that is not consumable by others... | 17:09 |
johnthetubaguy | totally makes sense | 17:09 |
*** achanda has joined #openstack-nova | 17:10 | |
*** kiranr has quit IRC | 17:12 | |
*** VW has quit IRC | 17:12 | |
mriedem | markus_z: here is a good example of tags i think https://bugs.launchpad.net/nova/+bug/1496235 | 17:13 |
openstack | Launchpad bug 1496235 in OpenStack Compute (nova) "Boot from volume faild with availability_zone option, in case of cinder do not have availability_zone" [Undecided,New] - Assigned to Ikuo Kumagai (i-kumagai) | 17:13 |
mriedem | with availability-zones | 17:13 |
mriedem | markus_z: since AZs are the special wrinkle there | 17:13 |
*** VW has joined #openstack-nova | 17:13 | |
*** tpeoples has quit IRC | 17:14 | |
*** ociuhandu has quit IRC | 17:15 | |
markus_z | mriedem: agreed | 17:15 |
*** ijw has joined #openstack-nova | 17:15 | |
*** e0ne has quit IRC | 17:16 | |
*** danpb has quit IRC | 17:16 | |
*** VW has quit IRC | 17:17 | |
*** nic has joined #openstack-nova | 17:17 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Make consolidate-console-api match impl https://review.openstack.org/199033 | 17:19 |
*** ijw has quit IRC | 17:20 | |
*** su_zhang_ has joined #openstack-nova | 17:21 | |
*** su_zhang has quit IRC | 17:21 | |
*** raildo is now known as raildo-afk | 17:25 | |
dansmith | ls | 17:25 |
dansmith | oops | 17:25 |
*** smatzek has quit IRC | 17:26 | |
*** ijw has joined #openstack-nova | 17:26 | |
*** smatzek has joined #openstack-nova | 17:26 | |
*** ankit has joined #openstack-nova | 17:27 | |
melwitt | alaski: did you see my question on https://review.openstack.org/#/c/176925/ ? just wanted to confirm whether the changes to the fake instances get function are still needed | 17:27 |
*** otter768 has joined #openstack-nova | 17:27 | |
ankit | hi all, Can someone please review https://review.openstack.org/#/c/185549/ | 17:28 |
*** yamahata has quit IRC | 17:29 | |
*** otter768 has quit IRC | 17:31 | |
*** ociuhandu has joined #openstack-nova | 17:32 | |
mriedem | bauzas: this is a fun AZ one https://bugs.launchpad.net/nova/+bug/1496235 | 17:33 |
openstack | Launchpad bug 1496235 in OpenStack Compute (nova) "Boot from volume faild with availability_zone option, in case of cinder do not have availability_zone" [Undecided,New] - Assigned to Ikuo Kumagai (i-kumagai) | 17:33 |
mriedem | bauzas: it's basically invalid | 17:33 |
*** rajesht has joined #openstack-nova | 17:33 | |
mriedem | if you try to boot from volume with source=image into a nova az and that same az isn't in cinder, the volume create fails | 17:34 |
alaski | melwitt: saw your question. I don't remember that code well enough to answer, let me test real quick | 17:35 |
*** su_zhang_ has quit IRC | 17:35 | |
*** su_zhang has joined #openstack-nova | 17:35 | |
*** atuvenie has quit IRC | 17:39 | |
*** hitalia has joined #openstack-nova | 17:41 | |
*** raildo-afk is now known as raildo | 17:41 | |
rajesht | hi cores, need one more +2 on code cleanup https://review.openstack.org/#/c/220142/ | 17:42 |
rajesht | https://review.openstack.org/#/c/221271/ | 17:42 |
*** vilobhmm11 has joined #openstack-nova | 17:43 | |
*** sudipto has joined #openstack-nova | 17:44 | |
*** emagana has joined #openstack-nova | 17:45 | |
*** rajesht has quit IRC | 17:45 | |
*** markus_z has quit IRC | 17:45 | |
*** mriedem has quit IRC | 17:46 | |
*** garyk has joined #openstack-nova | 17:47 | |
alaski | melwitt: removing the marker edition cause the test runner to use all memory and crash something on my system forcing me to reboot. So I think it's still necessary :) | 17:48 |
alaski | s/edition/addition/ | 17:48 |
*** rlrossit has quit IRC | 17:48 | |
melwitt | alaski: oh, sorry :( | 17:49 |
*** mriedem has joined #openstack-nova | 17:49 | |
*** ankit has quit IRC | 17:50 | |
alaski | melwitt: heh, no worries. it's my fault for tabbing away while the test ran and then I couldn't get back to it to kill it | 17:51 |
*** atuvenie has joined #openstack-nova | 17:51 | |
alaski | melwitt: I can fix the list comprehension if you'd like | 17:52 |
*** VW has joined #openstack-nova | 17:52 | |
*** rlrossit has joined #openstack-nova | 17:53 | |
*** yamahata has joined #openstack-nova | 17:54 | |
melwitt | alaski: nah, that's okay. just wrote it in the case of a respin. I didn't get how the existing test (not your new one) ends up with marker being not None (looking again) | 17:54 |
*** kiran-r has joined #openstack-nova | 17:54 | |
*** mriedem1 has joined #openstack-nova | 17:54 | |
*** mriedem has quit IRC | 17:54 | |
mriedem1 | doffm: https://blueprints.launchpad.net/nova/+spec/refresh-quotas-usage | 17:55 |
*** sudipto has quit IRC | 17:55 | |
*** e0ne has joined #openstack-nova | 17:56 | |
alaski | melwitt: the first time _get_paginated_instances is run it will pass marker=None to that method. After that marker is set to the uuid of the last instance returned | 17:57 |
melwitt | alaski: I guess with pagination, there's no chance you could get all of the instances in one call? it has to be two minimum? I was thinking of when you get all the first time | 17:57 |
*** krtaylor has quit IRC | 17:58 | |
alaski | melwitt: you could get them all in one call, but we don't know if that's the case or not | 17:58 |
melwitt | alaski: meaning... there's nothing that can tell you that you got them all, the first time | 17:58 |
melwitt | okay | 17:58 |
*** diogogmt has quit IRC | 17:58 | |
alaski | melwitt: nothing I can think of. If the results are less than limit we would know, but if we get exactly limit results we won't know | 17:59 |
melwitt | alaski: right.. | 17:59 |
*** diogogmt has joined #openstack-nova | 17:59 | |
dansmith | johnthetubaguy: presume you want the /topic changed once the mitaka patch lands? | 18:00 |
*** electrocucaracha has joined #openstack-nova | 18:01 | |
*** hemna is now known as hemnafk | 18:01 | |
*** andrewbogott has left #openstack-nova | 18:05 | |
*** e0ne has quit IRC | 18:06 | |
doffm | mriedem1: ta. | 18:07 |
*** jwcroppe has quit IRC | 18:08 | |
*** ijuwang has quit IRC | 18:08 | |
*** mriedem1 has quit IRC | 18:10 | |
*** READ10 has quit IRC | 18:10 | |
*** su_zhang has quit IRC | 18:11 | |
*** su_zhang has joined #openstack-nova | 18:11 | |
*** krtaylor has joined #openstack-nova | 18:11 | |
ndipanov | dansmith, got a nice little fix for you to -2^H^H review | 18:11 |
dansmith | well, now I'm in a good mood.. hit me | 18:12 |
*** e0ne has joined #openstack-nova | 18:13 | |
*** annegentle has joined #openstack-nova | 18:13 | |
*** mriedem has joined #openstack-nova | 18:13 | |
*** zhangjn has joined #openstack-nova | 18:14 | |
*** rushil has quit IRC | 18:15 | |
*** ctrath has quit IRC | 18:18 | |
*** zhangjn has quit IRC | 18:19 | |
*** whenry_ has quit IRC | 18:20 | |
*** gszasz has quit IRC | 18:20 | |
*** ctrath has joined #openstack-nova | 18:21 | |
bauzas | mriedem: okay, I'll look at the bug | 18:22 |
*** rushil has joined #openstack-nova | 18:22 | |
*** rushil has quit IRC | 18:23 | |
*** achanda has quit IRC | 18:26 | |
mriedem | bauzas: i posted to the ML on it also | 18:26 |
mriedem | not a priority though | 18:26 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: rebuild: Apply migration context before calling the driver https://review.openstack.org/226923 | 18:27 |
ndipanov | dansmith, ^ | 18:27 |
*** dims__ has joined #openstack-nova | 18:28 | |
*** vishwanathj has joined #openstack-nova | 18:29 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Re-propose spec for refresh-quotas-usage https://review.openstack.org/226925 | 18:30 |
*** achanda has joined #openstack-nova | 18:30 | |
bauzas | mriedem: ack | 18:30 |
*** jichen has joined #openstack-nova | 18:31 | |
mriedem | rlrossit: something to work on if you're looking for work https://review.openstack.org/#/c/226925/ | 18:31 |
*** dims_ has quit IRC | 18:31 | |
openstackgerrit | Mark Doffman proposed openstack/nova: DO NOT MERGE: Extra logging for lxc teardown fail. https://review.openstack.org/223125 | 18:32 |
bauzas | mriedem: so, MHO is that I wonder why Nova and Cinder would need to have the same AZs | 18:32 |
kashyap | make -j8 | 18:32 |
kashyap | Ugh, sorry | 18:32 |
*** aix has joined #openstack-nova | 18:33 | |
*** kiran-r has quit IRC | 18:33 | |
ndipanov | bah pep8 ! | 18:33 |
* bauzas needs to dig into cinder code... | 18:34 | |
*** rushil has joined #openstack-nova | 18:35 | |
openstackgerrit | Nikola Dipanov proposed openstack/nova: rebuild: Apply migration context before calling the driver https://review.openstack.org/226923 | 18:35 |
*** amotoki has quit IRC | 18:35 | |
kashyap | mriedem: Hi, did you get to the end of -- https://review.openstack.org/#/c/218355/ | 18:37 |
mriedem | kashyap: nope | 18:37 |
mriedem | i've lost hope on that one | 18:37 |
mriedem | at least until kilo-eol happens | 18:37 |
kashyap | Sorry, I couldn't spend time to understand the issue more, I'm buried with a few things for a couple of weeks. | 18:38 |
mriedem | i also couldn't reproduce it manually using novaclient in stable/kilo devstack | 18:38 |
mriedem | that's fine | 18:38 |
mriedem | at this point it's passing for liberty, even though i didn't change anything | 18:38 |
mriedem | but the initial volume attach in juno and kilo doesn't result in a partition showing up on the guest | 18:38 |
mriedem | for whatever magical crazy reason | 18:38 |
*** diogogmt has quit IRC | 18:39 | |
vilobhmm11 | alaski : ping | 18:40 |
alaski | vilobhmm11: hi | 18:40 |
vilobhmm11 | regarding https://review.openstack.org/#/c/222744/ | 18:41 |
kashyap | mriedem: Okay. I've subscribed to it, and will keep an eye on it. | 18:41 |
vilobhmm11 | alaski : was thinking of modeling quota by az and quota by az (and flavor) in the way flavor classes will be modelled… | 18:43 |
*** redbeard has quit IRC | 18:43 | |
vilobhmm11 | so how will the flavor classes have the information about the AZ ? | 18:44 |
vilobhmm11 | or is there a plan to introduce something like az groups or something ? | 18:44 |
alaski | vilobhmm11: flavors/flavor classes don't have knowledge of AZs | 18:45 |
alaski | I'm not sure why you need quotas on AZs though, rather than on flavors. If you set it up so that only flavor X (or flavor class X) can build in AZs 1 & 2 then you can apply quotas to the flavors rather than the AZ | 18:46 |
mriedem | dansmith: alaski: what do we do about moving liberty specs from approved to implemented in the nova-specs repo? just needs someone to go and do that? | 18:46 |
vilobhmm11 | alaski : that was my gut feeling | 18:47 |
alaski | mriedem: that's my recollection of what happened last time | 18:47 |
mriedem | alaski: ok | 18:47 |
bauzas | what happens with AZs suddently with all these bugs and discussions coming in ? o_O | 18:48 |
mriedem | bauzas: people love AZs now | 18:48 |
Piet | The OpenStack UX project team would like to invite you to participate in a survey to help drive product direction for the Neutron project. The survey should take no more than fifteen minutes to complete. | 18:48 |
Piet | We are specifically looking for folks with the following roles; Architect, Cloud Operator, System Administrator, Network Architect / Engineer / Administrator, Developer, or IT Manager / Director. In addition, they should be familiar with either Nova Networks and/or Neutron. | 18:48 |
Piet | You will also be entered in a raffle for one of two $100 US Amazon gift cards at the end of the survey. As always, the results from the survey will be shared with the OpenStack community. | 18:48 |
Piet | Please click on the following link to begin the survey. | 18:48 |
Piet | https://www.surveymonkey.com/r/osnetworking | 18:48 |
*** e0ne has quit IRC | 18:49 | |
alaski | we should record each cycles "hip new thing" somewhere. first NFV then containers now AZs | 18:50 |
*** su_zhang has quit IRC | 18:51 | |
*** penick has quit IRC | 18:51 | |
bauzas | alaski: like Spiderman, let's do a reboot for AZs ? | 18:52 |
bauzas | that's nolanesque | 18:53 |
bauzas | uh, nolanish rather | 18:53 |
mriedem | omg we got volume bugs under 100 https://bugs.launchpad.net/nova/+bugs?field.tag=volumes&orderby=status&start=0 | 18:55 |
*** lpetrut has quit IRC | 18:57 | |
*** rdopiera has quit IRC | 18:57 | |
*** diogogmt has joined #openstack-nova | 18:58 | |
openstackgerrit | Andrew Laski proposed openstack/nova: Neutron: update cells when saving info_cache https://review.openstack.org/226941 | 19:00 |
*** diogogmt_ has joined #openstack-nova | 19:02 | |
*** alex_klimov has joined #openstack-nova | 19:03 | |
*** diogogmt has quit IRC | 19:04 | |
*** diogogmt_ is now known as diogogmt | 19:04 | |
*** penick has joined #openstack-nova | 19:06 | |
*** Marga_ has quit IRC | 19:06 | |
*** Marga_ has joined #openstack-nova | 19:08 | |
*** Piet has quit IRC | 19:08 | |
*** e0ne has joined #openstack-nova | 19:12 | |
*** Marga_ has quit IRC | 19:13 | |
*** rushil has quit IRC | 19:13 | |
*** hitalia has quit IRC | 19:14 | |
*** exploreshaifali has quit IRC | 19:15 | |
*** rushil has joined #openstack-nova | 19:18 | |
*** hodos has joined #openstack-nova | 19:19 | |
dansmith | ndipanov: so I know the answer is probably "no", but would it be safer to stash/restore locally the current numa_topology instead of restoring the old one from the context? | 19:19 |
*** garyk has quit IRC | 19:20 | |
dansmith | ndipanov: like, I think that the assumption your context manager implies is that it will set the new one temporarily and then revert | 19:20 |
ndipanov | dansmith, well yes | 19:20 |
dansmith | ndipanov: but if the caller has instance.numa_topology set to something crazy, this will not restore that when the context manager exits | 19:20 |
ndipanov | hmmm | 19:20 |
ndipanov | what do you mean? | 19:20 |
dansmith | it would be trivial to just store that in local scope and restore right? | 19:20 |
dansmith | ndipanov: let's say I have set instance.numa_topology = None just before making this call with the mutated context manager | 19:21 |
dansmith | when it exits, it will set instance.numa_topology = migration_context.old_, not what I had it before | 19:21 |
dansmith | so I could go from something -> new -> old | 19:22 |
*** tonytan4ever has joined #openstack-nova | 19:22 | |
dansmith | when I think the caller has the expectation that you will *restore* it | 19:22 |
ndipanov | dansmith, yar! | 19:23 |
ndipanov | -1 it and I'll fix it tomorrow | 19:23 |
dansmith | okay | 19:23 |
dansmith | http://pastebin.com/DH4bnqhz | 19:23 |
dansmith | is what I mean | 19:23 |
ndipanov | yes | 19:24 |
ndipanov | I will just copy that :) | 19:24 |
*** dims__ has quit IRC | 19:24 | |
*** e0ne has quit IRC | 19:24 | |
*** dims_ has joined #openstack-nova | 19:25 | |
*** garyk has joined #openstack-nova | 19:25 | |
*** mriedem has quit IRC | 19:26 | |
dansmith | cool | 19:26 |
*** dane-fichter_ has quit IRC | 19:26 | |
*** otter768 has joined #openstack-nova | 19:28 | |
*** whenry_ has joined #openstack-nova | 19:28 | |
*** salv-orlando has quit IRC | 19:29 | |
openstackgerrit | Mark Doffman proposed openstack/nova: libvirt: Don't attempt to convert initrd images. https://review.openstack.org/225289 | 19:29 |
*** salv-orlando has joined #openstack-nova | 19:29 | |
*** penick has quit IRC | 19:31 | |
*** annegentle has quit IRC | 19:32 | |
*** mriedem has joined #openstack-nova | 19:32 | |
*** annegentle has joined #openstack-nova | 19:32 | |
*** hemnafk is now known as hemna | 19:33 | |
*** otter768 has quit IRC | 19:33 | |
*** derekh has joined #openstack-nova | 19:36 | |
*** garyk has quit IRC | 19:36 | |
*** Marga_ has joined #openstack-nova | 19:37 | |
*** penick has joined #openstack-nova | 19:41 | |
*** achanda has quit IRC | 19:41 | |
*** garyk has joined #openstack-nova | 19:41 | |
*** artom has quit IRC | 19:42 | |
*** mriedem has quit IRC | 19:43 | |
*** mriedem has joined #openstack-nova | 19:44 | |
*** ccarmack has quit IRC | 19:46 | |
*** ccarmack has joined #openstack-nova | 19:47 | |
*** Piet has joined #openstack-nova | 19:48 | |
*** su_zhang has joined #openstack-nova | 19:48 | |
*** Marga_ has quit IRC | 19:50 | |
*** matrohon has joined #openstack-nova | 19:51 | |
*** rlrossit1 has joined #openstack-nova | 19:52 | |
*** rlrossit has quit IRC | 19:52 | |
*** mwagner_lap has quit IRC | 19:54 | |
*** VW has quit IRC | 19:55 | |
*** VW has joined #openstack-nova | 19:55 | |
*** rlrossit1 has quit IRC | 19:57 | |
*** ctrath has quit IRC | 19:57 | |
*** ccarmack has quit IRC | 19:57 | |
openstackgerrit | Samuel Cassiba proposed openstack/nova-specs: Add support for FreeBSD as a compute node https://review.openstack.org/224998 | 19:59 |
*** VW has quit IRC | 19:59 | |
*** hodos|2 has joined #openstack-nova | 20:00 | |
*** ctrath has joined #openstack-nova | 20:00 | |
*** rlrossit has joined #openstack-nova | 20:01 | |
*** ccarmack has joined #openstack-nova | 20:01 | |
*** salv-orlando has quit IRC | 20:01 | |
*** fawadkhaliq has joined #openstack-nova | 20:02 | |
*** hodos has quit IRC | 20:02 | |
*** fawadkhaliq has quit IRC | 20:05 | |
*** garyk has quit IRC | 20:06 | |
openstackgerrit | Mark Doffman proposed openstack/nova: libvirt: Don't attempt to convert initrd images. https://review.openstack.org/225289 | 20:06 |
*** jwcroppe has joined #openstack-nova | 20:08 | |
*** stevemar has quit IRC | 20:11 | |
*** stevemar has joined #openstack-nova | 20:11 | |
*** angdraug has joined #openstack-nova | 20:13 | |
*** stevemar has quit IRC | 20:13 | |
*** salv-orlando has joined #openstack-nova | 20:14 | |
*** derekh has quit IRC | 20:16 | |
*** rushil has quit IRC | 20:17 | |
*** whenry_ has quit IRC | 20:18 | |
*** smatzek has quit IRC | 20:19 | |
*** hitalia has joined #openstack-nova | 20:21 | |
*** VW has joined #openstack-nova | 20:24 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Deprecate cinder.cross_az_attach option https://review.openstack.org/226977 | 20:25 |
mriedem | jgriffith: bauzas: alaski: scottda: ^ | 20:25 |
bauzas | mriedem: \o/ | 20:25 |
bauzas | jaypipes: ^ too | 20:25 |
scottda | mriedem: I am trying to figure out with hemna how to introduce returning VolumeIsBusy exception to Nova.... | 20:26 |
bauzas | jaypipes: long story here, we tried to understand why AZs neded to be provided to Cinder | 20:26 |
scottda | We're thinking with api-microversions, a new cinderclient would make a new call that gets the exception... | 20:26 |
scottda | but I see that Nova does not upper-pin the cinderclient. | 20:26 |
hemna | haven't exactly wrapped my head around dealing with new cinderclient and cinder that has support for new microversion, with an old nova, say L. | 20:26 |
mriedem | scottda: novaclient allows the caller to specify the microversion | 20:27 |
mriedem | and defaults to latest | 20:27 |
mriedem | scottda: so nova would just have a minimum required cinderclient, and then opt into microversions as needed | 20:27 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Set TrustedFilter as experimental https://review.openstack.org/194592 | 20:27 |
hemna | so the caller of cinderclient is forced to know the microversion # and ask for it. | 20:28 |
scottda | So, the novaclient microversion would take care of insuring that we used the minimum required cinderclient? sounds ok | 20:28 |
hemna | not cinderclient itself ? | 20:28 |
mriedem | sorry, on a call | 20:29 |
scottda | ok, we can figure this out later | 20:29 |
dansmith | hemna: I think the point is, you can tell novaclient that you're making a call foo() and that the microversion should be X, if you want to depend on certain semantics that you only get running >=X | 20:30 |
hemna | dansmith, so in our case, when nova calls cinderclient to take some action, it passes in a newer microversion that it knows about? | 20:31 |
hemna | then I presume cinderclient has to tell the caller it got X version | 20:31 |
dansmith | hemna: I wasn't following the full discussion, so I'm not sure exactly what you were shooting for | 20:31 |
hemna | np | 20:31 |
dansmith | hemna: are you trying to determine if you're running with a new-enough nova? | 20:31 |
bauzas | mriedem: jaypipes: johnthetubaguy: so I made the TrustedFilter experimental https://review.openstack.org/226977 | 20:31 |
scottda | dansmith: no | 20:31 |
hemna | we are looking to add microversioning support to cinder | 20:32 |
hemna | and hence cinderclient | 20:32 |
scottda | We are going to implement microversions in Cinder | 20:32 |
dansmith | oh, heh | 20:32 |
hemna | and one of the new microversions will return a new exception that nova doesn't know about currently | 20:32 |
scottda | This will allow us to change the cinder <-> nova API, we hope | 20:32 |
scottda | in a safe way | 20:32 |
dansmith | okay, so the way nova handles this, | 20:32 |
hemna | so we are trying to figure out how to deal with an L nova talking to an M cinder and not break :) | 20:32 |
dansmith | is that if the caller didn't tell the client that it knows about the microversion where that new exception was added, the client behaves like it did originally | 20:33 |
scottda | ok, that is good. And easy for a caller of cinder api going directly to cinder api... | 20:33 |
dansmith | like, we assume "no version" means "the behavior before the versioning was added" | 20:33 |
scottda | But what about, say 'nova volume-attach', which uses cinderclient, which then calls cinder api? | 20:33 |
hemna | I guess I need to look at how the caller of novaclient asks for a version | 20:33 |
dansmith | scottda: so, to clarify there are two cases: the client as a python client and the client as a CLI | 20:34 |
hemna | dansmith, I think we are concerned with the python client | 20:34 |
dansmith | scottda: I believe that as a python client, it acts as if "no version" means "original version" and if operating as a CLI, then "no version" means "the latest version" | 20:34 |
dansmith | right | 20:35 |
dansmith | so in that case, | 20:35 |
*** tjones1 has joined #openstack-nova | 20:35 | |
dansmith | L nova would be not specifying the version, | 20:35 |
mriedem | hemna: e.g. https://review.openstack.org/#/c/190346/ | 20:35 |
dansmith | which means cinderclient needs to behave like it does today | 20:35 |
mriedem | yeah, what dan said | 20:36 |
mriedem | then eventually nova starts asking for a certain cinder microversion | 20:36 |
hemna | ok | 20:36 |
dansmith | yeah, once we're new enough to pass the version, then cinderclient will start exposing the new behavior | 20:36 |
hemna | I'm starting to poke around in the novaclient code to see where there is an api for asking for a specific version of a call | 20:36 |
hemna | gah...have to run to a meeting.....bbl | 20:37 |
*** hemna is now known as hemnafk | 20:37 | |
*** tjones1 has left #openstack-nova | 20:37 | |
scottda | OK, I'll have to write some code and go from there. Thanks. | 20:40 |
*** Apoorva has quit IRC | 20:41 | |
*** achanda has joined #openstack-nova | 20:42 | |
*** step6829 has quit IRC | 20:43 | |
*** su_zhang_ has joined #openstack-nova | 20:45 | |
*** su_zhang has quit IRC | 20:45 | |
*** hemnafk is now known as hemna | 20:45 | |
*** su_zhang_ has quit IRC | 20:45 | |
*** IanGovett has quit IRC | 20:45 | |
*** salv-orlando has quit IRC | 20:45 | |
*** su_zhang has joined #openstack-nova | 20:45 | |
*** lucas-dinner has quit IRC | 20:46 | |
*** achanda has quit IRC | 20:47 | |
hemna | https://github.com/openstack/python-novaclient/blob/master/novaclient/api_versions.py | 20:47 |
hemna | so I guess that's the magic sauce then | 20:47 |
mriedem | hemna: yeah, see how https://review.openstack.org/#/c/190346/ does that | 20:48 |
mriedem | that change is pretty straight forward | 20:48 |
hemna | yah | 20:48 |
hemna | the caveat there though is the caller doesn't know which one he got ? | 20:48 |
hemna | I guess it doesn't really matter | 20:49 |
*** achanda has joined #openstack-nova | 20:49 | |
melwitt | this is the novaclient doc showing how to request the version when you create the client object http://docs.openstack.org/developer/python-novaclient/api.html | 20:49 |
hemna | melwitt, great thank you | 20:49 |
bauzas | melwitt: hah, I love the warning there | 20:49 |
bauzas | 'could shoot you in the foot" | 20:50 |
*** su_zhang has quit IRC | 20:50 | |
scottda | But the novaclient will request an API microversion from nova API. Doesn't that mean the Nova API will need a new function that will call something new via the cinderclient to make all of this work? | 20:50 |
*** dustins has quit IRC | 20:50 | |
melwitt | bauzas: heh, yeah | 20:51 |
*** su_zhang has joined #openstack-nova | 20:51 | |
*** garyk has joined #openstack-nova | 20:52 | |
dansmith | scottda: the rest api you mean? | 20:52 |
dansmith | oh, no | 20:52 |
*** raildo is now known as raildo-afk | 20:53 | |
dansmith | nova-api will need to change to request a specific version when it makes some cinderclient call, correct | 20:53 |
hemna | dansmith, well it will just ask for a version when it instantiates the cinderclient object no ? | 20:53 |
melwitt | yeah, I was thinking, do we add a microversion to the nova api that will request the new version from cinder I guess? | 20:54 |
mriedem | alaski: it's a good thing you approved that change already :) http://lists.openstack.org/pipermail/openstack-operators/2015-September/008224.html | 20:54 |
dansmith | hemna: it's usually per-call I think | 20:54 |
dansmith | melwitt: only if we have to change our behavior, right? | 20:54 |
dansmith | so it depends | 20:54 |
scottda | OK. So this is a bit complex because to the way calls to cinder for volume attach are intertwined with the nova manage code...Has this been done before for any other microversion change. | 20:54 |
hemna | dansmith, but this http://docs.openstack.org/developer/python-novaclient/api.html looks like it's per object | 20:54 |
hemna | ? | 20:54 |
melwitt | dansmith: hm, yeah | 20:54 |
*** d34dh0r53 has quit IRC | 20:55 | |
dansmith | hemna: yeah, it does | 20:55 |
*** achanda has quit IRC | 20:55 | |
*** d34dh0r53 has joined #openstack-nova | 20:55 | |
alaski | mriedem: heh. it's a harmless revert if necessary | 20:55 |
dansmith | hemna: I defer to melwitt who *actually* knows, as opposed to myself who just knows someone that knows :) | 20:55 |
*** achanda has joined #openstack-nova | 20:55 | |
alaski | mriedem: I can remove the +W if you want | 20:55 |
bauzas | mriedem: alaski: that's a deprecation, not exactly a removal :) | 20:56 |
dansmith | hemna: my point is, I think, that you might make one call at 2.1 and another at 2.3 | 20:56 |
mriedem | alaski: i know, just thought it was funny | 20:56 |
melwitt | hemna: yeah, it's per object, the version specified | 20:56 |
mriedem | 'feel free to comment, oh, nvm, f you!' | 20:56 |
hemna | ok, that makes sense | 20:57 |
alaski | mriedem: heh, I was so hyped I missed that thread | 20:57 |
hemna | so I suppose after cinder microversions land and the updated version that has VolumeIsBusy, we'll patch nova to ask for the new microversion | 20:57 |
mriedem | i'm glad i could help hype you up today | 20:57 |
hemna | and add all the try: blocks around the calls to cinderclient and deal w/ the new exception | 20:57 |
dansmith | hemna: yar | 20:57 |
hemna | the old nova won't ask the new cinderclient for the newer microversion and cinderclient/cinder has to behave as it does today | 20:58 |
bauzas | so I wonder how AZs can be helpful with containers for NFV | 20:58 |
mriedem | bauzas: great question | 20:59 |
hemna | scottda, follow that ? | 20:59 |
*** VW has quit IRC | 20:59 | |
*** VW has joined #openstack-nova | 21:00 | |
scottda | Yes, I get it at the conceptual level. I just ported the nova microversion code to cinder, so I get all that.... | 21:00 |
hemna | ok coolio | 21:00 |
hemna | I 'think' I get it | 21:00 |
hemna | heh | 21:00 |
scottda | I'm just concerned with putting the microversion wrappers in the right place in the Nova code. | 21:00 |
dansmith | scottda: meaning the try..except to handle the new exception? | 21:01 |
hemna | I don't think there is a heck of a lot to do on the nova side | 21:01 |
hemna | other than adding the try blocks and asking for a specific version when creating cinderclient object | 21:01 |
scottda | dansmith: yeah. Should those go in the manager.py code around calls to volume_api.attach()? | 21:01 |
*** penick has quit IRC | 21:01 | |
dansmith | scottda: well, I dunno what you're specifically trying to do, but .. sounds right ;) | 21:02 |
scottda | With a microversion > x wrapper? | 21:02 |
hemna | scottda, the microversion gets set at cinderclient creation time | 21:02 |
*** VW_ has joined #openstack-nova | 21:02 | |
dansmith | scottda: no microversion comparison there, just request the one you want, and then try..except around the call | 21:02 |
hemna | so that would just go here: https://github.com/openstack/nova/blob/master/nova/volume/cinder.py#L128 | 21:03 |
scottda | Well, that's right. If there are try blocks in the new code, it won't matter if you get an old Cinder server. Duh. | 21:03 |
dansmith | alaski: doesn't all this mean that you guys will have to finally deploy a newer-than-diablo cinder? :P | 21:03 |
*** VW has quit IRC | 21:03 | |
*** penick has joined #openstack-nova | 21:04 | |
alaski | dansmith: hah. I do believe we're up to Havana now sir | 21:04 |
dansmith | alaski: ooh, fancy pants | 21:04 |
*** hemna is now known as hemnafk | 21:05 | |
scottda | Well, if you've Havana Cinder, it won't recognize the microversion header, and you'll get the old behaviour. That's the point of all this, right? | 21:06 |
alaski | but yes, we would need to move more towards the middle of the alphabet | 21:06 |
dansmith | scottda: well, not really | 21:06 |
dansmith | scottda: they will have to be on a newer cinderclient to even take the parameter | 21:06 |
*** smatzek has joined #openstack-nova | 21:06 | |
dansmith | scottda: and then cinderclient should not agree to talk to an old cinder if the caller requested at least version X | 21:06 |
dansmith | scottda: otherwise nova is saying "I NEED a version that throws this exception if this happens" | 21:07 |
dansmith | and cinderclient would be lying if it agreed but didn't deliver | 21:07 |
* dansmith likes to anthropomorphize python code | 21:07 | |
alaski | we do run up to date cinderclients | 21:08 |
dansmith | yeah, but once we roll to this, you'll be stuck i think | 21:08 |
alaski | yeah, if Nova starts requiring the use of microversions | 21:08 |
alaski | when they're in the client | 21:08 |
dansmith | "Cinder microversion 2.3: Eff-up RAX" | 21:09 |
*** penick has quit IRC | 21:09 | |
alaski | heh | 21:10 |
jroll | HEY | 21:10 |
*** thorst1 has quit IRC | 21:10 | |
*** rlrossit has left #openstack-nova | 21:10 | |
jroll | alaski: also we're on icehouse \o/ | 21:10 |
alaski | woo! | 21:10 |
dansmith | icehouse cinder? | 21:10 |
jroll | yes | 21:11 |
mriedem | that's getting dangerously close to a stable branch maintained upstream | 21:11 |
dansmith | hot damn, party like it's 2013! | 21:11 |
*** salv-orlando has joined #openstack-nova | 21:11 | |
jroll | scottda: this is how we're dealing with adding microversions to nova-ironic interactions, if it helps https://review.openstack.org/#/c/221848/7 | 21:12 |
jroll | scottda: think of pre-microversions as 1.0 or whatever | 21:12 |
jroll | default to that | 21:12 |
jroll | if it's configured to use newer... do it | 21:12 |
*** bswartz has joined #openstack-nova | 21:13 | |
scottda | thanks | 21:15 |
jroll | scottda: oh, one more thing, require a newer cinderclient in nova that does that the version param | 21:17 |
scottda | Yeah, it looks like an older Novaclient will work with newer (microversioned Cinder) but a newer (microversioned) Novaclient will not work with older cinder. | 21:18 |
*** ZZelle_ has joined #openstack-nova | 21:18 | |
dansmith | scottda: ideally it would | 21:18 |
dansmith | scottda: it should, if possible, fall back to the other behavior if the cinder is too old | 21:19 |
jroll | why does novaclient come into the picture? does novaclient talk to cinder directly? | 21:19 |
dansmith | scottda: so in nova, we can detect if the server supports this stuff by probing | 21:19 |
jroll | I though it would be novaserver talking to cinder | 21:19 |
dansmith | jroll: in some cases yeah | 21:19 |
jroll | for real? :| | 21:19 |
mriedem | dansmith: we removed that... | 21:20 |
dansmith | I don't remember why, nor do I remember people being thrilled about it :) | 21:20 |
dansmith | ah | 21:20 |
* jroll adds ironic support to novaclient | 21:20 | |
mriedem | or deprecated for removal, it was my todo after YVR | 21:20 |
melwitt | yes | 21:20 |
*** tpeoples has joined #openstack-nova | 21:20 | |
mriedem | btw, dibs on removal in mitaka | 21:20 |
mriedem | which i guess is nowish | 21:20 |
*** smatzek has quit IRC | 21:20 | |
melwitt | jroll: nooo | 21:21 |
melwitt | novaclient: the everything client | 21:21 |
*** Marga_ has joined #openstack-nova | 21:21 | |
*** tjones has left #openstack-nova | 21:21 | |
jroll | hehe | 21:21 |
mriedem | except for translations | 21:21 |
melwitt | it was that and we're making it not that. or at least mriedem is | 21:21 |
mriedem | we aren't allowed to translate novaclient | 21:21 |
melwitt | oh, right | 21:22 |
* jroll backs away slowly | 21:22 | |
mriedem | hey! stop him! | 21:23 |
jroll | mriedem: do I actually want to know why that is not allowed? | 21:24 |
jroll | I feel like probably not, but curiousity and all | 21:24 |
dansmith | jroll: are you itching to translate more crap? | 21:24 |
mriedem | the argument is that there are higher priorities | 21:24 |
mriedem | which there are | 21:24 |
jroll | so don't bother. | 21:24 |
mriedem | but i don't see how that means you can't make it possible to translate novaclient | 21:24 |
jroll | huh. | 21:24 |
mriedem | yeah, can't win, don't try | 21:24 |
*** ndipanov has quit IRC | 21:25 | |
mriedem | plus, horizon and osc still proxy through novaclient | 21:25 |
mriedem | so it's not like it's deprecated | 21:25 |
melwitt | the ML thread said we can translate openstackclient, to just focus the efforts over there | 21:25 |
jroll | that means we can delete all of the low-priority blueprints, then, right? | 21:25 |
mriedem | melwitt: yeah, but there are things not using osc | 21:25 |
jroll | like all the people | 21:25 |
* jroll ducks | 21:25 | |
mriedem | but i agree osc is higher priority | 21:25 |
melwitt | yeah | 21:25 |
mriedem | but osc is not a python sdk | 21:26 |
mriedem | and we aren't supposed to use the sdk in the server projects | 21:26 |
mriedem | so we use the clients, or write our own rest service like tempest does | 21:26 |
mriedem | ah damn | 21:28 |
mriedem | # NOTE(mriedem): Remove this along with the deprecated commands in the first | 21:28 |
mriedem | # python-novaclient release AFTER the nova server 13.0.0 'M' release. | 21:28 |
mriedem | can't remove that volume stuff from novaclient until after mitaka | 21:28 |
*** jichen has quit IRC | 21:28 | |
melwitt | hells bells | 21:29 |
*** otter768 has joined #openstack-nova | 21:29 | |
dansmith | lol | 21:29 |
dansmith | melwitt: now you're just trying | 21:29 |
*** VW_ has quit IRC | 21:29 | |
mriedem | very curmudgeonly, well done | 21:30 |
melwitt | I was going to say "darn it" and mriedem name reminded me there are other options | 21:30 |
*** VW has joined #openstack-nova | 21:31 | |
*** Apoorva has joined #openstack-nova | 21:32 | |
*** achanda has quit IRC | 21:32 | |
*** otter768 has quit IRC | 21:34 | |
*** vladikr has quit IRC | 21:35 | |
*** matrohon has quit IRC | 21:36 | |
*** achanda has joined #openstack-nova | 21:36 | |
*** obedmr_ has joined #openstack-nova | 21:40 | |
*** hemnafk is now known as hemna | 21:40 | |
*** burgerk has quit IRC | 21:42 | |
*** obedmr has quit IRC | 21:43 | |
openstackgerrit | Chuck Carmack proposed openstack/nova: WIP: Disable security group tests in tempest-dsvm-cells-rc using a config option https://review.openstack.org/226043 | 21:43 |
melwitt | dansmith: I've been looking at this patch https://review.openstack.org/#/c/173723/, it seems correct but I had a hard time tracing how to verify that FixedIP 1.1 goes with Network 1.0 | 21:45 |
melwitt | dansmith: then I was thinking about how we could test for these. I wasn't clear if these will all be caught by the test_obj_relationships from now on and if there are errors we just have to fix them manually and detecting them would be hard? | 21:46 |
melwitt | (detecting old errors, I mean) | 21:47 |
openstackgerrit | Brian Haley proposed openstack/nova: Disable IPv6 on bridge devices https://review.openstack.org/198054 | 21:47 |
dansmith | melwitt: the easy thing to do is to punt until mitaka and then we can delete all the object relationship crap | 21:47 |
*** burt has quit IRC | 21:47 | |
dansmith | melwitt: and the problem goes away because the client declares all its versions | 21:47 |
melwitt | dansmith: okay. I missed that we won't need object relationships anymore | 21:49 |
dansmith | melwitt: yeah.. you're welcome :) | 21:49 |
melwitt | haha | 21:50 |
bauzas | dansmith: I thought we needed a V2 for each object? | 21:50 |
dansmith | bauzas: nope | 21:51 |
dansmith | bauzas: instancev2 depends on that functionality, | 21:51 |
dansmith | but once we're sure that we're cool, we can drop all relationship stuff from mitaka | 21:51 |
dansmith | because liberty clients will always ask for backports with a full list of their supported objects | 21:51 |
bauzas | dansmith: oh okay | 21:51 |
bauzas | dansmith: yeah saw the conductor change | 21:52 |
*** jaypipes has quit IRC | 21:52 | |
dansmith | we just couldn't remove that from liberty because kilo clients won't, so liberty has to continue to use the relationship maps | 21:52 |
dansmith | but no kilo nodes will send instance v2, so we're cool | 21:52 |
dansmith | it's a complex dance | 21:52 |
dansmith | so I forgive you all for not keeping it all in your heads :) | 21:52 |
melwitt | whew! :) | 21:53 |
bauzas | dansmith: gotcha | 21:54 |
bauzas | once mitaka opens, we're free to say that liberty and mitaka know the new stuff | 21:54 |
bauzas | so we can drop the old stufuf | 21:54 |
bauzas | well, mitaka opened actually | 21:55 |
* bauzas needs to update his mindset | 21:55 | |
mriedem | tonyb: are you going to mark the nova part of this bug as invalid? https://bugs.launchpad.net/nova/+bug/1415087 | 21:55 |
openstack | Launchpad bug 1415087 in OpenStack Compute (nova) "[OSSA 2015-011] Format-guessing and file disclosure in image convert (CVE-2015-1850, CVE-2015-1851)" [High,Triaged] - Assigned to Tony Breeds (o-tony) | 21:55 |
tonyb | mriedem: Hmmm I s'pose. | 21:56 |
mriedem | the thing is, people are freaking about publishing security advisories | 21:56 |
tonyb | mriedem: I suspect it's an issue but I'm not enough of a black hat to get to it :/ | 21:56 |
*** achanda has quit IRC | 21:57 | |
dansmith | bauzas: yeah, but I don't want to actually drop things until the grenade job is doing L->M | 21:57 |
bauzas | oh yeah | 21:58 |
tonyb | mriedem: I thought that was dealt with :( | 21:58 |
mriedem | so cinder had a change here https://review.openstack.org/#/c/191785/ | 22:00 |
*** achanda has joined #openstack-nova | 22:00 | |
mriedem | where qemu-img convert was called | 22:00 |
mriedem | and so does nova https://github.com/openstack/nova/blob/master/nova/virt/images.py#L71 | 22:01 |
*** eharney has quit IRC | 22:01 | |
*** rfolco has quit IRC | 22:02 | |
tonyb | mriedem: Sure I had a similar fix but was never able to show it helped | 22:03 |
*** flaper87 has quit IRC | 22:03 | |
*** nihilifer1 has joined #openstack-nova | 22:03 | |
*** nihilifer has quit IRC | 22:03 | |
*** alexpilotti has joined #openstack-nova | 22:04 | |
tonyb | mriedem: So how about we hold off on closeing the bug, I'll polish my patch and add it and then we can talk through it at the meeting ~20hours | 22:04 |
*** ccarmack has left #openstack-nova | 22:05 | |
*** flaper87 has joined #openstack-nova | 22:07 | |
openstackgerrit | Chuck Carmack proposed openstack/nova: WIP: Disable security group tests in tempest-dsvm-cells-rc using a config option https://review.openstack.org/226043 | 22:08 |
*** penick has joined #openstack-nova | 22:10 | |
*** mdrabe has quit IRC | 22:11 | |
*** VW has quit IRC | 22:11 | |
*** VW has joined #openstack-nova | 22:12 | |
*** stevemar has joined #openstack-nova | 22:13 | |
*** annegent_ has joined #openstack-nova | 22:14 | |
*** stevemar has quit IRC | 22:15 | |
*** openstackgerrit has quit IRC | 22:16 | |
*** otter768 has joined #openstack-nova | 22:16 | |
*** VW has quit IRC | 22:16 | |
*** openstackgerrit has joined #openstack-nova | 22:16 | |
*** VW has joined #openstack-nova | 22:17 | |
*** annegentle has quit IRC | 22:18 | |
*** smatzek has joined #openstack-nova | 22:19 | |
*** smatzek has quit IRC | 22:19 | |
*** salv-orlando has quit IRC | 22:20 | |
*** mestery has quit IRC | 22:21 | |
*** diogogmt has quit IRC | 22:21 | |
*** mestery has joined #openstack-nova | 22:22 | |
*** salv-orlando has joined #openstack-nova | 22:22 | |
openstackgerrit | Christine Wang proposed openstack/nova: Update name validation with expanded characters https://review.openstack.org/227039 | 22:26 |
*** mriedem is now known as mriedem_away | 22:27 | |
*** ctrath has quit IRC | 22:28 | |
*** otter768 has quit IRC | 22:31 | |
*** jamielennox is now known as jamielennox|away | 22:35 | |
*** su_zhang has quit IRC | 22:35 | |
*** su_zhang has joined #openstack-nova | 22:35 | |
*** alejandrito has quit IRC | 22:36 | |
*** otter768 has joined #openstack-nova | 22:36 | |
*** VW_ has joined #openstack-nova | 22:37 | |
*** leitan has quit IRC | 22:40 | |
*** VW has quit IRC | 22:41 | |
*** VW_ has quit IRC | 22:42 | |
*** dims__ has joined #openstack-nova | 22:43 | |
*** tjones1 has joined #openstack-nova | 22:44 | |
*** alexpilotti has quit IRC | 22:44 | |
*** dims_ has quit IRC | 22:46 | |
*** su_zhang has quit IRC | 22:46 | |
*** su_zhang has joined #openstack-nova | 22:46 | |
*** annegent_ has quit IRC | 22:49 | |
*** ZZelle_ has quit IRC | 22:52 | |
*** obedmr_ has quit IRC | 22:54 | |
*** alex_klimov has quit IRC | 22:54 | |
*** jerrygb has quit IRC | 22:55 | |
*** jwcroppe has quit IRC | 22:56 | |
*** jwcroppe has joined #openstack-nova | 22:57 | |
*** jwcroppe has quit IRC | 23:01 | |
*** asettle has joined #openstack-nova | 23:03 | |
*** annegentle has joined #openstack-nova | 23:04 | |
*** annegentle has quit IRC | 23:05 | |
*** tjones1 has left #openstack-nova | 23:07 | |
*** tonytan4ever has quit IRC | 23:09 | |
*** tpeoples has quit IRC | 23:09 | |
*** hemna is now known as hemnafk | 23:10 | |
*** xyang1 has quit IRC | 23:12 | |
*** stevemar has joined #openstack-nova | 23:13 | |
*** emagana has quit IRC | 23:18 | |
*** electrocucaracha has quit IRC | 23:19 | |
*** penick has quit IRC | 23:21 | |
*** zhangjn has joined #openstack-nova | 23:24 | |
*** zhangjn has quit IRC | 23:24 | |
*** penick has joined #openstack-nova | 23:28 | |
*** otter768 has quit IRC | 23:29 | |
*** achanda has quit IRC | 23:33 | |
*** tpeoples has joined #openstack-nova | 23:34 | |
*** takedakn has joined #openstack-nova | 23:35 | |
*** hodos|2 has quit IRC | 23:36 | |
*** jwcroppe has joined #openstack-nova | 23:37 | |
*** nic has quit IRC | 23:41 | |
*** shyu has joined #openstack-nova | 23:42 | |
*** jwcroppe has quit IRC | 23:45 | |
*** baoli_ has quit IRC | 23:51 | |
*** jamielennox|away is now known as jamielennox | 23:52 | |
*** takedakn1 has joined #openstack-nova | 23:53 | |
*** takedakn1 has quit IRC | 23:55 | |
*** takedakn1 has joined #openstack-nova | 23:55 | |
*** takedakn has quit IRC | 23:56 | |
*** takedakn1 is now known as takedakn | 23:56 | |
*** takedakn has quit IRC | 23:59 | |
*** zhangjn has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!