Thursday, 2017-05-18

*** inara` has quit IRC00:00
mriedemi wonder if we could bulk query the floating IPs by the list of fixed IPs and the port_id00:01
openstackgerritmelanie witt proposed openstack/nova master: Cache database and message queue connection objects  https://review.openstack.org/46504200:01
mriedem"When you specify multiple filters, the Networking API v2.0 returns only objects that meet all filtering criteria. The operation applies an AND condition among the filters."00:02
mriedemso that wouldn't really work,00:02
mriedembecause fixed_ip1 AND fixed_ip2 wouldn't resolve to a floating IP00:02
*** inara has joined #openstack-nova00:02
cfriesenmriedem: we get an event for each vif, and we have to get info for all vifs on each event, so  when we have N vifs, the live-migration overhead goes up as N^200:02
mriedemwe could trim the fat on the fields we actually need to get back00:02
cfriesenmriedem: it'd be better to have the vif specified in the network-changed event00:03
*** ijw has quit IRC00:03
cfriesenmriedem: and avoid having to scan all the vifs00:03
mriedemcfriesen: yeah i'm not sure how feasible that is from the neutron side00:03
mriedemi guess it would depend on how they calculate the network-changed event00:03
cfriesenmriedem: gotta run, family is calling for supper00:03
edleafeI've written a post to the dev list about continuing the discussion on claims in the scheduler: http://lists.openstack.org/pipermail/openstack-dev/2017-May/116949.html00:03
edleafe^^ cdent jaypipes dansmith bauzas johnthetubaguy00:04
edleafeIt has a link to my blog post and the patch series00:04
mriedemedleafe: ack00:04
edleafeAnd with that, I'm off to PyCon00:04
mriedemhave fun00:04
edleafeSee you all next week!00:04
edleafemriedem: oh, it's all work work work00:04
*** suresh12 has quit IRC00:10
*** suresh12 has joined #openstack-nova00:13
*** edleafe has quit IRC00:13
*** edleafe has joined #openstack-nova00:14
mriedemcfriesen: here is something to look into later - the network-changed event in neutron does have a port in scope, it just doesn't put it in the external event request https://github.com/openstack/neutron/blob/master/neutron/notifiers/nova.py#L16100:15
mriedembut we should be able to scope that event to the single vif00:15
mriedemcfriesen: it would be helpful if we had a bug to track some of this00:16
*** ericyoung has quit IRC00:19
*** ericyoung has joined #openstack-nova00:21
*** gianpietro has quit IRC00:25
*** jamielennox is now known as jamielennox|away00:27
*** ijw has joined #openstack-nova00:31
*** ijw has quit IRC00:31
*** ijw has joined #openstack-nova00:32
*** nic has quit IRC00:33
*** suresh12 has quit IRC00:34
*** suresh12 has joined #openstack-nova00:35
mriedemcfriesen: https://bugs.launchpad.net/neutron/+bug/1691602 for tracking00:41
openstackLaunchpad bug 1691602 in OpenStack Compute (nova) "live migration generates several network-changed events which lock up refreshing the nw info cache" [Undecided,New]00:41
*** jamielennox|away is now known as jamielennox00:41
*** Sukhdev has joined #openstack-nova00:43
*** Guest2661 has quit IRC00:45
*** ijw has quit IRC00:48
*** ijw has joined #openstack-nova00:49
*** ijw has quit IRC00:49
*** ijw has joined #openstack-nova00:50
*** zz_dimtruck is now known as dimtruck00:50
*** ijw has quit IRC00:51
*** ijw has joined #openstack-nova00:52
*** ijw has quit IRC00:53
*** ijw has joined #openstack-nova00:53
*** gianpietro has joined #openstack-nova00:56
*** thorst_afk has joined #openstack-nova00:58
*** ijw has quit IRC00:58
openstackgerritGhanshyam Mann proposed openstack/nova master: Add policy description for os-host  https://review.openstack.org/46545800:59
*** rfolco has joined #openstack-nova01:00
*** rfolco has quit IRC01:01
*** Apoorva_ has joined #openstack-nova01:01
*** thorst_afk has quit IRC01:02
*** Apoorva has quit IRC01:05
*** Sukhdev has quit IRC01:06
*** Apoorva_ has quit IRC01:06
*** kaisers_ has joined #openstack-nova01:10
*** Kevin_Zheng has joined #openstack-nova01:11
*** jaypipes has joined #openstack-nova01:12
*** kaisers has quit IRC01:14
*** tuanluong has joined #openstack-nova01:15
*** cNilesh has joined #openstack-nova01:15
*** felipemonteiro has joined #openstack-nova01:15
*** liverpooler has joined #openstack-nova01:15
*** cNilesh has quit IRC01:16
*** cnilesh has joined #openstack-nova01:16
*** cnilesh has quit IRC01:17
*** cNilesh has joined #openstack-nova01:17
openstackgerritAlex Xu proposed openstack/nova master: Limit the min length of string for integer JSON-Schema  https://review.openstack.org/41510301:18
*** Daisy has joined #openstack-nova01:18
*** jaypipes has quit IRC01:19
*** karthiks has joined #openstack-nova01:20
openstackgerritZhenyu Zheng proposed openstack/nova master: Catch exception.OverQuota when create image for volume backed instance  https://review.openstack.org/46454801:21
openstackgerritAlex Xu proposed openstack/nova master: Limit the min length of string for integer JSON-Schema  https://review.openstack.org/41510301:24
*** gjayavelu has quit IRC01:25
*** jamesden_ has joined #openstack-nova01:26
*** fragatina has quit IRC01:27
*** ijw has joined #openstack-nova01:30
*** Daisy has quit IRC01:32
*** rfolco has joined #openstack-nova01:33
*** ijw has quit IRC01:36
dikonoorpumaranikar: Hi01:36
dikonoorpumaranikar: I am trying to understand if this new support added is still experimental > https://github.com/openstack/nova/blob/master/nova/conf/service_token.py#L4301:39
dikonoorCan it be turned on ?01:39
mriedemdikonoor: yes it can be turned on for testing, we test with it on in the nova-next CI job01:40
mriedemdikonoor: would actually appreciate someone to run with it in rally or longer-running operation scenarios like with live migration01:41
mriedemand report back if it's helping01:41
*** thorst_afk has joined #openstack-nova01:41
*** p01nt3r75 has joined #openstack-nova01:43
*** jwcroppe has joined #openstack-nova01:43
*** thorst_afk has quit IRC01:43
*** jwcroppe has quit IRC01:44
*** jwcroppe has joined #openstack-nova01:44
*** psachin has joined #openstack-nova01:51
*** baoli has quit IRC01:56
*** baoli has joined #openstack-nova01:57
*** suresh12 has quit IRC01:58
*** coreywright has quit IRC02:05
*** gianpietro has quit IRC02:08
*** markvoelker has quit IRC02:08
*** markvoelker has joined #openstack-nova02:08
*** crushil has joined #openstack-nova02:10
*** Daisy has joined #openstack-nova02:12
*** gianpietro has joined #openstack-nova02:13
*** yamahata has joined #openstack-nova02:15
openstackgerritJay Pipes proposed openstack/nova master: Ensure image metadata when unshelving BFV instance  https://review.openstack.org/34706302:16
*** hongbin has joined #openstack-nova02:18
*** jaypipes has joined #openstack-nova02:22
*** rfolco has quit IRC02:22
jaypipesalex_xu: looks sensible to me: https://review.openstack.org/#/c/353862/2302:22
jaypipesalex_xu: XenAPI resource tracking fix.02:23
*** coreywright has joined #openstack-nova02:23
*** suresh12 has joined #openstack-nova02:23
*** dimtruck has quit IRC02:28
*** suresh12 has quit IRC02:28
*** vladikr has quit IRC02:29
*** dimtruck has joined #openstack-nova02:31
openstackgerritAlex Xu proposed openstack/nova master: Correct the error message for query parameter validation  https://review.openstack.org/46578502:34
openstackgerritTao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver.  https://review.openstack.org/41446302:35
mriedemoh bug smashing glory02:38
*** jamielennox is now known as jamielennox|away02:40
*** jaypipes has quit IRC02:41
mriedemkevinbenton: looking at this https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L33102:43
mriedemit looks like you can list resources by a list of ids02:43
*** crushil has quit IRC02:43
kevinbentonmriedem: yeah, i was actually just looking through the code that builds the filters02:44
kevinbentonbecause i do remember that being a list somewhere02:44
*** thorst_afk has joined #openstack-nova02:44
kevinbentonmriedem: i don't remember if it's just for IDs or for everything02:44
mriedemok, because i was wondering if this could be optimized https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L201702:44
mriedemthat's called from a for loop https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L209702:45
*** gjayavelu has joined #openstack-nova02:45
mriedemif i could just do something like GET /floatingips?fixed_ip_address=['fixedip1', 'fixedip2', ...]&port_id=port_id02:45
mriedemthat would be the bulk query i'd need02:45
kevinbentonmriedem: https://github.com/openstack/neutron/blob/master/neutron/db/_model_query.py#L191-L19402:46
mriedembut,02:46
kevinbentonmriedem: yeah, it's an OR list02:46
*** baoli has quit IRC02:46
*** gianpietro has quit IRC02:46
*** gouthamr has quit IRC02:47
mriedemhmm, but the API ref says "OpenStack Networking does not offer an OR mechanism for filters."02:47
kevinbentonmriedem: it wouldn't be the first time the docs were wrong :)02:47
mriedemi figured the client would unpack the list and turn that into fixed_ip_address=1&fixed_ip_address=2&port_id=port_id02:47
mriedemand AND them all02:47
*** gouthamr has joined #openstack-nova02:47
*** salv-orlando has joined #openstack-nova02:48
*** Shunli has joined #openstack-nova02:49
*** Daisy has quit IRC02:49
*** Daisy has joined #openstack-nova02:50
*** jamesden_ has quit IRC02:50
mriedemok i can post a wip for that too and see what happens,02:50
mriedemi don't know if there are any tests in tempest which would tickle that02:50
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Handle network-changed event for a specific port  https://review.openstack.org/46578702:52
mriedemkevinbenton: ^ is the kind of awful i was talking about earlier02:52
kevinbentonmriedem: yeah, repeats turn into the list which becomes the OR in that first query link i sent02:53
kevinbentonmriedem: here is the request parse https://github.com/openstack/neutron/blob/master/neutron/api/api_common.py#L46-L4702:53
mriedemcool02:53
*** gianpietro has joined #openstack-nova02:53
kevinbentonmriedem: yeah, not exactly a trivial change :)02:54
mriedemit would really be split into 2 changes, the first being a refactor02:54
mriedembut yeah02:54
mriedemi'm not going for pretty at 10pm02:54
kevinbentonmriedem: move to hawaii02:55
kevinbentonmore time in the day that way02:55
kevinbenton:)02:55
mriedemheh02:55
mriedemyou are a proven thought leader02:55
mriedemapparently we handle GET /floatingips results as a list when we filter on fixed IP and port,03:00
mriedembut wouldn't that really be 1:1?03:00
mriedemcan you have more than one floating IP associated to the same fixed ip and port?03:00
mriedemi'd think not03:00
kevinbentonno03:01
mriedemok good, this is just confusing nova code then03:01
kevinbentonwell let me confirm03:01
*** baoli has joined #openstack-nova03:02
*** thorst_afk has quit IRC03:04
kevinbentonmriedem: there actually may be a way to get there03:04
mriedemew03:05
kevinbentonmriedem: two different routers03:05
mriedemok03:05
kevinbentonmriedem: attached to two different ext nets03:05
kevinbentonmriedem: both attached to same internal03:05
kevinbentonmriedem: and both with a floating IP associated to the instance03:05
kevinbentonmriedem: and it would actually work okay if you setup some routing rules on the instance03:06
kevinbentonmriedem: so i think you'll have to preserve that use case :)03:06
mriedemdamn03:06
mriedemok03:06
kevinbentonmriedem: the constraint for floating IP is external_network + IP + port_id https://github.com/openstack/neutron/blob/master/neutron/db/models/l3.py#L10103:07
*** gianpietro has quit IRC03:07
*** nicolasbock has quit IRC03:09
mriedemi'll bake that into a comment in here03:09
*** felipemonteiro has quit IRC03:12
*** trinaths has joined #openstack-nova03:12
*** links has joined #openstack-nova03:14
mriedemif we only have one fixed ip per port then this doesn't buy us much03:16
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Bulk query floating IPs in _nw_info_get_ips  https://review.openstack.org/46579203:16
mriedembut here is the thing ^03:16
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Bulk query floating IPs in _nw_info_get_ips  https://review.openstack.org/46579203:17
*** crushil has joined #openstack-nova03:18
*** baoli has quit IRC03:19
openstackgerritJay Pipes proposed openstack/nova master: Ensure image metadata when unshelving BFV instance  https://review.openstack.org/34706303:29
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Don't warn on expected network-vif-unplugged events  https://review.openstack.org/46579403:32
*** jamielennox|away is now known as jamielennox03:33
*** salv-orlando has quit IRC03:34
*** Daisy has quit IRC03:34
*** Daisy has joined #openstack-nova03:34
*** felipemonteiro has joined #openstack-nova03:35
*** mriedem has quit IRC03:39
*** Daisy has quit IRC03:39
*** coreywright has quit IRC03:46
*** gianpietro has joined #openstack-nova03:48
*** yamamoto_ has joined #openstack-nova03:50
*** vks1 has joined #openstack-nova03:50
*** hongbin has quit IRC03:52
*** gianpietro has quit IRC03:53
openstackgerritTao Li proposed openstack/nova master: Optimize the logged message  https://review.openstack.org/46579703:54
*** markvoelker has quit IRC03:56
*** markvoelker has joined #openstack-nova03:57
openstackgerritLuo Gangyi proposed openstack/nova master: convert unicode to string before we connect to rados  https://review.openstack.org/46540403:57
*** coreywright has joined #openstack-nova03:59
cfriesenmriedem: you've been a busy beaver on get_instance_nw_info()...taking a look at your patch.04:00
*** litao__ has joined #openstack-nova04:01
*** markvoelker has quit IRC04:01
*** ijw has joined #openstack-nova04:01
*** gianpietro has joined #openstack-nova04:01
*** suresh12 has joined #openstack-nova04:05
*** ijw has quit IRC04:06
*** gjayavelu has quit IRC04:10
*** takashin has joined #openstack-nova04:10
*** Sukhdev has joined #openstack-nova04:13
*** suresh12 has quit IRC04:13
*** armax has quit IRC04:14
*** zerda3 has quit IRC04:14
*** gianpietro has quit IRC04:18
*** takashin has quit IRC04:19
*** links has quit IRC04:20
*** fragatina has joined #openstack-nova04:30
*** fragatina has quit IRC04:30
*** fragatina has joined #openstack-nova04:31
*** takashin has joined #openstack-nova04:32
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove unused exceptions  https://review.openstack.org/46452504:32
openstackgerritTakashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases  https://review.openstack.org/39543604:33
*** diablo_rojo has quit IRC04:34
openstackgerritTakashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls  https://review.openstack.org/41387604:35
*** prateek has joined #openstack-nova04:36
*** Daisy has joined #openstack-nova04:36
*** prateek has quit IRC04:38
*** dave-mccowan has quit IRC04:43
*** ayogi has joined #openstack-nova04:45
*** nurdie has joined #openstack-nova04:47
*** adisky_ has joined #openstack-nova04:50
*** bkopilov has joined #openstack-nova04:51
*** sree has joined #openstack-nova04:51
*** crushil has quit IRC04:52
*** Daisy has quit IRC04:53
*** Daisy has joined #openstack-nova04:53
*** crushil has joined #openstack-nova04:54
*** yamahata has quit IRC04:54
*** crushil has quit IRC04:56
*** trinaths has left #openstack-nova04:57
*** yamahata has joined #openstack-nova04:58
*** prateek has joined #openstack-nova04:58
*** links has joined #openstack-nova04:58
*** jaosorior_away is now known as jaosorior05:00
*** thorst_afk has joined #openstack-nova05:01
*** claudiub has joined #openstack-nova05:03
*** thorst_afk has quit IRC05:05
*** crushil has joined #openstack-nova05:06
*** lpetrut has joined #openstack-nova05:06
*** fragatina has quit IRC05:11
*** claudiub has quit IRC05:12
*** udesale has joined #openstack-nova05:18
*** thorst_afk has joined #openstack-nova05:22
*** crushil has quit IRC05:23
*** ratailor has joined #openstack-nova05:24
*** thorst_afk has quit IRC05:26
*** dikonoor has quit IRC05:26
*** salv-orlando has joined #openstack-nova05:27
*** dikonoor has joined #openstack-nova05:27
*** Sukhdev has quit IRC05:28
*** shaohe_feng has quit IRC05:34
*** psachin has quit IRC05:35
*** shaohe_feng has joined #openstack-nova05:35
*** felipemonteiro has quit IRC05:37
*** yamamoto_ has quit IRC05:39
*** Daisy has quit IRC05:39
*** Daisy has joined #openstack-nova05:43
*** salv-orlando has quit IRC05:44
openstackgerritEli Qiao proposed openstack/nova master: Re-worked ram filter and weights to support overcommit  https://review.openstack.org/39098405:45
*** rcernin has joined #openstack-nova05:46
*** lpetrut has quit IRC05:47
*** mamandle has joined #openstack-nova05:48
*** kaisers has joined #openstack-nova05:50
openstackgerritJay Pipes proposed openstack/nova master: Fix wrong disk size for sparse flat files  https://review.openstack.org/39687305:51
*** gyee_ has quit IRC05:52
openstackgerritLuo Gangyi proposed openstack/nova master: convert unicode to string before we connect to rados  https://review.openstack.org/46540405:53
*** kaisers_ has quit IRC05:53
openstackgerritTao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver.  https://review.openstack.org/41446305:55
*** mdnadeem has joined #openstack-nova05:56
*** ijw has joined #openstack-nova06:05
*** links has quit IRC06:08
*** nurdie has quit IRC06:10
*** ijw has quit IRC06:10
openstackgerritchristopher.uhler proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale  https://review.openstack.org/44395106:13
*** Oku_OS-away is now known as Oku_OS06:14
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959306:19
*** thorst_afk has joined #openstack-nova06:23
*** links has joined #openstack-nova06:23
*** pcaruana has joined #openstack-nova06:24
*** yamamoto_ has joined #openstack-nova06:26
*** thorst_afk has quit IRC06:27
*** p01nt3r75 has quit IRC06:29
openstackgerritEli Qiao proposed openstack/nova master: Re-worked ram filter and weights to support overcommit  https://review.openstack.org/39098406:30
*** Daisy has quit IRC06:33
*** Daisy has joined #openstack-nova06:34
*** satyar has joined #openstack-nova06:34
*** gouthamr has quit IRC06:36
*** Daisy has quit IRC06:38
*** Daisy has joined #openstack-nova06:40
*** rabel_b1 has joined #openstack-nova06:42
*** markus_z has joined #openstack-nova06:43
*** rabel_b1 has quit IRC06:45
*** Daisy has quit IRC06:45
*** dimtruck is now known as zz_dimtruck06:45
*** rabel_b1 has joined #openstack-nova06:45
*** salv-orlando has joined #openstack-nova06:45
*** Daisy has joined #openstack-nova06:46
*** salv-orlando has quit IRC06:50
*** Daisy has quit IRC06:50
*** ralonsoh has joined #openstack-nova06:50
*** Daisy has joined #openstack-nova06:56
*** gszasz has joined #openstack-nova06:56
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref  https://review.openstack.org/46584207:02
*** sofie_ has joined #openstack-nova07:05
*** ltomasbo|away is now known as ltomasbo07:08
*** belmoreira has joined #openstack-nova07:11
*** mnestratov has joined #openstack-nova07:15
*** karimb has joined #openstack-nova07:15
*** links has quit IRC07:16
*** rabel_b1 has quit IRC07:17
*** rabel_b1 has joined #openstack-nova07:18
*** Daisy has quit IRC07:18
*** thorst_afk has joined #openstack-nova07:24
*** Daisy has joined #openstack-nova07:25
*** rabel has joined #openstack-nova07:28
*** thorst_afk has quit IRC07:28
*** ltomasbo is now known as ltomasbo|away07:28
*** rabel_b1 has quit IRC07:29
*** adisky_ has quit IRC07:29
*** links has joined #openstack-nova07:30
*** Daisy has quit IRC07:30
*** rabel_b1 has joined #openstack-nova07:30
*** rabel has quit IRC07:32
*** zz_dimtruck is now known as dimtruck07:34
*** Daisy has joined #openstack-nova07:35
*** mlakat has joined #openstack-nova07:39
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref  https://review.openstack.org/46584207:40
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-quota-sets endpoint instead of stevedore  https://review.openstack.org/46163007:41
*** lpetrut has joined #openstack-nova07:41
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore  https://review.openstack.org/46164107:42
*** cfriesen has quit IRC07:42
*** jpena|off is now known as jpena07:44
*** dimtruck is now known as zz_dimtruck07:44
*** tovin07_ has joined #openstack-nova07:46
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore  https://review.openstack.org/46081807:46
*** salv-orlando has joined #openstack-nova07:46
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-certificates endpoint instead of stevedore  https://review.openstack.org/46081307:48
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore  https://review.openstack.org/46162807:50
*** salv-orlando has quit IRC07:52
*** sdatko has quit IRC07:53
*** dpawlik has quit IRC07:53
*** ijw has joined #openstack-nova07:54
*** ijw has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** sdatko has joined #openstack-nova08:00
*** dpawlik has joined #openstack-nova08:01
*** karimb has quit IRC08:07
*** adisky_ has joined #openstack-nova08:10
*** aarefiev_afk is now known as aarefiev08:16
*** lucas-afk is now known as lucasagomes08:16
openstackgerritAndrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values  https://review.openstack.org/44800808:20
openstackgerritsahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller  https://review.openstack.org/45974108:24
openstackgerritsahid proposed openstack/nova master: libvirt: update logic to configure device for scsi controller  https://review.openstack.org/45974008:24
openstackgerritsahid proposed openstack/nova master: libvirt: update LibvirtConfigGuestDeviceAddress to provide XML  https://review.openstack.org/45973708:24
openstackgerritsahid proposed openstack/nova master: libvirt: rearange how scsi controller is defined  https://review.openstack.org/45973908:24
openstackgerritsahid proposed openstack/nova master: libvirt: set full description of the controller used by disk  https://review.openstack.org/45973808:24
*** thorst_afk has joined #openstack-nova08:24
openstackgerritTao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver.  https://review.openstack.org/41446308:25
*** yassine has joined #openstack-nova08:26
*** yassine is now known as Guest1636708:26
*** gongysh has joined #openstack-nova08:27
*** takashin has quit IRC08:28
*** thorst_afk has quit IRC08:29
*** takashin has joined #openstack-nova08:31
*** ltomasbo|away is now known as ltomasbo08:31
lyarwoodbauzas, sfinucan, johnthetubaguy ; https://review.openstack.org/#/c/462348/ - A pretty straight forward rootwrap fix for something I broke a few weeks ago if you have time to review this morning, I might be feeling slightly guilty about this ;)08:32
* sfinucan looking08:32
*** swebster has quit IRC08:33
sfinucanlyarwood: Yup, that looks fine by me. +208:34
*** karimb has joined #openstack-nova08:35
*** zz_dimtruck is now known as dimtruck08:35
*** swebster has joined #openstack-nova08:35
lyarwoodsfinucan: thanks08:37
*** djohnsto has quit IRC08:37
*** djohnsto has joined #openstack-nova08:38
*** moshele has joined #openstack-nova08:38
*** zhurong has joined #openstack-nova08:40
openstackgerritjichenjc proposed openstack/nova master: remove unused policy from policy doc  https://review.openstack.org/46550508:43
*** avolkov has joined #openstack-nova08:44
*** dimtruck is now known as zz_dimtruck08:45
*** rabel__b1 has joined #openstack-nova08:45
*** derekh has joined #openstack-nova08:45
*** rabel_b1 has quit IRC08:46
*** salv-orlando has joined #openstack-nova08:48
*** salv-orlando has quit IRC08:53
*** jaosorior is now known as jaosorior_lunch08:57
*** gszasz has quit IRC08:57
*** gszasz has joined #openstack-nova08:58
*** salv-orlando has joined #openstack-nova09:00
*** yamahata has quit IRC09:02
*** zhurong has quit IRC09:05
*** salv-orlando has quit IRC09:05
mlakatHi Stackers - we have an interesting problem - basically we have a broken connection to an NFS server, which results that os.path.getsize() hangs forever on a mounted file. Do we have any timeout like functionality in OpenStack? Greenthreads seem to be blocked.09:08
mlakatthis is the bug btw: https://bugs.launchpad.net/nova/+bug/169113109:09
openstackLaunchpad bug 1691131 in OpenStack Compute (nova) "IO stuck causes nova compute agent outage" [Undecided,In progress] - Assigned to Daniel Gonzalez Nothnagel (dgonzalez)09:09
*** satyar has quit IRC09:10
*** ljjjustin has joined #openstack-nova09:12
*** nurdie has joined #openstack-nova09:15
kashyapartom: Looking at your patch09:15
artomkashyap, thanks :)09:16
artomAlso, I tried playing around with updating the volume serial number when we do volume-update, but I don't think that's supported in a live domain09:17
artomIe, we can only change it in the persistent XML09:17
kashyapartom: Yeah, I saw your note in the LP bug09:17
kashyapartom: I was also wondering, if that line in driver.py should be:09:17
kashyap flags |= copy and (libvirt.VIR_DOMAIN_BLOCK_REBASE_COPY | libvirt.VIR_DOMAIN_BLOCK_REBASE_COPY_DEV) or 009:17
kashyapBut maybe doing it as two lines is consistent with the rest09:18
artomAnd having the serial number not match the actual volume ID has apparently not been a problem until now, so I just left it09:18
artomkashyap, it's still bitwise or'ing all the flags, right? Might as well stay consistent with what's there already09:19
kashyapartom: Yes, it is bitwise OR09:19
kashyaps/is/is still/09:19
*** nurdie has quit IRC09:19
openstackgerritYongfeng Du proposed openstack/nova master: Update the file for IPv4-only or IPv6-only network  https://review.openstack.org/46589109:21
*** esberglu has joined #openstack-nova09:22
*** cdent has joined #openstack-nova09:24
*** thorst_afk has joined #openstack-nova09:25
*** esberglu has quit IRC09:26
*** Daisy has quit IRC09:27
*** Daisy has joined #openstack-nova09:28
*** djohnsto has quit IRC09:29
openstackgerritEli Qiao proposed openstack/nova master: Correct force host checking for LM and evacuation.  https://review.openstack.org/46589509:29
*** udesale__ has joined #openstack-nova09:31
*** udesale has quit IRC09:31
openstackgerritEli Qiao proposed openstack/nova master: Correct force host checking for LM and evacuation  https://review.openstack.org/46589509:32
*** Daisy has quit IRC09:32
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix uuid replacement in aggregate notification test  https://review.openstack.org/46589609:33
*** Shunli has quit IRC09:36
*** zz_dimtruck is now known as dimtruck09:36
*** djohnsto has joined #openstack-nova09:36
openstackgerritJianghua Wang proposed openstack/nova master: Skip policy rules on attach_network for none network allocation  https://review.openstack.org/46590309:41
*** tbachman has quit IRC09:41
*** cNilesh has quit IRC09:42
*** karimb has quit IRC09:42
openstackgerritSylvain Bauza proposed openstack/nova master: Scheduler returning alternatives per destination  https://review.openstack.org/46517509:43
openstackgerritSylvain Bauza proposed openstack/nova master: Delete existing allocations when retrying  https://review.openstack.org/46043309:43
openstackgerritSylvain Bauza proposed openstack/nova master: Don't delete allocation if instance being scheduled  https://review.openstack.org/46017709:43
openstackgerritSylvain Bauza proposed openstack/nova master: WIP: Allocate instance by the conductor  https://review.openstack.org/46045509:43
*** thorst_afk has quit IRC09:44
*** sambetts|afk is now known as sambetts09:44
*** dimtruck is now known as zz_dimtruck09:45
kashyapmdbooth: Noticed the other day, wow, this is merged: https://review.openstack.org/#/c/383859/ ("Fix races with nfs volume mount/umount")09:46
mdboothkashyap: Yeah :)09:47
* kashyap admires the persistence 09:47
*** jwcroppe_ has joined #openstack-nova09:48
*** tbachman has joined #openstack-nova09:49
*** jwcroppe has quit IRC09:50
*** tovin07_ has quit IRC09:53
*** rabel__b1 has quit IRC09:55
* cdent despairs at the need for so much09:59
*** ljjjustin has quit IRC10:00
*** jamielennox is now known as jamielennox|away10:04
*** tuanluong has quit IRC10:09
*** abalutoiu has joined #openstack-nova10:09
*** pjm6 has joined #openstack-nova10:12
pjm6good morning10:13
pjm6i have a problem with live migration, basically I put a cpu_mode: custom and cpu_model: SandyBridge10:13
pjm6when i migrate from compute01 to compute02 it works good10:13
pjm6but the other way arround no, because tthe compute02 has more flags10:13
pjm6but by forcing the cpu_model SandyBridge this should be working, no?10:13
kashyappjm6: Do you have the same cpu_{mode,model} config on all Compute nodes10:15
pjm6kashyap, yes I have10:16
kashyapBTW, what release are you trying this with?  Can't say more without details10:16
pjm6of course, sorry10:16
*** pewp has quit IRC10:16
pjm6i'm in Ocata version10:16
pjm6using nova 15.1.210:17
*** pewp has joined #openstack-nova10:17
pjm6kashyap, this is old but https://blog.codecentric.de/en/2015/03/true-kvm-live-migration-openstack-icehouse-ceph-based-vm-storage/ it seems that I have the problem #210:20
pjm6the error says: Unacceptable CPU info: CPU doesn't have compatibility10:23
*** tbachman has quit IRC10:23
kashyappjm6: That was in IceHouse10:24
*** karimb has joined #openstack-nova10:25
pjm6kashyap, yes I said that was an older version, but the problem seems similar10:25
pjm6for isntance10:25
pjm6the error log: https://pastebin.com/zd35WqqJ10:26
pjm6then I use the virsh cpu-baseline and I get this: https://pastebin.com/AxUx0kYf10:27
* kashyap clicks10:27
kashyappjm6: BTW, just a gentle note: this channel is for devel purposes, maybe you're already aware10:28
*** salv-orlando has joined #openstack-nova10:28
kashyappjm6: I'm just curious, any reason you don't use 'host-model'?10:30
kashyapWhich is actually the default, which use the closest available named-CPU model10:30
pjm6kashyap, i didn't use host-model because I though that if i wanted to use SandyBridge the cpu_mode should be custom10:31
pjm6or i can use cpu_mode: host-model and cpu_model: SandyBridge?10:31
kashyappjm6: Yes, that is true _if_ you specifically want the SandyBridge10:31
kashyappjm6: No, you can't do that10:31
pjm6kashyap, the problem is that i have two different cpus, one is Broadwell the other is SandyBridge10:32
pjm6i was trying to "downgrade" the instruction set in order to allow live migration10:32
pjm6but it seems that when i migrate for the newer to the old, he's comparing physical cpu specs from one server to the other, and then fails10:33
pjm62 line of the first logs10:33
*** salv-orlando has quit IRC10:33
kashyappjm6: Ah-ha, okay, your problem is clearer.  If using the lowest-common denominator CPU model between the two hosts, it should 'just work'10:34
pjm6yes, that's what gived to me the virsh cpu-baseline (which had the two CPU configs)10:34
*** rdo__ has joined #openstack-nova10:35
*** zz_dimtruck is now known as dimtruck10:36
*** rdo_ has quit IRC10:37
*** dpawlik has quit IRC10:38
kashyappjm6: That should clearly work.  Do you have a libvirt debug log with log filters enabled?  And please file a Nova bug with details10:38
*** dpawlik has joined #openstack-nova10:38
pjm6kashyap, for now i have debug disabled and don't have any filter log10:38
pjm6but i can activate the debug yes10:38
pjm6I also see that the dumpxml are using the correct cpu model10:39
kashyappjm6: BTW, to enable libvirt log filters, this is the way to go: https://kashyapc.fedorapeople.org/virt/openstack/request-nova-libvirt-qemu-debug-logs.txt10:40
kashyappjm6: What are the libvirt / QEMU versions?  Just in case if I happened to have an env. I can test it10:40
kashyap(To see if could reproduce the behaviour.)10:41
pjm6kashyap, will see that10:41
pjm6libvirt 2.5.0  | qemu 1:2.810:42
kashyappjm6: Do you have a libvirt guest XML for a VM started on both hosts?10:43
kashyap(Can you post them somewhere, please)10:43
pjm6kashyap, you want a dumpxml of a VM that was created in compute01 and compute02 right/10:44
kashyappjm6: Yep10:44
pjm61 sec please10:44
*** dimtruck is now known as zz_dimtruck10:46
*** cdent has quit IRC10:46
*** openstackgerrit has quit IRC10:48
*** phuongnh has joined #openstack-nova10:48
kashyappjm6: Also, one more: please get `virsh capabilities` from both hosts10:52
pjm6dumpxml: https://pastebin.com/xbuA2BFS and https://pastebin.com/NPRAEjbV10:54
*** nicolasbock has joined #openstack-nova10:56
*** jpena is now known as jpena|lunch10:57
pjm6virsh capabilities: https://pastebin.com/upCBBTu6 and https://pastebin.com/aLDeQipG10:57
kashyapThanks; will look in a few10:58
pjm6well kashyap10:58
pjm6i found a problem!10:58
pjm6compute node02 for some reason is using broadwell10:58
kashyappjm6: ?10:58
kashyapHehe10:58
pjm6well this is strange10:59
pjm6first i create in compute0110:59
pjm6and then migrate to compute0210:59
pjm6the was fine10:59
pjm6and the dumpxml was using sandybridge10:59
kashyappjm6: Can you get the real error from /var/log/libvirt/libvirtd.log, too?11:03
pjm6I don't have that file yet11:04
*** sdague has joined #openstack-nova11:06
sfinucan(My) life would be so much easier if every instance had a NUMA topology associated with it11:07
* sfinucan just discovered that there's a NUMATopologyLimits object that appears to have broken tests with it11:08
pjm6kashyap, i already activate as you have11:10
pjm6in the doc11:10
kashyappjm6: For the actual libvirt error, maybe it's redirected to systemd journal, do you see anything in:11:11
kashyapjournalctl -u libvirtd -l --since=yesterday -p err11:11
kashyaps/yesterday/today/11:11
pjm6yeah i see error11:12
pjm6but idk if is this11:12
kashyappjm6: Can you paste what it is11:12
kashyapsfinucan: What is causing grief?11:12
pjm6sure11:12
kashyapOh, you tell it in the following comment11:12
*** cdent has joined #openstack-nova11:13
pjm6internal error: unable to execute QEMU command 'query-cpu-definitions': The command query-cpu-definitions has not been found11:13
sfinucansfinucan: People changing things in patches to fix something, but neglecting to mention that they fixed said thing11:14
sfinucanWhoops, kashyap ^11:14
kashyapsfinucan: Yeah, I hear you11:14
*** udesale__ has quit IRC11:15
*** ssbljk has joined #openstack-nova11:18
*** ssbljk has left #openstack-nova11:19
kashyappjm6: I think that error comes as part of a series of commands libvirt invocates during some high-level operation like migration.11:22
*** smatzek has joined #openstack-nova11:22
kashyapBut anyway, ensuring your destination Compute node really is also using the custom CPU model should get you going.  As you've noticed the problem.11:22
* kashyap needs to get something to eat11:22
*** dikonoo has joined #openstack-nova11:23
*** liverpooler has quit IRC11:23
pjm6yeah for some reason idk why when creating a VM he don't update the XML11:23
*** smatzek has quit IRC11:23
*** smatzek has joined #openstack-nova11:24
*** dikonoor has quit IRC11:26
*** salv-orlando has joined #openstack-nova11:28
*** jaosorior_lunch is now known as jaosorior11:29
pjm6kashyap,11:29
pjm6it seems it's working now11:29
pjm6but the virsh dumpxml don't show anything related to cpu11:29
pjm6this is normal/11:29
*** ijw has joined #openstack-nova11:30
*** mamandle has quit IRC11:30
pjm6forget it appears, with less info11:30
*** gianpietro has joined #openstack-nova11:31
pjm6https://pastebin.com/gbC4inS11:31
pjm6i just restared the service11:31
*** salv-orlando has quit IRC11:33
*** ychuang has joined #openstack-nova11:34
*** ijw has quit IRC11:34
*** prateek has quit IRC11:35
*** prateek has joined #openstack-nova11:36
*** ratailor has quit IRC11:36
*** zz_dimtruck is now known as dimtruck11:37
*** gongysh has quit IRC11:39
*** phuongnh has quit IRC11:39
*** openstackgerrit has joined #openstack-nova11:40
openstackgerritDaniel Gonzalez Nothnagel proposed openstack/nova master: WIP: Get disk size of instance in separate process  https://review.openstack.org/46565311:40
*** Kevin_Zheng has quit IRC11:40
*** prateek has quit IRC11:40
*** thorst_afk has joined #openstack-nova11:44
*** dpawlik has quit IRC11:44
*** dimtruck is now known as zz_dimtruck11:47
*** sree has quit IRC11:49
*** gianpietro has quit IRC11:50
*** gianpietro has joined #openstack-nova11:51
*** ychuang has quit IRC11:51
*** sree has joined #openstack-nova11:52
*** dpawlik has joined #openstack-nova11:55
*** gianpietro has quit IRC11:55
*** gianpietro has joined #openstack-nova11:55
*** salv-orlando has joined #openstack-nova11:57
*** tbachman has joined #openstack-nova11:59
*** claudiub has joined #openstack-nova12:00
*** MVenesio has joined #openstack-nova12:01
*** litao__ has quit IRC12:01
*** jmlowe has quit IRC12:02
*** jpena|lunch is now known as jpena12:03
*** edmondsw has joined #openstack-nova12:03
*** yamamoto_ has quit IRC12:03
*** catintheroof has joined #openstack-nova12:09
*** ayogi has quit IRC12:17
*** markvoelker has joined #openstack-nova12:23
*** pchavva has joined #openstack-nova12:24
*** rfolco has joined #openstack-nova12:25
*** mriedem has joined #openstack-nova12:28
*** sree has quit IRC12:28
*** liverpooler has joined #openstack-nova12:29
*** gianpietro has quit IRC12:32
openstackgerritjichenjc proposed openstack/nova master: remove unused policy from policy doc  https://review.openstack.org/46550512:36
*** zz_dimtruck is now known as dimtruck12:38
*** vladikr has joined #openstack-nova12:39
*** salv-orlando has quit IRC12:39
*** eharney has joined #openstack-nova12:40
*** salv-orlando has joined #openstack-nova12:40
*** vks1 has quit IRC12:41
*** cdent has quit IRC12:41
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3)  https://review.openstack.org/45282012:42
*** liverpooler has quit IRC12:42
*** liverpooler has joined #openstack-nova12:42
*** sofie_ has quit IRC12:43
*** gianpietro has joined #openstack-nova12:44
*** yamamoto has joined #openstack-nova12:44
*** links has quit IRC12:45
*** salv-orlando has quit IRC12:45
*** Alex______ has quit IRC12:45
kashyappjm6: Your paste-bin has expired; but good if youve' got it working12:47
kashyapmdbooth: Did you notice the above patch? https://review.openstack.org/465653 :12:47
kashyap"[...] While getting instance disk information from the libvirt domain xml, it calls os.path.getsize on the disk path. If this path is unresponsive, e.g. because it is located on an overloaded NFS mount, the nova-compute service hangs and becomes unresponsive. [...]"12:47
kashyapBut they're using a separate process to query for each disk, which could be ... "interesting" if there are, say, 50 disks12:48
*** gianpietro has quit IRC12:49
*** catintheroof has quit IRC12:50
*** gianpietro has joined #openstack-nova12:52
*** kylek3h has joined #openstack-nova12:54
*** prateek has joined #openstack-nova12:55
*** prateek has quit IRC12:55
*** gouthamr has joined #openstack-nova12:58
mlakatkashyap, yes, but what other methods could you suggest?12:59
*** karimb has quit IRC12:59
kashyapmlakat: Hmm, I saw your comment on the thread.12:59
mlakatI also added some more info to today's meeting: https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting13:00
kashyapAnd, thanks for the link to that eventlet bug, useful to learn about it13:00
*** mamandle has joined #openstack-nova13:00
*** esberglu has joined #openstack-nova13:00
*** esberglu has quit IRC13:01
*** gianpietro has quit IRC13:01
*** esberglu has joined #openstack-nova13:01
*** esberglu has quit IRC13:01
*** gianpietro has joined #openstack-nova13:01
*** ftersin has joined #openstack-nova13:03
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595413:05
*** abalutoiu_ has joined #openstack-nova13:09
*** cdent has joined #openstack-nova13:10
*** mdrabe has joined #openstack-nova13:11
*** abalutoiu has quit IRC13:12
*** baoli has joined #openstack-nova13:13
*** dave-mccowan has joined #openstack-nova13:14
*** nurdie has joined #openstack-nova13:14
*** karimb has joined #openstack-nova13:15
*** esberglu has joined #openstack-nova13:18
*** esberglu has quit IRC13:18
*** esberglu has joined #openstack-nova13:18
*** ijw has joined #openstack-nova13:18
openstackgerritMatt Riedemann proposed openstack/nova master: Remove DeviceIsBusy exception  https://review.openstack.org/46572313:19
*** takashin has left #openstack-nova13:20
*** karimb has quit IRC13:22
*** nurdie has quit IRC13:23
*** ijw has quit IRC13:23
*** nurdie has joined #openstack-nova13:25
*** takashin has joined #openstack-nova13:26
*** yamahata has joined #openstack-nova13:27
*** jdillaman has quit IRC13:28
*** smatzek has quit IRC13:29
*** takashin has quit IRC13:32
*** takashin has joined #openstack-nova13:32
artommriedem, heya, thanks for +W'ing the bottom 2 tagged attach patches :) Question for you in https://review.openstack.org/#/c/376021/49 - I'm not sure what you mean by moving error handling lower down, as I don't see a way to do it consistently without handling at least 1 new exception in the API layer13:35
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595413:35
openstackgerritStephen Finucane proposed openstack/python-novaclient master: Remove various deprecated options  https://review.openstack.org/46550213:36
*** Yangyapeng has joined #openstack-nova13:37
mriedemartom: for example https://review.openstack.org/#/c/391941/42/nova/compute/api.py@378613:38
mriedemyou're now raising VolumeTaggedAttachToShelvedNotSupported from nova.compute.api.API.attach_volume13:38
*** takashin has left #openstack-nova13:38
*** takashin has joined #openstack-nova13:39
mriedembut not handling that exception in the REST API controller code https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/volumes.py#L33713:39
mriedemnot in that patch anyway13:39
mriedemand not handling it in the rest api controller code means it would result in a 50013:39
openstackgerritEdu Alcaniz proposed openstack/nova master: follow up  https://review.openstack.org/46597513:39
mriedemi realize you can't get that exception until the microversion because the tag isn't in the schema yet,13:40
artommriedem, oh, you mean move the error handling to the same *patch*13:40
artomNot the same layer13:40
mriedembut i'd prefer to see the error handling for the rest api done in the same methods that introduce those exceptions13:40
mriedemyes13:40
mriedemsame patch13:40
artomOK, sorry13:40
mriedemlower layer in the series of changes is what i meant13:40
artomOK, makes sense, will do that :)13:41
*** salv-orlando has joined #openstack-nova13:41
*** hshiina has joined #openstack-nova13:43
*** baoli_ has joined #openstack-nova13:43
mriedemmelwitt: replied on https://review.openstack.org/#/c/465042/ - i think we can just backport, resolve the conflict and note it in the commit message13:43
mriedemi don't think we need 2 patches on master for that13:44
cdentmriedem: I think edleafe is at pycon, jaypipes in china, and i have to go outside to wait for someone to come rescue my car, so I think bauzas (if he's around) will be your go to guy for today's meeting's scheduler update13:44
mriedemcdent: i can run it actually, kid is home sick so i'll be here to do it13:45
bauzasyup I'm there13:45
cdenti assumed you were running it, mriedem, just the scheduler part, which looks like bauzas has under control13:46
mriedemoh i see13:46
mriedemok13:46
*** salv-orlando has quit IRC13:46
*** baoli has quit IRC13:46
bauzasfine by me13:47
*** Yangyapeng has quit IRC13:47
*** lbragstad has joined #openstack-nova13:48
*** jamielennox|away is now known as jamielennox13:49
*** cdent has quit IRC13:49
*** dimtruck is now known as zz_dimtruck13:51
*** gianpietro has quit IRC13:51
*** moshele has quit IRC13:53
*** nurdie has quit IRC13:54
*** gianpietro has joined #openstack-nova13:55
*** psachin has joined #openstack-nova13:56
mriedemgrenade live migration job is basically 100% fail http://logs.openstack.org/42/465042/5/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/6aae0c3/console.html#_2017-05-18_00_39_25_00722313:57
mriedemFailed to restart devstack@g-api.service: Unit devstack@g-api.service not found.13:57
mriedemi assume because on master that code is expecting systemd, but started on ocata with screen13:57
*** gianpietro has quit IRC14:00
*** Swami has joined #openstack-nova14:00
*** gianpietro has joined #openstack-nova14:00
sdagueyeh, though that should have been pinned14:00
*** awaugama has joined #openstack-nova14:01
sdaguemriedem: oh, the live migration scripts are all super custom14:01
sdaguethey are going to need adjustments14:01
*** mlavalle has joined #openstack-nova14:02
*** dikonoo has quit IRC14:02
sdaguemriedem: grenade completed fine14:02
*** dikonoo has joined #openstack-nova14:02
sdaguethe scripts after that are making assumptions14:02
*** jmlowe has joined #openstack-nova14:03
mriedemsdague: will dig after the nova meeting14:03
*** coreywright has quit IRC14:06
*** felipemonteiro has joined #openstack-nova14:06
*** Yangyapeng has joined #openstack-nova14:06
*** hongbin has joined #openstack-nova14:08
*** felipemonteiro_ has joined #openstack-nova14:09
*** smatzek has joined #openstack-nova14:10
*** yamamoto has quit IRC14:11
*** mriedem has left #openstack-nova14:12
*** mriedem has joined #openstack-nova14:12
*** yamamoto has joined #openstack-nova14:12
*** felipemonteiro__ has joined #openstack-nova14:12
*** felipemonteiro has quit IRC14:12
*** krtaylor has quit IRC14:13
mriedemhttps://bugs.launchpad.net/nova/+bug/169176914:15
openstackLaunchpad bug 1691769 in OpenStack Compute (nova) "gate-grenade-dsvm-neutron-multinode-live-migration-nv fails in pike: "Failed to restart <email address hidden>: Unit <email address hidden> not found."" [Medium,Triaged]14:15
*** felipemonteiro_ has quit IRC14:15
*** yamamoto has quit IRC14:16
*** zz_dimtruck is now known as dimtruck14:17
*** diablo_rojo has joined #openstack-nova14:18
*** mkoderer has joined #openstack-nova14:19
*** coreywright has joined #openstack-nova14:19
*** gyee has joined #openstack-nova14:23
*** cfriesen has joined #openstack-nova14:29
*** tbachman has quit IRC14:31
*** nurdie has joined #openstack-nova14:32
*** yamamoto has joined #openstack-nova14:33
*** Kevin_Zheng has joined #openstack-nova14:33
*** tbachman has joined #openstack-nova14:37
takashindansmith: Would you review https://review.openstack.org/#/c/408955/ again?14:40
*** krtaylor has joined #openstack-nova14:42
*** READ10 has joined #openstack-nova14:42
*** crushil has joined #openstack-nova14:43
*** hongbin has quit IRC14:43
*** krtaylor has quit IRC14:44
mriedemtakashin: i'm confused,14:44
mriedemhttps://review.openstack.org/#/c/408964/49/nova/api/openstack/api_version_request.py and https://review.openstack.org/#/c/459483/3/nova/api/openstack/api_version_request.py are competing for the same microversion,14:44
mriedembut i thought the abort cold migration blueprint dependent on being able to list cold migrations?14:45
*** mdrabe has quit IRC14:45
mriedemoh nvm14:45
mriedemi'm thinking of a different blueprint14:45
mriedemthis is cold migration with a target host14:45
*** mdrabe has joined #openstack-nova14:45
*** lpetrut has quit IRC14:48
*** krtaylor has joined #openstack-nova14:48
*** claudiub has quit IRC14:48
*** tbachman has quit IRC14:49
*** mdrabe has quit IRC14:50
takashinmriedem: As you said, there is no dependency between "cold migration with target host" and "List/show all migration types" blueprint.14:51
*** karimb has joined #openstack-nova14:53
*** jwcroppe_ has quit IRC14:53
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix uuid replacement in aggregate notification test  https://review.openstack.org/46589614:55
mriedemsdague: i'm confused that USE_SCREEN=False on the old side of this live migration grenade job http://logs.openstack.org/42/465042/5/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/6aae0c3/logs/old/local_conf.txt.gz14:55
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix uuid replacement in aggregate notification test  https://review.openstack.org/46589614:55
mriedemi thought screen was still the default in devstack in ocata?14:55
*** armax has joined #openstack-nova14:56
mriedemi guess it is https://github.com/openstack-dev/devstack/blob/stable/ocata/stackrc#L8814:56
*** mdrabe has joined #openstack-nova14:56
*** nurdie has quit IRC14:57
*** gianpietro has quit IRC14:57
sdaguemriedem: the confusion is there are 3 operating modes right now14:58
sdagueUSE_SCREEN=False before a month ago, means direct exec everything in bash and hope you captured the pid correctly to kill it later14:59
sdaguewe've used that in the gate for the last 3 years because screen races14:59
sdagueand in the non repeat state, it's fine.14:59
sdagueit is a different home grown init system with different bad trade offs than screen15:00
*** jaosorior is now known as jaosorior_away15:00
*** yingjun has joined #openstack-nova15:01
*** psachin has quit IRC15:01
dpawlikHi.If you have some time pls review my PS https://review.openstack.org/#/c/459753/15:01
dpawlikthanks a lot15:02
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification  https://review.openstack.org/39621015:02
ftersinmriedem: hi. who could review https://review.openstack.org/#/c/407440/ ?15:04
*** gianpietro has joined #openstack-nova15:04
*** takashin has left #openstack-nova15:04
ftersinmriedem: who of you guys is more familiar with that area?15:04
mriedemftersin: mdbooth15:04
ftersinmriedem: thx15:05
*** gianpiet_ has joined #openstack-nova15:05
*** dpawlik has quit IRC15:05
ftersinmdbooth: hi. are you around?15:07
mdboothftersin: Yep15:07
*** Oku_OS is now known as Oku_OS-away15:07
ftersinmdbooth: could you look at https://review.openstack.org/#/c/407440/ again?15:07
mdboothYeah, sure.15:08
*** Guest16367 has quit IRC15:08
mdboothSo, my issue with that patch has always been the number of changes required in driver.py15:08
*** yamamoto has quit IRC15:08
*** gianpietro has quit IRC15:08
*** dpawlik has joined #openstack-nova15:08
mdboothWhich is obvious an artifact of the imagebackend interface being completely borked15:08
ftersinmdbooth: i addressed some your comments there. so, it'd be good to get new ones :)15:08
mdboothSomebody should clean that up... ;)15:09
mdboothAnyway, we also agreed at some point to move on with it anyway, so I'll try to look again in that light15:09
mriedemsdague: ok, so maybe i didn't ask the right question,15:09
mriedemi'm basically wondering what i can key off of to fix this grenade live migration issue15:10
sdaguemriedem: no15:10
sdagueyou are going to need more complex logic15:10
openstackgerritChris Friesen proposed openstack/nova master: correctly log port id in neutron api  https://review.openstack.org/46600315:10
cfriesen^ trivial fix to logging15:10
sdaguemriedem: https://github.com/openstack-dev/devstack/blob/6a62619cbc28625755a76eae08b529438eb4f281/stackrc#L166-L17315:11
sdaguethat's the gate level work around, and why grenade works15:11
cfriesencrap, let me respin that...an extra file snuck in15:11
*** annegentle has joined #openstack-nova15:11
*** jwcroppe has joined #openstack-nova15:12
*** claudiub has joined #openstack-nova15:12
sdaguemriedem: USE_SCREEN=False is always the case for the gate15:12
sdagueit's just that we decided to make the transition simpler by making that mean something else15:13
sdagueso 95% of things just worked15:13
dpawlikmriedem: thanks for review. DocImpact-Group: openstack/glance is because someone on this channel told me that docs are related to Openstack Glance. I will create a new PS soon. Thanks!15:13
mriedemsdague: ok i see that in devstack-gate localrc_set "$localrc_file" "USE_SCREEN" "False"15:13
sdaguethe grenade cases needed this one extra hit15:13
sdaguehint15:13
ftersinmdbooth: this patch does not bring a lot of changes for driver.py. Moreover, some changed might be separated out of this patch. E.g. see a series started with https://review.openstack.org/#/c/41194215:13
sdaguethen you are dealing with things that wrote their own service management logic outside of devstack15:13
sdaguewhich reacted poorly15:14
sdaguebecause their start/stop/restart calls aren't going through devstack code with vars that would have done the right things15:14
openstackgerritChris Friesen proposed openstack/nova master: correctly log port id in neutron api  https://review.openstack.org/46600315:14
sdagueso options are: 1) replicate special logic15:14
sdague2) redo them to use devstack functions15:15
sdague3) not care about this one job that's in a tarpit15:15
sdaguegiven that general live migration post upgrade is already tested, and multi backend live migration on master is already tested15:15
sdaguejust not the combo of the 215:15
*** aarefiev is now known as aarefiev_afk15:15
mriedemgeneral live migration post upgrade being the grenade multinode job?15:16
sdagueyes15:16
sdagueso the missing test coverage right now is post upgrade live migration with ceph / nfs15:17
mriedemonly ceph actually15:17
mriedemthe nfs stuff was never fixed15:17
sdagueok, so only ceph15:17
mriedemwhat's multibackend live migration?15:17
sdaguealso, note, it's not a vm that predates the migration15:17
mriedemthe back and forth thing?15:17
sdaguemriedem: the normal livemigration job15:17
sdaguethat tests local disk, and ceph, and I guess not nfs any more15:17
mriedemthe back and forth thing was meant to test live migrating from n to n-1 and back15:18
*** gomarivera_ has joined #openstack-nova15:18
sdagueoh, right, this is mixed computes?15:18
sdagueright so mixed computes with ceph, not tested15:18
sdaguethat's what's currently lost15:18
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification  https://review.openstack.org/39621115:19
openstackgerritArtom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment  https://review.openstack.org/41562215:20
openstackgerritArtom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface  https://review.openstack.org/43538715:20
*** cdent has joined #openstack-nova15:20
openstackgerritArtom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume  https://review.openstack.org/43508215:20
openstackgerritArtom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment  https://review.openstack.org/39194115:20
openstackgerritArtom Lifshitz proposed openstack/nova master: API support for tagged device attachment  https://review.openstack.org/37602115:20
mriedemsdague: i'm confused as to why i don't see live migration tempest tests running in the normal grenade multinode job then http://logs.openstack.org/42/465042/5/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/22cb09f/console.html15:21
mriedemoh b/c smoke15:21
mriedemyeah, that was another thing the grenade live miration job did, was bypass the smoke test stuff15:21
mriedemso the normal grenade multinode job is not running live migration tests since they aren't smoke tests15:21
*** moshele has joined #openstack-nova15:21
sdagueum... really?15:21
sdagueI thought we added those back in tehre15:22
*** slaweq has joined #openstack-nova15:22
mriedemyeah it's why we added this https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L169815:22
*** slaweq has quit IRC15:22
mriedemto the grenade live migration job15:22
sdagueah15:22
mriedemso we don't have mixed compute live migration back and forth from n to n-1 and back testing15:23
sean-k-mooneymriedem: QQ what is the hosting requirements(days to keep logs) for the 3rd party ci logs? the systemd change has tripled the log size so we are running out of space.15:23
*** slaweq has joined #openstack-nova15:23
mriedemsean-k-mooney: no idea15:23
mriedemsean-k-mooney: i'd ask in infra probably15:23
sdaguemriedem: ok, I honestly though we still had basic live migration in multinode grenade, maybe it wasn't reliable enough15:24
mriedemsean-k-mooney: https://docs.openstack.org/infra/system-config/third_party.html15:24
mriedem Note All test artifacts must be retained for one month.15:24
sean-k-mooneymriedem: we were aiming for 90 days before im looking into geting more storage but we might have to reduce for a bit while we get that set up15:24
mriedemsean-k-mooney: sounds like 1 month minimum15:24
sdaguemriedem: ok, so option 4) don't do the special ceph live migration job15:24
*** hongbin has joined #openstack-nova15:24
sdagueand just make that job run live migration grenade tests15:24
sean-k-mooneymriedem: cool thats what i taught. we might reduce to 45 days while we see if we can upgrade form 500GB to 1TB of hosting15:25
sdaguebecause it's the post_hook that's a bit hacking and a problem15:25
sean-k-mooneythe nova jobs are eating ~ 13.5GB per day post the systemd change15:25
mriedemsdague: the special ceph live migration job being the non-grenade live migration job that runs with ceph too? that one isn't a problem.15:26
sdaguemriedem: I'm saying it all got too compled15:26
sdaguethere is grenade-multi15:26
mriedemgate-grenade-dsvm-neutron-multinode-ubuntu-xenial15:27
mriedemyes15:27
sdaguesorry, let me blast this out15:27
* mriedem braces for impact15:27
sdaguegrenade-multi - no ceph, no live migration15:27
sdaguedsvm-multi - no ceph, live migration15:27
sdaguelive-migration - ceph, live migration (custom post_hook, fixed)15:27
sdaguegrenade-live-migration - ceph, live migration (custom post_hook as above, doesn't work across systemd transition)15:28
sdaguethere is no grenade-multi-full15:28
sdaguethat would give you working live migration testing today15:28
sdaguethe equiv of dsvm-multi15:29
*** jdillaman has joined #openstack-nova15:29
*** gomarivera_ has quit IRC15:29
mriedemyeah, now i'm trying to see what's setting live_migrate_back_and_forth in tempest.conf15:30
*** winston-d_ has joined #openstack-nova15:30
mriedembecause the dsvm-multi job isn't doing it15:30
mriedemah only set in the grenade-dsvm-neutron-multinode-live-migration job15:31
*** tbachman has joined #openstack-nova15:33
mriedembut i don't actually see that set in tempest.conf in the job, probably because of the hackery in the hooks in nova15:33
mriedemfor that job15:33
*** salv-orlando has joined #openstack-nova15:33
*** fragatina has joined #openstack-nova15:33
*** fragatina has quit IRC15:33
mriedemman i don't even know anymore, i might just give up15:34
*** fragatina has joined #openstack-nova15:34
*** nurdie has joined #openstack-nova15:37
*** moshele has quit IRC15:37
*** mamandle has quit IRC15:44
*** jamesdenton has joined #openstack-nova15:45
*** markus_z has quit IRC15:45
*** dikonoo has quit IRC15:45
*** dikonoor has joined #openstack-nova15:45
openstackgerritTao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver.  https://review.openstack.org/41446315:47
*** rcernin has quit IRC15:47
mriedemildikov: johnthetubaguy: i can't make the nova/cinder meeting today15:52
mriedemstvnoyes: ^15:53
mriedemi also haven't had a chance to run through steve's live migration wip changes yet15:53
ildikovmriedem: thanks for the heads up15:54
*** Apoorva has joined #openstack-nova15:55
sean-k-mooneymriedem: were you looking for where in the code the live_migration flag is set in the gate-grenade-dsvm-neutron-multinode-live-migration-nv or just what file it was set in15:55
mriedemsean-k-mooney: it's set in local.conf https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L170315:56
mriedemand should be set in tempest.conf during devstack setup here https://github.com/openstack-dev/devstack/blob/master/lib/tempest#L36415:57
kashyapcfriesen: Looked at your patch this morning, responded on it15:57
sean-k-mooneymriedem: ah you found it so i was goint to take a look and see if i could find it15:57
mriedembut i'm not actually seeing it set in tempest.conf in that job15:57
kashyapcfriesen: It looks fine to me, nice catch.  I presume you stumbled on it by testing15:57
sean-k-mooneyi am15:57
kashyap(I mean, in real env)15:57
*** dpawlik is now known as _dpawlik15:57
sean-k-mooneymriedem: http://logs.openstack.org/60/465160/2/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/c34cb8b/logs/new/tempest_conf.txt.gz15:57
*** belmoreira has quit IRC15:58
mriedemsean-k-mooney: i don't see live_migrate_back_and_forth in there15:58
sean-k-mooneyor sorry i taught you were just looking for live_migration = True sorry15:58
mriedemit's in the loca.conf though http://logs.openstack.org/42/465042/5/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/6aae0c3/logs/new/local.conf.txt.gz15:58
mriedemLIVE_MIGRATE_BACK_AND_FORTH=True15:58
dansmithmriedem: omg, is that really a flag?16:00
*** gszasz has quit IRC16:00
mriedemit is16:00
dansmithlove. it.16:00
mriedemi must be missing something16:01
cfriesenkashyap: thanks for the review...we did indeed trip over it16:01
sean-k-mooneymriedem: hum maybe it would be better to skip the macro and update the tempest conf via setting [[test-config|/opt/stack/new/tempest/etc/tempest.conf]] ... section in the local.conf16:01
mriedemsean-k-mooney: that didn't exist when this was added16:01
mriedemand this used to work, i'm pretty sure16:01
*** karthiks has quit IRC16:01
mriedemi see the devstack flag in the grenade log16:01
mriedemhttp://logs.openstack.org/42/465042/5/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/6aae0c3/logs/grenade.sh.txt.gz#_2017-05-18_00_34_34_95616:01
kashyapcfriesen: Should that be documented the format-domain for libvirt upstream, too?16:02
dansmithmriedem: probably unintentional, but just thinking of the potential "me and you and everyone we know" reference16:02
sean-k-mooneymriedem: well its only used once in devstack and its pretty clear what its trying to do so that is strange that its not working https://github.com/openstack-dev/devstack/blob/ac042d41222eea83bf360b4db89c4c7bf415457d/lib/tempest#L36416:02
mriedemwtf it should happen right here http://logs.openstack.org/42/465042/5/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/6aae0c3/logs/grenade.sh.txt.gz#_2017-05-18_00_21_27_06816:02
kashyapcfriesen: Okay, currently I see it documented under 'vcpu' -> 'cpuset' there16:02
*** gomarivera_ has joined #openstack-nova16:03
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology  https://review.openstack.org/40824216:03
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology  https://review.openstack.org/40824116:03
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology  https://review.openstack.org/46603016:03
kashyapcfriesen: Probably that's good enough: "Each element in that list is either a single CPU number, a range of CPU numbers, or a caret followed by a CPU number to be excluded from a previous range."16:03
sfinucandansmith: Finally got one of those working ^16:03
mriedemsean-k-mooney: i see the problem16:03
dansmithsfinucan: cool16:03
mriedemhttps://github.com/openstack-dev/devstack/blob/stable/ocata/lib/tempest16:03
mriedemthe devstack flag didn't get into devstack stable/ocata16:03
mriedemand tempest.conf is configured on the old side in the grenade run16:04
sfinucandansmith: ...however, 'nova.tests.unit.virt.test_hardware.HelperMethodsTestCase.test_dict_numa_topology_to_obj' is failing for the other two and I can't figure out why16:04
*** aloga has quit IRC16:04
*** aloga has joined #openstack-nova16:04
sean-k-mooneymriedem: ah ok that makes sense16:04
sfinucanIf you have a chance to look at that at some point, could you?16:04
sfinucanFor some reason, a dict is not behaving like a dict :S16:04
sfinucanstupid ovo16:05
cfriesenkashyap: yes, I think that's okay.  the implicit all-cpus was just in the nova code.16:05
mriedemsean-k-mooney: so i just needed to backport the fix to devstack https://review.openstack.org/#/c/466033/16:05
mriedemnow having done that, the grenade live migration job is still broken, as discussed with sdague earlier16:06
mriedemand i'm not sure i have the energy atm to untangle that mess16:06
sean-k-mooneymriedem: yes backport or swap to useing the test-config update to the file which is supported on ocata already https://github.com/openstack-dev/devstack/blob/stable/ocata/stack.sh#L140516:07
kashyapcfriesen: True.  Okay, I'll go cook food, and head to my Dutch class, then :-)16:07
*** yingjun has quit IRC16:07
*** mdnadeem has quit IRC16:08
*** jose-phillips has quit IRC16:08
sean-k-mooneymriedem: oh the flag is new. backporting makes sense then16:08
*** yamamoto has joined #openstack-nova16:09
*** jose-phillips has joined #openstack-nova16:09
*** burt has joined #openstack-nova16:12
sean-k-mooneysfinucan: how dare you disbarage ovo :P16:13
sfinucansean-k-mooney: I'm sure it's lovely, but I've just burnt through a full day looking at ovo-related issues. I ain't feeling it today ;)16:13
sean-k-mooneythere a child of nova object you have to treat them kindly but ya they can be a pain.16:14
*** Yangyapeng has quit IRC16:14
*** yamamoto has quit IRC16:15
alex_xumriedem: sdague sfinucan oomichi there are some nova bug fix in the etherpad from bugsmash https://etherpad.openstack.org/p/OpenStack-Bug-Smash-Pike-Suzhou-Bug-List, appreciate if you can review few of them when you have free time, thanks in advance!16:16
*** gianpietro has joined #openstack-nova16:16
openstackgerritAlex Xu proposed openstack/nova master: Correct the error message for query parameter validation  https://review.openstack.org/46578516:17
*** yingjun has joined #openstack-nova16:17
*** catintheroof has joined #openstack-nova16:19
dansmithmriedem: so, I'm getting this on my cells test: http://logs.openstack.org/37/458537/20/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/ab364da/logs/testr_results.html.gz16:19
*** gianpiet_ has quit IRC16:19
dansmithmriedem: but I'm failing because of lack of shared storage, which shouldn't be affected by what I'm doing16:19
dansmithoh, um, hmm..16:19
*** nic has joined #openstack-nova16:20
dansmithI wonder if it's because I'm mucking with the config for compute..16:20
*** pewp has quit IRC16:20
*** karimb has quit IRC16:21
*** karimb has joined #openstack-nova16:23
dansmithmriedem: isn't there some late hooky stuff going on from the nova tree for the live migration job?16:23
dansmithsdague: ^16:23
openstackgerritStephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'log' commands  https://review.openstack.org/45317616:23
openstackgerritStephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'host' commands  https://review.openstack.org/45317516:23
openstackgerritStephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'project', 'account' commands  https://review.openstack.org/45317416:23
sfinucanalex_xu: I'll trade you some of yours for some of mine ;) ^^^16:23
sfinucanartom: Addressed your comment in above change, if you're about ^16:24
*** nic has quit IRC16:24
zzzeeksdague: what is a "5 byte utf8" character ?16:24
artomsfinucan, I'm about16:25
*** fragatina has quit IRC16:25
*** salv-orlando has quit IRC16:25
*** salv-orlando has joined #openstack-nova16:26
*** salv-orlando has quit IRC16:27
*** pewp has joined #openstack-nova16:27
*** salv-orl_ has joined #openstack-nova16:27
sfinucanzzzeek: UTF-8 chars can vary in size from 1 to 6 bytes, it appears https://stackoverflow.com/q/5290182/16:28
sdaguezzzeek: there was a nova bug on it at some point16:28
zzzeeksdague sfinucan OK but unicode itself only needs four bytes16:29
zzzeeksdague sfinucan MySQL / Posgresql / anything only goes out to 4 bytes max16:29
sdagueok, good to know, it was in the realm of "um... so not able to go there"16:29
zzzeekhttps://en.wikipedia.org/wiki/UTF-8#Description16:29
zzzeekOK16:29
sean-k-mooneysfinucan: i thinkg its 1-4 bytes if i remember correctly16:30
zzzeeksdague: thougth that was weird, b.c. why not refer to "6 byte support" :)16:30
sdagueI guess I had misunderstood that unicode consortium had crossed that line16:30
sfinucansean-k-mooney: Are you saying Google lied to me? :O16:30
mdboothIIRC UTF-8 characters can be arbitrarily many bytes, because they may combine arbitrarily many codepoints16:30
*** yingjun has quit IRC16:30
zzzeekmdbooth: the hypotthetical max is 6 bytes16:31
*** Swami has quit IRC16:31
sfinucanAnd this is the point where I go chuck a ball about in the sunshine :)16:31
sean-k-mooneysfinucan: yes hehe ill double check but when i was working with it in the past i rember it being 4 bytes max for some reason16:31
openstackgerritDan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path  https://review.openstack.org/45863616:31
openstackgerritDan Smith proposed openstack/nova master: Move to proper target_cell calling convention  https://review.openstack.org/45863416:31
openstackgerritDan Smith proposed openstack/nova master: Make target_cell() yield a new context  https://review.openstack.org/45863516:31
openstackgerritDan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test  https://review.openstack.org/46151316:31
mdboothzzzeek: I think that's for a single codepoint?16:31
openstackgerritDan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack  https://review.openstack.org/45853716:31
openstackgerritDan Smith proposed openstack/nova master: Target cell on local delete  https://review.openstack.org/46405916:31
mdboothA character may be multiple codepoints16:31
zzzeekmdbooth: now that you have to show me16:32
zzzeek:)16:32
zzzeekmdbooth: im pretty sure no such voodoo is used in practice16:32
mdboothzzzeek: Like a-umlaut, for eg16:32
mdboothIIRC that's 2 code points, but 1 char16:32
zzzeekmdbooth: i mean, in databases16:32
*** jmlowe has quit IRC16:33
mdboothWell it's all strings ultimately.16:33
mdboothIt boils down to the definition of a character.16:33
dansmithplease don't let my snoring interrupt this riveting conversation16:34
zzzeekmdbooth: OK! well.  writing an email here, specifics, things that affect us right now :)16:34
sdaguezzzeek: I'll update the pg spec with that, it's good info16:35
*** gjayavelu has joined #openstack-nova16:35
*** dikonoor has quit IRC16:35
*** lucasagomes is now known as lucas-afk16:35
mdboothdansmith zzzeek: Heh, yeah, its unlikely to be of interest to us :) It's of interest when you need to display text and, for eg, select characters with a mouse. I read a fantastic article about it recently but I can't find it right now.16:39
sean-k-mooneysfinucan: looking at https://tools.ietf.org/html/rfc3629 section 10 "Security Considerations" we are both right.... the utf-8 encounding would support up to 6 bytes but such encounding produce illegal  and only  up to 4 bytes are vailid16:39
mdboothQuestions like: what should pressing backspace do in a text editor are actually really hard to answer.16:40
*** pcaruana has quit IRC16:40
*** pchavva has quit IRC16:41
*** smatzek_ has joined #openstack-nova16:42
*** burt has quit IRC16:44
*** smatzek has quit IRC16:46
*** yamahata has quit IRC16:46
*** burt has joined #openstack-nova16:46
sean-k-mooneysdague: zzzeek mdbooth if ye just define the field as  an NVARCHAR in sqlalchemy and set the coalation type for the table to utf8_bin or utf8_general_ci/utf8_unicode_ci the db should handel it its self16:47
*** gjayavelu has quit IRC16:50
openstackgerritJianghua Wang proposed openstack/nova master: Skip policy rules on attach_network for none network allocation  https://review.openstack.org/46590316:50
*** karthiks has joined #openstack-nova16:54
*** vks1 has joined #openstack-nova16:54
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925716:55
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177716:55
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions  https://review.openstack.org/46605116:55
*** salv-orl_ has quit IRC16:56
*** salv-orlando has joined #openstack-nova16:56
*** pchavva has joined #openstack-nova16:57
*** ltomasbo is now known as ltomasbo|away16:57
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925716:57
*** nurdie has quit IRC16:57
*** derekh has quit IRC16:58
*** fragatina has joined #openstack-nova16:59
*** mlakat has quit IRC16:59
*** ralonsoh has quit IRC17:00
*** Apoorva_ has joined #openstack-nova17:00
*** salv-orlando has quit IRC17:00
*** pchavva has quit IRC17:02
*** pchavva has joined #openstack-nova17:02
*** hshiina has quit IRC17:03
*** Apoorva has quit IRC17:03
*** sambetts is now known as sambetts|afk17:03
*** fragatina has quit IRC17:04
*** gomarivera_ has quit IRC17:05
*** psachin has joined #openstack-nova17:06
*** claudiub has quit IRC17:07
*** jpena is now known as jpena|off17:07
*** jmlowe has joined #openstack-nova17:07
*** jose-phillips has quit IRC17:08
*** jmlowe has quit IRC17:09
*** jmlowe has joined #openstack-nova17:09
*** yamamoto has joined #openstack-nova17:11
*** jose-phillips has joined #openstack-nova17:12
*** nurdie has joined #openstack-nova17:14
*** yamamoto has quit IRC17:16
*** jmlowe has quit IRC17:17
*** jmlowe has joined #openstack-nova17:18
*** nurdie has quit IRC17:18
*** karthiks has quit IRC17:19
*** claudiub has joined #openstack-nova17:20
*** fragatina has joined #openstack-nova17:22
*** burt has quit IRC17:22
*** yamahata has joined #openstack-nova17:23
*** salv-orlando has joined #openstack-nova17:29
*** nurdie has joined #openstack-nova17:29
*** harlowja has joined #openstack-nova17:30
*** jmlowe has quit IRC17:32
*** jmlowe has joined #openstack-nova17:32
*** salv-orlando has quit IRC17:34
*** avolkov has quit IRC17:37
*** ijw has joined #openstack-nova17:38
*** jmlowe has quit IRC17:38
*** jmlowe has joined #openstack-nova17:38
*** Apoorva_ has quit IRC17:39
*** eandersson has joined #openstack-nova17:39
*** Apoorva has joined #openstack-nova17:39
*** gyee_ has joined #openstack-nova17:41
*** jose-phillips has quit IRC17:43
*** gyee has quit IRC17:43
*** gyee_ has quit IRC17:43
*** gyee has joined #openstack-nova17:43
*** jose-phillips has joined #openstack-nova17:44
*** ijw has quit IRC17:51
*** ijw has joined #openstack-nova17:51
*** jmlowe has quit IRC17:54
*** jmlowe has joined #openstack-nova17:54
*** annegentle has quit IRC17:55
*** gjayavelu has joined #openstack-nova17:55
*** pchavva has quit IRC17:56
*** armax has quit IRC17:57
*** vks1 has quit IRC17:58
*** jmlowe has quit IRC17:59
*** jmlowe has joined #openstack-nova18:00
*** pchavva has joined #openstack-nova18:00
*** jmlowe has quit IRC18:00
*** harlowja has quit IRC18:01
*** jmlowe has joined #openstack-nova18:01
*** Kevin_Zheng has quit IRC18:03
*** fragatina has quit IRC18:07
*** jmlowe has quit IRC18:08
*** fragatina has joined #openstack-nova18:08
*** jmlowe has joined #openstack-nova18:08
*** jmlowe has quit IRC18:10
*** claudiub has quit IRC18:11
*** jmlowe has joined #openstack-nova18:11
*** yamamoto has joined #openstack-nova18:13
*** harlowja has joined #openstack-nova18:15
*** ZZelle_ has joined #openstack-nova18:15
*** gianpietro has quit IRC18:16
openstackgerritmelanie witt proposed openstack/nova master: Cache database and message queue connection objects  https://review.openstack.org/46504218:17
*** gianpietro has joined #openstack-nova18:18
ZZelle_sfinucan, hi18:18
*** yamamoto has quit IRC18:18
*** markvoelker has quit IRC18:19
*** markvoelker has joined #openstack-nova18:20
*** smatzek_ has quit IRC18:23
*** markvoelker has quit IRC18:24
*** jose-phillips has quit IRC18:25
*** MVenesio has quit IRC18:27
*** Sukhdev has joined #openstack-nova18:30
*** armax has joined #openstack-nova18:32
*** marst has joined #openstack-nova18:32
marstedleafe: are you around?18:34
*** claudiub has joined #openstack-nova18:34
openstackgerritchristopher.uhler proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale  https://review.openstack.org/44395118:36
openstackgerritDan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack  https://review.openstack.org/45853718:37
*** gyee has quit IRC18:38
*** thorst_afk has quit IRC18:43
*** eharney has quit IRC18:48
*** mnestratov has quit IRC18:48
*** gianpietro has quit IRC18:51
*** eharney has joined #openstack-nova18:51
cdentmarst: edleafe is probably on a plane on his way to pycon18:53
*** dillaman has joined #openstack-nova18:55
*** harlowja has quit IRC18:56
*** karthiks has joined #openstack-nova18:58
marstcdent: thanks! I've just read his blog post18:59
*** salv-orlando has joined #openstack-nova19:04
*** gianpietro has joined #openstack-nova19:05
*** READ10 has quit IRC19:05
*** salv-orlando has quit IRC19:05
*** annegentle has joined #openstack-nova19:06
*** harlowja has joined #openstack-nova19:07
*** cdent has quit IRC19:10
*** nurdie has quit IRC19:10
*** karimb has quit IRC19:12
*** karimb has joined #openstack-nova19:13
*** crushil has quit IRC19:14
*** yamamoto has joined #openstack-nova19:15
openstackgerritSean Dague proposed openstack/nova master: see if we can use pg wheels with no bindep  https://review.openstack.org/46608119:15
*** nurdie has joined #openstack-nova19:15
*** MVenesio has joined #openstack-nova19:19
*** jose-phillips has joined #openstack-nova19:19
*** adisky_ has quit IRC19:19
*** MVenesio has quit IRC19:19
*** yamamoto has quit IRC19:21
*** MVenesio has joined #openstack-nova19:22
*** jose-phillips has quit IRC19:22
*** jose-phillips has joined #openstack-nova19:24
mriedemdansmith: did you figure out http://logs.openstack.org/37/458537/20/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/ab364da/logs/testr_results.html.gz ?19:26
*** MVenesio has quit IRC19:30
*** dave-mccowan has quit IRC19:31
*** krtaylor has quit IRC19:33
*** crushil has joined #openstack-nova19:35
*** Sukhdev has quit IRC19:35
*** krtaylor has joined #openstack-nova19:37
*** gouthamr has quit IRC19:37
*** krtaylor has quit IRC19:40
*** krtaylor has joined #openstack-nova19:41
openstackgerritCedric Brandily proposed openstack/nova master: Correct _ensure_console_log_for_instance implementation  https://review.openstack.org/46608819:47
mriedemthis seems wrong http://logs.openstack.org/42/465042/6/check/gate-nova-tox-functional-py35-ubuntu-xenial/9829a45/console.html#_2017-05-18_18_43_03_02029719:48
mriedemb"KeyError: 'allocations'"19:49
*** liverpooler has quit IRC19:49
dansmithmriedem: I found the hook yeah, still working on a fix19:49
mriedemok,19:50
mriedemi'm working on disabling the grenade live migration job from running on pike until we fix it, or queens19:50
mriedemsince right now it's just burning nodes19:50
*** itlinux has joined #openstack-nova19:51
mriedemthis is very weird - http://logs.openstack.org/42/465042/6/check/gate-nova-tox-functional-py35-ubuntu-xenial/9829a45/console.html#_2017-05-18_18_43_03_020297 it's like in the middle of a functional test, we got a different response from placement19:51
mriedemlike it's a GET / response19:52
mriedembut was expecting GET /resource_providers/d41180aa-047e-4bc3-b0f3-11532683400d/allocations19:52
openstackgerritCedric Brandily proposed openstack/nova master: Correct _ensure_console_log_for_instance implementation  https://review.openstack.org/46608819:54
*** gouthamr has joined #openstack-nova19:56
*** gouthamr has quit IRC19:57
melwittmriedem: yeah, I didn't know what to make of that19:58
*** ijw has quit IRC20:00
*** mnestratov has joined #openstack-nova20:02
*** harlowja has quit IRC20:03
mriedemcould have something to do with running those services under eventlet in our functional tests?20:03
*** eharney has quit IRC20:05
*** annegentle has quit IRC20:07
*** pchavva has quit IRC20:09
*** kornica has joined #openstack-nova20:09
*** slaweq has quit IRC20:11
*** kornicameister has quit IRC20:11
*** slaweq has joined #openstack-nova20:11
*** awaugama has quit IRC20:11
*** itlinux has quit IRC20:12
*** annegentle has joined #openstack-nova20:13
*** ijw has joined #openstack-nova20:16
*** yamamoto has joined #openstack-nova20:17
*** esberglu has quit IRC20:18
efriedWhere does the hashy thing at the end of a release note .yaml's basename come from?20:20
openstackgerritDan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack  https://review.openstack.org/45853720:21
artomefried, the 'reno' command does it20:21
*** yamamoto has quit IRC20:21
efriedartom Is there a doc somewhere that describes the release note process?20:23
efried(Cause your answer just raised bigger questions ;-)20:23
artomefried, not that I know of, but if you have a devstack env somewhere reno should be in there20:23
openstackgerritMatt Riedemann proposed openstack/nova master: Skip ceph in grenade live migration job due to restart failures  https://review.openstack.org/46609720:23
mriedemsdague: i think i've found a poor man's compromise ^20:23
artomefried, in which case reno --help will help :)20:23
*** cdent has joined #openstack-nova20:23
*** winston-d_ has quit IRC20:23
mriedemefried: artom: https://docs.openstack.org/developer/nova/releasenotes.html20:24
efriedartom Yeah, found reno --help; it kinda assumes you know what youre doing already.20:24
efriedOkay, thank you artom20:24
mriedemthe hash is auto-generated20:24
mriedemtox -e venv -- reno new <your reno name>20:24
mriedemdhellmann said he's found that sometimes people copy renos and re-use the same hash which messes things up20:25
efriedI guessed that would be a no-no20:25
efriedThanks mriedem & artom20:25
mriedemyeah, plus reviewers wouldn't be able to eyeball that20:25
mriedemseems the reno docs build itself should fail if that happens20:25
*** pchavva has joined #openstack-nova20:27
*** gouthamr has joined #openstack-nova20:29
*** harlowja has joined #openstack-nova20:30
*** esberglu has joined #openstack-nova20:30
*** belmoreira has joined #openstack-nova20:32
*** krtaylor has quit IRC20:34
*** pchavva has quit IRC20:36
*** belmoreira has quit IRC20:37
*** itlinux has joined #openstack-nova20:39
*** gianpietro has quit IRC20:39
*** gianpiet_ has joined #openstack-nova20:39
*** harlowja has quit IRC20:39
*** thorst_afk has joined #openstack-nova20:39
*** thorst_afk has quit IRC20:40
*** gianpiet_ has quit IRC20:45
*** salv-orlando has joined #openstack-nova20:45
*** jbernard has quit IRC20:47
*** ZZelle_ has quit IRC20:49
*** thorst_afk has joined #openstack-nova20:50
*** Sukhdev has joined #openstack-nova20:50
*** jbernard has joined #openstack-nova20:50
*** hamzy has quit IRC20:52
*** xyang1 has joined #openstack-nova20:54
*** dave-mccowan has joined #openstack-nova20:57
*** jmlowe has quit IRC20:57
*** jbernard has quit IRC20:59
*** cdent has quit IRC21:01
*** jbernard has joined #openstack-nova21:02
*** gianpietro has joined #openstack-nova21:02
*** gianpietro has quit IRC21:02
*** sdague has quit IRC21:06
*** gianpietro has joined #openstack-nova21:08
*** armax has quit IRC21:09
*** baoli_ has quit IRC21:12
*** jamesdenton has quit IRC21:12
*** jhesketh_ has joined #openstack-nova21:14
*** karimb has quit IRC21:16
*** NightKhaos_ has joined #openstack-nova21:17
*** yamamoto has joined #openstack-nova21:18
*** jhesketh has quit IRC21:18
*** NightKhaos has quit IRC21:18
*** swamireddy has quit IRC21:18
*** knikolla has quit IRC21:18
*** NightKhaos_ is now known as NightKhaos21:18
*** rfolco has quit IRC21:18
*** yamamoto has quit IRC21:22
*** dimtruck is now known as zz_dimtruck21:24
openstackgerritAlex Szarka proposed openstack/nova master: Add method for verify multiple notifications  https://review.openstack.org/46552621:25
openstackgerritAlex Szarka proposed openstack/nova master: [WIP] Transform instance.exists notification  https://review.openstack.org/40366021:25
*** ijw has quit IRC21:29
*** ijw has joined #openstack-nova21:30
*** crushil_ has joined #openstack-nova21:30
openstackgerritMonty Taylor proposed openstack/nova master: WIP Show results of nested parameters in api-ref  https://review.openstack.org/46427721:33
openstackgerritMonty Taylor proposed openstack/nova master: WIP Add description of server addresses objects  https://review.openstack.org/46433121:33
*** gjayavelu has quit IRC21:34
*** ijw has quit IRC21:35
*** edmondsw_ has joined #openstack-nova21:36
*** gjayavelu has joined #openstack-nova21:36
*** baoli has joined #openstack-nova21:37
*** gouthamr has quit IRC21:37
*** edmondsw has quit IRC21:38
cfriesenmriedem: you around?21:39
*** fragatin_ has joined #openstack-nova21:39
mriedemsort of21:40
cfriesenmriedem: got a weird scenario...upgrades are in progress, one compute node hasn't been upgraded.  It gets rebooted without being upgraded and hits exception.ServiceTooOld21:40
cfriesenmriedem: are we just expecting people to upgrade in that case?21:41
openstackgerritAlex Szarka proposed openstack/nova master: Add method for verify multiple notifications  https://review.openstack.org/46552621:41
*** baoli has quit IRC21:42
cfriesenmriedem: actually, thinking about it a bit I'm not clear why the version of that compute node isn't being seen in the DB21:42
openstackgerritAlex Szarka proposed openstack/nova master: [WIP] Transform instance.exists notification  https://review.openstack.org/40366021:42
*** fragatina has quit IRC21:42
cfriesenmriedem: maybe the service is still being forced down by our management software, causing its version to be ignored by service_get_minimum_version()21:44
*** fragatin_ has quit IRC21:45
mriedemthat's a dansmith question21:45
*** fragatina has joined #openstack-nova21:46
mriedemcfriesen: it's blowing up here right? https://github.com/openstack/nova/blob/master/nova/objects/service.py#L30721:46
cfriesenmriedem: yep.21:47
*** catintheroof has quit IRC21:47
cfriesendown in the sql code we ignore the version of "forced-down" services, so maybe that's it21:47
*** thorst_afk has quit IRC21:47
mriedemright this query https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L49021:49
*** tbachman has quit IRC21:49
mriedemso all of your other computes are newton, and you've got a mitaka forced-down compute, and you're restarting it and it's blowing up because that mitaka compute is older than the minimum for all of the other computes, which is newton21:50
mriedemso upgrade that compute to newton21:51
cfriesenmriedem: yeah, I'm not certain it's forced down but it's the only thing that makes sense21:51
mriedemdo you have the service record from the db?21:52
*** slaweq has quit IRC21:52
*** annegentle has quit IRC21:52
cfriesenmriedem: could probably track it down.  would indeed be able to tell from that.   I'm just reviewing a proposed change, wasn't actually my issue.  one of the guys wants to bypass the check and I'm not a big fan.21:53
*** haplo37 has quit IRC21:53
*** crushil_ has quit IRC21:53
mriedemi mean you could PUT /os-services/force-down and mark that service as forced_down=False, which would change the min service version check to say mitaka is the oldest, but that might give you other issues if other things are running thinking that newton is the minimum21:53
cfriesenwe're not using auto-rpc versioning (yet, at least) so during an uprade I think it'd be okay as long as we haven't set the rpc version to newton yet.  will have to go get more info I think.21:54
*** mdrabe has quit IRC21:55
*** armax has joined #openstack-nova21:56
openstackgerritMatt Riedemann proposed openstack/nova master: Skip ceph in grenade live migration job due to restart failures  https://review.openstack.org/46609721:56
*** nurdie has quit IRC21:56
*** nic has joined #openstack-nova21:57
*** claudiub has quit IRC21:59
mriedemcfriesen: did you happen to try this out in your live migration recreate with the 16 ports and network-changed event issues? https://review.openstack.org/#/c/465787/22:00
mriedembefore i go too far down that rabbit hole (probably next week at this rate) it'd be good to know if it helps at all22:00
*** xyang1 has quit IRC22:01
*** baoli has joined #openstack-nova22:01
*** nic has quit IRC22:02
*** haplo37 has joined #openstack-nova22:02
*** efried has quit IRC22:02
*** itlinux has quit IRC22:03
*** itlinux has joined #openstack-nova22:04
*** edmondsw_ has quit IRC22:05
*** ijw has joined #openstack-nova22:05
*** edmondsw has joined #openstack-nova22:06
*** djohnsto has quit IRC22:06
*** slaweq has joined #openstack-nova22:06
*** edmondsw_ has joined #openstack-nova22:07
cfriesenmriedem: related to the other issue, it turns out that if you force the "old" nova-compute down, then try to remove the "force-down" it will fail because Service.save() checks the minimum version22:08
cfriesenmriedem: haven't tried that patch yet, but I plan to.22:08
mriedemoh that's a fun chicken and egg22:08
cfriesenisn't it just. :)22:08
*** Jeffrey4l_ has quit IRC22:08
*** edmondsw has quit IRC22:10
*** edmondsw_ has quit IRC22:11
*** crushil_ has joined #openstack-nova22:11
*** crushil_ has quit IRC22:12
*** kylek3h has quit IRC22:13
*** Jeffrey4l has joined #openstack-nova22:13
mriedemwell, seems like a bug to report. we could bypass the service min version check on save() i'd think, but i'm not sure what kind of weird situations that puts us in22:14
mriedemagain, like if you've already restarted control services that are caching what they think is the min service version (newton) and all of a sudden you have a running mitaka compute service22:15
cfriesenmriedem: just for fun, the original scenario was that we were trying to take down the compute node to upgrade it, so we marked nova-compute as forced-down, then we tried to down the neutron agent and hit an exception, so we tried to roll back and enable nova-compute and hit exception.ServiceTooOld22:15
*** slaweq has quit IRC22:15
cfriesennot sure there's a good solution here.22:16
mriedemcfriesen: why not just disable the service?22:16
*** efried has joined #openstack-nova22:16
mriedemPUT /os-services/disable-log-reason22:16
mriedemthat should take that nova-compute out of scheduling22:16
cfriesenmriedem: that's a good point...usually we only force down on a fault.  will follow up.22:17
*** yamamoto_ has joined #openstack-nova22:19
*** Sukhdev has quit IRC22:20
mriedemhonestly i'm not sure what the use case is for forced-down which is different from disabling the service22:21
*** tbachman has joined #openstack-nova22:21
mriedemi think it has to do with external tooling forcing the service to be 'down' rather than waiting for the service group api check to do it22:21
mriedemwe could document that better in the api ref22:22
mriedemlike, what does doing this actually mean, what impacts / side effects does it have, etc22:22
*** gyee has joined #openstack-nova22:22
mriedemcfriesen: ^ probably another bug to open for lack of clear docs on these22:22
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/liberty/implemented/mark-host-down.html was the spec22:22
mriedemoh i see, the idea is external service finds out the host is about to die, forces it down and then evacuates the instances off that host22:23
mriedemsince evacuate checks this flag22:23
mriedemwe should state in the api-ref that forced-down is not something for routine upgrade/maintenance22:24
*** yamamoto_ has quit IRC22:24
mriedemas documented here https://docs.openstack.org/ops-guide/ops-maintenance-compute.html#planned-maintenance22:24
*** diablo_rojo has quit IRC22:27
*** salv-orlando has quit IRC22:29
mriedemcfriesen: https://bugs.launchpad.net/nova/+bug/169187122:30
openstackLaunchpad bug 1691871 in OpenStack Compute (nova) "forced-down vs service disable is not documented well in the compute API reference" [Medium,Confirmed]22:30
*** gjayavelu has quit IRC22:30
*** salv-orlando has joined #openstack-nova22:34
*** Sukhdev has joined #openstack-nova22:36
cfriesenmriedem: it's actually kind of ironic (always feels funny using that word around openstack) since we were one of the groups pushing for the introduction of the "force-down" feature.  now looks like someone's misusing it.22:37
*** salv-orlando has quit IRC22:38
*** Yangyapeng has joined #openstack-nova22:39
cfriesenmriedem: in our environment we have something else that monitors compute node health, so it'll detect the loss of a compute node way earlier than nova will, hence the proper use of force-down22:39
cfriesenagree with your bug though22:40
*** itlinux has quit IRC22:44
*** thorst_afk has joined #openstack-nova22:48
*** mriedem1 has joined #openstack-nova22:53
openstackgerritMatt Riedemann proposed openstack/nova master: Skip ceph in grenade live migration job due to restart failures  https://review.openstack.org/46609722:55
mriedem1melwitt: dansmith: ^ "fixes" the grenade live migration job22:55
*** mriedem has quit IRC22:55
mriedem1we lose ceph grenade + live migration until the bug is fixed or we get to queens,22:55
mriedem1but we at least get back testing for mixed-level compute upgrade testing22:55
mriedem1*live migration testing i mean22:56
mriedem1on mixed computes22:56
mriedem1regular multinode grenade doesn't test live migration22:56
mriedem1ps2 had a clean run, ps3 just removed a debug echo statement22:56
*** zz_dimtruck is now known as dimtruck23:00
*** gouthamr has joined #openstack-nova23:01
*** Apoorva_ has joined #openstack-nova23:02
*** Apoorva has quit IRC23:04
*** thorst_afk has quit IRC23:06
*** mriedem1 is now known as mriedem23:09
*** armax has quit IRC23:13
*** gouthamr has quit IRC23:13
*** sree has joined #openstack-nova23:18
*** yamamoto_ has joined #openstack-nova23:20
*** gjayavelu has joined #openstack-nova23:21
*** salv-orlando has joined #openstack-nova23:22
*** armax has joined #openstack-nova23:22
*** gjayavelu has quit IRC23:22
*** baoli has quit IRC23:23
*** sree has quit IRC23:23
melwittmriedem: +223:24
melwittI'm interested in fixing the bug but I don't understand your comment about using common functions like in https://github.com/openstack-dev/devstack/commit/c006bbdeb26df2c60f43d222bdf918f9e24d551f23:24
*** yamamoto_ has quit IRC23:26
*** mnestratov has quit IRC23:26
*** salv-orlando has quit IRC23:26
*** gjayavelu has joined #openstack-nova23:28
mriedemsdague said there were some common process start/stop/restart functions in devstack and we could overhaul these scripts to use those,23:28
mriedemwhich would hide the screen vs systemd stuff i think23:28
melwittah, okay23:28
mriedembut if those are available, i'm not sure why these live migration scripts weren't using them in the first place23:28
mriedemso i assume there was a good reason those functions from devstack wouldn't work in the first place23:28
mriedembut without tdurakov anymore it's hard to say23:28
melwittyeah. okay, I shall endeavor to dig into that a bit. because on the surface, I was just thinking to do something dumb like "if ocata, then use old way"23:29
melwittif it's even possible to check "if ocata" in that script23:29
*** salv-orlando has joined #openstack-nova23:31
*** salv-orlando has quit IRC23:31
*** ijw has quit IRC23:35
mriedemthe branch is available, the devstack-gate vars are in there23:35
mriedembut yeah i thought similar, basically we'd need to revert a patch that removed the old functions it was using23:35
mriedemand then put in branching logic for doing old vs new things23:35
mriedemdid you get the quotas changes done that you were talking about?23:36
mriedemfor dict something or other23:36
*** yassine has joined #openstack-nova23:36
*** yassine is now known as Guest633423:37
*** esberglu has quit IRC23:37
*** Apoorva_ has quit IRC23:39
*** Apoorva has joined #openstack-nova23:40
*** mlavalle has quit IRC23:40
*** Fdaisuke_ has joined #openstack-nova23:43
*** Fdaisuke has quit IRC23:43
*** jmlowe has joined #openstack-nova23:44
melwittmriedem: still working through it now. ran into some snags (of course) on the bottom patch. now it should be mechanical changes going through the stack. so I hope to push it up in an hour or so23:51
melwittthis series has been such a pain23:52
mriedemok. i decided that between two evils on my todo list, doing forum session recaps was easier for now. :)23:54
mriedemand i can continue to say i'm not reviewing your series becaues i'm waiting for you to update it23:54
melwittyup, that you can.23:55
*** ijw has joined #openstack-nova23:56
*** ijw has quit IRC23:56
*** ijw has joined #openstack-nova23:57
*** ijw has quit IRC23:58
*** felipemonteiro__ has quit IRC23:59
*** ijw has joined #openstack-nova23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!