Thursday, 2017-06-29

*** acormier has joined #openstack-nova00:00
openstackgerritmelanie witt proposed openstack/nova master: Handle version for PUT and POST in PlacementFixture  https://review.openstack.org/47868300:00
*** chyka has joined #openstack-nova00:03
*** acormier has quit IRC00:04
*** Nel1x has joined #openstack-nova00:06
*** N3l1x has joined #openstack-nova00:07
*** gmann has quit IRC00:08
*** gmann has joined #openstack-nova00:08
openstackgerritmelanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota  https://review.openstack.org/47770000:10
openstackgerritmelanie witt proposed openstack/nova master: Add scatter gather utilities for cells  https://review.openstack.org/47595700:10
openstackgerritmelanie witt proposed openstack/nova master: Remove 'reserved' count from used limits  https://review.openstack.org/44624200:10
openstackgerritmelanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota  https://review.openstack.org/47769900:10
openstackgerritmelanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage  https://review.openstack.org/44624300:10
openstackgerritmelanie witt proposed openstack/nova master: Count instances to check quota  https://review.openstack.org/41652100:10
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094500:10
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094600:10
*** ssurana has joined #openstack-nova00:12
*** jaypipes has quit IRC00:14
*** Swami has quit IRC00:16
*** thorst has joined #openstack-nova00:18
*** yangyapeng has quit IRC00:18
*** dtp has quit IRC00:19
*** jdwidari has joined #openstack-nova00:21
*** thorst has quit IRC00:23
*** esberglu has joined #openstack-nova00:28
*** thorst has joined #openstack-nova00:31
*** yingjun has joined #openstack-nova00:32
*** rajathagasthya has quit IRC00:32
*** esberglu has quit IRC00:32
*** thorst has quit IRC00:33
*** rajathagasthya has joined #openstack-nova00:33
*** Jack_Iv has joined #openstack-nova00:33
*** yingjun has quit IRC00:35
*** rajathagasthya has quit IRC00:37
*** Jack_Iv has quit IRC00:37
*** ckopper has joined #openstack-nova00:39
*** Jack_Iv has joined #openstack-nova00:49
*** gjayavelu has quit IRC00:51
*** huanxie has joined #openstack-nova00:53
*** Jack_Iv has quit IRC00:53
*** tuanluong has joined #openstack-nova00:56
*** phuongnh has joined #openstack-nova01:04
*** Swami has joined #openstack-nova01:06
*** edmondsw has joined #openstack-nova01:07
*** liujiong has joined #openstack-nova01:10
*** edmondsw has quit IRC01:12
*** ssurana has quit IRC01:12
*** jmlowe has quit IRC01:19
*** Shunli has joined #openstack-nova01:19
*** litao__ has joined #openstack-nova01:20
*** kaisers__ has joined #openstack-nova01:21
*** yangyapeng has joined #openstack-nova01:22
*** esberglu has joined #openstack-nova01:22
*** esberglu has quit IRC01:22
*** kaisers_ has quit IRC01:24
*** chyka has quit IRC01:24
*** chyka has joined #openstack-nova01:25
*** yangyapeng has quit IRC01:26
*** yangyapeng has joined #openstack-nova01:27
*** masayukig has quit IRC01:28
*** masayukig has joined #openstack-nova01:29
*** krtaylor has joined #openstack-nova01:30
gaoyuananyone doing bp/remove-mox-pike online?01:30
gaoyuanI have a little question01:32
*** jmlowe has joined #openstack-nova01:32
*** Apoorva_ has joined #openstack-nova01:34
*** sbezverk has quit IRC01:34
*** iceyao has joined #openstack-nova01:34
*** crushil_ has joined #openstack-nova01:35
*** Jack_Iv has joined #openstack-nova01:35
*** Apoorva_ has quit IRC01:36
*** thorst has joined #openstack-nova01:37
*** thorst has quit IRC01:37
*** Swami has quit IRC01:37
*** Apoorva has quit IRC01:37
*** Jack_Iv has quit IRC01:40
*** lucasxu has joined #openstack-nova01:41
*** Apoorva has joined #openstack-nova01:45
*** yangyapeng has quit IRC01:48
*** yangyapeng has joined #openstack-nova01:48
*** Apoorva has quit IRC01:49
gmanngaoyuan, yes, what is question?\01:50
*** Jack_Iv has joined #openstack-nova01:51
*** zz_dimtruck is now known as dimtruck01:52
*** yamahata has quit IRC01:53
*** Apoorva has joined #openstack-nova01:53
*** Jack_Iv has quit IRC01:55
*** crushil_ has quit IRC01:56
*** dave-mccowan has joined #openstack-nova01:58
*** iceyao_ has joined #openstack-nova02:03
*** iceyao has quit IRC02:03
*** dave-mccowan has quit IRC02:05
*** dave-mccowan has joined #openstack-nova02:06
*** pewp has quit IRC02:07
*** youzp has joined #openstack-nova02:07
*** ssurana has joined #openstack-nova02:08
*** Apoorva has quit IRC02:10
*** pewp has joined #openstack-nova02:11
*** crushil_ has joined #openstack-nova02:11
gaoyuangmann: please see this review https://review.openstack.org/#/c/477812/202:12
*** bkopilov_ has quit IRC02:12
gaoyuanKen'ichi Ohmichi says he prefer another patch using mock02:12
*** ssurana has quit IRC02:13
gaoyuanwhat in the bp description of remove-mox-pike, it states Please note: self.stub_out does not make use of mox, it is there to replace to use of "self.stub.Set" in an easy to review way. Moreover, and proposals to remove the use of stub_out should not be part of this spec.02:13
gmanngaoyuan, checking02:13
gmanngaoyuan, left comment on https://review.openstack.org/#/c/477904/102:13
gmanngaoyuan, on preferable way of mocking things.02:13
*** gyee has quit IRC02:14
gaoyuanDoes this mean we should use self.stub_out to replace self.stub.Set?02:14
gaoyuanor I misunderstood02:14
gaoyuangmann, thanks for the updating.02:15
*** crushil_ has quit IRC02:15
*** esberglu has joined #openstack-nova02:16
gmanngaoyuan, yes, as of now we are replacing the self.stub.Set with self.stub_out02:16
gmanngaoyuan, because if there are lot of changes in review changing the  self.stub.Set to mock (i saw in some of the review) then it is very hard to review02:17
gaoyuangmann, but it's got a -1, and I already abandoned it, https://review.openstack.org/#/c/477384/, this one got +2 already02:17
gmanngaoyuan, in that case you pointed it is just 1 line and thats why may be kenichi prefer the mock instead of stub_out02:17
gaoyuanOK, I understand02:18
gmanngaoyuan, i saw, seems like other patch was initially proposed. but as i mentioned as it is just 1 line so it is +202:18
gmanngaoyuan, but if its too many changes then stub_out is preferable as of now02:19
*** sree has joined #openstack-nova02:19
gaoyuangmann, finally, if you still have time, https://review.openstack.org/#/c/451614/, please have a look at this one02:19
*** acormier has joined #openstack-nova02:20
*** esberglu has quit IRC02:20
gmanngaoyuan, you should comment on that patch instead of new patch (just matching the timing of your review on another patch and your patch :))02:20
gmanngaoyuan, but main idea is same as you understood.  self.stub.Set -> self.stub_out02:21
gmanngaoyuan, sure, i will check but may be in evening or tomorrow for sure. added in queue02:21
gaoyuangmman, in test_vmops.py, I used a function mock_out_object which is created by myself in nova/test.py02:21
gaoyuanOK, good day then02:22
gaoyuangmann, I hope I am not too much disturbing.02:22
gmanngaoyuan, but stub_out does not work there ?02:22
gmanngaoyuan, not at all, we really appreciate your help on fixing mox things and rally nice work.02:23
gaoyuanno02:23
gaoyuanI have commetns02:24
gaoyuanunless I am wrong02:24
gmanngaoyuan, ok, let me check the changes in detail ad02:24
gmannand will get back to you02:24
*** gcb has joined #openstack-nova02:24
gaoyuangmann, 2ks02:24
gaoyuan3ks02:24
*** acormier has quit IRC02:24
*** Shunli has quit IRC02:33
*** Shunli has joined #openstack-nova02:33
*** Shunli has quit IRC02:35
*** trungnv has quit IRC02:35
*** yangyapeng has quit IRC02:35
*** Shunli has joined #openstack-nova02:35
*** hongbin has joined #openstack-nova02:36
*** Jack_Iv has joined #openstack-nova02:37
*** Shunli has quit IRC02:40
*** Shunli has joined #openstack-nova02:40
*** yangyapeng has joined #openstack-nova02:41
*** Jack_Iv has quit IRC02:42
*** Shunli has quit IRC02:42
*** Shunli has joined #openstack-nova02:42
*** Shunli has quit IRC02:48
*** gouthamr has joined #openstack-nova02:51
*** bakbad has joined #openstack-nova02:53
*** dave-mccowan has quit IRC02:55
*** edmondsw has joined #openstack-nova02:56
*** xinliang_ has quit IRC02:59
*** xinliang has joined #openstack-nova03:00
*** edmondsw has quit IRC03:00
*** salv-orlando has joined #openstack-nova03:04
*** salv-orlando has quit IRC03:09
*** hongbin has quit IRC03:10
*** zhurong has quit IRC03:11
*** Shunli has joined #openstack-nova03:12
*** yamahata has joined #openstack-nova03:18
*** Jack_Iv has joined #openstack-nova03:23
*** chenying has joined #openstack-nova03:24
*** N3l1x has quit IRC03:26
*** Nel1x has quit IRC03:26
*** baoli has joined #openstack-nova03:27
*** Jack_Iv has quit IRC03:28
*** gouthamr has quit IRC03:30
*** bkopilov_ has joined #openstack-nova03:30
*** gouthamr has joined #openstack-nova03:31
*** udesale has joined #openstack-nova03:36
*** thorst has joined #openstack-nova03:38
*** gouthamr has quit IRC03:39
*** Jack_Iv has joined #openstack-nova03:39
*** thorst has quit IRC03:43
*** Jack_Iv has quit IRC03:43
*** gouthamr has joined #openstack-nova03:47
*** zhurong has joined #openstack-nova03:49
*** yamamoto has joined #openstack-nova03:50
*** Jack_Iv has joined #openstack-nova03:55
*** adisky__ has joined #openstack-nova03:56
*** takashin has joined #openstack-nova03:58
*** amotoki_away is now known as amotoki03:58
*** Jack_Iv has quit IRC04:00
*** john5223_ has joined #openstack-nova04:00
*** lucasxu has quit IRC04:01
*** salv-orlando has joined #openstack-nova04:05
*** cNilesh has joined #openstack-nova04:08
*** salv-orlando has quit IRC04:09
*** hemanth has joined #openstack-nova04:09
*** Jack_Iv has joined #openstack-nova04:10
*** vladikr_ has quit IRC04:10
*** huanxie has quit IRC04:14
*** Jack_Iv has quit IRC04:15
*** Jack_Iv has joined #openstack-nova04:26
*** trungnv has joined #openstack-nova04:28
*** Jack_Iv has quit IRC04:30
*** dixiaoli has joined #openstack-nova04:35
*** ekuris has quit IRC04:39
*** dixiaoli has quit IRC04:39
*** psachin has joined #openstack-nova04:40
*** edmondsw has joined #openstack-nova04:43
*** edmondsw has quit IRC04:49
*** liujiong has quit IRC04:49
*** links has joined #openstack-nova04:53
*** gouthamr has quit IRC04:54
*** esberglu has joined #openstack-nova04:59
*** youzp has quit IRC04:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix a missing classifier  https://review.openstack.org/47873305:01
*** links has quit IRC05:03
*** esberglu has quit IRC05:03
*** lpetrut has joined #openstack-nova05:04
*** mkrai_ has joined #openstack-nova05:06
*** chyka has quit IRC05:07
*** Jack_Iv has joined #openstack-nova05:12
*** pcaruana has joined #openstack-nova05:14
*** links has joined #openstack-nova05:15
*** gyee has joined #openstack-nova05:15
*** Jack_Iv has quit IRC05:16
*** moshele has joined #openstack-nova05:17
*** huanxie has joined #openstack-nova05:18
*** Shunli has quit IRC05:18
*** Shunli has joined #openstack-nova05:19
*** trinaths has joined #openstack-nova05:19
*** dixiaoli has joined #openstack-nova05:19
*** Shunli has quit IRC05:23
*** Shunli has joined #openstack-nova05:24
*** prateek has joined #openstack-nova05:25
*** ssurana has joined #openstack-nova05:26
openstackgerritAlex Xu proposed openstack/nova master: placement: add traits support to get resource_providers with shared  https://review.openstack.org/47846405:27
*** Jack_Iv has joined #openstack-nova05:27
*** dixiaoli has quit IRC05:28
*** pcaruana has quit IRC05:30
*** slaweq_ has joined #openstack-nova05:31
*** Jack_Iv has quit IRC05:32
*** pcaruana has joined #openstack-nova05:33
*** slaweq_ has quit IRC05:34
*** salv-orlando has joined #openstack-nova05:34
*** ekuris has joined #openstack-nova05:35
mkrai_Hi I have question regarding nova serial console. How does it work?05:38
mkrai_In Ironic, a new shellinabox process is started which serves the serial console via ipmitool. How is it done in Nova?05:38
mkrai_Link to code or any doc will be helpful.05:39
mkrai_Thanks in advance.05:39
*** thorst has joined #openstack-nova05:39
*** ssurana has left #openstack-nova05:39
*** pcaruana has quit IRC05:39
*** Jack_Iv has joined #openstack-nova05:43
*** Shunli has quit IRC05:43
*** Shunli has joined #openstack-nova05:44
*** thorst has quit IRC05:44
*** Shunli has quit IRC05:45
*** Shunli has joined #openstack-nova05:46
*** Jack_Iv has quit IRC05:47
*** rcernin_ has joined #openstack-nova05:48
*** Jack_Iv has joined #openstack-nova05:51
*** lpetrut has quit IRC05:51
*** ratailor has joined #openstack-nova05:52
*** jamesden_ has quit IRC05:52
*** jamesdenton has joined #openstack-nova05:53
*** zhurong has quit IRC05:56
*** karts has quit IRC05:56
*** mariusv has joined #openstack-nova05:59
*** mariusv has quit IRC05:59
*** mariusv has joined #openstack-nova05:59
*** trinaths has quit IRC05:59
*** rcernin_ is now known as rcernin06:04
*** andreas_s has joined #openstack-nova06:07
openstackgerritGao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.test_block_device  https://review.openstack.org/47874906:08
*** zhurong has joined #openstack-nova06:11
openstackgerritGao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vm_utils.py  https://review.openstack.org/47874906:11
*** slaweq_ has joined #openstack-nova06:12
*** artom has joined #openstack-nova06:13
*** slaweq_ has quit IRC06:13
*** damien_r has joined #openstack-nova06:13
*** Oku_OS-away is now known as Oku_OS06:15
*** zhurong has quit IRC06:19
*** zhurong has joined #openstack-nova06:22
*** salv-orlando has quit IRC06:25
*** salv-orlando has joined #openstack-nova06:25
*** vvargaszte has quit IRC06:26
*** salv-orlando has quit IRC06:30
*** edmondsw has joined #openstack-nova06:32
*** damien_r has quit IRC06:35
*** trinaths has joined #openstack-nova06:35
*** edmondsw has quit IRC06:36
*** rajathagasthya has joined #openstack-nova06:45
*** slaweq_ has joined #openstack-nova06:45
*** hshiina has joined #openstack-nova06:46
*** slaweq_ has quit IRC06:52
*** kaisers__ has quit IRC06:54
*** markus_z has joined #openstack-nova06:56
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI  https://review.openstack.org/47843706:56
*** trinaths has quit IRC06:57
*** vvargaszte has joined #openstack-nova07:00
*** vvargaszte has quit IRC07:01
*** kaisers_ has joined #openstack-nova07:03
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Factor out relocate_vm()  https://review.openstack.org/27011507:03
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances  https://review.openstack.org/27011607:03
*** Shunli has quit IRC07:04
*** trinaths has joined #openstack-nova07:04
*** Shunli has joined #openstack-nova07:05
*** trinaths1 has joined #openstack-nova07:05
*** trinaths has quit IRC07:09
*** Shunli has quit IRC07:09
*** Shunli has joined #openstack-nova07:10
*** Shunli has quit IRC07:11
*** yassine has joined #openstack-nova07:11
*** gyee has quit IRC07:12
*** cNilesh has quit IRC07:17
*** cNilesh has joined #openstack-nova07:17
*** damien_r has joined #openstack-nova07:18
gmannmordred, can you check this - https://review.openstack.org/#/c/459172/  (could not find you on #openstack)07:19
*** salv-orlando has joined #openstack-nova07:21
*** vvargaszte has joined #openstack-nova07:22
*** slaweq has quit IRC07:24
*** slaweq has joined #openstack-nova07:28
openstackgerritBéla Vancsics proposed openstack/nova master: [WIP] Transform live_migration_post_dest notification  https://review.openstack.org/46978407:30
*** tesseract has joined #openstack-nova07:31
*** pcaruana has joined #openstack-nova07:35
openstackgerritChason Chan proposed openstack/nova master: import the admin guide content from openstack-manuals  https://review.openstack.org/47749707:35
*** ssurana1 has joined #openstack-nova07:36
*** jpena|off is now known as jpena07:37
*** thorst has joined #openstack-nova07:40
*** cNilesh has quit IRC07:40
*** esberglu has joined #openstack-nova07:41
*** cNilesh has joined #openstack-nova07:41
*** lpetrut has joined #openstack-nova07:42
*** esberglu has quit IRC07:45
*** thorst has quit IRC07:47
*** tesseract has quit IRC07:47
*** huanxie has quit IRC07:47
*** openstackgerrit has quit IRC07:47
*** tesseract has joined #openstack-nova07:48
*** hemanth has quit IRC07:49
*** ralonsoh has joined #openstack-nova07:50
*** openstackgerrit has joined #openstack-nova07:52
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow  https://review.openstack.org/33028507:52
* bauzas bows to the community 07:52
bauzasnow I have long backlog of emails and other reviews to look after :(07:52
*** ssurana1 has left #openstack-nova07:57
*** zzzeek has quit IRC08:00
*** chenying has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** chenying has joined #openstack-nova08:01
*** junbo has quit IRC08:03
*** eliqiao has quit IRC08:04
*** moshele has quit IRC08:06
*** junbo has joined #openstack-nova08:06
*** klipka has quit IRC08:08
*** eliqiao has joined #openstack-nova08:09
*** sree has quit IRC08:10
*** dtantsur|afk is now known as dtantsur08:11
*** huanxie has joined #openstack-nova08:11
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI  https://review.openstack.org/47843708:11
*** rajathagasthya has quit IRC08:11
openstackgerritklyang proposed openstack/nova master: Fix the log information argument mistake  https://review.openstack.org/47879208:12
*** sree has joined #openstack-nova08:12
ebbexWhat's the status on the nova-metadata wsgi script? Is it ok to use nova/wsgi/nova-metadata.py, or has it been replaced with something more up-to-date?08:12
*** salv-orl_ has joined #openstack-nova08:13
*** derekh has joined #openstack-nova08:15
*** gouthamr has joined #openstack-nova08:16
*** salv-orlando has quit IRC08:17
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847308:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section  https://review.openstack.org/47847208:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847508:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775108:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847408:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848408:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead plugin  https://review.openstack.org/47846908:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848508:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove cruft from conf.py  https://review.openstack.org/47846808:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Create directory structure for docs migration  https://review.openstack.org/47847108:20
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead files  https://review.openstack.org/47847008:20
*** salv-orl_ has quit IRC08:25
*** salv-orlando has joined #openstack-nova08:28
openstackgerritgaryk proposed openstack/nova master: VMware: ensure that live migration attaches to correct interface  https://review.openstack.org/47879708:29
*** tesseract has quit IRC08:30
openstackgerritgaryk proposed openstack/nova master: VMware: ensure that live migration attaches to correct interface  https://review.openstack.org/47879708:31
*** tesseract has joined #openstack-nova08:32
*** carthaca_ has joined #openstack-nova08:37
*** esberglu has joined #openstack-nova08:37
*** lpigueiras has joined #openstack-nova08:39
lpigueirashello, could you take a look to this PR plz: https://review.openstack.org/#/c/476122/ ?08:39
lpigueirasit should solve this bug: https://bugs.launchpad.net/nova/+bug/1592167 for a cells deployment08:39
openstackLaunchpad bug 1592167 in OpenStack Compute (nova) mitaka "Deleted keypair causes metadata failure" [Medium,Fix released] - Assigned to Matt Riedemann (mriedem)08:39
*** damien_r has quit IRC08:40
*** damien_r has joined #openstack-nova08:40
*** esberglu has quit IRC08:42
*** Masahisa has left #openstack-nova08:43
*** slaweq_ has joined #openstack-nova08:49
*** moshele has joined #openstack-nova08:54
*** slaweq_ has quit IRC08:54
*** karimb has joined #openstack-nova08:57
*** sree has quit IRC08:59
moshelesfinucan: hi08:59
moshelesfinucan: can we release the os-vif?08:59
*** sree has joined #openstack-nova08:59
*** huanxie has quit IRC09:01
sfinucanmoshele: I think that's mriedem's decision09:01
sfinucanjaypipes or sean-k-mooney might have an idea, but neither of those are online :/09:01
moshelesfinucan: I will check09:02
*** mkucia has joined #openstack-nova09:02
openstackgerritAlex Xu proposed openstack/nova master: placement: add traits support to get resource_providers with shared  https://review.openstack.org/47846409:02
*** sree has quit IRC09:03
*** rmart04 has joined #openstack-nova09:09
alex_xugmann: just filed a bug for quota-class https://bugs.launchpad.net/nova/+bug/170121109:17
openstackLaunchpad bug 1701211 in OpenStack Compute (nova) "The network related quotas aren't deprecated in quota-class API" [Undecided,Confirmed]09:17
*** sambetts|afk is now known as sambetts09:17
gmannalex_xu, thanks. i will update spec and push ML soon09:18
alex_xugmann: thanks09:19
*** AK47 has joined #openstack-nova09:19
*** AK47 has quit IRC09:19
*** Jack_Iv has quit IRC09:20
*** AK47 has joined #openstack-nova09:20
AK47..09:22
*** hshiina has quit IRC09:23
*** yamahata has quit IRC09:23
bauzassfinucan: you could help some tech debt kill by looking at https://review.openstack.org/#/c/469037/ :)09:24
sfinucanbauzas: Oh, definitely. Nice work09:24
sfinucanJust finishing doc-rework patches, then I'll move onto that09:25
sfinucanOur man pages are really rubbish :(09:25
sfinucanexcept for nova-manage, funnily enough, heh09:25
bauzas++09:27
openstackgerritMoshe Levi proposed openstack/os-vif master: unplug_vf_passthrough: don't try to delete representor netdev  https://review.openstack.org/47882009:29
*** trinaths1 has left #openstack-nova09:30
*** Jack_Iv has joined #openstack-nova09:30
*** esberglu has joined #openstack-nova09:31
*** esberglu has quit IRC09:31
openstackgerritChason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme  https://review.openstack.org/47775709:32
*** avolkov has joined #openstack-nova09:33
*** AK47 has quit IRC09:34
*** dixiaoli has joined #openstack-nova09:34
*** tiendc has joined #openstack-nova09:36
*** huanxie has joined #openstack-nova09:38
*** liujiong has joined #openstack-nova09:38
sfinucanbauzas: What are we replacing rootwrap with in the longterm? oslo.policy?09:38
*** tesseract has quit IRC09:38
bauzassfinucan: privsep, no ?09:39
sfinucanyup, it is privsep09:39
sfinucanMy bad :)09:39
bauzasthat's fine, mikal isn't watching us :p09:39
*** tesseract has joined #openstack-nova09:40
*** yamamoto has quit IRC09:41
*** yamamoto has joined #openstack-nova09:43
*** yamamoto has quit IRC09:43
*** thorst has joined #openstack-nova09:43
*** thorst has quit IRC09:48
*** kaisers_ has quit IRC09:48
*** kaisers_ has joined #openstack-nova09:48
*** slaweq_ has joined #openstack-nova09:50
*** slaweq_ has quit IRC09:55
*** mvk has quit IRC09:58
sfinucanbauzas: Question - I'm working on patches for https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html09:59
sfinucanThat tells us to use doc-migration for the topic10:00
sfinucanBut I'm guessing I should link against a blueprint, and that's called os-manuals-migration10:00
sfinucanIf I do that, the topic is auto-set to bp/os-manuals-migration10:00
*** salv-orlando has quit IRC10:00
*** dixiaoli has quit IRC10:01
sfinucanSo - drop the blueprints lines, or create a spec-less 'doc-migration' blueprint for nova?10:01
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847310:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section  https://review.openstack.org/47847210:04
*** ralonsoh has quit IRC10:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847510:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775110:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847410:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848410:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead plugin  https://review.openstack.org/47846910:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848510:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove cruft from conf.py  https://review.openstack.org/47846810:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Create directory structure for docs migration  https://review.openstack.org/47847110:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead files  https://review.openstack.org/47847010:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847310:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section  https://review.openstack.org/47847210:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847510:06
sfinucanturns out I could do none of the above, because bp/doc-rework != doc-rework10:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775110:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847410:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848410:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead plugin  https://review.openstack.org/47846910:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848510:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove cruft from conf.py  https://review.openstack.org/47846810:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Create directory structure for docs migration  https://review.openstack.org/47847110:06
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead files  https://review.openstack.org/47847010:06
*** yamamoto has joined #openstack-nova10:06
sfinucanOK, I think that's it now10:07
bauzasoh LOL10:10
*** pewp has quit IRC10:14
*** ralonsoh has joined #openstack-nova10:15
*** mkrai_ has quit IRC10:16
openstackgerritIldiko Vancsa proposed openstack/nova master: Remove check_detach  https://review.openstack.org/44667110:17
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow  https://review.openstack.org/33028510:17
*** pewp has joined #openstack-nova10:18
*** Masahisa has joined #openstack-nova10:20
*** iceyao_ has quit IRC10:22
*** liujiong has quit IRC10:23
*** yangyapeng has quit IRC10:24
*** vvargaszte has quit IRC10:24
*** mvk has joined #openstack-nova10:25
*** karimb has quit IRC10:29
ildikovsfinucan: hi10:29
ildikovsfinucan: did you get a chance to take a look at the swap and/or live_migrate patches I pinged you about earlier this week?10:29
sfinucanildikov: Yes, but I haven't been able to wrap my head around it :(10:30
sfinucanI really don't know that code, so I'm having to learn it from scratch10:30
ildikovsfinucan: I know that feeling pretty well :( :)10:30
ildikovsfinucan: please let e know if I can help with anything10:31
*** artom has quit IRC10:38
*** artom has joined #openstack-nova10:38
sfinucangibi: Can you take a look at my comments on https://review.openstack.org/#/c/448779/30 when possible10:39
sfinucanParticularly the bit about disabling API features with config10:39
*** artom_ has joined #openstack-nova10:40
*** artom_ has quit IRC10:41
*** artom_ has joined #openstack-nova10:41
*** udesale has quit IRC10:44
gaurangtHi, I encountered a bug in nova where quota details is not updated when a failed instance is deleted. I see that there is a fix for that which is backported to ocata. https://review.openstack.org/#/c/445236/10:44
gaurangtSo, what should I do to update the quota to correct values now?10:45
*** artom has quit IRC10:45
*** phuongnh has quit IRC10:47
*** vvargaszte has joined #openstack-nova10:51
*** vvargaszte has quit IRC10:51
*** karimb has joined #openstack-nova10:51
gibisfinucan: looking...10:54
*** jpena is now known as jpena|lunch10:56
*** artom has joined #openstack-nova10:58
*** artom has quit IRC10:58
*** artom has joined #openstack-nova10:59
*** iceyao has joined #openstack-nova10:59
*** artom has quit IRC10:59
*** artom_ has quit IRC10:59
*** artom has joined #openstack-nova10:59
*** ssurana has joined #openstack-nova11:00
*** artom has quit IRC11:01
*** artom has joined #openstack-nova11:01
*** bkopilov_ has quit IRC11:02
*** ssurana has quit IRC11:04
gibisfinucan: responded to your comments in https://review.openstack.org/#/c/448779/3011:06
openstackgerritBalazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload  https://review.openstack.org/44877911:06
*** Jack_Iv has quit IRC11:08
*** eliqiao has quit IRC11:09
*** eliqiao has joined #openstack-nova11:11
*** vvargaszte has joined #openstack-nova11:12
*** artom has quit IRC11:15
*** sree has joined #openstack-nova11:15
*** artom has joined #openstack-nova11:16
*** artom has quit IRC11:16
*** esberglu has joined #openstack-nova11:19
*** vladikr_ has joined #openstack-nova11:21
*** esberglu has quit IRC11:23
*** acormier has joined #openstack-nova11:25
*** acormier has quit IRC11:25
*** acormier has joined #openstack-nova11:25
*** hemanth has joined #openstack-nova11:26
*** yangyapeng has joined #openstack-nova11:32
*** karimb has quit IRC11:33
*** edmondsw has joined #openstack-nova11:35
*** smatzek has joined #openstack-nova11:36
*** yamamoto has quit IRC11:43
*** kornicameister has quit IRC11:43
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709111:46
*** acormier has quit IRC11:50
*** acormier has joined #openstack-nova11:50
*** gszasz has joined #openstack-nova11:51
*** acormier_ has joined #openstack-nova11:52
*** slaweq_ has joined #openstack-nova11:52
*** acormier_ has joined #openstack-nova11:52
*** kornicameister has joined #openstack-nova11:53
*** kornicameister has quit IRC11:54
*** acormier has quit IRC11:55
*** yamamoto has joined #openstack-nova11:55
*** litao__ has quit IRC11:57
*** thorst has joined #openstack-nova11:57
*** slaweq_ has quit IRC11:57
*** nolwenn has quit IRC11:59
*** tuanluong has quit IRC11:59
*** jpena|lunch is now known as jpena12:00
*** yamamoto has quit IRC12:00
*** acormier_ has quit IRC12:03
*** tiendc has quit IRC12:04
*** ericyoung has joined #openstack-nova12:07
*** prateek has quit IRC12:08
*** dixiaoli has joined #openstack-nova12:09
*** yamamoto has joined #openstack-nova12:12
*** baoli has quit IRC12:19
*** mriedem has joined #openstack-nova12:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore  https://review.openstack.org/47486912:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore  https://review.openstack.org/47446312:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore  https://review.openstack.org/47849912:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore  https://review.openstack.org/47849812:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for volumes instead of stevedore  https://review.openstack.org/47850212:20
openstackgerritAlex Xu proposed openstack/nova master: Remove the test for the route '/resources.:(format)'  https://review.openstack.org/47890012:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore  https://review.openstack.org/47890112:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore  https://review.openstack.org/47890212:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore  https://review.openstack.org/47890312:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore  https://review.openstack.org/47890412:20
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes instead of stevedore  https://review.openstack.org/47890512:20
*** artom has joined #openstack-nova12:21
*** ratailor has quit IRC12:21
mriedemyar12:23
gibimriedem: hi! if you have some time, the next two patches of searchlight notification additions just need a second +212:24
gibimriedem: start here /review.openstack.org/#/c/41973012:24
gibimriedem, sfinucan: and the BDM patch (the third) needs some discussion between you two guys12:24
gibimriedem ,sfinucan: about the configuration option for turning on BDM in instance.action12:25
edleafemriedem: When you have time, please re-review https://review.openstack.org/#/c/473627/ - it really doesn't seem that an extra-specs value can be None.12:25
bauzasgibi: edleafe: AFAIK, mriedem is in China this week12:28
*** abhishekk has quit IRC12:28
edleafebauzas: yeah, that's why I'm trying to catch him before he goes to sleep :)12:29
bauzaswell, UTC+8 means 10:30pm :p12:29
mriedemgah12:30
mriedem8:30pm right now12:30
bauzaswe're not all night owls :p12:30
bauzaswhat? did I make some wrong calculation ? /me is offended12:30
bauzasfake news!12:30
*** artom has quit IRC12:32
*** esberglu has joined #openstack-nova12:32
gibibauzas: I don't want to keep mriedem awake, I just let him know a possibility in case of insomnia hits12:32
*** artom has joined #openstack-nova12:32
gaoyuanI have a mox3 problem? any one can help12:32
gaoyuan?12:32
*** vladikr_ is now known as vladikr12:33
bauzasso, instead of counting sheeps, counting lines ?12:33
bauzasgaoyuan: sure, how can I help ?12:33
gibibauzas: yeah... maybe counting +2s :)12:33
bauzasgibi: just provide a correct URL and I could help too :p12:34
* bauzas is lazy12:34
gaoyuanbaozas: I have a mox3 mock object, let's call it a. I did something like a(1, 2, 3).AndReturn(5)12:34
*** mriedem has quit IRC12:34
*** bkopilov_ has joined #openstack-nova12:34
gaoyuanhow can I see the value of 1,2,3 later in the code12:34
*** mriedem has joined #openstack-nova12:34
gaoyuanbauzas: using the a object12:34
bauzasgaoyuan: not sure I understand you12:35
bauzasgaoyuan: you wanna check the values?12:35
gaoyuanyes12:35
bauzasgaoyuan: you also know that if you write a new UT, we ask developers to use mock instead of mox ?12:35
gaoyuanbaouzas: I was debuging12:36
gaoyuanbaozas: I am working on remove-mox-pike12:36
bauzasgaoyuan: so, what do you want explicitely?12:36
bauzasdo you have some code to show up so I could understand more ?12:36
gaoyuanbauzas: wait a minute12:37
*** esberglu has quit IRC12:37
mriedemedleafe: that seems like a bug12:37
mriedembecause the object field isn't defined as nullable=True12:37
bauzasbecause writing 'a(1, 2, 3).AndReturn(5)' means that you functionnally expect the mocked object to accept those values as inputs and once called, being always returning 5 ?12:38
bauzas(without the question mark, my bad)12:38
mriedemedleafe: a bug in oslo.versionedobjects i mean12:38
edleafemriedem: I thought by default nullable=True12:39
mriedemno12:39
*** ltomasbo|away has quit IRC12:39
*** dmellado has quit IRC12:39
mriedemi would expect that behavior to be allowed only for DictOfNullableStringsField12:39
mriedemoh wait, now i'm confusing myself12:40
mriedemso DictOfNullableStringsField would allow {'test': None}12:40
*** MVenesio has joined #openstack-nova12:40
*** karimb has joined #openstack-nova12:40
gaoyuanbauzas: http://paste.openstack.org/show/614046/12:40
mriedemnullable defaults to False though12:40
gaoyuanbauzas: this is the code, I deleted something to make it clearer12:41
mriedemhow was i able to do this then? https://review.openstack.org/#/c/473627/8/nova/scheduler/filter_scheduler.py@19712:41
*** dmellado has joined #openstack-nova12:42
gibibauzas: ups sorry, here is the correct URL https://review.openstack.org/#/c/41973012:42
*** cNilesh has quit IRC12:42
gaoyuanbauzas: what I want, is to set a break point, in the actual code, to see the value of, let's say, connection_info12:42
*** cNilesh has joined #openstack-nova12:42
mriedemedleafe: ok removed my -112:43
mriedemedleafe: going to rebase https://review.openstack.org/#/c/473627/ ?12:43
bauzasedleafe: I have a shit ton of review backlog, any urgent things I should look at for placement?12:44
edleafemriedem: maybe because of https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L207 ?12:45
bauzaslike the above ?12:45
edleafebauzas: I would focus on Jay's series on allocation candidates12:45
bauzasedleafe: that's the obvious one12:45
edleafebauzas: he's changing how scheduler drivers work12:45
sfinucanmriedem: Assuming gibi hasn't already asked you, could you look at the discussion here https://review.openstack.org/#/c/448779/30/nova/notifications/objects/instance.py12:45
edleafebauzas: well, you missed that discussion on Tuesday12:46
mriedemedleafe: the object doesn't know anything about that12:46
bauzasedleafe: tl;dr then ?12:46
mriedemedleafe: plus sqla columns default to nullable=True12:46
edleafebauzas: drivers now return the entire list of HostState objects to the scheduler manager12:46
sfinucanwait - I just saw the "is in China" bit. At some point, in that case12:46
edleafebauzas: The claim is now done in the manager, not the driver12:47
mriedemsfinucan: yeah gibi mentioned it above12:47
bauzasedleafe: which claim ?12:47
bauzasedleafe: you mean consume_from_req() ?12:47
bauzasor did we merged something while I was in my cave?12:47
edleafebauzas: No, the posting of the allocation12:47
bauzasI should look at code then12:48
*** baoli has joined #openstack-nova12:48
edleafebauzas: that might help :)12:48
*** baoli has quit IRC12:48
*** masber has quit IRC12:48
*** baoli has joined #openstack-nova12:49
bauzasthat's where stupid paperwork rules about country-specific regulations ruins my productivity12:49
mriedemedleafe: rebase that extra specs change and i'll +2 before be12:50
mriedem*Bed12:50
*** mdrabe has joined #openstack-nova12:50
edleafemriedem: on it12:50
*** huanxie has quit IRC12:51
openstackgerritMoshe Levi proposed openstack/os-vif master: unplug_vf_passthrough: don't try to delete representor netdev  https://review.openstack.org/47882012:52
*** cNilesh has quit IRC12:53
bauzasgaoyuan: well, you can introspect connection_info whenever you want12:53
*** slaweq_ has joined #openstack-nova12:53
bauzasthat snippet you gave me just implies the assertion of what's passed to the function12:54
gaoyuanbauzas: maybe another time, I am chasing my bus12:56
bauzask12:56
*** prateek has joined #openstack-nova12:57
*** kylek3h has joined #openstack-nova12:57
*** ftersin has joined #openstack-nova12:58
*** slaweq_ has quit IRC12:58
moshelemriedem: hi12:58
moshelemriedem: can we release a new os-vif version? In os-vif I introduce hardware offload support for openvswitch https://review.openstack.org/#/c/460278/12:58
moshelemriedem: I need os-vif to be release so I can continue work on the nova side - https://review.openstack.org/#/c/398265/12:58
mriedemmoshele: sure12:59
mriedemmoshele: has anyone proposed a new release in the releases repo?12:59
*** eharney has joined #openstack-nova12:59
*** pchavva has joined #openstack-nova13:00
moshelemriedem: I am not familiar with the release procedure13:00
edleafemriedem: the merge conflict is not simple. So don't wait up just for that patch13:00
mriedemmoshele: ok i'll do it13:00
mriedemedleafe: i'll be up through the nova meeting13:00
mriedemso another 90 min at least13:00
mriedemi'm fing hardcore china style these days13:00
*** takashin_ has joined #openstack-nova13:01
* mriedem was actually asleep by 9:30 last night13:01
*** lucasxu has joined #openstack-nova13:01
moshelemriedem: thanks :)13:02
*** xyang1 has joined #openstack-nova13:02
*** takashin has quit IRC13:02
*** pchavva has quit IRC13:02
*** pchavva has joined #openstack-nova13:03
bauzasedleafe: I'm confused13:04
mriedemmoshele: https://review.openstack.org/47892313:04
bauzasedleafe: did we merged a POST /allocation call to placement somewhere in the scheduler code yet?13:04
bauzascan't find it in my git log13:04
bauzasor I'm blind13:04
*** coreywright has quit IRC13:05
bauzasI only see https://review.openstack.org/#/c/478291/ new which merged that night13:05
moshelemriedem: ok thanks13:05
mriedembauzas: no not merged yet13:05
mriedembauzas: current series starts here https://review.openstack.org/#/c/475448/13:05
bauzasmriedem: the below edleafe's quote confused me13:06
bauzas(14:47:00) edleafe: bauzas: The claim is now done in the manager, not the driver13:06
mriedemin the series13:06
edleafebauzas: AFAIK, the allocations is a PUT, if that helps13:06
mriedemas in, is now done in the series13:06
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add X-Openstack-Request-Id description  https://review.openstack.org/47484713:06
*** esberglu has joined #openstack-nova13:07
bauzaswell, I know I'm French, but when I use the "merge" verb, it has a very clear meaning to me :)13:07
bauzasanyway13:07
edleafebauzas: for background, the discussion moved from gerrit to IRC at around 14:20 here: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-06-27.log.html13:07
bauzasI loved those off days, but I hate how it hitted me13:07
* bauzas looking13:08
edleafeAnd to clarify, *in Jay's design*, the claim is now done in the manager13:08
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow  https://review.openstack.org/33028513:10
bauzasedleafe: just for my brain sanity, could we please use the word "proposed" instead of "done" if we speak of changes up ? :)13:12
bauzasemotional lift, y'know :p13:13
*** liverpooler has joined #openstack-nova13:13
*** sree has quit IRC13:13
gcbmriedem: Can you revisit https://review.openstack.org/#/c/466787.  it blocks  oslo.messaging stable/newton release13:13
*** sree has joined #openstack-nova13:14
*** eric_young has joined #openstack-nova13:15
*** eric_young has quit IRC13:15
*** artom has quit IRC13:16
*** sree has quit IRC13:17
*** liverpooler has quit IRC13:17
*** sree has joined #openstack-nova13:17
*** coreywright has joined #openstack-nova13:17
*** moshele has quit IRC13:18
*** ekuris has quit IRC13:19
*** liverpooler has joined #openstack-nova13:19
bauzasedleafe: so, FWIW, after reading all of this, including https://review.openstack.org/#/c/478291/ I honestly think it's totally understandable to return HostStates instead of freeform dicts13:21
bauzasthe interface was very poor within the scheduler code, so I don't think it hurts13:21
mriedemgcb: ok, i got really confused by the various backports for those changes, and they weren't consistent about the change id and some were abandoned, i.e. https://review.openstack.org/#/c/474456/13:21
bauzasthe only drawback I would have said would be about custom drivers, but mriedem already got the word13:22
bauzaswhich is that we no longer support custom drivers13:22
mriedembauzas: i already reported a docs bug abou that13:22
mriedemsince the docs tell you how to write your own scheduler driver13:22
*** lyan has joined #openstack-nova13:22
bauzasthat sounds reasonable13:22
bauzaswe could still opt-in custom drivers13:22
mriedembauzas: https://bugs.launchpad.net/nova/+bug/170110513:23
openstackLaunchpad bug 1701105 in OpenStack Compute (nova) "Customizing the OpenStack Compute (nova) Scheduler in Operations Guide" [Medium,Confirmed]13:23
bauzasafter all, it's just adding a choice to an opt13:23
mriedemif someone wants to hack a custom driver they are going to hack the config options, sure13:23
bauzasbut I strongly disapprove this13:23
mriedemwe just shouldn't advertise this as a built in capability that we support13:23
bauzasgiven it's merely not interoperable13:23
bauzashonestly, we did cut the possibility and nobody complainted13:24
bauzascomplained13:24
bauzasif someone comes with a good usecase about that, we could discuss13:24
bauzasbut for the moment, no need to open a debate like this13:24
*** salv-orlando has joined #openstack-nova13:24
bauzasand most of the operators wanting to run a specific scheduler can still use FilterScheduler and custom filters that call out to a 3rd party scheduler13:25
*** salv-orlando has quit IRC13:25
*** rmart04_ has joined #openstack-nova13:25
*** salv-orlando has joined #openstack-nova13:25
*** rmart04 has quit IRC13:26
*** rmart04_ is now known as rmart0413:26
openstackgerritMerged openstack/python-novaclient master: Cleanup duplicated methods  https://review.openstack.org/47649313:26
mriedemgcb: +W13:26
mriedembauzas: or they aren't upgraded to ocata yet13:27
mriedembut anyway13:27
bauzasmriedem: well, actually, that leaves me wonder one thing13:27
bauzasmriedem: in the past, you could just disable all filters but just run your custom one13:27
bauzasand then litterally do all nasty things you had in mind13:28
bauzaslike not counting against CPU or TAM13:28
bauzasRAM13:28
bauzasnow that we check that by placement, I do wonder if someone could come up and say they wouldn't like to count like this13:28
bauzaswell, actually, we did enforced the CPU and RAM verifications in the compute claims13:29
bauzasso, maybe it's just me overthinking13:29
bauzasanyway13:29
openstackgerritMerged openstack/python-novaclient master: Deprecate binary argument in nova service enable/disable/force-down CLIs  https://review.openstack.org/47721213:30
*** ckopper has quit IRC13:31
*** baoli has quit IRC13:32
*** karimb has quit IRC13:33
*** felipemonteiro has joined #openstack-nova13:33
*** P has joined #openstack-nova13:35
*** felipemonteiro_ has joined #openstack-nova13:35
*** P is now known as PP13:35
*** PP is now known as MMR13:35
*** MMR is now known as MR200013:35
MR2000Hi, Anybody there? I have a question about adding new subcommand to nova.13:36
*** karimb has joined #openstack-nova13:37
*** sree has quit IRC13:38
*** sree has joined #openstack-nova13:38
*** artom has joined #openstack-nova13:38
*** felipemonteiro has quit IRC13:38
*** rmart04 has quit IRC13:39
*** artom has quit IRC13:39
*** pchavva has quit IRC13:39
*** artom has joined #openstack-nova13:40
*** rmart04 has joined #openstack-nova13:41
*** ericyoung has quit IRC13:41
*** eric- has joined #openstack-nova13:42
mriedemalex_xu: i'm +2 on the tagged attach api change now https://review.openstack.org/#/c/376021/ - you were +2 before13:42
*** sree has quit IRC13:43
mriedemMR2000: it's ok to just ask13:43
artommriedem, is that our new coverage standard, "exxxxxxxtreme"? ;)13:44
artomAka - how tired are you right now?13:44
*** mtanino__ has joined #openstack-nova13:44
*** awaugama has joined #openstack-nova13:44
mriedemartom: only 2xtreme13:44
mriedemnot 7x13:45
artomOur Xs are like SLAs's 9s13:45
*** nolwenn has joined #openstack-nova13:46
*** eric- has quit IRC13:46
*** ericyoung has joined #openstack-nova13:47
*** zhurong has quit IRC13:49
*** ckopper has joined #openstack-nova13:49
*** pchavva has joined #openstack-nova13:50
sfinucanartom: mriedem likes his tests like he likes his pancake flipping13:51
*** rmart04 has quit IRC13:51
*** ujjain has quit IRC13:53
*** ujjain has joined #openstack-nova13:53
*** ujjain has quit IRC13:53
*** ujjain has joined #openstack-nova13:53
mriedemdansmith: why did we make instance.keypairs a list?13:53
mriedemam i missing something or can you only associate a single keypair with an instance when it's created13:54
*** slaweq_ has joined #openstack-nova13:54
dansmithmriedem: yeah, I think we just went future-looking13:54
dansmithI vaguely remember a conversation13:55
mriedemhmmm, ok because that's going to be codified in the versioned notifications https://review.openstack.org/#/c/419730/13:55
*** dimtruck is now known as zz_dimtruck13:55
dansmithis that a problem?13:55
mriedemthe future conan?13:55
mriedemno13:55
mriedemi was just comparing instance.key_name to keypairs13:55
mriedemsometimes i forget why we do what we do13:56
dansmithheh13:56
mriedemsfinucan: i don't get the pancake flipping thing13:57
mriedemyou should only flip a pancake once13:57
*** pchavva has quit IRC13:57
*** pchavva has joined #openstack-nova13:57
sfinucanmriedem: Pancake flipping was the most mundane thing I could think of at the time13:58
sfinucanBesides, only flipping once doesn't let you demonstrate your pancake flipping prowess to you non-flipping buddies13:58
*** slaweq_ has quit IRC13:59
*** rmart04 has joined #openstack-nova13:59
bauzasFWIW, creps are larger than pancakes hence harder to flip :p14:00
mriedemyou are going to have to work on some new analogies14:00
mriedemi like my tests like i like my mt dew14:00
mriedemexxxxxxxxxxxxxxxxxxxxtreme!14:00
mriedemthere14:00
mriedemthat's an analogy14:00
* sfinucan applauds analogy and hangs own head in shame14:01
*** hemanth has quit IRC14:02
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore  https://review.openstack.org/47890114:02
openstackgerritAlex Xu proposed openstack/nova master: Remove the test for the route '/resources.:(format)'  https://review.openstack.org/47890014:02
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore  https://review.openstack.org/47890314:02
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore  https://review.openstack.org/47890214:02
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore  https://review.openstack.org/47446314:03
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore  https://review.openstack.org/47849914:03
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore  https://review.openstack.org/47849814:03
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes instead of stevedore  https://review.openstack.org/47890514:03
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for volumes instead of stevedore  https://review.openstack.org/47850214:03
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore  https://review.openstack.org/47890414:03
*** tidwellr has joined #openstack-nova14:03
openstackgerritMerged openstack/nova master: _schedule_instances() supporting a RequestSpec object  https://review.openstack.org/46903714:04
openstackgerritClaudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM  https://review.openstack.org/47894314:06
*** zz_dimtruck is now known as dimtruck14:12
*** vks1 has joined #openstack-nova14:13
*** annegentle has joined #openstack-nova14:13
*** vks1 has quit IRC14:17
*** hongbin has joined #openstack-nova14:18
*** MR2000 has quit IRC14:19
*** vks1 has joined #openstack-nova14:19
*** iceyao has quit IRC14:19
*** acormier has joined #openstack-nova14:22
*** hongbin has quit IRC14:22
*** sean-k-mooney has joined #openstack-nova14:22
*** hongbin has joined #openstack-nova14:22
sean-k-mooneysfinucan: o/ can you take a look at https://review.openstack.org/#/c/467173/8 our requirement conflict witht he upstrem ones14:23
*** READ10 has joined #openstack-nova14:23
sean-k-mooneysfinucan: or rather we are missing some blacklisted package versions14:23
sfinucansean-k-mooney: Done and done14:23
sean-k-mooneysfinucan: cool thanks14:24
*** jaypipes has joined #openstack-nova14:24
*** baoli has joined #openstack-nova14:25
openstackgerritStephen Finucane proposed openstack/nova master: tests: Validate huge pages  https://review.openstack.org/39965314:25
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: create functional test base class  https://review.openstack.org/40705514:25
*** MR2000 has joined #openstack-nova14:28
*** baoli has quit IRC14:28
*** yangyape_ has joined #openstack-nova14:28
openstackgerritEd Leafe proposed openstack/nova master: Extract custom resource classes from flavors  https://review.openstack.org/47362714:29
*** rmart04 has quit IRC14:29
*** yangyapeng has quit IRC14:29
*** zhurong has joined #openstack-nova14:30
openstackgerritMerged openstack/os-vif master: Updated from global requirements  https://review.openstack.org/46717314:30
edleafemriedem: ^^14:31
*** ralonsoh has quit IRC14:32
mriedemoooo just in time14:32
*** rajathagasthya has joined #openstack-nova14:33
*** rajathagasthya has quit IRC14:34
*** vvargaszte has quit IRC14:34
mriedemwow that was a nasty diff14:35
edleafemriedem: yeah, the section I changed was moved to another file14:35
edleafemriedem: always fun!14:35
*** psachin has quit IRC14:36
*** psachin has joined #openstack-nova14:37
mriedemedleafe: dead code in the rebase14:39
*** baoli has joined #openstack-nova14:39
edleafemriedem: yeah?14:40
openstackgerritMatt Riedemann proposed openstack/nova master: Extract custom resource classes from flavors  https://review.openstack.org/47362714:40
*** zhurong has quit IRC14:40
*** lbragstad has quit IRC14:40
mriedemhttps://review.openstack.org/#/c/473627/10..11/nova/scheduler/utils.py14:40
edleafeugh14:40
edleafethanks for fixing that14:40
*** MR2000 has quit IRC14:41
edleafeodd that pep8 didn't catch that14:41
mriedem+214:41
mriedemnow it's time14:41
edleafesweet dreams14:42
mriedemo714:42
*** mriedem has quit IRC14:42
*** iceyao has joined #openstack-nova14:44
*** jmlowe has quit IRC14:46
*** vvargaszte has joined #openstack-nova14:46
*** ckopper has quit IRC14:46
sean-k-mooneyjaypipes: bauzas if ye get a chance can ye take a look at https://review.openstack.org/#/q/topic:enable-sriov-nic-features14:47
sean-k-mooneyjaypipes: bauzas https://review.openstack.org/#/c/449257/28 in partcalar has been stuck for about a month without any really feedback but that feature is more or less complete if we can get those last few pathces over the line14:48
*** iceyao has quit IRC14:48
bauzassean-k-mooney: sure, no promise but I'll try to look :)14:49
bauzassean-k-mooney: I was horribly off for a few days before so got a shit ton of things to do now :p14:49
openstackgerritMerged openstack/nova master: api-ref: Fix missing parameters in API Versions  https://review.openstack.org/47522614:50
sean-k-mooneybauzas: cool understood. life as a core gets hecktic espcially if your unwell for a few days. hope your feeling better14:50
openstackgerritMerged openstack/nova master: Stop using mox stubs in test_keypairs.py  https://review.openstack.org/47738414:50
*** lyan has quit IRC14:51
*** annegentle has quit IRC14:52
*** Swami has joined #openstack-nova14:53
*** tuanla_fujitsu has joined #openstack-nova14:53
*** tuanla__ has joined #openstack-nova14:54
*** slaweq_ has joined #openstack-nova14:55
*** annegentle has joined #openstack-nova14:56
*** lpetrut has quit IRC14:56
*** ckopper has joined #openstack-nova14:57
openstackgerritMerged openstack/nova master: Fix redundant BDM lookups during rebuild  https://review.openstack.org/47625814:57
*** slaweq_ has quit IRC15:00
*** jmlowe has joined #openstack-nova15:02
*** Oku_OS is now known as Oku_OS-away15:03
openstackgerritMerged openstack/nova master: Cleanup the plethora of libvirt live migration options  https://review.openstack.org/47183415:03
openstackgerritMerged openstack/nova master: Fix the note at the end of allocate_for_instance  https://review.openstack.org/47557815:04
*** mmehan has joined #openstack-nova15:06
*** mdrabe has quit IRC15:06
*** ckopper has quit IRC15:08
mmehanNeed a help from Numa experts, I have a unique sitatuation15:09
mmehanthe VM i want to spin is using two nics from different numa and has direct port15:10
mmehanthe VM when spinned gets  half cpus from Numa0 and other half from Numa115:10
sfinucanmmehan: It already gets cores from two NUMA nodes, or you want it to get them that way?15:10
sean-k-mooneymmehan: yes this is the default15:10
mmehanwhat the VM vendor want is even if the nic ports are across numa they want to give cpus and memory from one numa15:11
sfinucanmmehan: We can't do that...yet15:11
* sfinucan goes to find the spec that will make that possible15:11
*** takashin_ has left #openstack-nova15:12
sfinucanmmehan: You're going to need this https://review.openstack.org/#/c/361140/15:12
sfinucanbut it's not happening in Pike, so you'll need to wait for Queens at least15:12
jaypipessean-k-mooney: yup, will do today. today is review day while I sit on yet another 5 hour sales call.15:12
sean-k-mooneymmehan: currently it is expcitly not supported because that intorduce alot of qpi overhead and reduce the performace of the plathform as a whole. the spec sfinucan linked make it configurable15:12
sfinucan^ what he said :)15:13
mmehanis yes i understand QPI overhead, but thats needed to keep platform uniform as of now15:14
mmehanthanks for spec, i will go over it15:14
*** lbragstad has joined #openstack-nova15:14
*** ckopper has joined #openstack-nova15:14
sfinucanmmehan: I'm pretty sure there are patches up there that you could test against too15:15
dansmithjaypipes: edleafe are you both good with this now such that I should review? https://review.openstack.org/#/c/475448/15:17
dansmithbauzas: are you actively reviewing that one?15:17
jaypipesdansmith: I am, yes. obviously I can't vote +2 on it :)15:17
dansmithjaypipes: yep15:17
jaypipesthanks much to ed for fixing my style.15:17
mmehansfinucan: I will try to find the patches, if you have any insight please share15:18
cfriesenmmehan: I think this is the main patch: https://review.openstack.org/#/c/390520/15:18
sfinucanhttps://review.openstack.org/#/q/topic:bp/share-pci-between-numa-nodes15:18
*** MR2000 has joined #openstack-nova15:18
*** jistr is now known as jistr|afk15:20
strigaziHi cores, can you have a look at https://review.openstack.org/#/c/476122 ?15:20
edleafedansmith: was just giving it one last scan before re-applying my +115:22
*** yamahata has joined #openstack-nova15:26
*** yamahata has quit IRC15:27
*** yamahata has joined #openstack-nova15:27
*** rcernin has quit IRC15:28
sfinucancfriesen: If I'm running a RT guest, I need an RT host kernel, right?15:30
* sfinucan is reading through the "realtime kvm cpu affinities" mail at last15:30
*** slaweq_ has joined #openstack-nova15:32
*** kaisers_ has quit IRC15:33
*** kaisers_ has joined #openstack-nova15:33
bauzasdansmith: yup15:34
*** mdrabe has joined #openstack-nova15:34
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895515:34
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896415:36
cfriesensfinucan: well, technically no but your RT response won't be as good.15:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492615:36
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004215:37
cfriesensfinucan: so if you actually care about RT then yes.15:37
*** gyee has joined #openstack-nova15:38
sfinucancfriesen: Fair enough :) I'm trying to summarize this and will reply to the mail once I do15:38
*** slaweq_ has quit IRC15:38
*** mkucia has quit IRC15:39
*** Swami has quit IRC15:39
*** andreas_s has quit IRC15:40
sfinucancfriesen: So what I'm getting out of this is that 'emulator_threads_policy' is too resource intensive, and we should be instead allocating "overhead threads" on one or two host cores15:40
sfinucanIs that right, or do I need to delve deeper?15:41
*** yamamoto has quit IRC15:41
sfinucan /reread the thread15:41
cfriesensfinucan: lol15:41
cfriesensfinucan: basically that's the opinion of some people--that dedicated a whole separate pCPU to the emulator threads consumes too many pCPUs15:42
cfriesens/dedicated/dedicating15:42
*** ckopper has quit IRC15:43
sfinucanGrr. I said that exact thing during reviews of that spec, but didn't put it succinctly enough15:43
cfriesensfinucan: I'm not entirely convinced that sharing emulator threads on a single pCPU is *guaranteeed* to be safe, like if one instance starts seriously hammering on I/O or does live migration or something15:43
sfinucanI think I suggested using 'vcpu_pin_set', but another configuration option (a mask, from the looks of things?) is much better15:44
sfinucancfriesen: I've just reached that mail now, heh15:44
*** chyka has joined #openstack-nova15:47
cfriesensfinucan: hence my overcommit ratio suggestion, then operators could tune the number of RT instances per "emulator pCPU" to whatever they feel comfortable with..setting it to 1 gives you the behaviour of "emulator_threads_policy=isolate"15:47
*** ircuser-1 has quit IRC15:51
*** ckopper has joined #openstack-nova15:51
*** chyka_ has joined #openstack-nova15:51
*** damien_r has quit IRC15:53
*** chyka has quit IRC15:55
*** hshiina has joined #openstack-nova15:56
*** lyan has joined #openstack-nova15:58
*** sambetts is now known as sambetts|afk16:00
*** Apoorva has joined #openstack-nova16:00
*** markus_z has quit IRC16:01
*** jistr|afk is now known as jistr16:07
*** prateek_ has joined #openstack-nova16:08
openstackgerritMerged openstack/nova master: XenAPI: device tagging  https://review.openstack.org/33378116:08
efriedsfinucan Found bugs in https://review.openstack.org/#/c/476159/ :)16:08
*** cNilesh has joined #openstack-nova16:08
*** prateek has quit IRC16:09
*** lyan has quit IRC16:10
*** salv-orl_ has joined #openstack-nova16:14
*** salv-orlando has quit IRC16:17
*** kaisers_ has quit IRC16:17
openstackgerritDan Smith proposed openstack/nova master: Sanitize instance in InstanceMetadata to avoid un-pickleable context  https://review.openstack.org/47899116:17
*** kaisers_ has joined #openstack-nova16:18
*** ckopper has quit IRC16:20
*** ckopper has joined #openstack-nova16:21
*** dixiaoli has quit IRC16:22
*** ckopper has quit IRC16:25
sfinucanefried: How dare you16:25
sfinucanI'll take a look shortly :)16:25
* sfinucan still has to figure out that auto-sizing16:26
efriedsfinucan I didn't touch that one.16:27
efriedsfinucan BTW, I don't think your monkeypatch will play well with my explicit newlines and box sizes, except by coincidence.16:27
sfinucanI was planning to drop the newlines and most of the box sizes, tbh16:28
*** mvk has quit IRC16:28
sfinucanthe only box size I planned to keep was the one for the box that's currently truncated16:28
efriedsfinucan As you wish; though the squished-in bottoms make it look like something's truncated (even when it's not).16:29
*** karimb has quit IRC16:29
efriedAnd I think there might be two long boxes now.16:29
sfinucanefried: good call. Maybe I should keep all the sizes so and just strip the newlines?16:29
sfinucanGiven that they're not necessary16:29
efriedsfinucan Either way, unless you figure out how to autosize the boxes, it'll need to be trial and error to get it rendering right.16:30
sfinucanYup. Hopefully we can figure something out instead :(16:30
*** psachin has quit IRC16:33
*** pewp has quit IRC16:35
*** ckopper has joined #openstack-nova16:35
*** prateek has joined #openstack-nova16:38
*** cNilesh has quit IRC16:40
*** prateek_ has quit IRC16:40
*** yamamoto has joined #openstack-nova16:41
*** pewp has joined #openstack-nova16:44
openstackgerritMerged openstack/nova master: Add keypairs field to InstanceCreatePayload  https://review.openstack.org/41973016:44
openstackgerritMerged openstack/nova master: Add key_name field to InstancePayload  https://review.openstack.org/46300216:45
*** ckopper has joined #openstack-nova16:48
*** pcaruana has quit IRC16:48
*** jdwidari has quit IRC16:49
*** dtantsur is now known as dtantsur|afk16:50
*** derekh has quit IRC16:50
*** yamamoto has quit IRC16:53
*** artom has quit IRC16:56
*** artom has joined #openstack-nova16:57
*** artom has quit IRC16:57
*** tesseract has quit IRC16:58
*** artom has joined #openstack-nova16:58
*** artom has quit IRC16:58
*** baoli has quit IRC16:59
*** artom has joined #openstack-nova16:59
*** baoli has joined #openstack-nova17:00
*** artom has quit IRC17:00
*** jpena is now known as jpena|off17:00
*** artom has joined #openstack-nova17:00
*** artom has quit IRC17:02
*** artom has joined #openstack-nova17:03
dansmithartom: why in the metadata test did you subclass FakeDeviceMetadata and FakeDeviceBus ?17:03
dansmiththose aren't registered objects, which is a problem for deserializing an Instance, but it also doesn't seem to matter if I use the real devices17:04
dansmither real objects17:04
*** trinaths has joined #openstack-nova17:05
*** iceyao has joined #openstack-nova17:06
*** MR2000 has quit IRC17:07
*** artom has quit IRC17:09
jaypipessean-k-mooney: https://review.openstack.org/#/c/449257/ needs rebasing. can you handle? not sure where ralonso is today.17:10
*** iceyao has quit IRC17:10
sean-k-mooneyjaypipes: he went home about an hour ago17:11
jaypipessean-k-mooney: can you rebase or perhaps he can do in morning?17:11
*** hshiina has quit IRC17:11
sean-k-mooneyill ask him to do it in the morning17:11
sean-k-mooneyi can take a look at rebaseing too17:11
*** MR2000 has joined #openstack-nova17:11
sean-k-mooneyjaypipes: other then the merge conflict does it look ok?17:12
jaypipessean-k-mooney: mostly17:12
jaypipessean-k-mooney: it's just the bottom of the series so needs to be rebased in order to make progress on the series.17:13
*** trinaths has quit IRC17:16
*** vks1 has quit IRC17:16
*** annegentle has quit IRC17:16
*** rajathagasthya has joined #openstack-nova17:22
*** tuanla_fujitsu has quit IRC17:24
*** tuanla__ has quit IRC17:24
openstackgerritDan Smith proposed openstack/nova master: Sanitize instance in InstanceMetadata to avoid un-pickleable context  https://review.openstack.org/47899117:26
*** MR2000 has quit IRC17:29
jaypipessean-k-mooney: I found a number of issues with that patch. please see my comments.17:31
*** kaisers__ has joined #openstack-nova17:32
*** annegentle has joined #openstack-nova17:32
*** mvk has joined #openstack-nova17:33
sean-k-mooneyjaypipes: well thats why we have review :) ill ask rodolfo to adres them in the morning17:33
sean-k-mooneyjaypipes: thanks for taking a look17:33
*** kaisers_ has quit IRC17:34
*** nic has joined #openstack-nova17:36
*** annegentle has quit IRC17:36
*** baoli has quit IRC17:37
melwittI have a patch up to fix an intermittent functional test failure with placement PUT /allocations if anyone could please review https://review.openstack.org/#/c/478683/17:38
*** baoli has joined #openstack-nova17:38
*** annegentle has joined #openstack-nova17:48
*** yamahata has quit IRC17:48
*** prateek has quit IRC17:50
*** iceyao has joined #openstack-nova17:50
*** annegentle has quit IRC17:53
*** annegentle has joined #openstack-nova17:54
*** iceyao has quit IRC17:55
*** gjayavelu has joined #openstack-nova17:56
edleafemelwitt: looks good17:57
melwittedleafe: thanks for the review!17:58
*** salv-orl_ has quit IRC18:00
*** salv-orlando has joined #openstack-nova18:00
*** eharney has quit IRC18:03
*** eharney has joined #openstack-nova18:03
*** salv-orlando has quit IRC18:05
*** salv-orlando has joined #openstack-nova18:14
*** Apoorva_ has joined #openstack-nova18:18
*** lpetrut has joined #openstack-nova18:19
*** jmlowe has quit IRC18:19
*** ujjain has quit IRC18:21
dansmithmelwitt: I can split that test fix out if you want, but it's even more obscure without the change to the code I think18:21
*** Apoorva has quit IRC18:22
melwittdansmith: no, it's not that. fixing the test is cool, it's just I was looking for how is anything testing the sanitization18:23
openstackgerritSean Dague proposed openstack/nova master: remove the very old unmaintained wsgi scripts  https://review.openstack.org/47902718:23
dansmithoh I see18:23
sdaguedansmith: the devstack bits just merged, and it's running by default, so we can delete those files now18:23
*** yamahata has joined #openstack-nova18:23
melwittit would be ideal if we have a test that would catch it if we regress again in the future18:25
*** mtanino__ has quit IRC18:25
dansmithmelwitt: yeah, I can't easily reproduce the actual thing I think, but I can make sure we've replaced the instance18:26
openstackgerritDan Smith proposed openstack/nova master: Sanitize instance in InstanceMetadata to avoid un-pickleable context  https://review.openstack.org/47899118:26
dansmithmelwitt: thusly ^18:26
dansmithsdague: ack18:26
melwittdansmith: I think that's fine. just something that would fail with the old code and pass with the new18:26
dansmithgah, I reset the reproducer again18:28
*** moshele has joined #openstack-nova18:30
*** iceyao has joined #openstack-nova18:30
*** ujjain has joined #openstack-nova18:31
*** ujjain has quit IRC18:31
*** ujjain has joined #openstack-nova18:31
openstackgerritsean mooney proposed openstack/os-vif master: set mtu on all code paths  https://review.openstack.org/46995118:32
*** annegentle has quit IRC18:33
*** annegentle has joined #openstack-nova18:34
*** iceyao has quit IRC18:35
*** moshele has quit IRC18:41
*** salv-orlando has quit IRC18:47
*** salv-orlando has joined #openstack-nova18:48
*** pchavva has quit IRC18:49
*** ckopper has quit IRC18:51
*** xyang1 has quit IRC18:52
*** sbezverk has joined #openstack-nova18:52
*** tidwellr has quit IRC18:53
*** salv-orlando has quit IRC18:53
*** xyang1 has joined #openstack-nova18:54
*** adisky__ has quit IRC18:54
*** jmlowe_ has joined #openstack-nova18:55
*** Apoorva_ has quit IRC19:02
*** Apoorva has joined #openstack-nova19:03
*** jamesden_ has joined #openstack-nova19:04
*** jamesdenton has quit IRC19:05
*** acormier has quit IRC19:06
*** thorst has quit IRC19:06
*** acormier has joined #openstack-nova19:06
*** thorst has joined #openstack-nova19:08
*** acormier has quit IRC19:11
*** thorst has quit IRC19:13
*** iceyao has joined #openstack-nova19:13
*** iceyao has quit IRC19:15
*** iceyao has joined #openstack-nova19:15
openstackgerritJohn Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow  https://review.openstack.org/33028519:15
*** sbezverk has quit IRC19:16
*** iceyao has quit IRC19:20
*** dimak has quit IRC19:21
*** dimak has joined #openstack-nova19:22
*** thorst has joined #openstack-nova19:23
*** sbezverk has joined #openstack-nova19:23
openstackgerritmelanie witt proposed openstack/nova master: Request zero root disk for boot-from-volume instances  https://review.openstack.org/42848119:24
openstackgerritmelanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances  https://review.openstack.org/42850519:24
*** pchavva has joined #openstack-nova19:28
*** jmlowe_ has quit IRC19:28
*** salv-orlando has joined #openstack-nova19:34
*** jmlowe has joined #openstack-nova19:40
openstackgerritmelanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota  https://review.openstack.org/47770019:41
openstackgerritmelanie witt proposed openstack/nova master: Add scatter gather utilities for cells  https://review.openstack.org/47595719:41
openstackgerritmelanie witt proposed openstack/nova master: Remove 'reserved' count from used limits  https://review.openstack.org/44624219:41
openstackgerritmelanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota  https://review.openstack.org/47769919:41
openstackgerritmelanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage  https://review.openstack.org/44624319:41
openstackgerritmelanie witt proposed openstack/nova master: Count instances to check quota  https://review.openstack.org/41652119:41
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094519:41
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094619:41
*** iceyao has joined #openstack-nova19:42
*** salv-orlando has quit IRC19:44
*** salv-orlando has joined #openstack-nova19:44
*** jmlowe_ has joined #openstack-nova19:47
*** iceyao has quit IRC19:47
*** jmlowe has quit IRC19:49
*** melwitt has left #openstack-nova19:50
*** melwitt has joined #openstack-nova19:50
*** eharney has quit IRC19:56
*** sbezverk has quit IRC19:57
*** annegentle has quit IRC19:59
*** iceyao has joined #openstack-nova20:00
*** iceyao has quit IRC20:05
*** MVenesio has quit IRC20:08
*** eandersson has quit IRC20:10
*** eharney has joined #openstack-nova20:11
dansmithjaypipes: sdague: we broke metadata server a bit ago, this fix has been validated by the osa people, and just needs a +W: https://review.openstack.org/#/c/478991/20:13
*** sbezverk has joined #openstack-nova20:15
jaypipesdansmith: done20:16
dansmithjaypipes: thanks20:16
sdaguedansmith: double +A20:19
dansmithwoot20:19
sdagueso... why didn't that trip us up in integration tests?20:19
dansmithI get an embroidered blazer for that right?20:19
sdaguewe just don't touch the right bits?20:19
dansmithsdague: because we don't actually pickle I imagine20:19
*** dansmith has left #openstack-nova20:19
*** dansmith has joined #openstack-nova20:20
dansmithwtf20:20
dansmithsdague: sorry, integration tests.. because we're not configured to use memcache I imagine20:20
*** iceyao has joined #openstack-nova20:22
*** lpetrut has quit IRC20:25
*** liverpooler has quit IRC20:25
sdaguedansmith: ah, yeh, that definitely could be20:26
sdagueok, at least that's an answer :)20:26
*** iceyao has quit IRC20:27
*** eharney has quit IRC20:29
*** gszasz has quit IRC20:32
*** awaugama has quit IRC20:32
*** karimb has joined #openstack-nova20:34
*** iceyao has joined #openstack-nova20:42
*** lucasxu has quit IRC20:44
*** sbezverk has quit IRC20:44
*** hamzy has quit IRC20:44
*** sbezverk has joined #openstack-nova20:45
*** iceyao has quit IRC20:47
melwittsdague: fix for intermittent functional job fails in the gate (such as http://logs.openstack.org/91/478991/3/gate/gate-nova-tox-functional-py35-ubuntu-xenial/34d3264/testr_results.html.gz) needs a +W https://review.openstack.org/#/c/478683/20:53
sdaguemelwitt: nice fix20:54
sdague+W20:54
melwittsdague: thanks!20:55
*** dimtruck is now known as zz_dimtruck20:55
*** pjm6 has quit IRC20:55
openstackgerritEd Leafe proposed openstack/nova master: Add uuid to scheduler's HostState object  https://review.openstack.org/47663020:56
openstackgerritEd Leafe proposed openstack/nova master: placement: support GET /allocation_candidates  https://review.openstack.org/47544820:56
*** yamahata has quit IRC20:56
*** yamahata has joined #openstack-nova20:57
*** jamesden_ has quit IRC21:01
*** baoli has quit IRC21:02
*** jmlowe has joined #openstack-nova21:02
*** iceyao has joined #openstack-nova21:03
*** jamesdenton has joined #openstack-nova21:03
*** jmlowe_ has quit IRC21:03
*** iceyao has quit IRC21:07
*** smatzek has quit IRC21:09
*** pchavva has quit IRC21:12
*** acormier has joined #openstack-nova21:14
*** esberglu has quit IRC21:17
*** zz_dimtruck is now known as dimtruck21:23
*** pchavva has joined #openstack-nova21:25
*** jmlowe has quit IRC21:27
*** acormier has quit IRC21:28
*** acormier has joined #openstack-nova21:29
*** sbezverk has quit IRC21:31
*** READ10 has quit IRC21:32
*** sbezverk has joined #openstack-nova21:32
*** Swami has joined #openstack-nova21:34
*** kylek3h has quit IRC21:35
*** yassine has quit IRC21:38
*** eharney has joined #openstack-nova21:42
*** felipemonteiro_ has quit IRC21:52
*** takashin has joined #openstack-nova21:53
*** burt has quit IRC21:59
*** thorst has quit IRC22:01
*** esberglu has joined #openstack-nova22:02
openstackgerritTakashi NATSUME proposed openstack/nova master: Use PCIAddressField in oslo.versionedobjects  https://review.openstack.org/46620122:02
*** iceyao has joined #openstack-nova22:05
*** mdrabe has quit IRC22:05
*** esberglu has quit IRC22:06
openstackgerritMerged openstack/nova master: Handle version for PUT and POST in PlacementFixture  https://review.openstack.org/47868322:08
*** iceyao has quit IRC22:09
*** annegentle has joined #openstack-nova22:10
*** pchavva has quit IRC22:13
*** salv-orl_ has joined #openstack-nova22:14
*** salv-orlando has quit IRC22:18
*** acormier has quit IRC22:19
*** xyang1 has quit IRC22:20
*** annegentle has quit IRC22:22
*** gjayavelu has quit IRC22:29
*** jmlowe has joined #openstack-nova22:31
*** salv-orl_ has quit IRC22:41
*** nolwenn has quit IRC22:46
*** efried is now known as efried_out_til_J22:48
*** efried_out_til_J is now known as efried_back_July22:49
*** efried_back_July is now known as efried_backJuly522:49
* edleafe will be back July 5 also22:49
*** timello has quit IRC22:55
*** jamielennox has quit IRC22:57
*** jamielennox has joined #openstack-nova23:03
*** iceyao has joined #openstack-nova23:08
openstackgerritTakashi NATSUME proposed openstack/nova master: Add 'updated_at' field to InstancePayload in notifications  https://review.openstack.org/47527623:10
*** mmehan has quit IRC23:11
*** iceyao has quit IRC23:13
*** gjayavelu has joined #openstack-nova23:17
*** johnthetubaguy has quit IRC23:18
*** hongbin has quit IRC23:19
*** jaypipes has quit IRC23:20
*** Swami has quit IRC23:21
*** tojuvone has quit IRC23:23
openstackgerritJohn Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow  https://review.openstack.org/33028523:25
*** mtanino_ has joined #openstack-nova23:25
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060823:26
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948323:27
*** johnthetubaguy has joined #openstack-nova23:28
*** chyka_ has quit IRC23:28
*** chyka has joined #openstack-nova23:29
*** iceyao has joined #openstack-nova23:30
*** thorst has joined #openstack-nova23:32
*** chyka has quit IRC23:34
*** iceyao has quit IRC23:34
*** thorst has quit IRC23:37
*** rajathagasthya has quit IRC23:39
*** mvensky has joined #openstack-nova23:44
*** baoli has joined #openstack-nova23:48
*** aries has quit IRC23:50
*** baoli has quit IRC23:53
*** mvensky has quit IRC23:54
*** Apoorva_ has joined #openstack-nova23:58

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