Wednesday, 2017-07-05

openstackgerritMathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432200:05
*** acormier has quit IRC00:07
*** gmann has quit IRC00:09
*** gmann has joined #openstack-nova00:10
*** chyka has joined #openstack-nova00:14
*** takashin has joined #openstack-nova00:15
*** gcb has joined #openstack-nova00:15
*** edmondsw has joined #openstack-nova00:16
*** iceyao has joined #openstack-nova00:20
*** edmondsw has quit IRC00:20
*** iceyao has quit IRC00:25
*** thorst has joined #openstack-nova00:27
*** thorst has quit IRC00:27
*** arahal_ has quit IRC00:34
*** claudiub has quit IRC00:37
Jeffrey4lhi guys, is there any consumer for nova versioned notification messages?00:39
*** iceyao has joined #openstack-nova00:41
*** iceyao has quit IRC00:46
*** winston-d_ has joined #openstack-nova00:52
*** iceyao has joined #openstack-nova01:02
*** trungnv has quit IRC01:03
*** iceyao has quit IRC01:06
*** litao__ has joined #openstack-nova01:13
*** liujiong has joined #openstack-nova01:14
*** moshele has quit IRC01:17
*** iceyao has joined #openstack-nova01:22
*** Shunli has joined #openstack-nova01:24
*** tiendc has joined #openstack-nova01:27
openstackgerritMerged openstack/nova master: Don't delete neutron port when attach failed  https://review.openstack.org/40356801:29
Kevin_ZhengJeffrey4l: Searchlight is going to consume those01:35
*** markvoelker has joined #openstack-nova01:36
*** tiendc_ has joined #openstack-nova01:37
Jeffrey4lKevin_Zheng, is searchlight already finished the work?01:37
Kevin_Zhengvery close01:37
Jeffrey4land is there any other project ( ceilometer?) can consume versioned message?01:37
*** tiendc has quit IRC01:38
Kevin_Zhenga good idea, bu no one worked on it in ceilometer01:39
*** abalutoiu has quit IRC01:52
*** bkopilov has quit IRC01:59
*** iceyao has quit IRC02:02
*** iceyao has joined #openstack-nova02:03
*** edmondsw has joined #openstack-nova02:04
*** takashin has quit IRC02:05
*** chyka has quit IRC02:08
*** markvoelker has quit IRC02:08
*** edmondsw has quit IRC02:09
*** zhurong has joined #openstack-nova02:12
*** lbragstad has joined #openstack-nova02:17
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification  https://review.openstack.org/45949302:27
*** thorst has joined #openstack-nova02:28
*** yamamoto has joined #openstack-nova02:33
*** thorst has quit IRC02:33
*** tuanluong has joined #openstack-nova02:33
*** gongysh has joined #openstack-nova02:33
Nel1xwhat consecuences using admin_auth_url => port 5000? and not 35537 in nova => [neutron]02:34
*** dixiaoli has joined #openstack-nova02:52
*** hongbin has joined #openstack-nova03:00
*** takashin has joined #openstack-nova03:03
*** markvoelker has joined #openstack-nova03:06
openstackgerritZhenyu Zheng proposed openstack/nova master: copy image id to image metadata from volume  https://review.openstack.org/31990503:08
*** bkopilov has joined #openstack-nova03:13
*** amotoki is now known as amotoki_away03:17
*** takashin has quit IRC03:33
*** sridharg has joined #openstack-nova03:34
*** markvoelker has quit IRC03:39
*** mdnadeem has joined #openstack-nova03:40
*** links has joined #openstack-nova03:44
*** adisky__ has joined #openstack-nova03:46
*** trinaths has joined #openstack-nova03:50
*** iceyao has quit IRC03:53
*** gongysh has quit IRC03:54
*** lbragstad has quit IRC03:55
*** thorst has joined #openstack-nova03:58
*** sree has joined #openstack-nova04:02
*** thorst has quit IRC04:03
*** cNilesh has joined #openstack-nova04:06
*** lpetrut has joined #openstack-nova04:09
*** sapcc-bot1 has quit IRC04:11
*** sapcc-bot has joined #openstack-nova04:11
*** lpetrut has quit IRC04:12
*** tojuvone has quit IRC04:14
*** iceyao has joined #openstack-nova04:15
*** baoli has quit IRC04:16
*** amotoki_away is now known as amotoki04:17
*** Nel1x has quit IRC04:18
*** _pewp_ has quit IRC04:18
*** lpetrut has joined #openstack-nova04:19
*** _pewp_ has joined #openstack-nova04:19
*** iceyao has quit IRC04:20
*** Nel1x has joined #openstack-nova04:23
*** hongbin has quit IRC04:25
*** gongysh has joined #openstack-nova04:25
*** tojuvone has joined #openstack-nova04:26
*** amotoki is now known as amotoki_away04:28
*** psachin has joined #openstack-nova04:32
*** iceyao has joined #openstack-nova04:33
*** hshiina has joined #openstack-nova04:34
*** claudiub has joined #openstack-nova04:35
*** markvoelker has joined #openstack-nova04:36
*** iceyao has quit IRC04:38
*** iceyao has joined #openstack-nova04:42
*** mtreinish has quit IRC04:44
*** yangyapeng has joined #openstack-nova04:50
*** mtreinish has joined #openstack-nova04:51
*** yangyapeng has quit IRC04:57
*** yangyapeng has joined #openstack-nova04:58
*** dixiaoli has quit IRC05:00
*** lpetrut has quit IRC05:02
*** sree has quit IRC05:03
*** sree has joined #openstack-nova05:03
*** yangyapeng has quit IRC05:05
*** yangyapeng has joined #openstack-nova05:06
*** yangyape_ has joined #openstack-nova05:07
*** sree has quit IRC05:07
*** prateek has joined #openstack-nova05:07
*** sree has joined #openstack-nova05:08
*** markvoelker has quit IRC05:08
*** ratailor has joined #openstack-nova05:08
*** yangyapeng has quit IRC05:11
*** moshele has joined #openstack-nova05:18
*** yangyape_ has quit IRC05:20
*** yangyapeng has joined #openstack-nova05:21
*** lpetrut has joined #openstack-nova05:25
*** dixiaoli has joined #openstack-nova05:27
*** takashin has joined #openstack-nova05:29
*** iceyao has quit IRC05:39
*** edmondsw has joined #openstack-nova05:40
*** yangyapeng has quit IRC05:40
*** yangyapeng has joined #openstack-nova05:41
*** Nel1x has quit IRC05:41
*** moshele has quit IRC05:45
*** edmondsw has quit IRC05:45
*** udesale has joined #openstack-nova05:46
*** Oku_OS-away is now known as Oku_OS05:56
*** iceyao has joined #openstack-nova05:58
*** xiefp has joined #openstack-nova05:58
*** thorst has joined #openstack-nova05:59
*** iceyao has quit IRC06:02
*** thorst has quit IRC06:05
*** markvoelker has joined #openstack-nova06:05
*** sshwarts has joined #openstack-nova06:06
*** iceyao has joined #openstack-nova06:09
*** armax has joined #openstack-nova06:12
*** iceyao has quit IRC06:13
*** amotoki_away is now known as amotoki06:20
*** carthaca_ has joined #openstack-nova06:22
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959306:23
*** karthiks_afk is now known as karthiks06:23
*** lpetrut has quit IRC06:27
*** amotoki is now known as amotoki_away06:30
masberhi, is this the right place to ask about the integration of nova with sr-iov?06:35
*** iceyao has joined #openstack-nova06:36
*** belmoreira has joined #openstack-nova06:36
*** markvoelker has quit IRC06:39
*** iceyao has quit IRC06:41
*** markus_z has joined #openstack-nova06:42
*** moshele has joined #openstack-nova06:45
*** ratailor has quit IRC06:46
*** amotoki_away is now known as amotoki06:46
*** ratailor has joined #openstack-nova06:46
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix arguments in calling _delete_nic_metadata  https://review.openstack.org/48036906:46
*** udesale__ has joined #openstack-nova06:49
*** udesale has quit IRC06:49
*** lpetrut has joined #openstack-nova06:50
*** diga has joined #openstack-nova06:53
*** vvargaszte has quit IRC07:00
*** sree has quit IRC07:02
*** tesseract has joined #openstack-nova07:02
*** sree has joined #openstack-nova07:03
*** sree has quit IRC07:08
bauzasalex_xu: stephenfin: follow-up on https://bugs.launchpad.net/nova/+bug/1702304 with https://review.openstack.org/#/c/480198/ needing reviews07:12
openstackLaunchpad bug 1702304 in OpenStack Compute (nova) "MISC_SHARES_VIA_AGGREGATE not found still show up in the functional test" [Critical,In progress] - Assigned to Sylvain Bauza (sylvain-bauza)07:12
alex_xubauzas: cool, will check soon07:12
*** jpena|off is now known as jpena07:12
openstackgerritAlex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP  https://review.openstack.org/48037907:13
alex_xubauzas: ^ another bug fix for placement07:13
bauzasalex_xu: ack, looking07:13
gibiJeffrey4l: besides searchlight I think watcher also consumes nova versioned notifications07:14
*** gouthamr has joined #openstack-nova07:15
gibiJeffrey4l: for example https://github.com/openstack/watcher/blob/3a3ce0268bbb900c0ca3fe40379beda574a4d548/watcher/decision_engine/model/notification/nova.py07:15
*** lpetrut has quit IRC07:16
*** rcernin has joined #openstack-nova07:19
*** alexchadin has joined #openstack-nova07:20
*** lpetrut has joined #openstack-nova07:23
*** edmondsw has joined #openstack-nova07:29
*** cNilesh has quit IRC07:29
*** lpetrut has quit IRC07:32
*** edmondsw has quit IRC07:33
Jeffrey4lgibi, roger. thanks a lot.07:35
*** vvargaszte has joined #openstack-nova07:36
*** markvoelker has joined #openstack-nova07:36
*** oanson has quit IRC07:38
*** oanson has joined #openstack-nova07:38
*** damien_r has quit IRC07:42
*** damien_r has joined #openstack-nova07:43
*** ralonsoh has joined #openstack-nova07:43
*** slaweq has quit IRC07:43
*** slaweq has joined #openstack-nova07:44
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach  https://review.openstack.org/48039207:47
*** slaweq has quit IRC07:48
*** andreas_s has joined #openstack-nova07:49
openstackgerritChason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme  https://review.openstack.org/47775707:54
*** yamahata has quit IRC07:55
*** zzzeek has quit IRC08:00
*** takashin has left #openstack-nova08:00
*** vvargaszte1 has joined #openstack-nova08:01
*** thorst has joined #openstack-nova08:01
*** zzzeek has joined #openstack-nova08:01
*** vvargaszte has quit IRC08:03
*** vvargaszte1 is now known as vvargaszte08:03
*** thorst has quit IRC08:05
*** markvoelker has quit IRC08:08
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI  https://review.openstack.org/47843708:13
*** xiefp has quit IRC08:14
*** mpaolino has joined #openstack-nova08:15
stephenfinbauzas: https://review.openstack.org/#/c/480198 reviewed. Moving onto the next one now08:17
stephenfinwait - the next one's the same :)08:17
bauzaswhat's the next one ? :p08:17
stephenfinyou pinged me last night to review https://review.openstack.org/#/c/480198, then again this morning08:18
* stephenfin thought it was pings for different patches :)08:18
bauzasha-ah08:18
*** chenying has quit IRC08:19
*** chenying has joined #openstack-nova08:19
stephenfinDamn, this place is quiet without ze yanks around - I can actually read the scrollback on one page :O08:19
gibibauzas: hi! there is a doc generation improvement that only needs a second +2. If you have time could you take a look? https://review.openstack.org/#/c/475349/08:24
bauzasgibi: sure08:24
gibibauzas: thanks!08:25
*** slaweq has joined #openstack-nova08:26
bauzasgibi: ack my comment and I +W https://review.openstack.org/#/c/475349/108:27
bauzasbased on takashi's valid point08:28
*** efoley has joined #openstack-nova08:28
*** diga has quit IRC08:31
*** lucas-afk is now known as lucasagomes08:32
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI  https://review.openstack.org/47843708:35
*** slaweq has quit IRC08:35
stephenfinbauzas: I don't think you +W'd that yet, if you plan to?08:35
*** slaweq has joined #openstack-nova08:35
bauzasstephenfin: because I'd like to see gibi adding a follow-up08:36
stephenfinaha - gotcha08:36
*** crushil has quit IRC08:36
*** links has quit IRC08:37
*** crushil has joined #openstack-nova08:37
*** slaweq has quit IRC08:37
stephenfinbauzas: or gibi could just fix it in the same patch, seeing as we're both online08:37
stephenfineither-or08:37
*** slaweq has joined #openstack-nova08:38
gibibauzas, stephenfin: answered. I will propose a follow where I try to enable pep8 on these source files08:38
stephenfingibi: Eh, maybe you could leave that for a bit while we finish up doc-migration?08:39
*** crushil has quit IRC08:39
stephenfinOr at least, push it on top of https://review.openstack.org/#/c/479358/08:39
stephenfinI've modified doc/source/conf.py in a few of those, and I really don't want to rebase a 15 patch series if I can help it :P08:40
*** iceyao has joined #openstack-nova08:40
*** crushil has joined #openstack-nova08:40
stephenfinbut that's your call, of course08:40
gibistephenfin: I don't think that your series and my patch conflicts as I dont touch your the files you touch08:41
gibistephenfin: mine only changes doc/ext/versioned_notifications.py08:41
gibistephenfin: the followup pep8 patch might conflict with yours but then I will push that top of your series08:42
*** yangyape_ has joined #openstack-nova08:42
stephenfingibi: yup - I'm referring to the follow-up08:42
gibistephenfin: cool, then we are on the same page.08:42
stephenfin(y)08:42
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI  https://review.openstack.org/47843708:43
*** cdent has joined #openstack-nova08:43
*** yangyape_ has quit IRC08:46
*** yangyapeng has quit IRC08:46
openstackgerritChason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme  https://review.openstack.org/47775708:47
*** yangyapeng has joined #openstack-nova08:48
*** hshiina has quit IRC08:51
openstackgerritMerged openstack/nova master: Reset the traits sync flag in the placement fixtures  https://review.openstack.org/48019808:51
*** links has joined #openstack-nova08:53
*** ducnc has joined #openstack-nova08:56
*** yangyapeng has quit IRC08:59
*** cNilesh has joined #openstack-nova08:59
*** yangyapeng has joined #openstack-nova09:00
*** slaweq has quit IRC09:04
*** slaweq has joined #openstack-nova09:04
*** yangyapeng has quit IRC09:04
*** yangyapeng has joined #openstack-nova09:05
*** abalutoiu has joined #openstack-nova09:05
*** markvoelker has joined #openstack-nova09:06
*** ducnc has quit IRC09:07
*** slaweq has quit IRC09:08
*** Shunli has quit IRC09:12
*** sanfern has joined #openstack-nova09:13
*** sfinucan has joined #openstack-nova09:14
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959309:16
*** trinaths has left #openstack-nova09:16
*** edmondsw has joined #openstack-nova09:17
bauzassfinucan: test courtesy ping09:18
*** sambetts|afk is now known as sambetts09:18
openstackgerritSilvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true  https://review.openstack.org/48046409:20
*** edmondsw has quit IRC09:21
openstackgerritSilvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true  https://review.openstack.org/48046409:22
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925709:23
openstackgerritSilvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true  https://review.openstack.org/48046409:23
*** qwmkucia has joined #openstack-nova09:23
*** slaweq has joined #openstack-nova09:24
bauzassfinucan: test courtesy ping209:24
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980009:25
openstackgerritBalazs Gibizer proposed openstack/nova master: Enable pep8 on doc generation code  https://review.openstack.org/48046909:25
gibistephenfin, bauzas: I pushed the doc gen follow up patch ^^09:25
bauzasgibi: mmm, sounds okay to me09:27
openstackgerritSilvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true  https://review.openstack.org/48046409:27
stephenfingibi: likewise09:28
stephenfinbauzas: If you were really bored today, the rest of that series sure could do with reviews ;)09:29
*** trungnv has joined #openstack-nova09:29
*** trungnv_ has joined #openstack-nova09:29
bauzasstephenfin: https://review.openstack.org/#/q/topic:doc-migration ?09:30
*** kcem has joined #openstack-nova09:30
stephenfinYup. There are patches up there for nova, python-novaclient, os-vif and os-traits09:30
stephenfinBut the nova changes are the most important ones right now, IMO09:30
sanfernhi, my vm instance is not getting ip, how can I login into VM from hypervisor and check logs09:31
* stephenfin wants to start dragging in stuff from openstack-manuals asap09:31
stephenfinsanfern: See the channel message - you'd have better luck asking that on #openstack09:31
openstackgerritMatthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination  https://review.openstack.org/46252109:33
openstackgerritMatthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration  https://review.openstack.org/47980209:33
sanfernstephenfin, thanks09:33
gibibauzas: could you put your +A on https://review.openstack.org/#/c/475349 ?09:33
bauzasgibi: looks like you're gonna conflict with https://review.openstack.org/#/c/480469/109:36
bauzasgibi: ping me when you rebase09:36
gibibauzas: https://review.openstack.org/#/c/480469/ will go in later as it is top of the doc migration series09:37
*** dixiaoli has quit IRC09:38
gibibauzas: thanks for the +A I will handle the rebase as the conflict appears09:38
bauzasgibi: sure09:39
*** markvoelker has quit IRC09:39
*** trungnv_ has quit IRC09:43
*** trungnv has quit IRC09:43
*** trungnv has joined #openstack-nova09:43
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources  https://review.openstack.org/47977709:44
*** slaweq has quit IRC09:47
*** alexchad_ has joined #openstack-nova09:47
openstackgerritlan proposed openstack/nova master: Switch from oslosphinx to openstackdocstheme  https://review.openstack.org/48048709:48
*** links has quit IRC09:49
*** alexchadin has quit IRC09:50
*** vvargaszte has quit IRC09:52
*** vvargaszte has joined #openstack-nova09:52
*** yangyapeng has quit IRC09:55
*** yangyapeng has joined #openstack-nova09:55
*** thorst has joined #openstack-nova10:02
openstackgerritSilvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true  https://review.openstack.org/48046410:02
*** links has joined #openstack-nova10:03
*** abalutoiu_ has joined #openstack-nova10:05
*** thorst has quit IRC10:06
*** tuanluong has quit IRC10:07
*** liujiong has quit IRC10:07
*** abalutoiu has quit IRC10:08
*** iceyao has quit IRC10:08
*** qwmkucia has quit IRC10:10
*** yangyapeng has quit IRC10:12
*** alexchad_ has quit IRC10:14
*** iceyao has joined #openstack-nova10:19
*** iceyao has quit IRC10:23
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification  https://review.openstack.org/42045310:23
*** udesale__ has quit IRC10:31
*** hferenc has quit IRC10:32
openstackgerritVladyslav Drok proposed openstack/nova master: Fix incorrect docstrings in neutron network API  https://review.openstack.org/48016710:32
*** hieulq_ has joined #openstack-nova10:36
*** markvoelker has joined #openstack-nova10:36
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification  https://review.openstack.org/42045310:42
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance-live_migration_post notification  https://review.openstack.org/48011910:42
*** priteau has joined #openstack-nova10:45
*** zhurong has quit IRC10:45
openstackgerritVladyslav Drok proposed openstack/nova master: Fix incorrect docstrings in neutron network API  https://review.openstack.org/48016710:45
*** iceyao has joined #openstack-nova10:46
*** smatzek has joined #openstack-nova10:46
openstackgerritMerged openstack/nova master: add missing notification samples to dev ref  https://review.openstack.org/47534910:46
*** yamamoto_ has joined #openstack-nova10:46
*** thorst has joined #openstack-nova10:47
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources  https://review.openstack.org/47977710:47
openstackgerritAlex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP  https://review.openstack.org/48037910:47
*** thorst has quit IRC10:49
*** yamamoto has quit IRC10:50
*** hieulq_ has quit IRC10:50
*** iceyao has quit IRC10:50
*** sanfern has quit IRC10:52
*** bkopilov has quit IRC10:59
*** mpaolino has quit IRC11:00
*** thorst has joined #openstack-nova11:00
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext  https://review.openstack.org/47935811:00
openstackgerritStephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues  https://review.openstack.org/48051611:00
*** edmondsw has joined #openstack-nova11:05
*** lucasagomes is now known as lucas-afk11:06
*** edmondsw has quit IRC11:09
*** gongysh has quit IRC11:09
*** markvoelker has quit IRC11:10
*** hferenc has joined #openstack-nova11:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Enable pep8 on doc generation code  https://review.openstack.org/48046911:15
*** iceyao has joined #openstack-nova11:22
*** jaypipes has joined #openstack-nova11:23
*** thorst has quit IRC11:36
*** efried_backJuly5 is now known as efried11:40
sean-k-mooneystephenfin: dumb quest but use use git review -d to checkout your patches right? what is the correct syntax to checkout https://review.openstack.org/#/c/466132/11:43
sean-k-mooney*question11:43
sean-k-mooneystephenfin: should i be useing the change id?11:44
*** acormier has joined #openstack-nova11:45
sean-k-mooneystephenfin: the yes i should never mind...11:45
*** slaweq has joined #openstack-nova11:46
*** tbachman has quit IRC11:47
mdboothsfinucan: I had another look at that errors_out_migration patch. I started by writing a better commit message, and as often happens writing docs leads to better understanding :) I've made a couple of changes to affected functions, but hopefully they're now better described and justified.11:47
*** slaweq has quit IRC11:48
mdboothI was really tempted, btw, to change things around in _finish_resize, because it's weird that we do networky things before sending the start notification.11:48
*** slaweq has joined #openstack-nova11:49
*** mkucia has joined #openstack-nova11:49
mdboothI resisted the urge, though, which has resulted in a slightly untidy return value from that function.11:49
efriedjaypipes Why do you prefer set(1, 2, 3) over the {1, 2, 3} notation?11:51
*** ratailor has quit IRC11:52
*** slaweq has quit IRC11:53
*** acormier has quit IRC11:53
stephenfinmdbooth: Sounds good. I'll check it out after lunch :)11:54
*** acormier has joined #openstack-nova11:54
stephenfinAlso, I'm using stephenfin instead of sfinucan now. You should have got an away message, but it seems some clients don't show them :(11:54
stephenfinsean-k-mooney: So you're good?11:54
sean-k-mooneystephenfin: i think so one question though11:54
stephenfinsean-k-mooney: it's 'git review -d 466132' just in case11:55
stephenfinhit me11:55
sean-k-mooneystephenfin: when submiting if i use -f will it restore the original topic?11:55
* cdent moos at jaypipes 11:55
sean-k-mooneystephenfin: it checked it out as review/sean_mooney/kolla-build-external-dockerfiles but normally the branch would be just  kolla-build-external-dockerfiles11:55
stephenfinsean-k-mooney: It does some odd stuff. If any of the patches have 'blueprint' mentioned in them, the topic will be set to 'bp/{title}'. Ditto for bug ('bug/{bugID}')11:56
stephenfinsean-k-mooney: In general though, if your branch is called 'review/{username}/{topic}', only the '{topic}' piece is kept (i.e. nothing changes)11:57
sean-k-mooneyhum ok well i can change it in the gerrit ui if id does the wronge thing might as well just see what it does.11:57
*** jpena is now known as jpena|lunch11:57
stephenfinefried: I would guess it's because the set notation is not that common11:57
sean-k-mooneystephenfin: i usually checkout everything manually but i think it about time i start using git review for that too11:58
*** acormier has quit IRC11:58
stephenfinI would say it's very rare in the nova code base, while the 'set(1, 2, 3)' notation is very common11:58
efriedstephenfin I ask because I prefer the latter - and in fact my IDE complains if you use set() in certain circumstances where {...} will do.11:58
stephenfinsean-k-mooney: I've used nothing but git-review for the past two years and it's rarely, if ever, let me down11:58
efriedstephenfin If it's a "nova codebase consistency" thing, okay.11:58
sean-k-mooneystephenfin: yep git review -f work as i wanted thanks :)11:59
efriedsean-k-mooney stephenfin +1 git-review11:59
stephenfinsean-k-mooney: two tips for you too. #1 - if you want to download a single patch with dependencies, use '-x' instead of '-d'11:59
stephenfin(very helpful when someone has uploaded a new revision of a patch in the middle of a series, are you want to rebase the successors)12:00
stephenfinand #2 - use '{change},{revision}' to download a previous revision12:00
*** gcb has quit IRC12:01
stephenfine.g. 'git review -d 466132,1'12:01
*** yangyapeng has joined #openstack-nova12:02
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources  https://review.openstack.org/47977712:03
sean-k-mooneyefried: ya i use it for upload becase "git push gerrit HEAD:refs/change/466132" is annorying but i havent been useing the other features12:03
*** vishwanathj has joined #openstack-nova12:03
sean-k-mooneystephenfin: cool that is good to know12:03
*** vishwanathj has quit IRC12:04
*** avishay has joined #openstack-nova12:04
*** avishay has left #openstack-nova12:04
*** slaweq has joined #openstack-nova12:06
*** markvoelker has joined #openstack-nova12:07
*** yangyapeng has quit IRC12:07
*** lucas-afk is now known as lucasagomes12:08
*** sanfern has joined #openstack-nova12:09
*** liverpooler has joined #openstack-nova12:09
*** gongysh has joined #openstack-nova12:11
*** sanfern has quit IRC12:14
*** sanfern has joined #openstack-nova12:15
*** yamamoto_ has quit IRC12:15
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709112:15
*** markvoelker has quit IRC12:16
*** markvoelker has joined #openstack-nova12:17
*** lyan has joined #openstack-nova12:21
*** edmondsw has joined #openstack-nova12:22
openstackgerritVladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver  https://review.openstack.org/41997512:23
openstackgerritVladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver  https://review.openstack.org/41997512:26
*** jmlowe has quit IRC12:28
*** dims has quit IRC12:29
*** baoli has joined #openstack-nova12:31
*** yamamoto has joined #openstack-nova12:33
*** mdnadeem has quit IRC12:35
*** yangyapeng has joined #openstack-nova12:37
*** gongysh has quit IRC12:38
jaypipesefried: because {1, 2, 3} is too easy to mistake for a dict.12:39
efriedjaypipes It is?12:40
*** bkopilov has joined #openstack-nova12:40
jaypipescdent: I would moo back but suffering from allergies badly today :(12:40
*** slaweq has quit IRC12:40
cdentbummer dood12:41
jaypipesefried: perhaps not for you. for others it is :)12:41
*** slaweq has joined #openstack-nova12:41
efriedjaypipes Okay, good to know.12:41
*** eharney has joined #openstack-nova12:41
alex_xujaypipes: cdent two fixes for AllocationCandidates.get_by_filters https://review.openstack.org/480379 and https://review.openstack.org/48013012:42
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification  https://review.openstack.org/48056112:43
* cdent needs to catch up on the allocation candidates stuff12:43
*** mdrabe has joined #openstack-nova12:43
cdentI've been pretty good at not paying attention for a while12:43
alex_xucdent: welcome back :)12:43
*** thorst has joined #openstack-nova12:44
cdentthanks alex_xu12:44
*** dims has joined #openstack-nova12:44
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification  https://review.openstack.org/48056112:44
alex_xunova api meeting is in 15 mins at #openstack-meeting-412:45
*** thorst_ has joined #openstack-nova12:45
*** slaweq has quit IRC12:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations  https://review.openstack.org/47093312:47
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages  https://review.openstack.org/45010512:47
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations  https://review.openstack.org/45055012:47
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages  https://review.openstack.org/48056312:47
*** thorst has quit IRC12:48
*** jpena|lunch is now known as jpena12:49
openstackgerritAndrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values  https://review.openstack.org/44800812:51
*** alexchadin has joined #openstack-nova12:52
*** baoli has quit IRC12:52
*** kcem_ has joined #openstack-nova12:54
*** kcem_ has quit IRC12:54
*** kcem has quit IRC12:55
*** kcem has joined #openstack-nova12:55
*** gcb has joined #openstack-nova12:57
*** lucasxu has joined #openstack-nova13:00
*** esberglu has joined #openstack-nova13:01
*** jmlowe has joined #openstack-nova13:06
*** mpaolino has joined #openstack-nova13:07
*** tbachman has joined #openstack-nova13:09
*** kcem has quit IRC13:13
*** baoli has joined #openstack-nova13:15
openstackgerritSilvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true  https://review.openstack.org/48046413:18
*** catintheroof has joined #openstack-nova13:18
cdentavolkov: can you give me a quick summary on where things are with placement api-ref? Has there been any progress on publishing for real?13:19
*** acormier has joined #openstack-nova13:21
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925713:22
*** mriedem has joined #openstack-nova13:22
*** felipemonteiro_ has joined #openstack-nova13:23
bauzasjaypipes: I'd really love to see your insight in https://review.openstack.org/#/c/480379/213:23
*** links has quit IRC13:24
*** slaweq has joined #openstack-nova13:24
*** felipemonteiro__ has joined #openstack-nova13:24
*** felipemonteiro_ has quit IRC13:28
*** xyang1 has joined #openstack-nova13:28
*** tiendc_ has quit IRC13:33
*** cNilesh has quit IRC13:33
cdentyou've infected me jaypipes, my allergies are spinning up13:35
*** slaweq has quit IRC13:43
*** zhurong has joined #openstack-nova13:44
stephenfinsean-k-mooney: Any ideas on https://bugs.launchpad.net/nova/+bug/1698010 ?13:46
openstackLaunchpad bug 1698010 in OpenStack Compute (nova) "neutron-based instances should not use the nova-network 'dhcp_domain' option" [High,Confirmed]13:46
stephenfinralonsoh too ^13:46
*** tbachman has quit IRC13:46
*** tbachman has joined #openstack-nova13:47
ralonsohstephenfin: let me take a look at this one13:48
mriedemstephenfin: that is a duplicate,13:48
mriedemjust need to find the other bug13:48
stephenfinI can get network info from the instance using 'Instance.get_network_info', but it just returns a list of 'os_vif.objects.vif.VIFBase' subclass instances13:48
stephenfinmriedem: Oh hai13:48
mriedemthe dhcp_domain option is read in the metadata api code and that's how it gets into config drive13:48
mriedemas noted by bnemec in the bug report it's showing up via cloud-init13:48
stephenfinYup :( I was trying to see where I could get the neutron equivalent from, but we don't seem to cache it anywhere13:49
stephenfinand I don't want to call out to neutron for every instance (assuming that is even an option)13:49
mriedemwe already call to neutron for dns info when we create or attach to a port13:50
stephenfinHmm, so is that a viable path to pursue?13:51
bauzasmriedem: permission to apologize, captain. https://bugs.launchpad.net/nova/+bug/170245413:51
openstackLaunchpad bug 1702454 in OpenStack Compute (nova) "Transforming the RequestSpec object into legacy dicts doesn't support the requested_destination field" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)13:51
bauzasmriedem: tl;dr: evacuations providing a target don't verify that target since newton13:52
* bauzas now hides13:52
mriedemi thought the bug said that was only for legacy specs or whatever?13:53
mriedemsomeone from suse was just fixing bugs in stable related to evacuate with a targeted host, so something must work13:53
bauzasmriedem: see the description13:54
bauzasmriedem: that's because we turn into dicts only for evacuate13:54
mriedemstephenfin: could we just make dhcp_domain default to None? that would change how https://github.com/openstack/nova/blob/0b9bacb/nova/api/metadata/base.py#L546-L549 behaves13:54
bauzasmriedem: and tbh, the bug was reported internally on our Newton branch13:55
bauzasso I doubt it's fixed upstream but just master because of https://review.openstack.org/#/c/469037/13:55
bauzasthat's where tech debt kills us13:55
bauzasand I wouldn't like to backport https://review.openstack.org/#/c/469037/13:56
bauzasrather fixing the problem with the transformating13:56
*** awaugama has joined #openstack-nova13:56
stephenfinmriedem: It would change it, but I don't think it would deliver the desired behavior. bnemec did that, and cloudinit added a default 'localdomain' domain instead of what neutron had configured ('nemebean.com')13:56
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276613:58
*** acormier has quit IRC13:58
*** udesale has joined #openstack-nova14:01
*** jwcroppe has quit IRC14:01
*** jwcroppe has joined #openstack-nova14:02
*** winston-d_ has quit IRC14:03
*** alexchadin has quit IRC14:05
dansmithmriedem: are we going to go ahead with this? https://review.openstack.org/#/c/468227/214:05
*** jwcroppe has quit IRC14:06
mriedemdansmith: related to bauzas' evacuate in newton clusterfunnery above?14:07
mriedemis there a rhosp newton evacuate red alert going off this morning?14:07
bauzasnot really14:08
dansmithmriedem: is it related to that? I don't think so14:08
bauzasAFAIK14:08
bauzasjust some internal BZ triage I did14:08
*** hongbin has joined #openstack-nova14:08
dansmithI don't know about bauzas' thing so I assume unrelated14:08
mriedemdansmith: but yes we should get that in - the evacuate stuff with forced target was broken since newton i think14:08
bauzasyup, unrelated14:08
*** takedakn has joined #openstack-nova14:09
bauzasmriedem: not forced14:09
dansmithmriedem: yeah it's definitely broken14:09
dansmithmriedem: okay then I'll review14:09
bauzasmriedem: just 'passed'14:09
bauzasdansmith: mriedem: I need to write a change, should be quick14:09
*** zhurong has quit IRC14:09
bauzasdansmith: context is https://bugs.launchpad.net/nova/+bug/170245414:09
openstackLaunchpad bug 1702454 in OpenStack Compute (nova) "Transforming the RequestSpec object into legacy dicts doesn't support the requested_destination field" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)14:09
mriedemok i guess i don't know how the evacuate api handles passing a host and forcing a host differently14:09
bauzasmriedem: that's simple14:10
bauzassince 2.29 (or 2.30, don't remember which one), if you pass a target to the API, then we verify that target by the scheduler14:10
*** tbachman has quit IRC14:11
bauzasunless you pass a specific parameter to the API, namely 'force'14:11
bauzasin that case, we fully bypass the scheduler call and we directly cast the compute service with the hostname14:11
sean-k-mooneystephenfin: without designate im not sure how you can have instance use teh neutron dns_domain unless we add it to vif_binding details in some way.14:11
mriedemok there was something else we found when investigating that other evacuate bug,14:13
stephenfinsean-k-mooney: We are able to fetch that from neutron though, right? https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1131-L113314:14
mriedemwas we can pass force=None here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/evacuate.py#L12014:14
mriedembut the compute api assumes it's a boolean https://github.com/openstack/nova/blob/master/nova/compute/api.py#L417014:14
mriedembut since it can be None that's not quite right14:14
mriedemanyway, we didn't fix that as part of bug 1658070 since the claim issue was fixed in the compute14:15
openstackbug 1658070 in OpenStack Compute (nova) newton "Failed SR_IOV evacuation with host" [High,In progress] https://launchpad.net/bugs/1658070 - Assigned to Matt Riedemann (mriedem)14:15
mriedemi have to take maya to daycare14:15
mriedembbiab14:15
sean-k-mooneystephenfin: you can but it would be good to include this in the os-vif network object so that you can jsut read it from there. that would require neutron to populated it in the vif binding details14:15
*** jwcroppe has joined #openstack-nova14:16
stephenfinsean-k-mooney: Yeah, that's a good idea. I'll take a look and see how awkward it is14:16
sean-k-mooneystephenfin: well this is what you would need to extend in os-vif https://github.com/openstack/os-vif/blob/master/os_vif/objects/network.py#L28-L3914:17
*** prateek has quit IRC14:18
sean-k-mooneystephenfin: this is the neutron extention that provides access to the dns name https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/dns_domain_ports.py14:18
*** karthiks has quit IRC14:20
stephenfinsean-k-mooney: https://bugs.launchpad.net/os-vif/+bug/170249114:23
openstackLaunchpad bug 1702491 in os-vif "[RFE] 'os_vif.objects.network.Network' should expose dns_name" [Undecided,New]14:23
stephenfinI'll take a shot at that, using your MTU changes as a blueprint14:23
sean-k-mooneystephenfin: you might want to add the info here https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L2186 and then extend https://github.com/openstack/nova/blob/0b9bacb3ef29f59f1461ce62f65f9d1f6efab761/nova/network/os_vif_util.py#L212 to populate the os-vif version14:25
*** arahal_ has joined #openstack-nova14:26
*** yangyape_ has joined #openstack-nova14:29
*** ratailor has joined #openstack-nova14:30
*** ralonsoh has quit IRC14:30
*** felipemonteiro__ has quit IRC14:30
*** yangyapeng has quit IRC14:32
openstackgerritStephen Finucane (stephenfin) proposed openstack/nova master: metadata: Don't use nova-net DNS values with neutron  https://review.openstack.org/48061614:33
*** prateek has joined #openstack-nova14:34
*** yamamoto_ has joined #openstack-nova14:41
*** tbachman has joined #openstack-nova14:42
*** crushil_ has joined #openstack-nova14:43
*** yamamoto has quit IRC14:44
*** acormier has joined #openstack-nova14:45
openstackgerritStephen Finucane (stephenfin) proposed openstack/os-vif master: objects: Add 'dns_domain' attribute to 'Network'  https://review.openstack.org/48063014:46
*** tuanla_fujitsu has joined #openstack-nova14:47
*** tuanla__ has joined #openstack-nova14:47
mriedemdansmith: replied to your comments in https://review.openstack.org/#/c/468227/14:49
*** takedakn has quit IRC14:52
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix arguments in calling _delete_nic_metadata  https://review.openstack.org/48036914:53
mriedemugh14:54
mriedemsomeone want to +W that ^ ?14:55
*** markus_z has quit IRC14:56
*** mmehan has joined #openstack-nova14:58
*** crushil_ has quit IRC14:58
*** crushil_ has joined #openstack-nova14:59
*** tbachman has quit IRC14:59
*** lucasxu has quit IRC15:00
*** Oku_OS is now known as Oku_OS-away15:01
*** liujiong has joined #openstack-nova15:03
*** liujiong has quit IRC15:04
*** tbachman has joined #openstack-nova15:05
*** lbragstad has joined #openstack-nova15:05
*** belmoreira has quit IRC15:06
*** moshele has quit IRC15:12
mriedemsfinucan: i think https://review.openstack.org/#/c/480616/ needs a tweak,15:12
mriedemseems we should have always had a CONF.dhcp_domain or '' in the last substitution in that code15:13
mriedemif CONF.dhcp_domain was set to None in nova.conf it would result in a weird hostname15:13
*** bkopilov has quit IRC15:14
mriedemanyway, don't really need to worry about that for nova-net if people are just using the default of novalocal15:14
avolkovcdent: I finished with api ref for all methods (except POST inventories) and want to update api ref tool15:18
*** yamahata has joined #openstack-nova15:18
*** udesale has quit IRC15:18
avolkovcdent: nothing about publish job, but I think it's easy to fix15:19
*** nic has joined #openstack-nova15:21
*** rcernin has quit IRC15:21
gibimriedem: hi! could you express your opinion about binary vs. service here https://review.openstack.org/#/c/476538/4/nova/notifications/objects/base.py ?15:21
*** tbachman has quit IRC15:28
mriedemugh15:28
mriedemcan i not have an opinion?15:28
*** Nel1x has joined #openstack-nova15:29
gibimriedem: can you just +2 it blindly? :)15:29
*** hieulq_ has joined #openstack-nova15:29
*** andreas_s has quit IRC15:30
bauzasmriedem: you're the chief and we blindly follow you15:30
mriedemnova-api isn't a binary15:30
mriedemcalling it such isn't really correct15:30
mriedemand just causes confusing15:30
mriedemthe api binary is actually nova-osapi_compute15:30
*** cNilesh has joined #openstack-nova15:31
*** ratailor has quit IRC15:31
mriedemwhen is the queens ptl election...15:31
gibimriedem: what about calling to 'source'15:32
openstackgerritMatt Riedemann proposed openstack/nova master: Fix comment for API binary name in WSGIService  https://review.openstack.org/48064815:34
*** lucasxu has joined #openstack-nova15:35
sean-k-mooneysfinucan: mriedem what does the hydrate fuction do? https://github.com/openstack/nova/blob/bdd6876b8feca2532fa5d9e2e98028b4d19ba6b6/nova/network/model.py#L492-L496 i didnt know object could be thristy15:35
mriedemsean-k-mooney: deserializes the json blob15:36
*** gyee has joined #openstack-nova15:36
mriedem*serialized json string i mean15:36
*** crushil_ has quit IRC15:36
*** tbachman has joined #openstack-nova15:36
mriedemthat thing gets stored in the instance_info_cache.network_info column as a serialized json string15:36
sean-k-mooneymriedem: ok and its not called load or deserialize because?15:36
mriedembecause hydrate is cooler?15:36
mriedemidk15:36
mriedemwe should call it explode()15:37
*** psachin has quit IRC15:37
mriedemand implode()15:37
sean-k-mooneyok well some new i learned today.15:37
*** r2j has joined #openstack-nova15:38
*** dixiaoli has joined #openstack-nova15:39
*** bkopilov has joined #openstack-nova15:40
*** tbachman has quit IRC15:43
*** dixiaoli has quit IRC15:44
*** tbachman has joined #openstack-nova15:44
*** fragatina has joined #openstack-nova15:45
mriedemgibi: comments inline15:45
*** prateek has quit IRC15:45
gibimriedem: thanks, looking15:47
mriedemno sdague this week?15:48
mriedemclarkb: can you peek at the assertions about compute endpoint format being made in here? https://review.openstack.org/#/c/479818/15:48
mriedemhttps://review.openstack.org/#/c/479818/6/novaclient/v2/versions.py15:48
ericyoungwe have a patch for nova, to use ScaleIO for ephemeral storage. It's passed CI and mdbooth had offered to review it for us.15:48
ericyoung We haven't been able to get ahold of him, can someone else take a look for us at https://review.openstack.org/#/c/407440/15:48
mriedemericyoung: just fyi, there are 35 blueprints that need review yet for pike https://blueprints.launchpad.net/nova/pike15:49
clarkbmriedem: you can do unversioned if the endpoint supports version discovery which nova does do?15:50
clarkbmordred: ^15:50
*** Apoorva has joined #openstack-nova15:50
mriedemlooking at https://developer.openstack.org/api-ref/compute/#api-versions it's not totally clear to me15:50
*** damien_r has quit IRC15:51
clarkbmriedem: / list all major versions <- pretty sure that means /compute/ is valid rather than /compute/vWhatever15:51
ericyoungmriedem: this blueprint was already approved for pike15:51
mriedemi remember the openstacksdk team having a thread in the ML last year struggling with the plethora of possible endpoint url formats in the wild15:51
mriedemericyoung: yes, so are those other 3415:52
mriedemericyoung: that's my point15:52
*** hieulq__ has joined #openstack-nova15:52
mriedemclarkb: yeah i remember having reported this to the sdk team https://bugs.launchpad.net/python-openstacksdk/+bug/163148015:52
openstackLaunchpad bug 1631480 in OpenStack SDK "session endpoint lookup doesn't handle endpoints with paths" [Undecided,New]15:52
mriedembecause we had a network endpoint like this: publicURL: http://localhost:5000/network             |15:53
*** hieulq_ has quit IRC15:53
*** felipemonteiro_ has joined #openstack-nova15:55
*** sridharg has quit IRC15:56
mriedemdansmith: will we be cellsv2 meeting today?15:56
mriedembecause i have developments15:56
dansmithmriedem: sure15:56
ericyoungmriedem: understood. I'm not trying to be a pain, just wondering if there was someone else who could look or if we should just sit tight.15:57
mriedemericyoung: you're basically in the latter camp at this point15:57
cdentanybody want to put eyes on https://review.openstack.org/#/c/460147/ has one +2 already. bauzas ? mriedem ?15:59
mriedemi've been -1 on that one a few times now, so i can look at it again16:00
gibimriedem: I will try to do a binary->source renaming with the major version bump. I think it is not that hard in this particular case.16:01
*** mkucia has quit IRC16:01
*** jamesden_ has joined #openstack-nova16:04
*** fragatina has quit IRC16:04
*** jamesdenton has quit IRC16:04
bauzascdent: looking16:05
*** rajathagasthya has joined #openstack-nova16:06
*** Sukhdev has joined #openstack-nova16:06
*** abalutoiu__ has joined #openstack-nova16:07
*** burt has joined #openstack-nova16:09
*** abalutoiu_ has quit IRC16:11
bauzascdent: jaypipes: we're not passing back the generation bit when deleting ? https://review.openstack.org/#/c/460147/15/nova/scheduler/client/report.py@66616:11
cdentthe response body is empty when deleting16:12
cdent(in that case)16:12
*** lucasagomes is now known as lucas-afk16:12
mriedemyeah, so our generation + 1 could actually be wrong16:13
mriedembut,16:13
openstackgerritMathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432216:13
*** moshele has joined #openstack-nova16:13
mriedemi think when we try to use that incorrect generation we'll fail later, and get it corrected16:13
mriedemsomewhere in here16:13
mriedemin _get_inventory_and_update_provider_generation16:14
stephenfinmriedem: Random question. Is there any reason we would want 'refresh_cache' to be False for '_has_port_binding_extension'? https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L101316:14
mriedemand _update_inventory_attempt16:14
mriedemstephenfin: no idea16:14
stephenfinI ask because I note that _has_multi_provider_extension always refreshes the cache, while _has_auto_allocate_extension never does (outside of tests)16:14
stephenfinAh :)16:15
mriedemthere is no rhyme or reason to when we refresh the cache of neutron extensions16:15
mordredmriedem, clarkb: what's the question about unversioned endpoints?16:15
*** moshele has quit IRC16:15
cdentmriedem: yeah, it will get correct if wrong, what's being doing there is an optimistic sort of "this is more likely to be correct than it would be if we did nothing"16:15
mriedemthe auto allocation extension was new in mitaka and i believe not optional, or so i was told,16:15
mriedembut we check for it's existence to see if we're talking to older neutron16:15
bauzasmriedem: cdent: well, I'd say it's unfortunate but workable given we retry16:15
mriedemmordred: https://review.openstack.org/#/c/479818/6/novaclient/v2/versions.py16:16
bauzascdent: yeah, from a REST PoV that's understandable16:16
bauzascdent: but I'm just thinking that we assert things a way too much hard16:16
mordredmriedem: looking now16:16
mriedemparsing that sentence is a way too much hard16:16
mriedem:)16:16
stephenfinmriedem: Given that the refresh_cache function actually does some sanity checks before firing off requests suggests we could _always_ refresh the cache https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1003-L100516:17
*** smatzek has quit IRC16:17
mordredmriedem: the endpoints in the catalog should either be example.com/compute or example.com/compute/v2.1 if the operator puts a versioned endpoint into the catalog16:17
stephenfin...though maybe the auto allocation extension could be removed altogether. I'll have a look-see16:17
mordredmriedem: we just added a crap ton of code to handle these things and tests for them in keystoneauth16:18
mriedemstephenfin: yeah and that defaults to refresh every 10 minutes by default16:18
mriedemstephenfin: is this cache refresh really a problem?16:18
*** efoley has quit IRC16:18
mriedemare operators enabling and disabling extensions ad hoc and it's causing real issues?16:18
stephenfinI've no idea and was hoping you could enlighten me16:19
mriedemmordred: and the sdk has a crap ton of code for it too :)16:19
mriedemstephenfin: i assume operators are not doing that16:19
mordredmriedem: yah - well, we should be porting the sdk over to use the code in ksa now so that there is only one place - but yeah16:19
stephenfinmriedem: I would assume that too. I'm just adding a '_has_dns_extension' function and wanted to know if the cache check should be there or not16:19
mordredmriedem: anyway - I'll leave some comments ...16:20
stephenfinI'm going to stick it in to be safe but then go on a hunt for more info (likely goose chase - meh)16:20
mriedemkevinbenton: do you know if the dns extension in neutron is optional?16:20
stephenfinmriedem: I'm pretty sure sean-k-mooney said it was16:20
stephenfinYou need to have designate deployed for it to work16:21
sean-k-mooneymost of the extentions in neutron are optional.16:21
stephenfinsean-k-mooney: Any idea where we'd find a definitive list?16:22
* stephenfin 's Google-fu is failing him16:22
*** adisky__ has quit IRC16:22
mordredmriedem: OH - this is in novacient16:22
mordredmriedem: I thought this was in nova server16:23
sean-k-mooneywell running  openstack extension list | grep dns only gave me os-floating-ip-dns16:23
sean-k-mooneyso on ocata its not required.16:23
mordredmriedem: as soon as we cut a release, you can straight-up delete this code16:23
mordredmriedem: I'll make you a patch RSN16:23
sean-k-mooneyill check master but i dont think its mandatoary16:23
*** markvoelker has quit IRC16:24
*** igordcard has quit IRC16:24
mriedemsean-k-mooney: os-floating-ip-dns is from the compute api16:24
mriedemi.e. nova16:25
mriedemi.e. nova-network16:25
mriedemi don't know where or how openstack extension list gets it's data16:25
*** claudiub has quit IRC16:26
*** abalutoiu__ has quit IRC16:27
mordredefried: https://review.openstack.org/#/c/479818 is worth looking at - it's right in the midst of all of the discovery work16:27
*** slaweq has joined #openstack-nova16:27
*** slaweq has quit IRC16:27
*** lucasxu has quit IRC16:27
*** jamesdenton has joined #openstack-nova16:29
*** jamesden_ has quit IRC16:29
openstackgerritIldiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to swap_volume  https://review.openstack.org/45697116:29
openstackgerritIldiko Vancsa proposed openstack/nova master: Remove check_detach  https://review.openstack.org/44667116:29
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028516:29
dansmithefried: solidarity my brother.16:29
*** igordcard has joined #openstack-nova16:30
*** markvoelker has joined #openstack-nova16:30
*** igordcard has quit IRC16:33
*** rajathag_ has joined #openstack-nova16:34
*** igordcard has joined #openstack-nova16:35
dansmithmriedem: have you done any looking at this yet? https://review.openstack.org/#/c/416521/16:35
dansmithdoesn't look like it16:35
*** hieulq__ has quit IRC16:36
*** rajathagasthya has quit IRC16:37
*** fragatina has joined #openstack-nova16:38
mriedemdansmith: i looked at the LOC16:40
mriedemand then i looked away16:40
mriedembecause i was flying over the pacific16:40
dansmithack16:41
*** sambetts is now known as sambetts|afk16:44
*** dtp has joined #openstack-nova16:45
*** tjones has joined #openstack-nova16:49
*** smatzek has joined #openstack-nova16:53
*** itlinux has joined #openstack-nova16:55
*** tuanla_fujitsu has quit IRC16:56
*** tuanla__ has quit IRC16:56
*** sanfern has quit IRC16:57
*** sshwarts has quit IRC16:58
openstackgerritStephen Finucane proposed openstack/nova master: Always refresh cache when checking extensions  https://review.openstack.org/48067416:59
openstackgerritStephen Finucane proposed openstack/nova master: Add '_has_dns_extension' function  https://review.openstack.org/48067516:59
openstackgerritStephen Finucane proposed openstack/nova master: Use dns_domain value from neutron in os-vif wiring  https://review.openstack.org/48067616:59
*** sanfern has joined #openstack-nova17:00
dansmithmriedem: cells meeting?17:01
mriedemyeah17:01
*** eharney has quit IRC17:02
*** iceyao has quit IRC17:03
*** yamahata has quit IRC17:03
*** eharney has joined #openstack-nova17:04
*** fragatin_ has joined #openstack-nova17:04
*** fragatina has quit IRC17:07
*** cNilesh has quit IRC17:08
*** hieulq has quit IRC17:09
*** cNilesh has joined #openstack-nova17:09
*** trungnv has quit IRC17:09
sean-k-mooneymriedem: stephenfin running neutron ext-list on master also dos not have the dns extention enabled by default17:09
*** chyka has joined #openstack-nova17:10
*** igordcard has quit IRC17:10
*** igordcard has joined #openstack-nova17:11
*** igordcard has quit IRC17:11
*** igordcard has joined #openstack-nova17:12
*** cNilesh has quit IRC17:17
*** jpena is now known as jpena|off17:17
sean-k-mooneyigordcard: o/17:18
*** cNilesh has joined #openstack-nova17:18
melwittdansmith: what did you mean by "get back the instance" before conductor is called? like if they did a GET and got an instance that was made from a BuildRequest?17:19
dansmithmelwitt: well, I mean: I do a create, I get back my instance uuid, which previously meant that I passed my quota check already, so I could wait for it to either go active or error17:20
dansmithmelwitt: now, I get that back, and for a split second, the instance continues to be in my list, but then goes away17:20
dtpmriedem was your comment on https://review.openstack.org/#/c/464280/ for me?17:20
melwittoh17:21
melwitthmm17:21
dansmithmelwitt: I have scripts that poll instance list for the instance I just created17:21
dansmithwhich will be confused as the instance will now just disappear17:21
dansmithmelwitt: definitely a behavioral change, so I think going to error instead of deleted is required here, right?17:22
dansmithwhich will put them over budget by one, but it's a shell of an instance anyway17:22
melwittdansmith: yeah, I need to look at it in more detail bc what I'm doing there is deleting the record from the instances table, which isn't what you would have been seeing in your instance list prior to conductor finishing its work. but whatever exception handling that happens from the OverQuota raise might probably wipe out the BuildRequest that was giving the instance back before17:23
dansmithyou're following the same pattern as the other resources, but those are created and deleted synchronously17:24
*** gjayavelu has joined #openstack-nova17:24
sean-k-mooneymriedem: stephenfin  so if i deploy designate i get dns-integration added when i run neutron ext-list17:24
dansmithmelwitt: yeah, I think the buildreq will be gone, but you're not wiping out the instance record,17:24
*** hieulq has joined #openstack-nova17:25
dansmithmelwitt: you're setting it to deleted17:25
*** trungnv has joined #openstack-nova17:25
dansmithmelwitt: which is still findable via the instance map until purge17:25
melwittyeah, I know. the instances table record is created synchronously there too, it's just that prior to that, a BuildRequest is used to present an instance to keep the same behavior we had before moving instance create to conductor17:25
dansmithmelwitt: synchronous to the api call Imea17:25
dansmith*I mean17:25
melwittokay, yeah17:25
sean-k-mooneymriedem: stephenfin this adds some dns field to the port info http://paste.openstack.org/show/614483/17:26
melwittdansmith: so yeah, probably that's the only thing we can do, set it to ERROR17:26
dansmithmelwitt: aye17:26
dansmithmelwitt: and we need an instance action or fault message or something so people can tell why17:27
dansmithmelwitt: otherwise they'll just go create again, which I guess will probably fail immediately, but still17:27
*** lyan has quit IRC17:27
*** lyan has joined #openstack-nova17:27
melwittdansmith: right. yeah, good point17:28
*** rajathag_ has quit IRC17:29
*** iceyao has joined #openstack-nova17:30
*** fragatin_ has quit IRC17:31
*** fragatina has joined #openstack-nova17:32
melwittdansmith: I have a crazy idea of maybe counting BuildRequests as part of the instances count. not sure if there's any gotchas there17:32
melwittto make it so we dont return an instance if we go over quota during a race in the first place17:33
dansmithmelwitt: meaning counting both and taking the sum?17:33
melwittyeah17:33
dansmithI don't think you can do that naively unless you filter out the dupes, because we do have dupes for a bit right?17:33
sean-k-mooneystephenfin: if i boot a new instace after designate is there it correct populates the server name from the create command into the hostname and fqdn fields http://paste.openstack.org/show/614486/17:33
dansmithmelwitt: we create the instance and then delete the buildreq so the api code starts looking at the actual instance, IIRC17:34
melwittdansmith: hm, I didn't realize it dupes17:34
melwittyeah17:34
melwittdangit17:34
dansmithmelwitt: two different databases, so it can't be atomic17:34
*** iceyao has quit IRC17:34
*** yamahata has joined #openstack-nova17:40
*** mpaolino has quit IRC17:41
*** igordcard has quit IRC17:41
*** igordcard has joined #openstack-nova17:42
*** ijw has joined #openstack-nova17:42
*** igordcard has quit IRC17:44
*** igordcard has joined #openstack-nova17:44
*** rajathagasthya has joined #openstack-nova17:44
*** ralonsoh has joined #openstack-nova17:45
*** ijw has quit IRC17:47
*** igordcard has quit IRC17:48
*** igordcard has joined #openstack-nova17:48
*** cNilesh has quit IRC17:52
jaypipesbauzas, cdent, dansmith: sorry for being absent today, guys :( been struggling with allergies all day. can't seem to shake em.17:52
cdentjaypipes: hot baths and spicy foods17:53
jaypipescdent: yes, but not together.17:53
cdentspicy foods in the hot tub!17:53
* cdent is living large17:53
dansmithjaypipes: s'okay, I'm allergic to florida too17:54
jaypipesdansmith: ack17:54
* cdent gives dansmith some thai food17:54
dimsLOL17:55
*** Swami has joined #openstack-nova17:56
*** tjones has quit IRC18:00
*** tjones has joined #openstack-nova18:02
*** Guest39045 is now known as med_18:05
*** med_ has joined #openstack-nova18:05
*** litao__ has quit IRC18:17
*** slaweq has joined #openstack-nova18:19
*** yamahata has quit IRC18:20
*** yamahata has joined #openstack-nova18:20
*** dave-mccowan has joined #openstack-nova18:21
*** sanfern has quit IRC18:25
*** sanfern has joined #openstack-nova18:26
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925718:26
*** slaweq has quit IRC18:30
*** slaweq has joined #openstack-nova18:30
*** sanfern has quit IRC18:30
*** sanfern has joined #openstack-nova18:31
*** iceyao has joined #openstack-nova18:35
*** slaweq has quit IRC18:35
*** Sukhdev has quit IRC18:36
*** ijw has joined #openstack-nova18:37
*** tbachman has quit IRC18:37
*** iceyao has quit IRC18:39
*** ijw has quit IRC18:41
*** baoli has quit IRC18:41
*** ijw has joined #openstack-nova18:41
*** slaweq has joined #openstack-nova18:42
*** cdent has quit IRC18:42
*** jaypipes has quit IRC18:42
*** baoli has joined #openstack-nova18:44
*** slaweq has quit IRC18:46
*** slaweq has joined #openstack-nova18:47
*** rcernin has joined #openstack-nova18:47
*** hieulq has quit IRC18:49
*** trungnv has quit IRC18:49
ildikovmriedem: hi18:49
ildikovmriedem: patches are nice and green if you would happen to have time to review: https://review.openstack.org/#/q/topic:bp/cinder-new-attach-apis :)18:50
*** priteau has quit IRC18:51
*** jamesdenton has quit IRC18:55
*** jamesdenton has joined #openstack-nova18:56
*** slaweq has quit IRC18:57
*** tbachman has joined #openstack-nova18:58
*** slaweq has joined #openstack-nova18:58
*** jamesdenton has quit IRC18:59
*** abalutoiu has joined #openstack-nova18:59
*** jamesdenton has joined #openstack-nova18:59
*** iceyao has joined #openstack-nova19:02
*** trungnv has joined #openstack-nova19:04
*** hieulq has joined #openstack-nova19:04
*** slaweq has quit IRC19:06
*** jamesdenton has quit IRC19:06
*** iceyao has quit IRC19:06
*** Sukhdev has joined #openstack-nova19:08
*** jamesdenton has joined #openstack-nova19:11
openstackgerritMatt Riedemann proposed openstack/nova master: Fix error message when support matrix entry is missing a driver  https://review.openstack.org/48070819:18
*** xyang1 has quit IRC19:20
*** yamamoto_ has quit IRC19:21
openstackgerritRick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API  https://review.openstack.org/44928819:22
*** sanfern has quit IRC19:26
*** ijw has quit IRC19:32
*** yamamoto has joined #openstack-nova19:35
*** mkucia has joined #openstack-nova19:39
*** yamamoto has quit IRC19:40
*** claudiub has joined #openstack-nova19:43
openstackgerritMatt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432219:46
efriedmordred https://review.openstack.org/#/c/479818 looks like exactly a case where your new stuff needs to be used.19:47
*** amotoki is now known as amotoki_away19:47
*** amotoki_away is now known as amotoki19:47
mordredefried: yup. if you don't beat me to it - I'll write a patch19:47
efriedmordred And all the supporting code is merged at this point?19:48
efriedAnd in a ksa release that's percolated through g-r?19:48
mordredefried: yes - merged. no - not released yet - need to respond to some reviews on the docs update patch19:48
mordredefried: and I was hoping I could get the service-types support in19:48
efriedmordred Then the novaclient patch has to wait for that, neh?19:49
mordredefried: yah - although if  it's urgent we could likely talk to folks about cutting a release once the docs patch lands, then doing another one once service-types is in19:49
*** tesseract has quit IRC19:56
*** slaweq has joined #openstack-nova20:02
*** rcernin has quit IRC20:05
*** Nel1x has quit IRC20:07
*** Nel1x has joined #openstack-nova20:08
*** jamesdenton has quit IRC20:10
*** xyang1 has joined #openstack-nova20:11
*** Apoorva has quit IRC20:11
*** Apoorva has joined #openstack-nova20:12
*** Apoorva has quit IRC20:13
*** Apoorva has joined #openstack-nova20:14
*** rcernin has joined #openstack-nova20:14
*** ralonsoh has quit IRC20:17
*** tjones has left #openstack-nova20:21
*** hieulq has quit IRC20:26
*** trungnv has quit IRC20:26
*** trungnv has joined #openstack-nova20:27
*** ijw has joined #openstack-nova20:30
openstackgerritMathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432220:33
*** yamamoto has joined #openstack-nova20:38
*** ijw has quit IRC20:38
*** hieulq has joined #openstack-nova20:42
*** yassine has quit IRC20:43
*** yamamoto has quit IRC20:43
*** ijw has joined #openstack-nova20:49
*** liverpooler has quit IRC20:49
*** slaweq has quit IRC20:53
*** slaweq has joined #openstack-nova20:54
*** thorst_ has quit IRC20:55
*** esberglu has quit IRC20:55
*** thorst has joined #openstack-nova20:57
*** slaweq has quit IRC20:58
*** slaweq has joined #openstack-nova20:59
*** slaweq_ has joined #openstack-nova21:00
*** swamireddy has joined #openstack-nova21:00
*** jmlowe has quit IRC21:01
*** thorst has quit IRC21:01
*** slaweq_ has quit IRC21:01
*** r2j has quit IRC21:06
*** slaweq_ has joined #openstack-nova21:06
*** slaweq has quit IRC21:10
*** awaugama has quit IRC21:10
*** Sukhdev has quit IRC21:16
*** smatzek has quit IRC21:16
*** slaweq_ has quit IRC21:20
*** baoli has quit IRC21:20
*** slaweq has joined #openstack-nova21:23
*** jamesdenton has joined #openstack-nova21:27
*** ijw has quit IRC21:28
*** ijw has joined #openstack-nova21:28
*** edmondsw has quit IRC21:31
*** thorst has joined #openstack-nova21:31
*** edmondsw has joined #openstack-nova21:33
*** thorst has quit IRC21:35
*** edmondsw has quit IRC21:38
*** eharney has quit IRC21:39
*** trungnv has quit IRC21:44
*** hieulq has quit IRC21:45
*** iceyao has joined #openstack-nova21:48
*** slaweq_ has joined #openstack-nova21:50
*** takashin has joined #openstack-nova21:52
*** iceyao has quit IRC21:53
*** slaweq_ has quit IRC21:55
*** claudiub has quit IRC21:58
*** felipemonteiro_ has quit IRC22:00
*** gjayavelu has quit IRC22:01
*** burt has quit IRC22:01
*** trungnv has joined #openstack-nova22:04
*** hieulq has joined #openstack-nova22:05
*** trungnv has quit IRC22:09
*** edmondsw has joined #openstack-nova22:11
*** edmondsw has quit IRC22:16
*** ijw has quit IRC22:16
*** esberglu has joined #openstack-nova22:21
*** acormier has quit IRC22:22
*** acormier has joined #openstack-nova22:22
*** trungnv has joined #openstack-nova22:23
*** lyan has quit IRC22:24
*** esberglu has quit IRC22:26
*** acormier has quit IRC22:27
*** slaweq has quit IRC22:30
*** slaweq has joined #openstack-nova22:31
*** slaweq has quit IRC22:36
*** jwcroppe has quit IRC22:43
*** jwcroppe has joined #openstack-nova22:43
*** rcernin has quit IRC22:45
*** jwcroppe has quit IRC22:47
*** ijw has joined #openstack-nova22:48
*** iceyao has joined #openstack-nova22:50
*** itlinux has quit IRC22:50
*** iceyao has quit IRC22:54
openstackgerritMichael Still proposed openstack/nova master: Only setup iptables for metadata if using nova-net  https://review.openstack.org/48076522:57
*** jwcroppe has joined #openstack-nova23:00
*** jwcroppe has quit IRC23:01
*** jwcroppe has joined #openstack-nova23:01
*** ijw has quit IRC23:04
*** xyang1 has quit IRC23:05
*** jwcroppe has quit IRC23:06
*** iceyao has joined #openstack-nova23:11
*** gjayavelu has joined #openstack-nova23:11
*** thorst has joined #openstack-nova23:12
*** thorst has quit IRC23:12
*** hongbin has quit IRC23:15
*** iceyao has quit IRC23:15
*** thorst has joined #openstack-nova23:19
*** hieulq has quit IRC23:21
*** _pewp_ has quit IRC23:22
*** trungnv has quit IRC23:23
*** Swami has quit IRC23:23
*** hemna is now known as assbutt23:25
*** _pewp_ has joined #openstack-nova23:25
*** assbutt is now known as buttass23:26
*** buttass is now known as butt23:26
*** iceyao has joined #openstack-nova23:31
*** jwcroppe has joined #openstack-nova23:31
*** chyka has quit IRC23:34
*** hieulq has joined #openstack-nova23:35
*** trungnv has joined #openstack-nova23:35
*** iceyao has quit IRC23:36
*** jwcroppe has quit IRC23:36
*** abalutoiu has quit IRC23:38
*** rajathagasthya has quit IRC23:45
*** hoonetorg has quit IRC23:47
*** catintheroof has quit IRC23:47
*** iceyao has joined #openstack-nova23:52
*** owalsh has quit IRC23:53
*** iceyao has quit IRC23:56
*** gjayavelu has quit IRC23:58

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