Thursday, 2017-06-08

*** salv-orlando has joined #openstack-nova00:00
*** jwnx has quit IRC00:03
*** tbachman has joined #openstack-nova00:03
*** salv-orlando has quit IRC00:05
openstackgerritArtom Lifshitz proposed openstack/nova master: Run api sample tests against 2.latest  https://review.openstack.org/43035200:10
openstackgerritArtom Lifshitz proposed openstack/nova master: Add skip_latest_microversion decorator  https://review.openstack.org/43358500:10
mriedemcomstud: only when it's about babies00:11
*** ckopper has quit IRC00:12
mriedemmikal: heh that reminds me we said at the ptg we were going to kill the localfs driver00:13
*** mtanino_ has quit IRC00:13
* mriedem makes a todo00:13
comstudmriedem: i see00:14
mikalmriedem: that code is bonkers, I'd love it to die. That said, its still used by xenapi.00:14
mriedemmikal: yeah https://review.openstack.org/#/c/324720/00:14
mriedemwe're leaving the xenapi using stuff in00:14
*** mtanino_ has joined #openstack-nova00:15
mikalIf you're going to do that it would be nice to move the localfs stuff that xenapi into their driver and then remove the central thing so its clear its not a thing00:16
mriedemall good comments00:17
mriedemmikal: maybe note that in the review since it's old and we forgot about it at least a few times00:17
mikalSure, I can add that in the commit message00:18
mikalOr I could just do it I suppose00:18
mikalIs it cheating to put that at the end of the chain?00:18
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: fix type for hypervisor_marker  https://review.openstack.org/47197000:19
*** mtanino_ has quit IRC00:20
*** r-daneel has quit IRC00:26
openstackgerritArtom Lifshitz proposed openstack/nova master: api-ref: Add tags to server(s) detail response  https://review.openstack.org/43114900:27
*** masber has quit IRC00:32
*** lyan has joined #openstack-nova00:32
*** masber has joined #openstack-nova00:32
openstackgerritMichael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916600:34
openstackgerritMichael Still proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197200:34
openstackgerritmelanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project()  https://review.openstack.org/44624600:36
openstackgerritmelanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project()  https://review.openstack.org/44624700:36
openstackgerritmelanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object  https://review.openstack.org/44624400:36
openstackgerritmelanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts()  https://review.openstack.org/44624500:36
openstackgerritmelanie witt proposed openstack/nova master: Remove 'reserved' count from used limits  https://review.openstack.org/44624200:36
openstackgerritmelanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage  https://review.openstack.org/44624300:36
openstackgerritmelanie witt proposed openstack/nova master: Count server groups to check quota  https://review.openstack.org/44624000:36
openstackgerritmelanie witt proposed openstack/nova master: Count networks to check quota  https://review.openstack.org/44624100:36
openstackgerritmelanie witt proposed openstack/nova master: Count instances to check quota  https://review.openstack.org/41652100:36
openstackgerritmelanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db  https://review.openstack.org/45785800:36
openstackgerritmelanie witt proposed openstack/nova master: Count server group members to check quota  https://review.openstack.org/45785900:36
openstackgerritmelanie witt proposed openstack/nova master: Count security groups to check quota  https://review.openstack.org/45786000:36
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094500:36
openstackgerritmelanie witt proposed openstack/nova master: Count fixed ips to check quota  https://review.openstack.org/45786100:36
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094600:36
openstackgerritmelanie witt proposed openstack/nova master: Count floating ips to check quota  https://review.openstack.org/45786200:36
openstackgerritmelanie witt proposed openstack/nova master: Default to 0 when merging values in limit check  https://review.openstack.org/47197300:36
*** lbragstad has joined #openstack-nova00:37
openstackgerritMerged openstack/nova master: Remove unused CONF import from placement/auth.py  https://review.openstack.org/47171600:38
melwittgrr why doesn't the infra user update bugs for me anymore00:39
melwittof course, right after I say that it does it. several minutes after I uploaded the patch00:40
*** rfolco has quit IRC00:44
*** armax has quit IRC00:48
*** Shunli has joined #openstack-nova00:48
*** lyan has quit IRC00:49
*** mdnadeem has joined #openstack-nova00:52
*** nic1 has quit IRC00:53
*** gjayavelu has joined #openstack-nova00:54
openstackgerritArtom Lifshitz proposed openstack/nova master: api-ref: Add tags to server(s) detail response  https://review.openstack.org/43114900:55
*** armax has joined #openstack-nova00:56
openstackgerritShunli Zhou proposed openstack/nova master: Transform servergroup.create notification  https://review.openstack.org/45402300:57
openstackgerritArtom Lifshitz proposed openstack/nova master: Test InstanceNotFound handling in 'nova usage'  https://review.openstack.org/46851400:58
* artom got flooded with emails from Jenkins informing him of apparent merge conflicts00:58
* artom is going through his patches to rebase and fix those00:58
artomExcept they don't actually exist00:58
artomEverything rebases cleanly00:59
*** armax has quit IRC00:59
*** salv-orlando has joined #openstack-nova01:01
*** liujiong has joined #openstack-nova01:04
*** salv-orlando has quit IRC01:06
*** jamesdenton has joined #openstack-nova01:08
*** cNilesh has joined #openstack-nova01:08
*** rfolco has joined #openstack-nova01:08
*** gjayavelu has quit IRC01:13
*** smatzek has joined #openstack-nova01:14
*** thorst_afk has joined #openstack-nova01:14
mriedemartom: yeah some known issue from earlier today01:15
*** sree has joined #openstack-nova01:18
*** litao__ has joined #openstack-nova01:19
*** thorst_afk has quit IRC01:19
*** mriedem has quit IRC01:22
*** sree has quit IRC01:22
*** rfolco has quit IRC01:25
*** jamesdenton has quit IRC01:25
*** crushil has joined #openstack-nova01:31
*** smatzek has quit IRC01:33
*** winston-d_ has joined #openstack-nova01:35
*** Apoorva has joined #openstack-nova01:36
*** wlfightup has joined #openstack-nova01:36
*** Apoorva_ has quit IRC01:40
*** Apoorva has quit IRC01:40
*** yamahata has quit IRC01:40
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref  https://review.openstack.org/46584201:42
*** lbragstad has quit IRC01:42
*** phuongnh has joined #openstack-nova01:43
*** thorst_afk has joined #openstack-nova01:47
*** thorst_afk has quit IRC01:47
*** zhurong has joined #openstack-nova01:49
*** jichen has joined #openstack-nova01:58
*** xinliang has quit IRC02:00
*** armax has joined #openstack-nova02:02
*** jwnx has joined #openstack-nova02:03
*** jamielennox is now known as jamielennox|away02:04
*** jaypipes has joined #openstack-nova02:09
*** gomarivera has joined #openstack-nova02:09
openstackgerritHuan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is already deprecated  https://review.openstack.org/46827302:10
*** bkopilov has quit IRC02:12
*** coreywright has quit IRC02:12
*** thorst_afk has joined #openstack-nova02:12
*** thorst_afk has quit IRC02:12
*** xinliang has joined #openstack-nova02:12
*** yamamoto_ has quit IRC02:14
*** jamielennox|away is now known as jamielennox02:15
*** yamamoto_ has joined #openstack-nova02:17
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959302:17
*** lei-zh has joined #openstack-nova02:17
openstackgerritfalseuser proposed openstack/nova master: Delete the instance files after revert a resize task  https://review.openstack.org/46981402:25
*** gomarivera has quit IRC02:27
*** gyee has quit IRC02:27
*** jaypipes has quit IRC02:27
*** coreywright has joined #openstack-nova02:30
*** hongbin has quit IRC02:36
*** Sukhdev has joined #openstack-nova02:36
*** gomarivera has joined #openstack-nova02:44
*** liusheng has quit IRC02:46
*** liusheng has joined #openstack-nova02:46
*** crushil_ has joined #openstack-nova02:47
*** liuqing has quit IRC02:47
*** liuqing has joined #openstack-nova02:47
*** slaweq has quit IRC02:53
*** ckopper has joined #openstack-nova02:54
*** ckopper has quit IRC02:55
*** edmondsw has joined #openstack-nova02:55
*** edmondsw has quit IRC02:59
*** wlfightup has quit IRC03:04
*** yamamoto_ has quit IRC03:04
*** crushil has quit IRC03:05
*** crushil has joined #openstack-nova03:06
*** rfolco has joined #openstack-nova03:11
*** rfolco has quit IRC03:12
*** Shashi has joined #openstack-nova03:12
*** thorst_afk has joined #openstack-nova03:13
*** yamahata has joined #openstack-nova03:15
*** sridharg has joined #openstack-nova03:15
*** dikonoor has joined #openstack-nova03:17
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref  https://review.openstack.org/46584203:20
*** dimtruck is now known as zz_dimtruck03:23
*** rfolco has joined #openstack-nova03:27
*** rfolco has quit IRC03:27
*** gomarivera has quit IRC03:30
*** gomarivera has joined #openstack-nova03:31
*** gomarivera has quit IRC03:31
*** gomarivera has joined #openstack-nova03:32
*** bkopilov has joined #openstack-nova03:32
*** thorst_afk has quit IRC03:32
*** links has joined #openstack-nova03:34
*** zsli_ has joined #openstack-nova03:36
*** Shunli has quit IRC03:39
*** vks1 has joined #openstack-nova03:42
*** zhurong has quit IRC03:45
*** dikonoor has quit IRC03:48
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959303:49
*** tojuvone has quit IRC03:52
*** crushil_ has quit IRC03:54
*** crushil_ has joined #openstack-nova03:54
*** gomarivera has quit IRC03:56
*** gomarivera has joined #openstack-nova03:57
*** tojuvone has joined #openstack-nova03:57
*** yingjun_1 has joined #openstack-nova03:58
*** gomarivera has quit IRC04:01
*** psachin has joined #openstack-nova04:02
*** itlinux has joined #openstack-nova04:03
*** slaweq has joined #openstack-nova04:25
*** dave-mccowan has quit IRC04:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895504:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896404:27
openstackgerritTakashi NATSUME proposed openstack/nova master: Use PCIAddressField in oslo.versionedobjects  https://review.openstack.org/46620104:28
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004204:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492604:29
*** thorst_afk has joined #openstack-nova04:29
openstackgerritTakashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls  https://review.openstack.org/41387604:29
*** pcaruana has joined #openstack-nova04:31
*** mamandle has joined #openstack-nova04:32
*** zhurong has joined #openstack-nova04:33
*** thorst_afk has quit IRC04:37
*** slaweq has quit IRC04:38
*** salv-orlando has joined #openstack-nova04:39
*** crushil_ has quit IRC04:41
*** yamamoto has joined #openstack-nova04:48
*** sree has joined #openstack-nova04:51
*** elady has quit IRC04:54
*** adisky__ has joined #openstack-nova04:55
*** pcaruana has quit IRC04:57
*** jaosorior_away is now known as jaosorior05:05
*** salv-orlando has quit IRC05:05
*** salv-orlando has joined #openstack-nova05:06
*** ltomasbo|away is now known as ltomasbo05:09
openstackgerritHuan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration  https://review.openstack.org/45165705:11
*** jamielennox is now known as jamielennox|away05:12
*** prateek has joined #openstack-nova05:12
*** jamielennox|away is now known as jamielennox05:19
*** itlinux has quit IRC05:21
*** PM_ has joined #openstack-nova05:25
*** claudiub has joined #openstack-nova05:26
*** gomarivera has joined #openstack-nova05:27
*** udesale has joined #openstack-nova05:28
*** ratailor has joined #openstack-nova05:30
*** Sukhdev has quit IRC05:35
*** kwathore has joined #openstack-nova05:35
*** ratailor has quit IRC05:44
*** ratailor has joined #openstack-nova05:45
*** trinaths has joined #openstack-nova05:46
*** ayogi has joined #openstack-nova05:56
*** jamielennox is now known as jamielennox|away05:56
*** dikonoor has joined #openstack-nova05:58
*** jamielennox|away is now known as jamielennox06:03
*** armax has quit IRC06:08
*** armax has joined #openstack-nova06:08
*** armax has quit IRC06:09
*** diga has joined #openstack-nova06:09
*** Oku_OS-away is now known as Oku_OS06:10
*** rcernin has joined #openstack-nova06:15
*** oanson has quit IRC06:20
openstackgerritBéla Vancsics proposed openstack/nova master: Use more specific asserts in tests  https://review.openstack.org/46660906:21
*** oanson has joined #openstack-nova06:21
openstackgerritjichenjc proposed openstack/nova-specs master: add z/VM Driver  https://review.openstack.org/46491506:22
*** thorst_afk has joined #openstack-nova06:29
*** zhurong has quit IRC06:32
*** thorst_afk has quit IRC06:34
*** PM_ has quit IRC06:34
*** gomarivera has quit IRC06:41
*** gomarivera has joined #openstack-nova06:42
openstackgerritChris Friesen proposed openstack/nova master: Add validation for cpu_realtime_mask  https://review.openstack.org/46820306:42
openstackgerritChris Friesen proposed openstack/nova master: Tweak the cpu_realtime_mask handling slightly  https://review.openstack.org/46145606:42
openstackgerritBéla Vancsics proposed openstack/nova master: Remove reimported  https://review.openstack.org/46665806:43
openstackgerritAlex Xu proposed openstack/nova-specs master: Fix to return server groups quota in the quota-classes API  https://review.openstack.org/46861906:45
*** gomarivera has quit IRC06:47
*** esberglu has joined #openstack-nova06:49
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref  https://review.openstack.org/46584206:50
*** esberglu has quit IRC06:54
*** pcaruana has joined #openstack-nova06:57
*** jwnx has quit IRC06:58
openstackgerritBhagyashri Shewale proposed openstack/nova master: Don't create instance backup image if rotation is 0  https://review.openstack.org/40964406:58
*** Hosam has quit IRC07:03
*** damien_r has joined #openstack-nova07:07
openstackgerritsahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller  https://review.openstack.org/45974107:12
openstackgerritsahid proposed openstack/nova master: libvirt: update logic to configure device for scsi controller  https://review.openstack.org/45974007:12
*** tesseract has joined #openstack-nova07:12
openstackgerritBéla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622507:12
*** mingyu has joined #openstack-nova07:13
*** markus_z has joined #openstack-nova07:14
openstackgerritChris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details  https://review.openstack.org/43514107:15
*** oanson has quit IRC07:20
*** Hosam has joined #openstack-nova07:20
*** diga has quit IRC07:20
*** oanson has joined #openstack-nova07:21
*** mingyu has quit IRC07:22
*** klipka has joined #openstack-nova07:29
openstackgerritmelanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens  https://review.openstack.org/32538107:30
openstackgerritmelanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations  https://review.openstack.org/32541407:30
openstackgerritmelanie witt proposed openstack/nova master: Add console connection object  https://review.openstack.org/32006307:30
openstackgerritmelanie witt proposed openstack/nova master: Add access_url_base to console_auth_tokens table  https://review.openstack.org/33461407:30
openstackgerritmelanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation  https://review.openstack.org/33399007:30
*** mingyu has joined #openstack-nova07:30
*** thorst_afk has joined #openstack-nova07:30
*** oanson has quit IRC07:31
*** oanson has joined #openstack-nova07:31
*** mlakat has joined #openstack-nova07:32
*** jpena|off is now known as jpena07:34
*** thorst_afk has quit IRC07:35
*** alexchadin has joined #openstack-nova07:36
*** gcb has joined #openstack-nova07:39
*** mingyu has quit IRC07:40
*** jaosorior has quit IRC07:42
*** nicolasbock has joined #openstack-nova07:45
*** zhurong has joined #openstack-nova07:47
*** bkopilov has quit IRC07:47
*** ralonsoh has joined #openstack-nova07:48
*** mnestratov has joined #openstack-nova07:49
*** ralonsoh has quit IRC07:49
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352607:52
*** jaosorior has joined #openstack-nova07:54
*** ralonsoh has joined #openstack-nova07:56
*** tuanluong has joined #openstack-nova07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:01
*** yingjun_1 has quit IRC08:07
*** zhurong has quit IRC08:13
*** salv-orlando has quit IRC08:13
*** litao__ has quit IRC08:13
*** salv-orlando has joined #openstack-nova08:14
*** zhurong has joined #openstack-nova08:15
*** qwertyco has joined #openstack-nova08:17
*** salv-orlando has quit IRC08:17
*** lucas-afk is now known as lucasagomes08:18
*** litao__ has joined #openstack-nova08:23
*** links has quit IRC08:25
*** prateek_ has joined #openstack-nova08:28
*** prateek has quit IRC08:30
*** thorst_afk has joined #openstack-nova08:31
*** prateek_ has quit IRC08:32
*** prateek has joined #openstack-nova08:33
*** prateek has quit IRC08:33
*** prateek has joined #openstack-nova08:34
*** prateek has quit IRC08:35
*** prateek has joined #openstack-nova08:35
*** frickler_ is now known as frickler08:36
*** esberglu has joined #openstack-nova08:37
*** links has joined #openstack-nova08:38
*** esberglu has quit IRC08:42
*** fragatina has joined #openstack-nova08:44
*** prateek has quit IRC08:44
*** salv-orlando has joined #openstack-nova08:45
openstackgerritStephen Finucane proposed openstack/nova master: Add PCIWeigher  https://review.openstack.org/37952408:45
openstackgerritStephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances  https://review.openstack.org/37962508:45
openstackgerritPooja Jadhav proposed openstack/nova master: Nova flavor-list swap should be 0 when no value is specified  https://review.openstack.org/47171508:46
*** efoley has joined #openstack-nova08:47
*** xiefp has joined #openstack-nova08:48
*** thorst_afk has quit IRC08:50
openstackgerritStephen Finucane proposed openstack/nova master: Remove **kwargs passing in payload __init__  https://review.openstack.org/41848908:50
openstackgerritStephen Finucane proposed openstack/nova master: remove get_nw_info_for_instance from compute.utils  https://review.openstack.org/45138708:53
openstackgerritStephen Finucane proposed openstack/nova master: Remove useless API tests  https://review.openstack.org/46101108:55
openstackgerritStephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group  https://review.openstack.org/46246908:55
openstackgerritMikhail Feoktistov proposed openstack/nova master: Remove fstype param from ploop init  https://review.openstack.org/44497008:56
*** yamamoto has quit IRC08:58
*** roeyc has joined #openstack-nova08:58
openstackgerritStephen Finucane proposed openstack/nova master: Convert additional disassociate tests to mock  https://review.openstack.org/47140808:59
*** mvk has joined #openstack-nova09:02
openstackgerritMikhail Feoktistov proposed openstack/nova master: Add is_vz_container function  https://review.openstack.org/44594709:03
*** gszasz has joined #openstack-nova09:04
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925709:06
openstackgerritBalazs Gibizer proposed openstack/nova master: Improve assertJsonEqual error reporting  https://review.openstack.org/42819909:09
openstackgerritBéla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification  https://review.openstack.org/46062509:11
*** ftersin has joined #openstack-nova09:13
openstackgerritBalazs Gibizer proposed openstack/nova master: Add separate instance.create payload type  https://review.openstack.org/46300109:15
openstackgerritBalazs Gibizer proposed openstack/nova master: Add keypairs field to InstanceCreatePayload  https://review.openstack.org/41973009:17
*** qwertyco has quit IRC09:23
*** prateek has joined #openstack-nova09:25
openstackgerritBalazs Gibizer proposed openstack/nova master: Add key_name field to InstancePayload  https://review.openstack.org/46300209:27
*** Hosam has quit IRC09:29
Kevin_ZhengHi, I'm working on https://review.openstack.org/#/c/465842/ which adds tags field to instance list details/update/show API and related functional tests, and tests for list details/show already exists in https://github.com/openstack/nova/tree/master/nova/tests/functional/api_sample_tests/api_samples/os-server-tags/v2.26 under os-server-tags tests, as it is tests for servers API, I'd think it should be under servers09:30
Kevin_Zhengtests, so, should I add test for PUT also to os-server-tags or move them all to servers?09:30
*** Hosam has joined #openstack-nova09:30
*** cNilesh has quit IRC09:30
*** cNilesh has joined #openstack-nova09:30
*** alexchadin has quit IRC09:31
*** alexchadin has joined #openstack-nova09:31
*** zsli_ has quit IRC09:32
*** gongysh has joined #openstack-nova09:32
*** crushil has quit IRC09:33
openstackgerritBalazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload  https://review.openstack.org/44877909:33
*** crushil has joined #openstack-nova09:33
*** Hosam has quit IRC09:34
*** efoley has quit IRC09:34
openstackgerritjichenjc proposed openstack/nova master: propagate OSError to MigrationPreCheckError  https://review.openstack.org/46977909:34
*** esberglu has joined #openstack-nova09:35
*** moshele has joined #openstack-nova09:35
*** cNilesh has quit IRC09:36
*** cNilesh has joined #openstack-nova09:36
openstackgerritAndrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref for allocations  https://review.openstack.org/47093309:37
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for aggregates  https://review.openstack.org/47093209:37
*** edand has joined #openstack-nova09:37
*** sambetts|afk is now known as sambetts09:38
*** alexchadin has quit IRC09:40
*** esberglu has quit IRC09:40
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification  https://review.openstack.org/39621009:43
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification  https://review.openstack.org/39621109:43
*** cNilesh has quit IRC09:47
*** thorst_afk has joined #openstack-nova09:47
*** cNilesh has joined #openstack-nova09:47
openstackgerritMerged openstack/nova master: Remove useless API tests  https://review.openstack.org/46101109:49
*** rmart04 has joined #openstack-nova09:49
*** mdnadeem has quit IRC09:49
*** thorst_afk has quit IRC09:51
*** diga has joined #openstack-nova09:52
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref  https://review.openstack.org/46584209:53
*** cdent has joined #openstack-nova09:54
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352609:54
*** cNilesh has quit IRC09:57
openstackgerritChris Dent proposed openstack/nova master: [placement] Disambiguate resource provider conflict message  https://review.openstack.org/46892809:57
*** klipka has quit IRC09:57
openstackgerritChris Dent proposed openstack/nova master: [placement] Update placement devref to modern features  https://review.openstack.org/46904709:59
openstackgerritChris Dent proposed openstack/nova master: [placement] Use util.extract_json in allocations handler  https://review.openstack.org/47106710:00
*** kaisers has quit IRC10:00
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276610:01
*** jichen has quit IRC10:02
*** lei-zh has quit IRC10:03
*** klipka has joined #openstack-nova10:03
*** liujiong has quit IRC10:06
*** sdague has joined #openstack-nova10:11
openstackgerritsahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller  https://review.openstack.org/45974110:12
*** kaisers has joined #openstack-nova10:16
*** kwathore_ has joined #openstack-nova10:21
*** kwathore has quit IRC10:21
*** phuongnh has quit IRC10:24
openstackgerritMikhail Feoktistov proposed openstack/nova master: raise exception if create Virtuozzo container with swap disk  https://review.openstack.org/39782910:24
*** markvoelker has quit IRC10:27
*** markvoelker has joined #openstack-nova10:27
*** mnestratov|3 has joined #openstack-nova10:29
*** dtantsur|afk is now known as dtantsur10:31
*** med_ has quit IRC10:31
*** cristicalin has joined #openstack-nova10:31
*** mnestratov has quit IRC10:32
*** cristicalin has quit IRC10:33
*** med_ has joined #openstack-nova10:35
*** med_ is now known as Guest7261210:35
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177710:38
*** sree has quit IRC10:39
*** Gayi has joined #openstack-nova10:39
GayiHello everyone..10:40
*** Gayi has quit IRC10:40
*** dikonoor has quit IRC10:43
*** salv-orlando has quit IRC10:44
*** salv-orlando has joined #openstack-nova10:44
openstackgerritfalseuser proposed openstack/nova master: Clear instance root_device_name in rebuild operation  https://review.openstack.org/46499110:48
*** smatzek has joined #openstack-nova10:53
*** winston-d_ has quit IRC10:55
openstackgerritRitesh proposed openstack/nova master: rbd: flatten images when unshelving  https://review.openstack.org/45788610:55
*** kaisers has quit IRC11:06
*** udesale has quit IRC11:06
*** jpena is now known as jpena|lunch11:07
mikalmdbooth: You around? Not sure what timezone you're in.11:08
*** carthaca_ has joined #openstack-nova11:10
*** mvk has quit IRC11:16
mdboothmikal: I'm around. UK.11:19
mdboothWhat's up?11:19
* cdent will see mdbooth at the polls11:20
mikalI was confused about some "pty flush" code in the libvirt driver...11:20
*** Hosam has joined #openstack-nova11:21
mdboothmikal: I'll probably be similarly confused :)11:21
mikalBut I think I've got it now. I'm fiddling with file permissions while I sneak up with privsep and I didn't understand why the flush needed a chown first11:21
mikalIt turns out it doesn't its the call _after_ that which needs the chown11:21
mikalSo yeah, I think I just wasted your time11:21
mikalSorry11:21
mdboothcdent: Too late: https://goo.gl/photos/31jQijVTf5wsAuFi6 :)11:21
cdent+many11:21
mdboothmikal: Hehe, np.11:22
*** dikonoor has joined #openstack-nova11:23
*** kaisers has joined #openstack-nova11:23
*** Hosam_ has joined #openstack-nova11:25
*** lucasagomes is now known as lucas-hungry11:26
*** Hosam has quit IRC11:28
openstackgerritMerged openstack/nova master: Fix html_last_updated_fmt for Python3  https://review.openstack.org/47067611:28
*** Hosam has joined #openstack-nova11:29
*** Hosam_ has quit IRC11:32
*** trinaths has left #openstack-nova11:32
*** dikonoor has quit IRC11:33
*** mingyu has joined #openstack-nova11:35
*** dikonoor has joined #openstack-nova11:36
*** liuqing has quit IRC11:36
*** liuqing has joined #openstack-nova11:36
*** gongysh has quit IRC11:37
*** CristinaPauna has quit IRC11:37
*** dtantsur is now known as dtantsur|sick11:37
*** edmondsw has joined #openstack-nova11:40
*** mikal has quit IRC11:41
*** gomarivera has joined #openstack-nova11:43
*** mikal has joined #openstack-nova11:43
*** dikonoor has quit IRC11:45
*** mingyu has quit IRC11:45
*** mingyu has joined #openstack-nova11:46
*** thorst_afk has joined #openstack-nova11:46
*** mvk has joined #openstack-nova11:47
*** gomarivera has quit IRC11:47
*** dikonoor has joined #openstack-nova11:49
*** kwathore_ has quit IRC11:51
*** abalutoiu has joined #openstack-nova11:52
*** mingyu has quit IRC11:53
*** litao__ has quit IRC11:53
openstackgerritMerged openstack/nova master: Remove **kwargs passing in payload __init__  https://review.openstack.org/41848911:54
*** hwoarang_ is now known as hwoarang11:56
*** tbachman has quit IRC11:57
*** jpena|lunch is now known as jpena11:57
*** mingyu has joined #openstack-nova11:58
*** kwathore has joined #openstack-nova11:58
*** Shashi has quit IRC12:05
*** jwnx has joined #openstack-nova12:07
openstackgerritMichael Still proposed openstack/nova master: Move the last_bytes util method to the libvirt driver.  https://review.openstack.org/47222812:08
openstackgerritMichael Still proposed openstack/nova master: Avoid chowing console logs in libvirt  https://review.openstack.org/47222912:08
*** lucas-hungry is now known as lucasagomes12:10
*** mingyu has quit IRC12:11
*** mingyu has joined #openstack-nova12:11
*** vks1 has quit IRC12:12
openstackgerritMikhail Feoktistov proposed openstack/nova master: Add is_vz_container function  https://review.openstack.org/44594712:13
*** mriedem has joined #openstack-nova12:13
*** lyan has joined #openstack-nova12:14
mriedemit's p-2 day12:16
*** mingyu has quit IRC12:16
mriedemany burning regressions or bugs we need to get fixed before we tag it?12:16
*** damien_r has left #openstack-nova12:18
*** jamesdenton has joined #openstack-nova12:22
*** tuanluong has quit IRC12:23
bauzasmriedem: honestly, I'm still looking at all the bugs12:23
bauzasfor the moment, none12:23
bauzasbut that's a today triage day for me12:23
mriedemlyarwood: i just noticed the git author thing again while rebasing https://review.openstack.org/#/c/436094/12:25
mriedemi pulled that down with git review -d12:25
mriedemrebased it to master: git rebase -i master12:25
*** krtaylor has quit IRC12:25
mriedemresolved merge conflicts, added the resolved files and did git commit, then git rebase --continue12:26
mriedemand then i was the author12:26
*** jwcroppe has joined #openstack-nova12:26
mriedemi do'nt know what's going on there12:27
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup update_instance cell mapping handling  https://review.openstack.org/46738012:29
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells  https://review.openstack.org/46738312:29
openstackgerritMatt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance  https://review.openstack.org/46738212:29
kashyapcfriesen: Have you booted an UEFI image lately?12:29
kashyap(Or if ever)12:29
openstackgerritMatthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready  https://review.openstack.org/47177312:29
openstackgerritMatt Riedemann proposed openstack/nova master: Pull out code that builds VIF in _build_network_info_model  https://review.openstack.org/46644912:30
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Handle network-changed event for a specific port  https://review.openstack.org/46578712:30
*** derekh has joined #openstack-nova12:31
*** catintheroof has joined #openstack-nova12:32
*** karimb has joined #openstack-nova12:34
*** sree has joined #openstack-nova12:35
*** zhurong has quit IRC12:36
sdaguemriedem: https://bugs.launchpad.net/nova/+bug/1687581 seems like something we should probably fix sooner rather than later12:36
openstackLaunchpad bug 1687581 in OpenStack Compute (nova) "Newton > Ocata upgrade. secret_uuid empty" [High,New]12:36
sdaguepeople upgrading newton -> ocata ceph apparently can't start newton guests12:37
*** tbachman has joined #openstack-nova12:37
*** liverpooler has joined #openstack-nova12:37
mriedemi know what caused that12:37
mriedemhttps://review.openstack.org/#/c/389399/12:38
mriedemi think12:38
*** krtaylor has joined #openstack-nova12:38
*** ratailor has quit IRC12:40
mriedemso it looks like in newton they had rbd auth enabled on the cinder side, but didn't have the secret uuid coming from cinder, it was coming from nova.conf12:40
*** rfolco has joined #openstack-nova12:41
sdagueright12:42
mriedemi can work up a patch12:42
sdaguemriedem: there is a patch in the bug12:42
sdaguethey just didn't gerrit it12:42
sdaguehttp://paste.ubuntu.com/24497744/12:43
mriedemi know, but that needs tests and comments, etc12:45
mriedemi can co-author them12:45
*** jroll has left #openstack-nova12:47
*** ayogi has quit IRC12:48
bauzasclaudiub: see https://bugs.launchpad.net/nova/+bug/169668512:48
openstackLaunchpad bug 1696001 in OpenStack Compute (nova) ocata "duplicate for #1696685 nova-manage cell_v2 map_cell0 can create invalid connection URL" [Undecided,In progress] - Assigned to Sylvain Bauza (sylvain-bauza)12:48
bauzasclaudiub: you can review the stable/ocata change12:49
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot  https://review.openstack.org/40038412:50
bauzasmriedem: we have a super weird issue for stable/newton with db migrations when you have different TZs https://bugs.launchpad.net/nova/+bug/169641712:50
openstackLaunchpad bug 1696417 in OpenStack Compute (nova) "nova-manage db online_data_migrations can fail when upgrading to newton under certain conditions" [Undecided,Incomplete]12:50
bauzasmriedem: for sure we could just say "meh"12:50
lyarwoodmriedem: huh, are you using an alias for git commit or something?12:50
*** Hosam has quit IRC12:51
*** Hosam has joined #openstack-nova12:52
sdaguemriedem: git author or git committer?12:52
lyarwoodsdague: both, https://review.openstack.org/#/c/441204/ is an example.12:54
mriedemlyarwood: nope12:55
mriedemjust git commit12:55
mriedemsdague: author12:55
sdaguemriedem: author shouldn't change unless you --reset-author12:55
*** yamamoto has joined #openstack-nova12:55
mriedemwhich i didn't do12:55
mriedemjust git review12:55
mriedemgit rebase12:55
mriedemgit commit12:55
lyarwoodyeah that's weird12:55
mriedemsdague: somehow stole author from lyarwood on https://review.openstack.org/#/c/441204/12:55
mriedemand noticed the same this morning when rebasing a change of dan's in devstack12:55
mriedembut caught it this time12:55
sdaguemriedem: and nothing in the ~/.gitconfig?12:56
*** cleong has joined #openstack-nova12:56
mriedemuser@ubuntu:~/git/nova$ cat ~/.gitconfig12:56
mriedem[user]12:56
mriedemname = Matt Riedemann12:56
mriedememail = mriedem.os@gmail.com12:56
mriedem[gitreview]12:56
mriedemusername = mriedem12:56
mriedem[core]12:56
mriedemexcludesfile = /home/user/.gitignore12:56
*** crushil has quit IRC12:56
sdagueand this is all at the cli and not using some other tool?12:56
bauzasmriedem: possibly merging https://bugs.launchpad.net/nova/+bug/1696605 before pike-2 too12:56
openstackLaunchpad bug 1696605 in OpenStack Compute (nova) "TypeError: unorderable types: int() < NoneType() possible during quota limit check" [High,In progress] - Assigned to melanie witt (melwitt)12:56
*** Hosam has quit IRC12:56
*** mriedem1 has joined #openstack-nova12:59
*** udesale has joined #openstack-nova12:59
*** mriedem has quit IRC13:00
*** mriedem1 is now known as mriedem13:02
*** baoli has joined #openstack-nova13:02
openstackgerritMatt Riedemann proposed openstack/nova master: Log a warning if there is only one cell when listing instances  https://review.openstack.org/44065213:03
*** cristicalin has joined #openstack-nova13:03
*** gongysh has joined #openstack-nova13:03
*** gongysh has quit IRC13:03
*** CristinaPauna has joined #openstack-nova13:03
*** mdrabe has joined #openstack-nova13:04
*** esberglu has joined #openstack-nova13:06
bauzasdoes the VM really supports the order of NICs that you provided when booting ?13:07
*** crushil has joined #openstack-nova13:07
bauzaslooks to me hard to respect13:07
*** pchavva has joined #openstack-nova13:07
bauzashttps://bugs.launchpad.net/nova/+bug/169666413:07
openstackLaunchpad bug 1696664 in OpenStack Compute (nova) "Order of the interfaces is not honored when using virt and SR-IOV interfaces" [Undecided,New]13:07
bauzassfinucan: any idea of the above ? ^13:07
* sfinucan looking13:07
bauzasI mean, I suspect that as a non-issue13:07
bauzasbut it's not really related to SR-IOV13:08
sfinucanbauzas: I concur - we don't say anywhere that the order is important13:08
sfinucan /respected13:09
*** jamesdenton has quit IRC13:10
bauzaswell, we don't say anything in https://developer.openstack.org/api-ref/compute/?expanded=create-server-detail#create-server13:10
bauzasand when looking at code, we're just getting the dict13:10
claudiubbauzas: ah, cool, ty. It's in merge conflict though. :) https://review.openstack.org/#/c/471790/13:10
*** yamamoto has quit IRC13:10
bauzassfinucan: so I'm pretty sure we're not exactly verifying the order13:10
sfinucanbauzas: Agreed. You can close it as invalid, IMO13:10
bauzassfinucan: will do13:11
bauzasif the user cares about the order, that would be a feature to me13:11
bauzasand good luck with that.13:11
bauzasclaudiub: oh good point, will rebase it13:11
*** dave-mccowan has joined #openstack-nova13:12
*** lei-zh has joined #openstack-nova13:12
*** yamamoto has joined #openstack-nova13:14
*** crushil has quit IRC13:15
mriedembauzas: sfinucan: there is some network sorting in the neutronv2.api code13:15
*** links has quit IRC13:15
mriedemhttps://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L252213:15
*** lucasxu has joined #openstack-nova13:16
mriedemthat's for networks, but when given ports we get the networks for those ports too so maybe that's what they are thinking13:16
bauzashumpg13:19
bauzasmriedem: so there are two possible outcomes13:20
bauzasmriedem: either we say "sorry, but we don't sort NICs when you have both network and port requests"13:20
bauzasmriedem: or we say it's a valid bug that requires resolution13:21
mriedemi don't want to say anything13:22
bauzasmriedem: so I'll keep the bug open13:24
bauzas:)13:24
bauzasanyone can just look at it13:24
*** sbezverk has joined #openstack-nova13:25
*** smatzek has quit IRC13:29
*** diga has quit IRC13:29
*** igordcard has quit IRC13:31
mriedemi replied with some details too and code links13:31
mriedemlooks like we do only preserve the order of requested networks, not the networks associated with a requested port13:31
mriedemadding the port.network_id to the list of ordered_networks would be easy, but would probably break something else,13:31
mriedemlike i bet that would end up having it create another port on that other port's network or something13:32
mriedemit just gets very complicated13:32
mriedemwe could build a 3rd list of networks :)13:32
*** igordcard has joined #openstack-nova13:34
*** awaugama has joined #openstack-nova13:34
*** sbezverk has quit IRC13:34
*** gomarivera has joined #openstack-nova13:36
*** gomarivera has quit IRC13:36
*** gomarivera has joined #openstack-nova13:37
*** udesale has quit IRC13:38
*** lucasxu has quit IRC13:38
*** tbachman has quit IRC13:38
*** prateek has quit IRC13:40
*** lucasxu has joined #openstack-nova13:40
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: handle missing rbd_secret_uuid from old connection info  https://review.openstack.org/47226613:41
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: handle missing rbd_secret_uuid from old connection info  https://review.openstack.org/47226613:42
mriedemsdague: here is that fix for the rbd secret uuid ^13:42
mriedemsdague: which reminds me of something i've thought about for awhile,13:42
mriedemwhich is adding an admin api to force a bdm.connection_info refresh13:43
openstackgerritAlex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications  https://review.openstack.org/46552613:43
mriedemsince if you changed things somewhere on the cinder side, or in your underlying storage, i think you have to detach the volume and re-attach it, or reboot the instance to refresh the connection info13:43
mriedemthis has come up when people recycle secrets in ceph13:43
mriedemand the only way to update the existing bdm connection info is via straight db hackery13:44
mriedemso we could do something as simple as an rpc cast to the compute, call os-initialize_connection in cinder to get the connection_info and then update the bdm record13:45
mriedemmelwitt: haven't you and i talked about something like this before? ^ or at least a periodic task13:45
*** Jack_Iv has joined #openstack-nova13:46
*** jwnx has quit IRC13:46
mriedemthe problem is i bet initialize_connection in cinder isn't the same (idempotent) for all volume backends13:46
*** roeyc has quit IRC13:47
*** tbachman has joined #openstack-nova13:48
*** burt has joined #openstack-nova13:48
sdaguemriedem: could that come from cinder in the admin event interface?13:49
*** edand has quit IRC13:49
sdagueit would be best if cinder was the one that said to do that when needed13:49
*** smatzek has joined #openstack-nova13:50
*** lucasxu has quit IRC13:51
mriedemit could13:52
mriedembut,13:52
mriedemsend what? just telling us to do it? because that api doesn't take something like a big json blob13:52
mriedemwe could get a "refresh-connection-info" event from cinder though yeah, like we do for network-changed in neutron13:53
*** igordcard has quit IRC13:54
*** gbarros has joined #openstack-nova13:54
*** igordcard has joined #openstack-nova13:55
sdagueyeh, that's kind of what I was thinking13:57
sdagueI guess it should be instance oriented?13:57
*** igordcard has quit IRC13:57
mriedemyes it would be13:58
mriedemjust sent to the ML13:58
mriedemmight be a duplicate idea i've already brought up before13:58
mriedemanyway, bbiab13:58
*** igordcard has joined #openstack-nova13:58
*** READ10 has joined #openstack-nova13:59
*** jamesdenton has joined #openstack-nova13:59
*** tidwellr has joined #openstack-nova14:00
*** krtaylor has quit IRC14:01
*** igordcard has quit IRC14:01
*** igordcard has joined #openstack-nova14:02
*** lucasxu has joined #openstack-nova14:02
*** ZZelle has quit IRC14:03
*** zz_dimtruck is now known as dimtruck14:03
*** ZZelle has joined #openstack-nova14:03
*** crushil has joined #openstack-nova14:06
*** mtanino has joined #openstack-nova14:06
*** sbezverk has joined #openstack-nova14:07
*** dikonoor has quit IRC14:09
*** ZZelle has quit IRC14:12
*** jamesdenton has quit IRC14:12
*** ZZelle has joined #openstack-nova14:12
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology  https://review.openstack.org/46603014:13
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology  https://review.openstack.org/40824214:13
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology  https://review.openstack.org/40824114:13
*** lbragstad has joined #openstack-nova14:14
*** ZZelle has quit IRC14:14
*** eharney has joined #openstack-nova14:16
*** r-daneel has joined #openstack-nova14:17
*** felipemonteiro_ has joined #openstack-nova14:18
sfinucandansmith: I think I finally figured out the remaining kinks with the above ^^^ patches, if you want to take a look at some point14:18
* sfinucan goes to implement an '__eq__' method for 'nova.objects.base.NovaObject'14:19
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Check a last error of power action  https://review.openstack.org/47075414:20
bauzassfinucan: I think the consensus is not to have __eq__14:20
*** Yangyapeng has joined #openstack-nova14:20
dansmithsfinucan: nak on the __eq__ yeah :)14:20
sfinucanbauzas: Oh? Any particular reason?14:20
bauzassfinucan: when you want to verify o.vo objects, you need to verify the fields14:20
*** burt has quit IRC14:21
sfinucanbauzas: Yeah - we do that in nova/objects/pci_device.py and nova/objects/numa.py14:21
*** burt has joined #openstack-nova14:21
sfinucan...and we could also do with it in nova/objects/instance_numa_topology.py14:21
bauzassfinucan: there are multiple reasons, one being that objects can be semantically identical without being the same, like if they're passed over RPC14:22
*** tbachman has quit IRC14:22
sfinucanbauzas: You mean there'd be differences other than timestamps and the likes?14:24
bauzaswell, take an object, pass it as an argument to a remote method, you won't get the same object at the end14:25
*** jamesdenton has joined #openstack-nova14:25
*** sbezverk has quit IRC14:26
*** awaugama_ has joined #openstack-nova14:27
*** lei-zh has quit IRC14:28
*** lei-zh has joined #openstack-nova14:28
*** hieulq_ has joined #openstack-nova14:29
*** awaugama has quit IRC14:29
sfinucanhmm, well I'm going to try something to see if it makes sense14:29
sfinucanit might not, but then I'll know :)14:30
*** felipemonteiro__ has joined #openstack-nova14:30
openstackgerritMatt Riedemann proposed openstack/nova master: WIP/POC Implement new attach Cinder flow  https://review.openstack.org/33028514:31
openstackgerritMatt Riedemann proposed openstack/nova master: Skip microversion discovery check for update/delete volume attachments  https://review.openstack.org/46966814:31
openstackgerritMatt Riedemann proposed openstack/nova master: Use 3.27 microversion when creating new style volume attachments  https://review.openstack.org/46960914:31
openstackgerritMatt Riedemann proposed openstack/nova master: Use microversions for new style volume attachments  https://review.openstack.org/46957914:31
mriedemsdague: could you add the bottom 3 changes here to your review queue? ^14:31
mriedemthe new cinder stuff depends on that14:31
*** moshele has quit IRC14:32
sdagueyeh, can do14:33
sdagueit will be a minute, in the middle of unrelated stuff14:33
*** felipemonteiro_ has quit IRC14:34
mriedemsure, thanks14:34
*** jwnx has joined #openstack-nova14:34
mriedemdansmith: looks like your devstack fleet week patch needs to depend on the check min service version in all cells patch14:34
mriedemhttp://logs.openstack.org/94/436094/32/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0d3a7e2/console.html#_2017-06-08_13_33_37_46195714:34
mriedemi think...14:34
*** gbarros has quit IRC14:34
*** coreywright has quit IRC14:35
*** jamesdenton has quit IRC14:35
bauzassfinucan: the consensus is basically about providing a general __eq__ rule14:38
bauzassfinucan: semantically, the equivalence really depends on the object type14:38
bauzassfinucan: like for example, you could assume some fields are irrelevant for that14:38
sfinucanYeah, that sounds fair. It does seem pretty contextual14:38
bauzasthe same goes with __hash__14:39
bauzassince objects can be very different, we try hard to not define __hash__ on a global basis14:39
bauzassfinucan: that said, you know we have an obj_equal_prims() right?14:40
sfinucanbauzas: I've just spotted that, alright14:41
bauzascool14:41
bauzasthat's the only thing we can really do on a general basis14:41
bauzasif primitives are equal, fine14:42
bauzasbut standing that obj_1 == obj_2 is a total different deal14:42
bauzassame goes for obj1 in an objectlist14:42
*** sree has quit IRC14:43
*** claudiub has quit IRC14:43
*** salv-orl_ has joined #openstack-nova14:44
dansmithmriedem: aye, not sure why I wasn't hitting that before14:46
*** salv-orlando has quit IRC14:47
mriedemdansmith: i think the test was recently-ish merged14:47
mriedemwell,14:47
dansmithah14:47
*** armax has joined #openstack-nova14:47
mriedemand it required the ssh validation stuff being turned on, which happened globally about 2 weeks ago14:47
dansmithmriedem: thanks for fixing the bottom patch, I just realized I needed to get back to it14:47
dansmithis the merge conflict because of all the infra drama yesterday?14:48
mriedemyes, i've rechecked14:48
mriedemwell, i assume it is, i haven't tried rebasing14:48
dansmithcool14:48
dansmithit's running it so I think it must've been14:48
dansmithotherwise it'd kick right out14:48
*** coreywright has joined #openstack-nova14:48
cfriesenkashyap: I haven't, one of our people might have and I'm pinging for confirmation.  We're just going GA with Newton though, so we're not exactly bleeding edge.14:49
*** hongbin has joined #openstack-nova14:50
johnthetubaguymriedem: was just taking a peak at the cinder patches (https://review.openstack.org/#/c/469668/3)14:52
johnthetubaguymriedem: any reason not to cache the availablity of a microversion, once we find its available?14:53
*** abalutoiu has quit IRC14:53
*** jaypipes has joined #openstack-nova14:54
cfriesenjohnthetubaguy: couldn't they reboot the cinder service at any time?14:54
cfriesenjohnthetubaguy: or are you expecting monotonic increases14:55
johnthetubaguycfriesen: yeah, but we don't really support downgrades14:55
johnthetubaguycfriesen: your right in general though, just wondering if we can be simpler14:55
johnthetubaguyactually, yeah, downgrading past this point just isn't going to work, as far as I understand it14:56
*** psachin has quit IRC14:57
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NovaComparableObject mixin  https://review.openstack.org/47228514:58
sfinucanbauzas, dansmith: What about that? ^14:59
mriedemjohnthetubaguy: replied inline14:59
sfinucanjohnthetubaguy: Oh hai14:59
mriedemjohnthetubaguy: in general because caching of versions gets complicated14:59
mriedemespecially when nova as the client might need to request multiple different versions based on which operation we're doing14:59
*** sree has joined #openstack-nova14:59
mriedemlike if we need 3.27 today for something but 3.33 tomorrow for something else14:59
* sfinucan feels like he hasn't seen johnthetubaguy in aaaages14:59
dansmithsfinucan: why not use equal_prims for that?14:59
mriedemjohnthetubaguy: for now i'm just keeping it simple14:59
dansmithsfinucan: more "is this equal" helpers seems bad to me15:00
johnthetubaguymriedem: I was thinking caching was simpler, but you are right15:00
*** jmlowe has quit IRC15:00
mriedemit's simple in the simple case :)15:00
johnthetubaguymriedem: what you have is clearly correct15:00
sfinucandansmith: I was hoping you'd tell me :) I'm not confident enough in that code to say they're interchangeable15:00
* johnthetubaguy waves at sfinucan as he comes back from the dead15:00
sfinucanIf they are, I can take that route15:00
mriedemjohnthetubaguy: when talking about this last week, mordred was also saying that at some point i think ksa can provide this info, so we don't need the extra call to cinder15:01
mriedemto get the version doc15:01
mriedemksa would just provide it15:01
sfinucan(by "that code", I mean "all things object'y")15:01
* edleafe runs away from zombie johnthetubaguy 15:01
johnthetubaguymriedem: good point, he is cooking things in that area, they can do the caching15:01
mriedemjohnthetubaguy: i.e. when we get the ksa session (which is cached...)15:01
dansmithsfinucan: I'm saying if you want to have a mixin that implements __eq__, use the existing equal_prims thing to determine equality in your handler, don't define another one15:01
*** Yangyape_ has joined #openstack-nova15:02
mriedemjohnthetubaguy: but thanks for reviewing, and welcome back15:02
*** sree has quit IRC15:02
*** Yangyapeng has quit IRC15:02
mordredmriedem: yes15:02
mordredmriedem: this is correct15:02
*** gbarros has joined #openstack-nova15:06
openstackgerritRick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API  https://review.openstack.org/44928815:07
*** lbragstad has quit IRC15:08
*** awaugama_ has quit IRC15:08
openstackgerritChris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details  https://review.openstack.org/43514115:10
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NovaComparableObject mixin  https://review.openstack.org/47228515:10
mordredjohnthetubaguy, mriedem: the ksa patches for all of this are under review, so hopefully should be able to take legit forward aciton in the next couple of weeks15:10
sfinucandansmith: Like that?15:10
*** gbarros_ has joined #openstack-nova15:11
*** yamahata has quit IRC15:11
*** slaweq has joined #openstack-nova15:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add docstring for test_limit_check_project_and_user_zero_values  https://review.openstack.org/47228715:12
mriedemcdent: bauzas: melwitt: ^15:12
*** gbarros has quit IRC15:14
*** awaugama has joined #openstack-nova15:15
cfriesenkashyap: apparently we have booted UEFI on Newton by specifying ‘--property hw_firmware_type=uefi” on the image and then booting as normal.15:16
*** rcernin has quit IRC15:16
kashyapcfriesen: Thank you for confirming15:16
kashyapcfriesen: I just checked with Fedora 25, and current Nova Git15:16
kashyapcfriesen: I thought the Q35 machine type was _mandatory_ for OVMF15:17
kashyapBut, apparently it worked even with stock F25; which was QEMU 2.7 or something15:17
cfriesenkashyap: we're using CentOS 7's qemu-kvm-ev-2.6.0-28.el7_3.9.1, so who knows what all they've done. :)15:19
openstackgerritsean mooney proposed openstack/os-vif master: set mtu on all code paths  https://review.openstack.org/46995115:19
mnaserstable/newton is broken with the new release of oslo.db that drops sqlite_db (it was removed in stable/ocata and beyond but not in stable/newton apparently): https://review.openstack.org/#/c/472290/15:20
mnasernot sure why a notification didn't pop up here but if a small eye could be kept on that :)15:20
kashyapcfriesen: Heh15:20
*** jmlowe has joined #openstack-nova15:21
mriedemmnaser: i'm not sure why something dropped in ocata would break newton,15:21
mriedemsince we should be using upper-constraints15:21
mnasermriedem: maybe it was my mistake, but pretty much i did a pip install from the stable/newton repo and there doesn't seem to be a limit for oslo.db in requirements.txt15:22
mnaserso it downloaded the latest version of the library which has this commit from a few days ago => https://github.com/openstack/oslo.db/commit/cc64f7010522e812a696159009a72e59e2b017a815:23
mriedemmnaser: it'd be in tox.ini15:23
mnasermriedem ok i see it15:23
*** rmart04 has quit IRC15:23
mriedemmnaser: yeah that's 4.19.0, but we're not using that in newton https://github.com/openstack/requirements/blob/stable/newton/upper-constraints.txt#L21515:23
mriedemhttps://github.com/openstack/nova/blob/stable/newton/tox.ini#L12 should have you covered15:24
mnasermriedem: sorry about that then, should i abandon that patch in that case?15:24
openstackgerritAlex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update  https://review.openstack.org/46629615:24
mriedemmnaser: yes please15:25
*** tbachman has joined #openstack-nova15:25
mnasermriedem: done, thanks for the lesson in requirements and sorry for noise15:25
mriedemnot a problem15:25
cfriesenspeaking of requirements...I'm trying to run current master devstack and it's giving me a series of version issues:  ContextualVersionConflict: (os-brick 1.12.0 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('os-brick>=1.13.1'), set(['cinder']))15:28
cfriesenisn't it supposed to handle upgrading packages as needed automatically?15:28
*** yamamoto has quit IRC15:30
*** moshele has joined #openstack-nova15:31
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for aggregates  https://review.openstack.org/47093215:32
*** lei-zh1 has joined #openstack-nova15:33
*** lei-zh has quit IRC15:34
*** tbachman has quit IRC15:34
*** yamamoto has joined #openstack-nova15:35
*** jgriffith_ is now known as jgriffith15:36
*** moshele has quit IRC15:37
*** tbachman has joined #openstack-nova15:38
*** yamamoto has quit IRC15:39
mriedemmelwitt: can we use an IN clause to delete all server group members using the list of instance uuids at once? https://review.openstack.org/#/c/457858/1615:39
*** armax has quit IRC15:40
*** chyka has joined #openstack-nova15:40
*** Jack_Iv has quit IRC15:41
*** Jack_Iv has joined #openstack-nova15:41
*** Jack_Iv has quit IRC15:41
*** Jack_Iv has joined #openstack-nova15:42
*** Jack_Iv has quit IRC15:42
*** gyee has joined #openstack-nova15:42
*** slaweq has quit IRC15:43
*** Jack_Iv has joined #openstack-nova15:43
*** mdrabe has quit IRC15:44
*** lei-zh1 has quit IRC15:45
*** Jack_Iv has quit IRC15:47
*** mdrabe has joined #openstack-nova15:50
*** clayton has quit IRC15:51
*** markus_z has quit IRC15:51
*** Apoorva has joined #openstack-nova15:52
*** Apoorva has quit IRC15:52
sfinucanmelwitt: Could you take another look at https://review.openstack.org/#/c/379524/ today, please? :)15:52
*** hshiina has joined #openstack-nova15:52
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538515:54
*** dikonoor has joined #openstack-nova15:54
*** dtp has joined #openstack-nova15:54
*** mlakat has quit IRC15:54
*** clayton has joined #openstack-nova15:54
openstackgerritMerged openstack/python-novaclient master: Better handle key=value pair not being key=value  https://review.openstack.org/46998815:56
*** eharney has quit IRC15:57
*** lbragstad has joined #openstack-nova15:57
*** gyee has quit IRC15:58
*** jaosorior has quit IRC16:00
*** Oku_OS is now known as Oku_OS-away16:00
*** Apoorva has joined #openstack-nova16:01
*** gomarivera has quit IRC16:01
cfriesenre: my comment above, I just hit another one...neutron wants os-xenapi>=0.2.0.  Shouldn't devstack handle the pip upgrade itself?16:04
sdaguejohnthetubaguy: https://dogpilecache.readthedocs.io/en/latest/api.html#memory-backends if we wanted to go down that path16:08
*** eharney has joined #openstack-nova16:08
sdaguecfriesen: RECLONE=True ?16:08
sdaguebecause if not you can hit requirements files out of sync with upper-constraints16:09
*** nic1 has joined #openstack-nova16:09
cfriesensdague: thanks, will give that a try.  I thought that was the default, for some reason.16:12
sdagueno, because then it hard resets people's code16:13
cfriesenwonder if I should try PIP_UPGRADE=True16:13
*** gyee has joined #openstack-nova16:16
sdagueI don't think that does anything any more with constraints in play16:17
*** jamesdenton has joined #openstack-nova16:17
mriedemkashyap: is there any reason that nova "needs" to set the 'serial' field in the disk config for the libvirt domain xml?16:19
mriedemreading https://libvirt.org/formatdomain.html#elementsDisks16:19
mriedemit says,16:19
mriedem"If present, this specify serial number of virtual hard drive.           For example, it may look           like <serial>WD-WMAP9A966149</serial>.           Not supported for scsi-block devices, that is those using           disk type 'block' using device 'lun'           on bus 'scsi'.           Since 0.7.1"16:19
mriedemhowever, we make no distinction of when we set that in the disk config, block type or not16:20
mriedemand it's always the cinder volume id i believe16:20
mriedemdoes qemu actually care about it?16:20
johnthetubaguysdague: yeah, good call, spied your comment on that16:21
mriedemexcept avoid pickle16:22
*** lucasagomes is now known as lucas-afk16:26
*** slaweq has joined #openstack-nova16:27
cfriesensdague: with RECLONE=True I still hit it:  ContextualVersionConflict: (pyroute2 0.4.13 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('pyroute2>=0.4.15'), set(['neutron']))16:27
sdaguecfriesen: I don't know16:28
sdaguedefinitely going to need more than a single error log to know how you got there16:28
sdaguego through the log and see why there is a disconnect there16:29
*** sbezverk has joined #openstack-nova16:29
*** mingyu has joined #openstack-nova16:29
cfriesensdague: http://paste.openstack.org/show/612007/16:29
sdaguecfriesen: and the value in /opt/stack/requirements/upper-constraints.txt16:30
cfriesensdague: pyroute2===0.4.1516:30
sdaguehmmm... idk16:31
sdagueI thought that was always supposed to override16:32
*** tesseract has quit IRC16:32
cfriesensdague: it choked on pyroute2 again even with PIP_UPGRADE=True16:35
cfriesenI'm tempted to just blow everything away and start from scratch16:36
*** yamamoto has joined #openstack-nova16:37
sdagueyeh, I'm rerunning locally to try to figure out if I see anything16:37
sdaguethere could be a pip regression16:38
*** Yangyape_ has quit IRC16:42
*** derekh has quit IRC16:42
*** rajathagasthya has joined #openstack-nova16:42
*** armax has joined #openstack-nova16:43
*** yamamoto has quit IRC16:43
*** clayton has quit IRC16:43
openstackgerritMerged openstack/nova master: Default to 0 when merging values in limit check  https://review.openstack.org/47197316:45
*** smatzek has quit IRC16:46
*** clayton has joined #openstack-nova16:46
*** mamandle has quit IRC16:48
*** itlinux has joined #openstack-nova16:48
openstackgerritAlex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update  https://review.openstack.org/46629616:49
*** rajathagasthya has quit IRC16:52
dansmithmriedem: looks like that service version set is good now16:55
mriedemdansmith: did you see artom's questions in the top change?16:56
dansmithnope16:56
dansmithoh, yeah, I was just fixing the device metadata thing here16:57
mriedemthe min version is checked in 2 places in conductor16:57
dansmithI can do the others now16:57
dansmithartom: ^16:57
mriedemone is safely top-level super conductor,16:57
*** nic1 is now known as nic16:57
mriedemthe other is rpc, so it's not possibly (i think)16:57
dansmithyeah, the rpc one he points to needs to stay single-cell, but I'll look to see if maybe we should do it higher16:57
cfriesensdague: I have no idea what's going on.  I blew away ~/devstack and cloned it again, and blew away /opt/stack, and I'm still hitting version conflicts.16:57
*** hshiina has quit IRC16:58
dansmithmriedem: we may also be able to drop lots of these16:58
*** sambetts is now known as sambetts|afk16:58
mriedemi'd like to drop paging from the hypervisors api16:59
dansmithheh16:59
*** pcaruana has quit IRC17:00
*** cdent has quit IRC17:00
*** rajathagasthya has joined #openstack-nova17:01
clarkbcfriesen: sdague fwiw a trivial attempt at reproducing using a venv with 0.4.13 installed then an install of 0.4.15 using constraints works ok17:01
mriedembtw i'm out tomorrow17:02
clarkbso whatever it is seems more complicated than just pyroute2 and constraints17:03
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366017:03
*** ralonsoh has quit IRC17:03
openstackgerritAlex Szarka proposed openstack/nova master: Refactor instance-exists docstring and unit test  https://review.openstack.org/46273817:03
*** dtp has quit IRC17:04
*** baoli has quit IRC17:04
clarkbcfriesen: maybe try with pip -v and/or --log ?17:05
*** baoli has joined #openstack-nova17:05
cfriesenclarkb: I can upgrade this manually no problem....but I expect devstack to be doing the upgrading and I don't understand why it isn't17:05
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.resize.error notifications  https://review.openstack.org/41788217:06
clarkbcfriesen: so if you run the command yourself it works fine?17:06
cfriesenclarkb: I'm hitting this with multiple packages.  If I manually upgrade one package it continues on until it hits another one that needs upgrading.17:06
mriedemdansmith: are you ok with this now after the cells v1 rpc version bump? https://review.openstack.org/#/c/461532/17:06
cfriesenclarkb: something like this works fine: sudo pip install --upgrade ovsdbapp==0.4.017:07
dansmithmriedem: with that approach yeah, I need to look at the rest of the patch though17:07
sdaguecfriesen: I for installed pyroute2==0.4.13 then ran ./clean.sh && ./stack.sh and everthing works fine17:07
clarkbperhaps something with the -H then and pip caching things? But also getting output from pip -v and/or --log would likely be helpful17:07
sdaguethis is a 16.04 box17:08
cfriesensdague: I hadn't tried clean.sh, will give that a shot17:08
cfriesensdague: same here17:09
*** mamandle has joined #openstack-nova17:10
openstackgerritMerged openstack/nova master: code comments incorrectness  https://review.openstack.org/44941617:11
cfriesenmriedem: if you've got suggestions on how to properly mock the api_version in https://review.openstack.org/#/c/435141/ I'd be happy to make changes.  I couldn't figure it out short of modifying each testcase using the "list" command.17:12
*** lucasxu has quit IRC17:14
*** ltomasbo is now known as ltomasbo|away17:14
*** sridharg has quit IRC17:15
*** ericyoung has quit IRC17:16
cfriesensdague: ./clean.sh wasn't enough.  just hit ContextualVersionConflict with neutron-lib17:16
cfriesensdague: clarkb: http://paste.openstack.org/show/612034/17:18
*** rmart04 has joined #openstack-nova17:18
*** jpena is now known as jpena|off17:23
*** baoli has quit IRC17:23
*** baoli has joined #openstack-nova17:24
*** mamandle has quit IRC17:24
*** fragatina has quit IRC17:24
*** baoli has quit IRC17:25
*** amotoki is now known as amotoki_away17:28
*** rmart04 has quit IRC17:30
*** dtp has joined #openstack-nova17:31
artomdansmith, huh, didn't notice the ping - so basically the other places where get_min_service_version is called will be fixed in different patches?17:32
dansmithartom: yeah, I was just fixing that one place in device metadata because of mriedem's bug he opened17:32
dansmithbut I17:32
dansmithwill take a look at the rest in a few17:32
artomdansmith, makes sense :)17:33
artomKeep commits topical17:33
*** cdent has joined #openstack-nova17:33
*** mkucia has quit IRC17:33
*** baoli has joined #openstack-nova17:34
*** amotoki_away is now known as amotoki17:35
*** ihrachys has joined #openstack-nova17:35
ihrachysseems like port detach issues still happens even with https://review.openstack.org/#/c/441204/ in17:36
mriedemi like my creams just like i like my commits17:36
ihrachysmriedem, ^17:36
ihrachysexample: http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=TRACE#_Jun_08_16_58_01_39598917:36
mriedemhmm, that's not fun17:37
*** nicolasbock has quit IRC17:37
mriedemihrachys: it's always this same test right http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/console.html#_2017-06-08_17_13_16_844927 ?17:38
mriedemthe hotpluc nic one?17:38
mriedemand usually on the linuxbridge job?17:38
artommriedem, the word you're looking for is "ointments" - it's up there with "moist" as the most shudder-inducing word in the English language17:38
mriedemyeah it's like 99% LB job17:38
ihrachysI think yes17:38
mriedemi wonder what's different about LB in thiscase17:38
mriedemmaybe the vif isn't unplugging how we'd expect before we detach from the guest?17:38
mriedemartom: i think "topical cream"17:39
mriedemnot "topical ointment"17:39
ihrachysafaik for linuxbridge case, neutron now manages the whole plugging cycle ?17:39
ihrachyssince late switch to tap device type17:39
*** hieulq_ has quit IRC17:39
mriedemihrachys: i don't know anything about that17:39
ihrachysI mean https://review.openstack.org/#/c/447150/17:39
*** hieulq_ has joined #openstack-nova17:39
*** cdent has quit IRC17:39
artommriedem, because of you my search history is weird now17:39
mriedemartom: search on your phone, not your work system17:40
*** hieulq_ has quit IRC17:40
mriedemnothing weird about grown man pain reliefe17:40
mriedem*relief17:40
mriedemjust ask shaq17:40
* artom wonders if we need a DNM patch that shows us the actual libvirt error17:41
mriedemshould be in the libvirtd logs wouldn't it?17:41
mriedemi also don't understand why libvirt considers this a transient domain17:43
* ihrachys doesn't even know what transient domain is17:43
mriedemhttps://wiki.libvirt.org/page/VM_lifecycle#Transient_guest_domains_vs_Persistent_guest_domains17:43
artomI *think* it does both?17:43
mriedemme neither17:43
artomAs in, a running domain has the persistent part, and the transient part17:44
artomkashyap would help here, but I think he's gone17:44
melwittI think it's something like it creates a transient domain on-the-fly and uses that for the migration? something like that?17:44
ihrachysoh it's just a in-ram domain?17:44
mriedemreading ^ makes me think all of our guests are transient17:44
mriedemthis isn't a migration though17:44
melwittoh17:45
*** jamesdenton has quit IRC17:45
mriedemhttp://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_network_basic_ops.py#n47517:45
*** itlinux has quit IRC17:45
cfriesenmriedem: I think we generally make the xml file and tell libvirt to make a guest from it, making them persistent17:46
artomWait, that's not a libvirt error17:48
melwittone thing to know is a detach from transient domain is optional, that is, the guest can ignore it17:49
artomhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L442-L44917:50
cfriesenmelwitt: that sounds like a terrible idea.17:50
mriedemartom: oh nice spot17:50
mriedemwe should be checking the persistent flag there17:50
ihrachysehm, why in the snippet do we raise unconditionally?17:50
mriedemihrachys: it's part of a retry loop17:51
mriedemsee the decorator17:51
mriedemit retries on that expected error until the device conf is gone or we timeout17:51
artomIt retries until it stops raising?17:51
ihrachysah I see17:51
cfriesenmelwitt: mriedem: so under what circumstances would nova create a transient domain?17:51
melwittI don't think nova does it, I thought it was something with qemu but I can't remember what kashyap told me. maybe I'm totally off on that17:52
cfriesenlooks like maybe while doing a snapshot17:52
mriedemwe're not using os-vif http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_19_92244517:53
mriedembut that probably doesn't matter17:53
*** nicolasbock has joined #openstack-nova17:53
artomHrmm, that could be a libvirt error though17:56
artomIf we've failed to detach and reach the max retry count17:56
* artom is confused17:56
mriedemit's got to be something with linuxbridge and this tap vif type17:56
*** jwcroppe has quit IRC17:56
mriedemnot a problem for ovs17:56
mriedemand a race, since it's not 100% fail17:57
*** jwcroppe has joined #openstack-nova17:57
openstackgerritMerged openstack/nova master: Use microversions for new style volume attachments  https://review.openstack.org/46957917:57
kfox1111so... whats the easiest way to list out all vms in all tenants that use a particular flavor?17:58
mriedemSuccessfully detached device <nova.virt.libvirt.config.LibvirtConfigGuestInterface object at 0x7f86a0051650> from guest. Persistent? 1. Live? True {{(pid=24466) _try_detach_device /opt/stack/new/nova/nova/virt/libvirt/guest.py:407}}17:58
*** gjayavelu has joined #openstack-nova17:58
melwittyeah, initially I was thinking we must have reached the max retry count because the guest wasn't responding, but I'm not sure how to tell from the log, if we can look at a timestamp delta at all to see how long it took before the error was raised17:58
mriedemkfox1111: nova list --all-tenants --flavor x ?17:58
ihrachysmriedem, you think we may try to revert the tap switch patch for the time being?17:59
kfox1111doh. there it was, right in front of me.17:59
ihrachyswill it possibly help17:59
kfox1111thanks. :)17:59
mriedemihrachys: the neutron one?17:59
*** jwcroppe_ has joined #openstack-nova17:59
mriedemihrachys: we could see if that's the cause by rechecking that a few times17:59
ihrachysat this point, we can't land much in neutron, and we have a line of gate fixes, so anything that could help..17:59
ihrachysok lemme revert and see where it goes17:59
mriedemi don't know enough about the low level networking stuff here to know why this would cause issues17:59
mriedemi'm going to go libvirtd log diving17:59
*** lucasxu has joined #openstack-nova18:00
*** jwcroppe has quit IRC18:01
mriedemright before we detach in the driver, we get the network-vif-unplugged event from neutron18:03
*** gszasz has quit IRC18:04
mriedembut we don't do anything with that18:05
mriedem"Successfully detached device <nova.virt.libvirt.config.LibvirtConfigGuestInterface object at 0x7f86a0051650> from guest. Persistent? 1. Live? True"18:06
mriedemso i guess the domain is persistent on the first detach_device call to the guest18:06
*** smatzek has joined #openstack-nova18:07
openstackgerritMerged openstack/nova master: Use 3.27 microversion when creating new style volume attachments  https://review.openstack.org/46960918:08
mriedemhmm, could we be hitting one of these libvirt errors and raising DeviceNotFound?18:08
mriedemhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L40918:09
melwittcompute manager ignores DeviceNotFound though, just considers detach already done, right?18:09
openstackgerritMerged openstack/nova master: Skip microversion discovery check for update/delete volume attachments  https://review.openstack.org/46966818:09
mriedemfor volumes18:09
mriedemthe driver ignores DeviceNotFound during detach_interface18:09
mriedemso that's probably not it18:10
*** jamesdenton has joined #openstack-nova18:10
mriedemwtf, i don't think it's even retrying18:10
*** pewp has quit IRC18:10
mriedemso we detach the guest here http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_25_26681418:11
mriedems/guest/device18:11
mriedemthat's https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L40518:11
mriedemon the first attempt18:11
mriedemwhere persistent=118:11
mriedemthen we start our retry loop http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_25_26722318:11
mriedemthen we try detach again http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_30_27831718:11
mriedemthis time persistent=False18:11
mriedembecause https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L44518:12
*** pewp has joined #openstack-nova18:12
mriedemoh nvm it does retry18:13
ihrachysit may be interesting that tap3e81295f-c1 never shows in l2 agent log since 08 16:56:20.783179 which is long before the detach. should we have seen the agent to e.g. get a detach event from neutron-server and untangling in from the integration bridge, or smth? I assume nova talks to neutron when detaching?18:13
mriedemthe retry decorator doesn't have the logging context for the request id18:13
mriedemihrachys: nova will only update the port to unbind it18:13
mriedemotherwise nova just unplugs the vif and relies on neutron agent picking that up and processing it18:14
mriedemand then neutron sends the network-vif-unplugged event18:14
*** Sukhdev has joined #openstack-nova18:16
mriedemi wonder if we're not checking the right things here18:17
mriedemhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L247-L25618:17
mriedemthis is our xml18:18
mriedemhttp://paste.openstack.org/show/612041/18:18
mriedemfrom LibvirtConfigGuestInterface18:18
ihrachysmriedem, what do you mean unplugging the VIF? from the domain? because with tap vif type, everything else is probably on neutron to do right? meaning, you request unbind and then we handle it all?18:19
ihrachyskevinbenton, are you up already?18:20
mriedemihrachys: i mean this https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L145018:20
ihrachyswhich is empty for lb18:22
mriedemihrachys: no it's going here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L90618:23
ihrachysoh. I was looking at unplug in os-vif18:24
*** mvk has quit IRC18:24
ihrachysfor lb plugin18:24
mriedemyeah it's not using os-vif18:24
mriedembecause there is no os-vif tap conversion in nova yet18:24
mriedemhttp://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_19_92244518:25
mriedemNo conversion for VIF type tap yet18:25
mriedemso nova does: sudo nova-rootwrap /etc/nova/rootwrap.conf ip link delete tap3e81295f-c118:25
*** salv-orl_ has quit IRC18:25
*** salv-orlando has joined #openstack-nova18:26
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: fix alternative_device_name for detaching interfaces  https://review.openstack.org/47237118:30
*** salv-orlando has quit IRC18:30
mriedemihrachys: i also noticed this comment which is probably from danpb https://github.com/openstack/nova/blob/master/nova/virt/libvirt/designer.py#L5618:31
mriedembut that's the method called to build the device config for this type of tap device18:31
ihrachysbtw I misinterpreted the logs before. I now see that neutron logs operations on the tap device at the same time nova starts the decorated func18:31
mriedemthis is all i see about it in the libvirt docs https://libvirt.org/formatdomain.html#elementsNICSEthernet18:32
mriedemi assume it's discouraged because "Provides a means for the administrator to execute an arbitrary script       to connect the guest's network to the LAN."18:32
mriedemas far as i can tell though nova doesn't pass through any script path18:33
mriedemhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/designer.py#L6918:34
mriedemoh i think that means if not provided, it defaults to /etc/qemu-ifup18:34
*** jamesdenton has quit IRC18:34
*** jamesdenton has joined #openstack-nova18:35
*** gbarros_ has quit IRC18:35
*** dikonoor has quit IRC18:35
*** itlinux_ has joined #openstack-nova18:38
mriedemihrachys: i'm going to push a change that catches the final exception, dumps a bunch of debug info to the logs, and then re-raises18:38
mriedemto see what's actually in the guest xml when we fail compared to the device xml we're trying to remove18:38
*** moshele has joined #openstack-nova18:44
*** nicolasbock has quit IRC18:45
*** nicolasbock has joined #openstack-nova18:48
*** egonzalez has joined #openstack-nova18:49
*** crushil has quit IRC18:50
*** nicolasbock has quit IRC18:50
*** nicolasbock has joined #openstack-nova18:50
*** cdent has joined #openstack-nova18:53
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: dump debug info when interface detach times out  https://review.openstack.org/47237718:54
mriedemihrachys: i'll push a neutron change that depends on this ^ to debug, i see you pushed a revert18:54
*** amotoki is now known as amotoki_away18:55
openstackgerritCorey Bryant proposed openstack/nova master: Make nova-placement-api default config less strict  https://review.openstack.org/47237818:56
*** mnestratov|3 has quit IRC18:56
*** dtp has quit IRC18:58
*** jwcroppe_ has quit IRC18:58
*** dave-mccowan has quit IRC18:58
*** eharney has quit IRC18:58
cfriesenif instances have attached volumes, that information would be in the block_device_mapping table of the nova db, right?18:59
mriedemcfriesen: yes19:00
*** cristicalin has quit IRC19:00
*** nicolasbock has quit IRC19:00
*** gbarros has joined #openstack-nova19:01
*** dave-mccowan has joined #openstack-nova19:02
*** jwcroppe has joined #openstack-nova19:03
*** cdent has quit IRC19:04
*** jwcroppe has quit IRC19:04
*** jwcroppe has joined #openstack-nova19:05
mriedemwe have an intermittent KeyError in functional tests https://bugs.launchpad.net/nova/+bug/169683419:05
openstackLaunchpad bug 1696834 in OpenStack Compute (nova) "Intermittent "KeyError: 'allocations'" in functional tests" [Medium,Confirmed]19:05
*** nicolasbock has joined #openstack-nova19:05
melwittyeah, seems to hit pretty often too19:05
mriedem14 in 7 days19:05
*** jmlowe_ has joined #openstack-nova19:07
*** jmlowe has quit IRC19:08
*** Jack_Iv has joined #openstack-nova19:09
*** pewp has quit IRC19:13
*** yamahata has joined #openstack-nova19:17
mriedemdansmith: melwitt: can one of you put this through so i don't have to work it into my hypervisors uuid api series? https://review.openstack.org/#/c/471970/19:17
mriedemi have to build on that one19:17
mriedemto change the type19:17
dansmithdone19:17
openstackgerritCorey Bryant proposed openstack/nova master: Make nova-placement-api default config less strict  https://review.openstack.org/47237819:17
mriedem<319:17
*** pewp has joined #openstack-nova19:17
mriedemfleet week failed again,19:18
mriedembut something different this time http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/console.html#_2017-06-08_18_39_03_92916819:18
*** nicolasbock has quit IRC19:18
*** nicolasbock has joined #openstack-nova19:18
dansmitho.O19:19
mordredefried: https://review.openstack.org/#/q/project:openstack/keystoneauth+topic:version-discovery is updated from reviews if you wanna poke19:21
efriedmordred Roger wilco.19:21
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get  https://review.openstack.org/46153219:21
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API  https://review.openstack.org/47189919:21
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Use uuid for id in Services API  https://review.openstack.org/46428019:21
*** rmart04 has joined #openstack-nova19:21
mordredefried: still missing interface lists and service type aliases - but we're getting real close19:21
efriedneat19:21
mriedemdansmith: unrelated19:22
mriedemthe test is posting a password to the metadata server19:23
dansmithgood, because.......19:23
*** nicolasbock has quit IRC19:23
mriedemand then checks to see it shows up19:23
mriedemwhich seems super racy19:23
mriedemthat's totally async so tempest should probably be polling here19:23
mriedemmaybe i'm not sure,19:26
mriedemit sets a password field in the instance sys meta19:26
mriedemit's a POST,19:26
mriedembut i'm not sure where the metadata service returns19:26
openstackgerritMerged openstack/nova master: api-ref: fix type for hypervisor_marker  https://review.openstack.org/47197019:26
kevinbentonihrachys: hey19:26
*** gbarros has quit IRC19:27
kevinbentonihrachys: looking at backlog. l2-agent never deletes a tap device, that's left up to nova19:28
kevinbentonmriedem: ^^19:28
kevinbentonbut it looks like you already found that19:28
kevinbentonanything i can help with?19:28
*** itlinux_ has quit IRC19:28
*** awaugama has quit IRC19:31
*** sbezverk has quit IRC19:32
mriedemdansmith: i guess the metadata POST is synchronous19:32
*** rajathagasthya has quit IRC19:34
*** moshele has quit IRC19:35
mriedemdansmith: aha http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/screen-n-api.txt.gz#_Jun_08_18_36_06_54974819:36
mriedemthe cache fs us again19:36
mriedemwe update inst.system_metadata, then tempest calls the rest api to get the password, and it looks like things are going through the metadata cache19:37
mriedemnot sure why we wouldn't update the cache when we update/set the password...19:37
dansmithbut that's still unrelated right?19:38
dansmithbut yeah, that metadata stuff needs fixin'19:38
mriedemwtf, we're getting an InstanceNotFound19:38
mriedemoooooo19:38
mriedemit is cells19:38
mriedemwe're not targeted19:38
mriedemmofo19:38
mriedemopening a bug19:38
dansmithwe shouldn't be targeted19:39
mriedemactually the instance should be targeted when we look it up19:39
mriedemi mean, we look up the mapping and then target the instance don't we?19:40
dansmithmetadata should be a cell thing so it shouldn't be using target_cell, is my point19:40
dansmithmaybe that's the problem19:40
dansmithdoes metadata run as a separate process in devstack?19:40
* dansmith looks19:40
mriedemnot here, it's part of nova-api19:41
openstackgerritMerged openstack/nova master: Add docstring for test_limit_check_project_and_user_zero_values  https://review.openstack.org/47228719:41
*** mvk has joined #openstack-nova19:41
mriedemhttps://bugs.launchpad.net/nova/+bug/169684319:42
openstackLaunchpad bug 1696843 in OpenStack Compute (nova) "metadata: POST /openstack/2013-10-17/password fails with InstanceNotFound" [Undecided,New]19:42
mriedemkevinbenton: i've got a debug patch up to see if i can recreate in a neutron patch and get some dumps of the related domain xml19:43
dansmithmriedem: well, that's going to be a problem then19:43
*** claudiub has joined #openstack-nova19:44
mriedemand ihar has a patch up with a revert of the lb tap change to see if that is any different19:44
*** itlinux_ has joined #openstack-nova19:44
*** yamahata has quit IRC19:44
*** gbarros has joined #openstack-nova19:44
mriedemdansmith: right here https://github.com/openstack/nova/blob/b535c7432d02366f11fad7d1b1a3e3a8b8ac5415/nova/api/metadata/password.py#L7019:45
mriedemwhy wouldn't we lookup the instance mapping via that uuid19:45
mriedemand then target the get_by_uuid to the cell19:45
dansmithmriedem: because metadata is in the cell and we can't do that19:45
mriedemi thought it was global19:45
mriedemactually i don't think i've ever heard of anyone ever talk about the metadata service wrt cells v219:45
dansmithnot sure why it would be global,19:46
dansmithbut also,19:46
dansmithnot sure from a neutron perspective19:46
dansmithI always think of the world like nova net multihost19:46
dansmithmaybe we can ask kevinbenton since he's here19:46
mriedemwhy wouldn't it be global if the api is global?19:46
mriedemi guess because you want the guests confined to the cell19:47
mriedem?19:47
dansmithhmm, we already target in metadata one place19:47
kevinbentonask me what?19:47
mriedemyeah...19:47
mriedemif metadata can't be global then it's going to have to know what cell it's in19:47
dansmithmriedem: in multihost you run one per compute node so you scale 1:1 with computes19:47
*** gszasz has joined #openstack-nova19:47
dansmithmriedem: I don't think I understand why that is19:48
*** jaypipes has quit IRC19:48
dansmithmriedem: however, question would be how the neutron proxy thing finds the metadata api19:48
dansmithkevinbenton: ^19:48
mriedemit's configured in neutron.conf i though19:48
mriedem*thought19:48
kevinbentondansmith: ah, it's just a hardcoded config19:48
kevinbentondansmith: of a single http endpoint19:48
dansmithbut neutron config per compute node right?19:48
mriedemno19:48
mriedemwell i can't answer that19:49
kevinbentonit's on the network node that runs the proxy19:49
*** gbarros has quit IRC19:49
kevinbentonso not compute node19:49
dansmithkevinbenton: so is it a single config for the whole deployment?19:49
kevinbentondansmith: essentially yes19:49
kevinbentondansmith: you could have different configs per network node19:49
dansmithokay, not sure that's a good thing, although it does make this easier19:49
kevinbentonbut i don't think that would do anything useful19:49
mriedemkevinbenton: where is that in http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/etc/neutron/neutron.conf.txt.gz ?19:49
kevinbentonsince networks nodes aren't correlated with cells19:49
mriedemoh nvm http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/etc/neutron/metadata_agent.ini.txt.gz19:49
mriedemnova_metadata_ip = 10.40.184.14019:50
kevinbentonmriedem: +119:50
dansmithfrom a scale point of view I mean, but since we already effectively have it global I guess that's a totally different conversation19:50
*** rmart04 has quit IRC19:50
mriedemdon't people already disable metadata service b/c of perf issues, and security19:50
mriedemi know that's irrelevant19:51
dansmithdepends on their goals, but some do yeah19:51
dansmithalright anyway, I'll fix that up19:51
openstackgerritDan Smith proposed openstack/nova master: Remove old service version check for mitaka  https://review.openstack.org/47239519:51
*** mingyu has quit IRC19:52
mriedemi suppose this meta bug has to be backported19:52
mriedemalthough i'm only seeing it in the multi-cell devstack change19:52
*** rajathagasthya has joined #openstack-nova19:56
kevinbentondansmith, mriedem: is there a way we can know which cell to send something to?19:56
kevinbentonwe could alter the proxy logic to route in some manner because it does do a port lookup to inject headers19:57
dansmithmriedem: I don't think we need to backport it19:57
*** salv-orlando has joined #openstack-nova19:57
dansmithkevinbenton: you don't and shouldn't know any of that19:57
kevinbentondansmith: ok, just checking if this is an issue we need to fix on our metadata proxy logic19:58
mriedemi wonder how cells v1 people are doing the metadata service19:59
dansmithkevinbenton: no it sounds like you're effectively forcing a global metadata server now, which makes a fix for this easy on our end19:59
dansmithmriedem: not sure it works in cellsv1, or they just run it in a cell I'd imagine19:59
mriedemi'll ask in ops19:59
kevinbentonjust put a load balancer in front of it that fans every request out to a cell until it doesn't get a 404 :)19:59
*** liangy has joined #openstack-nova19:59
*** lyan has quit IRC20:01
mriedemyou don't get a 404 here,20:02
mriedemyou get a 500 :(20:02
mriedemwe probably also need to fix that20:02
mriedemhttp://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/screen-n-api.txt.gz#_Jun_08_18_36_06_54974820:02
mriedem10.1.0.11,10.40.184.140 "POST /openstack/2013-10-17/password HTTP/1.1" status: 500 len: 139 time: 0.0445061: InstanceNotFound: Instance 6cd26f78-9c8a-4b37-8c62-38ad003a095f could not be found.20:03
mriedemi'll open a separate bug for that20:03
mriedemwhich is backportable20:04
*** pchavva has quit IRC20:04
mriedemhttps://bugs.launchpad.net/nova/+bug/169684820:05
openstackLaunchpad bug 1696848 in OpenStack Compute (nova) "POST /openstack/2013-10-17/password returns 500 if instance is not found" [Medium,Triaged]20:05
*** harlowja has quit IRC20:08
mriedemclaudiub: are you doing anything with this? https://blueprints.launchpad.net/nova/+spec/hyperv-sriov20:08
mriedemelse i'm going to defer it to queens20:08
*** liverpooler has quit IRC20:09
mriedemgdi, you know what's awesome,20:13
mriedemoverriding a method and tests start failing, you spend too much time trying to figure out why,20:13
mriedemand realize you're not returning the results of the method you're now delegating too20:13
mriedem*to20:13
*** pchavva has joined #openstack-nova20:17
*** jmlowe_ has quit IRC20:17
*** Jack_Iv has quit IRC20:17
*** Jack_Iv has joined #openstack-nova20:18
openstackgerritMaciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_spice_channel  https://review.openstack.org/47078720:18
openstackgerritMaciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_video_device  https://review.openstack.org/47078320:18
openstackgerritMaciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_memory_balloon  https://review.openstack.org/47078620:18
openstackgerritMaciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_watchdog_action  https://review.openstack.org/47078520:18
openstackgerritMaciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_pci_devices  https://review.openstack.org/47078420:18
melwittdoes anyone know if it matter whether you give a sqlalchemy in_() call a list or a set? is there any benefit in a set vs a list?20:18
melwitt*matters20:19
*** sbezverk has joined #openstack-nova20:19
*** Jack_Iv has quit IRC20:22
*** cheran has joined #openstack-nova20:23
*** owalsh is now known as owalsh-pto20:23
bauzasmelwitt: AFAIK, it just needs a container20:25
bauzasie. anything having an __in__20:26
melwittbauzas: yeah, I know it accepts list, tuple, set. I was just curious if there was a benefit of any over others. I didn't see anything when I googled so far20:26
dansmithisn't it generating a text query?20:27
dansmithsuch that you need iter() not __in__ ?20:27
dansmithin that case, I can't imagine it would matter, other than that a set avoids potential dupes20:27
bauzasmmmm, http://docs.sqlalchemy.org/en/latest/core/sqlelement.html#sqlalchemy.sql.operators.ColumnOperators.in_20:28
melwittyeah, that would make sense why there no mention about it mattering20:28
bauzasit just says lists and tuples20:28
melwitt*there's. guh, I can't type today20:28
bauzashonestly, I think it's not a big deal20:28
melwittI assumed it's not a big deal, I was just curious about it20:28
* bauzas shrugs20:29
bauzasmelwitt: if it accepts sets, then cool, as dansmith said20:30
melwittI know. I know what it accepts. I was curious how it worked and decided to ask if anyone happened to know20:30
bauzasfor dupes I mean20:30
*** mnestratov|3 has joined #openstack-nova20:30
bauzasunrelated, could someone tell me the cases where when we have a live-migration, the instance can be shutoff ?20:31
bauzasAFAIK, when it's timing out, it's not shut the instance off, right?20:32
*** Apoorva_ has joined #openstack-nova20:33
*** cleong has quit IRC20:33
*** jmlowe has joined #openstack-nova20:35
dansmithbauzas: afaik live migration only works on a running instance, and if you want to live migrate it you need to start it paused to do so20:35
*** Apoorva has quit IRC20:36
openstackgerritDan Smith proposed openstack/nova master: Fix lookup of instance mapping in metadata set-password  https://review.openstack.org/47240520:36
*** purplerbot has quit IRC20:36
bauzasdansmith: sorry, the question is about the output of a live-migration20:37
bauzasdansmith: I'm trying to understand https://bugs.launchpad.net/nova/+bug/169602520:37
openstackLaunchpad bug 1696025 in OpenStack Compute (nova) "live-migration sometimes failed" [Undecided,New]20:37
bauzasbut I'll ask for more details20:37
* dansmith shrugs20:37
*** purplerbot has joined #openstack-nova20:38
*** hamzy has quit IRC20:38
*** Apoorva_ has quit IRC20:41
*** Apoorva has joined #openstack-nova20:41
*** karimb has quit IRC20:42
*** jamesdenton has quit IRC20:42
*** Dksad has joined #openstack-nova20:44
openstackgerritMerged openstack/nova master: Add PCIWeigher  https://review.openstack.org/37952420:44
*** salv-orl_ has joined #openstack-nova20:44
openstackgerritSteve Noyes proposed openstack/nova master: WIP: update live migration to use v3 cinder api  https://review.openstack.org/46398720:46
*** Dksad has left #openstack-nova20:47
*** pchavva has quit IRC20:47
*** salv-orlando has quit IRC20:47
*** cdent has joined #openstack-nova20:51
*** slaweq has quit IRC20:52
*** slaweq has joined #openstack-nova20:53
*** takashin has joined #openstack-nova20:54
*** tidwellr has quit IRC20:54
melwittthis is weird, api sample test fail because of the v2.1 in the url? http://logs.openstack.org/59/457859/17/check/gate-nova-tox-functional-py35-ubuntu-xenial/6e6bd37/testr_results.html.gz20:56
*** tidwellr has joined #openstack-nova20:56
mriedemdansmith: you can live migrate a paused instance20:57
mriedemwe have a tempest test for it20:57
dansmithmriedem: right20:57
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API  https://review.openstack.org/47189920:58
*** slaweq has quit IRC20:58
sdaguemelwitt: that actually looks like the template didn't fill for some reason20:58
*** Apoorva_ has joined #openstack-nova20:59
melwittsdague: oh, I see. so it shouldn't be showing the placeholders20:59
sdagueno, or they should match20:59
*** dtp_ has joined #openstack-nova21:00
openstackgerritCorey Bryant proposed openstack/nova master: Pass CLI options to nova-placement-api  https://review.openstack.org/47237821:00
mriedemmeeting time21:00
melwittyeah. everything looks to match, except possibly the "v2.1" part. but that parameter is called "versioned_compute_endpoint" so it seems like it would encompass the v2.121:00
*** tidwellr has quit IRC21:01
*** Apoorva has quit IRC21:02
openstackgerritmelanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project()  https://review.openstack.org/44624621:02
openstackgerritmelanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project()  https://review.openstack.org/44624721:02
openstackgerritmelanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object  https://review.openstack.org/44624421:02
openstackgerritmelanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts()  https://review.openstack.org/44624521:02
openstackgerritmelanie witt proposed openstack/nova master: Remove 'reserved' count from used limits  https://review.openstack.org/44624221:02
openstackgerritmelanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage  https://review.openstack.org/44624321:02
*** thorst_afk has quit IRC21:02
openstackgerritmelanie witt proposed openstack/nova master: Count networks to check quota  https://review.openstack.org/44624121:02
openstackgerritmelanie witt proposed openstack/nova master: Count instances to check quota  https://review.openstack.org/41652121:02
openstackgerritmelanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db  https://review.openstack.org/45785821:02
openstackgerritmelanie witt proposed openstack/nova master: Count server group members to check quota  https://review.openstack.org/45785921:02
openstackgerritmelanie witt proposed openstack/nova master: Count security groups to check quota  https://review.openstack.org/45786021:02
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094521:02
openstackgerritmelanie witt proposed openstack/nova master: Count fixed ips to check quota  https://review.openstack.org/45786121:02
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094621:02
openstackgerritmelanie witt proposed openstack/nova master: Count floating ips to check quota  https://review.openstack.org/45786221:03
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform instance.live_migration_rollback notification  https://review.openstack.org/40212421:03
*** thorst_afk has joined #openstack-nova21:04
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform instance.soft_delete notifications  https://review.openstack.org/46002921:04
*** xyang1 has joined #openstack-nova21:05
bauzascdent: you could be interested in https://bugs.launchpad.net/nova/+bug/169683021:06
openstackLaunchpad bug 1696830 in OpenStack Compute (nova) "nova-placement-api default config files is too strict" [Low,In progress] - Assigned to Corey Bryant (corey.bryant)21:06
bauzascdent: I'm honestly not opiniated but given oslo.config supports this, placement should accept that too21:07
cdentbauzas: I don't have much of an opinion either. sdague wrote that code if I remember right.21:07
* cdent looks at the patch21:08
cdenthmm, not clear on how that fix will work21:08
*** thorst_afk has quit IRC21:09
*** lucasxu has quit IRC21:11
*** Sukhdev has quit IRC21:14
bauzascdent: seems like the patch is not fully done yet21:17
bauzascdent: just pointed you the bug report rather21:17
*** Sukhdev has joined #openstack-nova21:17
openstackgerritMerged openstack/nova master: Add get_minimum_version_all_cells() helper for service  https://review.openstack.org/47143121:17
openstackgerritMerged openstack/nova master: Fix device metadata service version check for multiple cells  https://review.openstack.org/47143221:18
openstackgerritMerged openstack/nova master: Move rebuild notification tests into separate method  https://review.openstack.org/45178921:19
*** Sukhdev has quit IRC21:21
*** tbachman has quit IRC21:25
*** rfolco has quit IRC21:26
*** esberglu has quit IRC21:27
*** thorst_afk has joined #openstack-nova21:27
*** esberglu has joined #openstack-nova21:27
efriedCan anyone explain why e.g. https://review.openstack.org/#/c/471796/ is showing a merge conflict?21:29
efriedIt appears to be right at the tip of the newton branch.21:29
bauzasefried: I need to rebase it21:29
efriedbauzas I tried that.  It's already where it should be, as far as I can tell.21:30
efriedThere are several like this.21:30
bauzasefried: lemme test again21:31
*** thorst_afk has quit IRC21:31
*** esberglu has quit IRC21:32
mikalThe CI system was telling lies yesterday21:32
mriedemyes it was21:33
efriedYeah, but how do we get it to straighten up?21:33
bauzasmikal: mmm, so recheck then ?21:33
mriedemefried: just needs a recheck21:34
mriedemwhich i just did21:34
efriedk, cool21:34
* bauzas facepalms21:34
bauzasjust did too21:34
*** jwnx has quit IRC21:34
mriedemNOOOOOOO21:34
mriedemdouble recheck21:34
mriedemsends it to the black hole21:35
bauzasrecheck recheck recheck21:35
bauzas(candyman)21:35
jgriffithmriedem FYI, it appears that cinder-migrate is broken anyway and would seem that it has been for some time now.21:36
mriedemjgriffith: great21:36
mriedemjgriffith: not sure if scottda's tempest changes to test retype in a multinode job ever landed21:36
mriedembut he was working on that21:37
jgriffithmriedem  not sure.. but anyway, I started on this:  https://docs.google.com/document/d/10IfIeZ4luPbPOd2oi57FByQ5ByN223KN4j81EskOIM0/edit#21:37
jgriffithildikov ^^21:37
ildikovjgriffith: thank you!21:38
jgriffithand I was thinking.. "hey, we don't update the bdm on the volume-id swap nonsense it seems". so tried it out and, well... it's borked21:38
jgriffithanyway... back at it, let ya know if/when I make some progress21:38
*** tbachman has joined #openstack-nova21:38
*** fragatina has joined #openstack-nova21:38
jgriffithit may work using the retype path... I'll try that next21:39
mriedemmelwitt: so the IN thing is all good?21:40
mriedemjgriffith: i figured retype was the same from a nova pov21:40
melwittmriedem: looks like it, yeah. worked in the db func test. the only thing I wasn't 100% about is what to use for the synchronize_session parameter, most of the bulk deletes I found use False21:40
mriedemmaybe a zzzeek question21:41
melwitthas to do with whether anything else in the session expects to use those objects after deletion (whether to remove them from session after delete) but I thought that shouldn't be the case that anything else is using the session21:41
mriedems/maybe/definitely/ s/question/conjuring/21:41
jgriffithmriedem nope, sadly... the retype uses a different rpc call to get to the manager21:42
jgriffithanyway21:42
*** smatzek has quit IRC21:42
jgriffithf'ing shit code21:43
jgriffithjust incase I haven't mentioned my distaste a hundred times already21:43
mriedemheh21:45
*** mriedem is now known as mriedem_afk21:48
*** mriedem_afk has quit IRC21:48
*** catintheroof has quit IRC21:50
*** tbachman has quit IRC21:53
*** slaweq has joined #openstack-nova21:54
*** xyang1 has quit IRC21:57
*** slaweq has quit IRC21:59
*** xyang1 has joined #openstack-nova21:59
*** adisky__ has quit IRC22:00
*** edmondsw has quit IRC22:02
*** baoli has quit IRC22:02
*** baoli has joined #openstack-nova22:02
*** luos has quit IRC22:04
*** lyarwood has quit IRC22:04
*** luos has joined #openstack-nova22:04
*** lyarwood has joined #openstack-nova22:04
*** mnestratov|3 has quit IRC22:05
*** tbachman has joined #openstack-nova22:05
*** dave-mccowan has quit IRC22:05
scottdamriedem this tempest test merged: https://review.openstack.org/#/c/326681/22:06
scottdaand this project-config: https://review.openstack.org/#/c/330678/22:06
*** FL1SK has quit IRC22:07
*** baoli has quit IRC22:07
*** mdrabe has quit IRC22:11
*** salv-orl_ has quit IRC22:19
*** salv-orlando has joined #openstack-nova22:20
*** Apoorva_ has quit IRC22:21
*** jdurgin has quit IRC22:21
*** cheran has quit IRC22:21
*** Apoorva has joined #openstack-nova22:21
*** cdent has quit IRC22:22
cfriesenhow do I restart services in devstack now?22:22
cfriesennever mind, I was looking for the wrong name in the systemd service list22:23
*** salv-orlando has quit IRC22:24
*** esberglu has joined #openstack-nova22:29
*** jdurgin has joined #openstack-nova22:30
*** dtp_ has quit IRC22:32
*** esberglu has quit IRC22:33
*** xyang1 has quit IRC22:39
*** jwcroppe has quit IRC22:39
*** liangy has quit IRC22:51
*** mingyu has joined #openstack-nova22:53
*** tonygunk has quit IRC22:53
*** slaweq has joined #openstack-nova22:55
openstackgerritmelanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations  https://review.openstack.org/32541422:57
openstackgerritmelanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation  https://review.openstack.org/33399022:57
*** mingyu has quit IRC22:58
*** slaweq has quit IRC23:00
*** rajathagasthya has quit IRC23:00
*** tonygunk has joined #openstack-nova23:01
*** itlinux_ has quit IRC23:03
*** rajathagasthya has joined #openstack-nova23:04
*** felipemonteiro__ has quit IRC23:05
*** fragatina has quit IRC23:07
*** sdague has quit IRC23:08
openstackgerritMerged openstack/nova master: Count server groups to check quota  https://review.openstack.org/44624023:09
*** armax_ has joined #openstack-nova23:12
*** egonzalez has quit IRC23:12
*** Hosam has joined #openstack-nova23:14
*** armax has quit IRC23:14
*** armax_ is now known as armax23:14
*** claudiub has quit IRC23:15
*** d9k has quit IRC23:16
*** d9k has joined #openstack-nova23:17
*** Jack_Iv has joined #openstack-nova23:18
*** pewp has quit IRC23:21
*** pewp has joined #openstack-nova23:22
*** Jack_Iv has quit IRC23:22
*** rajathagasthya has quit IRC23:25
*** Apoorva has quit IRC23:25
*** rajathagasthya has joined #openstack-nova23:28
*** awaugama has joined #openstack-nova23:28
*** mhenkel has quit IRC23:32
*** armax_ has joined #openstack-nova23:33
*** armax has quit IRC23:33
*** armax_ is now known as armax23:33
*** markvoelker has quit IRC23:34
*** sbezverk has quit IRC23:38
*** Apoorva has joined #openstack-nova23:40
*** rajathagasthya has quit IRC23:43
*** jaypipes has joined #openstack-nova23:48
*** jaypipes has quit IRC23:48
*** tonygunk has quit IRC23:53
*** chyka has quit IRC23:56
*** slaweq has joined #openstack-nova23:56
*** harlowja has joined #openstack-nova23:57
*** jamielennox is now known as jamielennox|away23:57

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