Wednesday, 2017-04-12

openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc  https://review.openstack.org/41644400:02
openstackgerritTakashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases  https://review.openstack.org/39543600:02
openstackgerritTakashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases  https://review.openstack.org/39473800:02
mgagneso... about that nova-specs gate breakage... "sphinx.errors.SphinxWarning: WARNING: no "man_pages" config value found; no manual pages will be written" is there any change I can rebase on to get it fixed?00:06
*** thorst has quit IRC00:09
*** gjayavelu has quit IRC00:10
*** crushil has quit IRC00:12
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.rescue notifications  https://review.openstack.org/38564400:22
*** dtp has quit IRC00:24
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.rescue notifications  https://review.openstack.org/38564400:27
*** armax has quit IRC00:29
*** rfolco has joined #openstack-nova00:30
*** karimb has quit IRC00:34
*** thorst has joined #openstack-nova00:35
*** winston-d_ has joined #openstack-nova00:37
clarkbmgagne: did that fail earlier today? there was an issue breaking docs builds and may hve affected nova-specs?00:39
clarkbmgagne: you might just try recheckign as it was a test node issue00:39
*** zhurong has joined #openstack-nova00:39
mgagnewill recheck right now00:39
*** armax has joined #openstack-nova00:41
*** Apoorva_ has quit IRC00:42
*** Apoorva has joined #openstack-nova00:43
*** thorst has quit IRC00:46
*** coreywright has quit IRC00:50
*** cNilesh has joined #openstack-nova00:54
*** mdrabe has joined #openstack-nova00:56
*** tovin07_ has joined #openstack-nova00:56
mgagneclarkb: no luck after a recheck https://review.openstack.org/#/c/453272/01:00
*** coreywright has joined #openstack-nova01:04
*** XueFeng has joined #openstack-nova01:06
*** crushil has joined #openstack-nova01:07
*** armax has quit IRC01:08
*** Jack_Iv has joined #openstack-nova01:10
*** Jack_Iv has quit IRC01:14
mriedemclarkb: was failing all afternoon01:14
mriedemmaybe a new sphinx release and we're legitimately broken01:15
*** thorst has joined #openstack-nova01:17
*** liverpooler has quit IRC01:19
*** takashin has quit IRC01:19
*** NikhilS has joined #openstack-nova01:20
*** phuongnh has joined #openstack-nova01:23
*** lucasxu has joined #openstack-nova01:24
*** thorst has quit IRC01:24
*** MasterOfBugs has quit IRC01:25
*** pramodrj07 has quit IRC01:25
mriedemhmm, sphinxcontrib-fulltoc 1.2.0 released on 4/1001:28
mriedemwe should be using 1.1 though https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L47901:28
mriedemmaybe pbr 2.1.0?01:30
*** gcb has joined #openstack-nova01:32
*** Oku_OS is now known as Oku_OS-away01:32
*** dixiaoli has joined #openstack-nova01:33
mriedembingo https://github.com/openstack-dev/pbr/commit/2d7c0045f402b31d6223437ba25423eeb32278e101:33
mriedemclarkb: ^01:33
mriedemit's pbr 2.1.001:33
mriedemhttps://bugs.launchpad.net/pbr/+bug/168198301:36
openstackLaunchpad bug 1681983 in PBR "2.1.0: sphinx.errors.SphinxWarning: WARNING: no "man_pages" config value found; no manual pages will be written" [Undecided,New]01:36
*** kaisers_ has joined #openstack-nova01:36
*** lucasxu has quit IRC01:38
mriedemblacklist patch https://review.openstack.org/45586501:39
*** kaisers has quit IRC01:39
*** lucasxu has joined #openstack-nova01:39
clarkbyou cant blacklist pbr01:41
clarkbits installed bfore anyrhing that could blacklist is executed01:41
mriedemshirt01:42
mriedemheh01:42
clarkbit may work in this case because its a post install step though but generally ya01:42
*** kaisers_ has quit IRC01:42
*** lucasxu has quit IRC01:46
*** lucasxu has joined #openstack-nova01:46
*** Sukhdev has quit IRC01:47
mordredclarkb, mriedem: I +A'd the u-c blacklist, but also what clarkb said is correct so I don't expect it'll do much - while the pbr issue gets worked, commenting out the man_pages line in conf.py fixes theissue01:48
*** dillaman has quit IRC01:49
*** nic has quit IRC01:50
*** vladikr has joined #openstack-nova01:59
*** dillaman has joined #openstack-nova02:04
*** jamielennox is now known as jamielennox|away02:07
*** rfolco has quit IRC02:08
*** gongysh has joined #openstack-nova02:09
*** Apoorva_ has joined #openstack-nova02:09
*** fandi has joined #openstack-nova02:10
*** ssurana has quit IRC02:12
*** mdrabe has quit IRC02:12
*** Apoorva has quit IRC02:13
*** Apoorva_ has quit IRC02:14
*** crushil has quit IRC02:20
*** thorst has joined #openstack-nova02:21
*** lucasxu has quit IRC02:22
*** mtanino has joined #openstack-nova02:26
*** crushil has joined #openstack-nova02:29
*** mriedem1 has joined #openstack-nova02:30
*** mriedem has quit IRC02:32
*** mriedem1 is now known as mriedem02:36
*** kaisers has joined #openstack-nova02:37
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Workaround pbr 2.1.0 man_pages bug  https://review.openstack.org/45588502:38
*** thorst has quit IRC02:41
*** liverpooler has joined #openstack-nova02:44
*** gouthamr has quit IRC02:44
*** Kevin_Zheng has joined #openstack-nova02:54
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Deprecate multinic, os-virtual-interfaces, and floating IP action APIs  https://review.openstack.org/38426102:54
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Nova support for attached volume extend  https://review.openstack.org/45327202:54
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Amend spec for placement project/user association  https://review.openstack.org/45435202:55
Kevin_Zhengping mriedm02:58
*** Apoorva has joined #openstack-nova02:58
Kevin_Zhengping mriedem02:58
mriedemKevin_Zheng: hi02:59
Kevin_Zhengmriedem: Hi, about https://review.openstack.org/#/c/455183/2 , I don't quite get your suggestion03:00
mriedemKevin_Zheng: i left a reply, see if that makes sense03:02
Kevin_Zhengmriedem thanks03:02
mriedemif the for loop does not break, then we raise exception.ConsoleNotAvailable()03:02
mriedemif the for loop does break, pty is set03:03
*** nicolasbock has quit IRC03:03
*** udesale has joined #openstack-nova03:05
*** peter-hamilton has quit IRC03:05
*** cNilesh has quit IRC03:05
*** cNilesh has joined #openstack-nova03:07
Kevin_Zhengmriedem: thanks, I just searched it, that makes sense :)03:10
mriedemcool03:11
mriedemalex_xu: gmann: have you ever noticed that the host power actions on the os-hosts API are all GETs?03:13
mriedemGET /os-hosts/{host_name}/reboot03:13
mriedemthat's really odd03:13
gmannmriedem: yea all start etc are GET03:15
*** takashin has joined #openstack-nova03:16
*** lyan has joined #openstack-nova03:17
*** gongysh has quit IRC03:21
*** jamielennox|away is now known as jamielennox03:23
*** Apoorva has quit IRC03:24
openstackgerritZhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance  https://review.openstack.org/45518303:26
*** fragatin_ has joined #openstack-nova03:26
*** kaisers has quit IRC03:27
*** fragatina has quit IRC03:29
*** fragatin_ has quit IRC03:30
mriedemgmann: i also see there are Tempest negative tests that hit the os-hosts enable/disable/reboot/shutdown/start APIs, those should probably all be moved out of tempest and into nova unit tests, if we don't already have that coverage, but I bet we do03:31
mriedemi'll mention that in the spec to deprecate os-hosts03:32
*** gongysh has joined #openstack-nova03:35
gmannmriedem: yes we can do and i think defcore might not be using those as admin APIs and we have positive cases covered in api sample tests and if any negative cases missing in unit tests we can add easily.03:37
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Deprecate the os-hosts API (spec)  https://review.openstack.org/45589603:39
mriedemgmann: alex_xu: ^03:39
*** armax has joined #openstack-nova03:41
*** crushil has quit IRC03:44
*** krtaylor has quit IRC03:45
*** mriedem has quit IRC03:48
*** huanxie has joined #openstack-nova03:50
*** huanxie has joined #openstack-nova03:51
*** baoli has quit IRC03:53
*** dave-mccowan has joined #openstack-nova03:57
*** rcernin has joined #openstack-nova03:59
*** rcernin is now known as rcernin|wfh|mtg03:59
*** armax has quit IRC04:01
*** armax has joined #openstack-nova04:02
*** links has joined #openstack-nova04:03
*** zhurong has quit IRC04:06
*** fragatina has joined #openstack-nova04:10
*** kaisers has joined #openstack-nova04:12
*** ratailor has joined #openstack-nova04:13
*** fragatina has quit IRC04:14
*** kaisers has quit IRC04:15
*** dave-mccowan has quit IRC04:16
*** dave-mccowan has joined #openstack-nova04:17
*** crushil has joined #openstack-nova04:17
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/45596004:17
*** mtanino has quit IRC04:18
*** kaisers has joined #openstack-nova04:18
*** trinaths has joined #openstack-nova04:18
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/45104904:19
*** kaisers has quit IRC04:21
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/45600204:21
*** lyan has quit IRC04:21
EmilienMdansmith: when you have time, any chance to help with this trace? http://logs.openstack.org/15/359215/85/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/0040e01/logs/undercloud/var/log/nova/nova-api.txt.gz#_2017-04-12_04_03_47_78504:29
EmilienMI'm probably wrong but it looks like related to https://review.openstack.org/#/c/442162/04:29
*** huanxie has quit IRC04:30
EmilienMhttps://bugs.launchpad.net/tripleo/+bug/168200104:33
openstackLaunchpad bug 1682001 in tripleo "[promotion] nova fails to start a server - host is not mapped to any cell" [Critical,Triaged]04:33
*** zhurong has joined #openstack-nova04:34
*** psachin has joined #openstack-nova04:36
*** lucasxu has joined #openstack-nova04:36
*** thorst has joined #openstack-nova04:37
*** yamamoto has quit IRC04:38
*** mdnadeem has joined #openstack-nova04:40
*** ishant has quit IRC04:40
*** guchihiro has joined #openstack-nova04:41
*** yamamoto has joined #openstack-nova04:41
*** xinliang has quit IRC04:42
*** thorst has quit IRC04:42
*** fragatin_ has joined #openstack-nova04:44
*** bmace has quit IRC04:45
*** kaisers has joined #openstack-nova04:46
*** crushil has quit IRC04:48
*** dave-mccowan has quit IRC04:50
*** ayogi has joined #openstack-nova04:50
*** kaisers has quit IRC04:51
*** huanxie has joined #openstack-nova04:51
*** kaisers has joined #openstack-nova04:52
*** kaisers has quit IRC04:57
*** adisky_ has joined #openstack-nova04:57
*** crushil has joined #openstack-nova05:01
*** iceyao has joined #openstack-nova05:05
*** sridharg has joined #openstack-nova05:08
*** diga has joined #openstack-nova05:08
*** bmace has joined #openstack-nova05:10
*** Jack_Iv has joined #openstack-nova05:11
*** rcernin|wfh|mtg has quit IRC05:11
openstackgerrit赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram  https://review.openstack.org/45605005:13
*** coreywright has quit IRC05:13
*** jaosorior_away is now known as jaosorior05:13
*** Jack_Iv has quit IRC05:15
*** crushil has quit IRC05:16
*** gjayavelu has joined #openstack-nova05:17
*** prateek has joined #openstack-nova05:18
*** ekuris has joined #openstack-nova05:19
*** Sukhdev has joined #openstack-nova05:20
*** rcernin has joined #openstack-nova05:20
*** prateek is now known as prateek_05:22
*** prateek_ is now known as prateek05:22
*** prateek is now known as prateek_05:23
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action  https://review.openstack.org/45605305:23
*** mdnadeem_ has joined #openstack-nova05:24
*** mdnadeem has quit IRC05:24
*** abhi89 has joined #openstack-nova05:26
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action  https://review.openstack.org/45605305:28
*** MasterOfBugs has joined #openstack-nova05:32
*** pramodrj07 has joined #openstack-nova05:32
*** coreywright has joined #openstack-nova05:33
*** Jack_Iv has joined #openstack-nova05:38
*** thorst has joined #openstack-nova05:39
*** lucasxu has quit IRC05:40
*** thorst has quit IRC05:43
*** markus_z has joined #openstack-nova05:46
*** hshiina has joined #openstack-nova05:46
*** kaisers has joined #openstack-nova05:53
*** Sukhdev has quit IRC05:58
*** claudiub has joined #openstack-nova05:59
*** garyk has joined #openstack-nova06:02
*** Dinesh_Bhor has quit IRC06:04
*** Oku_OS-away is now known as Oku_OS06:05
*** gjayavelu has quit IRC06:08
*** ociuhandu has quit IRC06:08
*** gyee has quit IRC06:08
*** browne has quit IRC06:11
*** diga has quit IRC06:23
*** tbachman has quit IRC06:23
*** voelzmo has joined #openstack-nova06:23
*** hieulq has joined #openstack-nova06:24
*** diga has joined #openstack-nova06:24
*** Jack_Iv has quit IRC06:25
*** fragatin_ has quit IRC06:25
*** voelzmo has quit IRC06:26
*** hieulq has quit IRC06:26
*** voelzmo has joined #openstack-nova06:26
*** pcaruana has joined #openstack-nova06:30
*** huanxie has quit IRC06:32
*** pramodrj07 has quit IRC06:32
*** MasterOfBugs has quit IRC06:32
*** Jack_Iv has joined #openstack-nova06:37
*** Dinesh_Bhor has joined #openstack-nova06:38
*** Jack_Iv has quit IRC06:42
openstackgerritZhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance  https://review.openstack.org/45518306:49
*** belmoreira has joined #openstack-nova06:49
*** tesseract has joined #openstack-nova06:49
*** gjayavelu has joined #openstack-nova06:50
*** huanxie has joined #openstack-nova06:53
*** andreykurilin has quit IRC06:54
*** andreykurilin has joined #openstack-nova06:54
*** Kimmo_ has quit IRC06:54
*** jistr has quit IRC06:54
*** adreznec has quit IRC06:55
*** brad[] has quit IRC06:55
*** luos has quit IRC06:55
*** MarkMielke has quit IRC06:55
*** luos has joined #openstack-nova06:55
*** markus_z has quit IRC06:55
*** BobBall has quit IRC06:55
*** BobBall has joined #openstack-nova06:56
*** corvus has quit IRC06:56
*** jistr has joined #openstack-nova06:56
*** BobBall is now known as Guest2797506:56
*** adreznec has joined #openstack-nova06:56
*** corvus has joined #openstack-nova06:56
*** MarkMielke has joined #openstack-nova06:57
*** john51_ has quit IRC06:57
*** john51 has joined #openstack-nova06:57
*** sridharg has quit IRC06:59
*** rcernin has quit IRC06:59
*** Jack_Iv has joined #openstack-nova07:00
*** markus_z has joined #openstack-nova07:01
*** hieulq has joined #openstack-nova07:01
*** diga has quit IRC07:01
*** Jack_Iv has quit IRC07:01
*** prateek_ has quit IRC07:02
*** prateek has joined #openstack-nova07:02
*** Jack_Iv has joined #openstack-nova07:03
*** Jack_Iv has quit IRC07:03
*** Jack_Iv has joined #openstack-nova07:03
openstackgerritYuYang proposed openstack/nova master: api-ref: Nova Update Compute services Link  https://review.openstack.org/45608507:06
*** karimb has joined #openstack-nova07:14
openstackgerrit赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram  https://review.openstack.org/45605007:14
*** rcernin has joined #openstack-nova07:16
*** kaisers has quit IRC07:17
*** brad[] has joined #openstack-nova07:24
*** fragatina has joined #openstack-nova07:26
*** karimb has quit IRC07:29
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification  https://review.openstack.org/45580107:36
*** rcernin has quit IRC07:37
*** fragatina has quit IRC07:37
*** rcernin_ has joined #openstack-nova07:37
*** rcernin_ has quit IRC07:38
*** rcernin_ has joined #openstack-nova07:38
*** thorst has joined #openstack-nova07:40
openstackgerritGergely Csatari proposed openstack/nova master: api-ref: adminPass is optional in Create Server  https://review.openstack.org/45149707:41
*** tuanluong has joined #openstack-nova07:41
*** thorst has quit IRC07:45
*** abalutoiu_ has quit IRC07:47
*** trinaths has left #openstack-nova07:47
*** garyk has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** karimb has joined #openstack-nova08:01
*** ralonsoh has joined #openstack-nova08:02
*** lucas-afk is now known as lucasagomes08:04
*** sridharg has joined #openstack-nova08:07
*** kaisers has joined #openstack-nova08:13
*** xinliang has joined #openstack-nova08:15
*** guchihiro has quit IRC08:20
*** jpena|off has joined #openstack-nova08:22
*** dmellado has joined #openstack-nova08:23
*** aarefiev_afk is now known as aarefiev08:24
*** gjayavelu has quit IRC08:25
*** Jack_Iv has quit IRC08:26
*** Jack_Iv has joined #openstack-nova08:26
*** huanxie has quit IRC08:28
openstackgerritGergely Csatari proposed openstack/nova master: api-ref: adminPass is optional in Create Server  https://review.openstack.org/45149708:33
*** efoley has joined #openstack-nova08:34
*** links has quit IRC08:35
*** efoley_ has joined #openstack-nova08:36
*** trinaths1 has joined #openstack-nova08:37
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: List/show all server migration types  https://review.openstack.org/40723708:38
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration  https://review.openstack.org/33473208:38
*** efoley has quit IRC08:39
*** thorst has joined #openstack-nova08:41
*** priteau has joined #openstack-nova08:49
*** yonglihe has quit IRC08:49
*** gszasz has joined #openstack-nova08:51
*** yamahata has quit IRC08:51
*** links has joined #openstack-nova08:52
*** Kimmo_ has joined #openstack-nova08:54
*** abalutoiu has joined #openstack-nova08:54
*** ociuhandu has joined #openstack-nova08:55
*** cdent has joined #openstack-nova08:56
Dinesh_Bhoralex_xu: ping08:58
*** thorst has quit IRC09:00
Dinesh_Bhoralaski: Hi, Does this mean that If there are some issues then we are not going to fix them? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/scheduler_hints.py#L67-L7009:03
Dinesh_Bhoralaski: I can see your ML thread: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067996.html09:03
johnthetubaguyDinesh_Bhor: can I help with context?09:09
gibimgagne: I left some comment in the spec https://review.openstack.org/#/c/453272/09:10
johnthetubaguyDinesh_Bhor: we allow custom scheduler filters and weighers, they can have their own hints, if we blocked those in the API, it would break things. Does that help?09:10
openstackgerrit赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram  https://review.openstack.org/45605009:11
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory  https://review.openstack.org/45521509:11
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory  https://review.openstack.org/45521409:11
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory  https://review.openstack.org/45521309:11
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories  https://review.openstack.org/45270809:11
*** lpetrut has joined #openstack-nova09:12
openstackgerritBéla Vancsics proposed openstack/nova master: Transform instance.resize_confirm notification  https://review.openstack.org/39065409:13
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory  https://review.openstack.org/45521509:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory  https://review.openstack.org/45521409:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory  https://review.openstack.org/45521309:16
*** iceyao has quit IRC09:16
alex_xuDinesh_Bhor: johnthetubaguy yea, we relax that strict validation, so it already fixed09:17
johnthetubaguyalex_xu: +1 thats fixed a while back, not sure what the question was really09:18
Dinesh_Bhoralex_xu, johnthetubaguy: ok, understood. One more question if any user tries to pass invalid custom scheduler hint then is it the job of nova to handle it correctly?09:19
openstackgerritStephen Finucane proposed openstack/nova-specs master: Mitigate 'no "man_pages" config value' error  https://review.openstack.org/45612809:19
openstackgerritStephen Finucane proposed openstack/nova-specs master: doc: Remove crud from Sphinx conf.py  https://review.openstack.org/45612909:19
Dinesh_Bhorinvalid means in wrong format. I mean actually I passed '{"os:scheduler_hints": {"key": null}'09:20
*** links has quit IRC09:20
Dinesh_Bhoralex_xu, johnthetubaguy: ^^ which caused ValueError at a-api09:20
alex_xuDinesh_Bhor: it is open for custom scheduler, your custom filter will handle the invalid hint09:20
alex_xuDinesh_Bhor: that is kind of thing should be fixed09:24
alex_xuDinesh_Bhor: we should limit the input for hints is string09:25
*** trinaths1 has left #openstack-nova09:26
Dinesh_Bhoralex_xu: ok, understood, thanks09:27
alex_xujohnthetubaguy Dinesh_Bhor : the requestspec break the API a little, it limits the schedule hints to string09:28
johnthetubaguylimiting to a string seems reasonable09:28
johnthetubaguylonger term, I we need the registration system we spoke about before09:29
alex_xujohnthetubaguy: longer term, the scheduler hints will be removed? I guess placement won't support that09:30
alex_xu?09:30
johnthetubaguyalex_xu: not sure myself, I can see the scheduler bit always living in Nova and having hints09:30
johnthetubaguyalex_xu: I am unclear there, what I mean it, nova still picks the "best" of the possible hosts placement says09:30
alex_xujohnthetubaguy: I guess scheduler bits always living Nova for some filters can be implemented by placement?09:30
alex_xus/can be/can't be/09:31
johnthetubaguyalex_xu: maybe, yeah09:31
johnthetubaguyI guess it needs a better name09:31
johnthetubaguy"preference engine" maybe?09:31
alex_xu:)09:31
*** links has joined #openstack-nova09:33
*** djohnsto has quit IRC09:35
*** egonzalez has joined #openstack-nova09:37
openstackgerritGhanshyam Mann proposed openstack/nova master: Add description to policies in security_groups.py  https://review.openstack.org/45230909:37
openstackgerritJens Rosenboom proposed openstack/nova master: Fix network metadata information - part 2/3  https://review.openstack.org/45021209:38
*** tbachman has joined #openstack-nova09:39
*** kaisers has quit IRC09:50
*** sambetts|afk is now known as sambetts09:52
openstackgerritGhanshyam Mann proposed openstack/nova master: Add description to policies in hypervisors.py  https://review.openstack.org/45129409:53
*** cdent has quit IRC09:55
*** thorst has joined #openstack-nova09:57
*** tovin07_ has quit IRC09:59
*** thorst has quit IRC10:01
*** tbachman has quit IRC10:02
*** nkorabli has joined #openstack-nova10:04
*** nkorabli has quit IRC10:09
*** tbachman has joined #openstack-nova10:13
openstackgerritLee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU  https://review.openstack.org/43707010:13
lyarwoodkashyap: ^ updated, just noticed that I didn't call out the lack of support for network volumes so feel free to -1 if you think that's required10:14
kashyaplyarwood: Will look in a few10:16
*** hferenc has quit IRC10:20
*** sdague has joined #openstack-nova10:22
*** cdent has joined #openstack-nova10:25
*** hshiina has quit IRC10:28
*** nicolasbock has joined #openstack-nova10:33
*** efoley__ has joined #openstack-nova10:35
*** efoley_ has quit IRC10:38
*** tuanluong has quit IRC10:43
*** phuongnh has quit IRC10:43
*** trinaths has joined #openstack-nova10:44
*** kaisers has joined #openstack-nova10:46
*** karimb has quit IRC10:52
*** efoley_ has joined #openstack-nova10:54
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: List/show all server migration types  https://review.openstack.org/40723710:55
*** karimb has joined #openstack-nova10:57
*** efoley__ has quit IRC10:58
openstackgerritIldiko Vancsa proposed openstack/nova master: Remove check_detach  https://review.openstack.org/44667111:00
*** trinaths has left #openstack-nova11:01
*** takashin has left #openstack-nova11:06
*** dixiaoli has quit IRC11:06
*** abhi89 has quit IRC11:08
*** udesale has quit IRC11:13
*** cNilesh has quit IRC11:14
*** rfolco has joined #openstack-nova11:15
*** nkorabli has joined #openstack-nova11:16
*** karimb has quit IRC11:16
*** iceyao has joined #openstack-nova11:20
*** liverpooler has quit IRC11:20
*** zhurong has quit IRC11:26
openstackgerritIldiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls  https://review.openstack.org/43875011:26
openstackgerritAndrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider  https://review.openstack.org/45617611:28
*** iceyao has quit IRC11:30
*** mvk has quit IRC11:31
*** lucasagomes is now known as lucas-hungry11:32
*** hferenc has joined #openstack-nova11:32
*** haplo37_ has quit IRC11:36
*** haplo37_ has joined #openstack-nova11:36
*** thorst has joined #openstack-nova11:41
*** tbachman has quit IRC11:44
*** abhi89 has joined #openstack-nova11:47
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/45596011:49
*** baoli has joined #openstack-nova11:49
*** prateek_ has joined #openstack-nova11:49
*** psachin has quit IRC11:51
*** prateek has quit IRC11:52
*** abhishek has joined #openstack-nova11:52
*** abhi89 has quit IRC11:55
*** dane-fichter has joined #openstack-nova12:01
*** smatzek has joined #openstack-nova12:02
*** dane-fichter has quit IRC12:02
*** mvk has joined #openstack-nova12:03
*** psachin has joined #openstack-nova12:08
*** peter-hamilton has joined #openstack-nova12:08
*** diana_clarke has left #openstack-nova12:09
*** garyk has joined #openstack-nova12:10
*** ecelik has joined #openstack-nova12:10
*** krtaylor has joined #openstack-nova12:11
*** edmondsw has joined #openstack-nova12:13
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925712:13
*** abhishek has quit IRC12:13
*** karimb has joined #openstack-nova12:14
*** abhishek has joined #openstack-nova12:14
*** lucas-hungry is now known as lucasagomes12:14
*** gouthamr has joined #openstack-nova12:14
*** gongysh has quit IRC12:15
*** abhishek has quit IRC12:17
*** gongysh has joined #openstack-nova12:17
*** ecelik has left #openstack-nova12:17
*** armax has quit IRC12:17
*** abhishek has joined #openstack-nova12:17
*** liverpooler has joined #openstack-nova12:19
*** abhishek has quit IRC12:20
*** abalutoiu has quit IRC12:20
*** abhishek has joined #openstack-nova12:20
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Fix placement-api-ref check tool  https://review.openstack.org/45619812:23
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory  https://review.openstack.org/45521512:25
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory  https://review.openstack.org/45521412:25
openstackgerritAndrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider  https://review.openstack.org/45617612:25
*** haplo37 has quit IRC12:25
*** haplo37_ is now known as haplo3712:25
*** mriedem has joined #openstack-nova12:31
*** NikhilS has quit IRC12:32
*** haplo37 has quit IRC12:32
*** winston-d_ has quit IRC12:33
*** kaisers has quit IRC12:33
*** kaisers has joined #openstack-nova12:33
sfinucansdague: When we deprecate an API in a microversion, does there come a point where we can delete all the code behind it?12:34
sfinucanfor example, is the deprecated nova-network API code still in the code base but not exercised?12:35
*** Jack_Iv has quit IRC12:35
sfinucanalex_xu: You might be able to answer that for me too ^ :)12:35
*** Jack_Iv has joined #openstack-nova12:36
mriedemsfinucan: nova-network is still used for cells v112:36
sdaguesfinucan: nova-net is kind of a special case12:36
mriedemsfinucan: also, we deprecate an api at a microversion so things don't work at that microversion or later, but 2.1 still works12:37
*** yasemin has joined #openstack-nova12:37
sfinucanmriedem: Right, but what about for other APIs. Would we ever delete the code behind the APIs you'd like to deprecate this cycle, e.g. os-hosts?12:37
*** markvoelker has joined #openstack-nova12:37
*** ratailor has quit IRC12:38
*** haplo37 has joined #openstack-nova12:38
sfinucanI imagine we can't because then microversion {version before deprecation} wouldn't respond the same way12:38
*** haplo37_ has joined #openstack-nova12:38
sfinucan...but, if so, what's the point of deprecating anything if we can't eventually remove the code12:38
sdagueyou deprecate things as well to signal folks not to use that bit because it's not well maintained12:39
*** baoli has quit IRC12:39
*** Jack_Iv has quit IRC12:40
*** yasemin has left #openstack-nova12:41
sfinucansdague: but crucially we had to keep that code ad infinitum and keep testing it?12:41
*** Jack_Iv has joined #openstack-nova12:43
openstackgerritStephen Finucane proposed openstack/nova master: Fix network metadata information - part 3/3  https://review.openstack.org/45029712:43
*** eharney has joined #openstack-nova12:43
openstackgerritdane-fichter proposed openstack/nova master: Remove backend dependency for key types  https://review.openstack.org/45620412:47
mriedemsfinucan: if we ever start raising required minimums,12:47
mriedemthen we could delete some older code12:48
mriedemsfinucan: ironic wants to do that, and i think it's going to be discussed at the forum12:48
sfinucanmriedem: Ohhh, now that you mention it I do recall that being discussed at...the PTG (?) Thanks for the info :)12:49
*** cdent has quit IRC12:49
*** dane-fichter has joined #openstack-nova12:50
*** hferenc has quit IRC12:50
dane-fichtermriedem: pushed a fix for that cursive bug12:50
dane-fichtermriedem: do we need to do the same for the hash algs too?12:50
sfinucanjohnthetubaguy: As our (my) resident "is reno required" expert - thoughts on https://review.openstack.org/#/c/450211 ?12:51
* johnthetubaguy looks at review12:51
mriedemdane-fichter: comments in https://review.openstack.org/#/c/456204/112:51
johnthetubaguysfinucan: that looks like a breaking API change, so feels like users will need a heads up about that12:52
fricklerjohnthetubaguy: sfinucan: reno for all three patches is in the final one: https://review.openstack.org/#/c/450297/312:52
sfinucanfrickler: Aha - I didn't spot that :) Thanks for the heads up12:53
*** hferenc has joined #openstack-nova12:53
johnthetubaguysfinucan: I find it hard to review anything in the metadata right now, due to the general poor testing around that whole thing12:53
fricklersfinucan: added it for you after your earlier review ;)12:53
sfinucanjohnthetubaguy: Aye, that's on my backlog of things to do this cycle12:53
johnthetubaguysfinucan: that looks like an API change without a spec, the reno covers the details I guess...12:55
fricklersfinucan: not sure the W+1 on the first patch is intentional? doesn't seem to fit with your -1 review12:55
alex_xusfinucan: yea, as mriedem said, we can raise the min version of microversion. but we won't raise the min version easily12:55
sfinucanjohnthetubaguy: Think there's a spec required?12:55
sfinucandammit12:55
alex_xunova api meeting is in 5 mins at #openstack-meeting-412:55
openstackgerritStephen Finucane proposed openstack/nova master: Fix network metadata information - part 1/3  https://review.openstack.org/45021112:55
*** moreless_ has joined #openstack-nova12:55
openstackgerritStephen Finucane proposed openstack/nova master: Fix network metadata information - part 2/3  https://review.openstack.org/45021212:56
johnthetubaguysfinucan: I duno, that would be my default take for an API change, just so we have agreement on the change, we did talk about adding this in a new metadata version only12:56
openstackgerritStephen Finucane proposed openstack/nova master: Fix network metadata information - part 3/3  https://review.openstack.org/45029712:56
mriedemthe release note makes it sound like the changes are additive, but yeah it's hard when we don't test the thing and it's an unversioned json blob12:57
johnthetubaguyfrickler: its a good change, don't get me wrong, just getting my head around protecting the folks that relay on the old behaviour in some way12:59
sfinucanYeah, the changes to the metadata API are all additive (by design, from the looks of things)13:00
*** armax has joined #openstack-nova13:00
sfinucanI don't personally think a spec is necessary for this but I'll leave that to others to decide13:00
moreless_I have some error in my nova log files, I think it is crediterncails problem but I am not sure if it is my username and password or authentication mechanism it says it is using "AMQPLAIN" is it how it should be?13:00
alex_xunova api meeting is running13:00
fricklerwell, the third one isn't, it changes the type ipv6_dhcp so something else, but then nobody IMO can be using it in a reasonable way currently, else the change wouldn't be necessary13:01
mriedemfrickler: i'd probably ask clarkb, mordred, jroll and mgagne to take a look at the changes13:02
*** baoli has joined #openstack-nova13:02
mriedemmgagne was recently updating the nw metadata json in an old spec that introduced it, so i know he cares13:02
mordredfrickler: we definitey care - but we'd LOVE for it to be something sane13:03
*** trinaths has joined #openstack-nova13:03
*** crushil has joined #openstack-nova13:03
*** awaugama has joined #openstack-nova13:03
bhagyashrisHi all, Is there any way to clean up the database table records for eg. I am trying to clean up/remove instances table records (instances entries from instances table) but haven't found any ways, Can any one suggest me how I clean up the records from table?13:03
*** lyan has joined #openstack-nova13:03
mordredmain thing will be ability to detect if the data has new form or old form13:03
fricklermordred: well, the first two patches only add data, if you can live without it, your code just has to ignore it, that should be simple13:04
mordredfrickler: canyou point me at the patches?13:05
*** iceyao has joined #openstack-nova13:05
fricklermordred: for the network type, currently it is always shown as ipv6_dhcp even when it is slaac instead, so you cannot use it for configuration at all13:05
fricklermordred: https://review.openstack.org/#/c/450211/3 and the two on top of it13:05
openstackgerritdane-fichter proposed openstack/nova master: Remove backend dependency for key types  https://review.openstack.org/45620413:06
mordredfrickler: awesome. yes - I think we'd very much like to see this change13:06
openstackgerritJohn Garbutt proposed openstack/nova master: Add description to policies in servers.py  https://review.openstack.org/45196613:06
*** psachin has quit IRC13:06
mriedembhagyashris: nova-manage db archive_deleted_rows13:07
mordredfrickler: you will prevent clarkb from continuing to complain that "ipv6_dhcp doesn't always mean ipv6 using dhcp" :)13:07
mriedembhagyashris: that moves 'deleted' instances into the shadow tables13:07
mriedemso they aren't purged from the database, but they are out of the main query paths13:07
mriedembhagyashris: we don't have a real purge command yet13:07
fricklermordred: yeah, that's the main issue, the others are just random bits I found when looking at the data I got while testing13:08
sdaguefrickler: why did all the new keys pop up in the test here - https://review.openstack.org/#/c/450211/3/nova/tests/unit/network/test_network_info.py ?13:15
sdaguethat's not obvious to me13:15
*** prateek_ has quit IRC13:16
*** dane-fichter has quit IRC13:16
*** ayogi has quit IRC13:17
fricklersdague: as I state in the commit message, the old behaviour is to omit all that information if the type is dhcp. I claim however, that people still may want static configuration on their host even when dhcp is enabled13:17
*** links has quit IRC13:17
fricklersdague: so the patch effectively adds all that information indeed13:17
*** esberglu has joined #openstack-nova13:18
*** xyang1 has joined #openstack-nova13:18
mordredfrickler: amusingly enough - the first two patches make me almost want to change glean to just do static network config if the data is available instead of dhcp - just because I know in openstack land that the ip isn't actually going to _change_ - mainly to simplify the code paths - but my sysadmin background can't quite be on board with that fully for reasons I can't explain13:19
openstackgerritAndy McCrae proposed openstack/nova master: Allow CONTENT_LENGTH to be present but empty  https://review.openstack.org/45571013:19
*** abalutoiu has joined #openstack-nova13:20
mordredsdague: yah - I mean, the way we dhcp in openstack is just a mechanism to get a statically pre-defined ip to a host - but if we also put it in the config-drive then even though it's not how _I_ would have liked to have gotten there, it would wind up being a thing you could count on being available in every cloud regardless of operator deployment decisions13:21
mordredand that's neat, whichever way it goes13:21
*** p01nt3r75 has joined #openstack-nova13:22
fricklermordred: I would be sceptical on always doing that static config, yes. But we do have use cases for it, like an instance connected to two networks both presenting a default route. in order for floating IPs to work, one needs to configure the other interface without the default route13:22
*** erlon has joined #openstack-nova13:23
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/45596013:23
mordredfrickler: wow - that seems like a TERRIBLE scenario! :)13:23
bhagyashrismriedem: ok, Thank you for info.13:23
sdaguefrickler: is there any other concern about other consumers of this data?13:23
mordredsdague: currently the field is meaningless, as it's always set to ipv6_dhcp no matter if that's what's happening or not - so other consumers cannot be making decisions based on it13:24
sdaguemordred: not this field, all the extra stuff going into the dhcp responses13:24
mordredah - gotcha13:24
mordredthat I can't say - I know it won't break glean - but I can't say if someone is doing some sort of strict schema based thing that might throw them if extra fields show up13:25
mordred(I obviously think those folks are doing things wrong, but people do things wrong all the time)13:25
*** fragatina has joined #openstack-nova13:25
sdagueyeh, I'm more concerned that we've now got the 2 clocks problem where there is authoritative source, but we're back channeling it as well13:26
sdaguebecause it does seem like if the answer is use dhcp, then use it13:26
sdaguedon't say use dhcp, and also here is all the info from dhcp in a different channel13:26
*** awaugama has quit IRC13:27
*** awaugama has joined #openstack-nova13:27
*** psachin has joined #openstack-nova13:27
*** catintheroof has joined #openstack-nova13:27
*** dimtruck is now known as zz_dimtruck13:29
*** zz_dimtruck is now known as dimtruck13:29
lyarwoodildikov: are you working on the tests for jgriffith's cinderv3 detach change? https://review.openstack.org/#/c/438750/ ./me is losing track of who is working on this change.13:31
*** mtanino has joined #openstack-nova13:32
*** efoley__ has joined #openstack-nova13:33
fricklersdague: cloud-init should be fine looking at https://git.launchpad.net/cloud-init/tree/cloudinit/sources/helpers/openstack.py#n571 , but I agree we can be discussing that patch seperately from the others, so I'm going to do a bit of reshuffling13:33
sdaguefrickler: ok, cool13:34
sdagueyeh, I want to get the type fixes in there, this is just a bigger issue about expanding the dhcp return13:34
lyarwoodmriedem: as the UCA devstack change looks like it *might* land is there any chance of another review of my QEMU LUKS spec ahead of the deadline tomorrow? https://review.openstack.org/#/c/43707013:34
sdagueso lets get those type fixes first, land them, we can discuss the other thing13:34
mriedemlyarwood: i plan on going through specs today13:34
lyarwoodmriedem: cool thanks13:35
mriedemlyarwood: to be clear, my main concerns in there were the api13:36
sdaguemriedem: speaking of which, do you have final concerns before I pull the trigger on merging the UCA patch and us running libvirt 2.5 by default in the gate?13:36
*** cdent has joined #openstack-nova13:36
*** al498u has joined #openstack-nova13:36
*** efoley_ has quit IRC13:37
*** markvoelker has quit IRC13:37
lyarwoodmriedem: right, I removed the additional encryption provider constant and just left it up to the local nova-compute to decide which approach to use based on the available QEMU and Libvirt versions.13:37
ildikovlyarwood: both of us are looking at it, like who has more bandwidth13:37
*** burt has joined #openstack-nova13:37
*** al498u_ has joined #openstack-nova13:38
*** djohnsto has joined #openstack-nova13:38
ildikovlyarwood: I checked on the patch and fixed the nits, we might move some parts to follow up one if we cannot decide on it quickly13:38
lyarwoodildikov: no problem, if you don't have time just let me know and I can take a look ahead of the meeting tomorrow.13:38
ildikovlyarwood: sure, Tnx13:39
*** markvoelker has joined #openstack-nova13:40
*** al498u has quit IRC13:41
*** tbachman has joined #openstack-nova13:42
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback notification  https://review.openstack.org/40212413:43
*** dimtruck is now known as zz_dimtruck13:43
mriedemsdague: i don't13:43
mriedemi thought it was already done13:43
*** mriedem has quit IRC13:44
*** mriedem has joined #openstack-nova13:44
*** garyk has quit IRC13:45
*** zhurong has joined #openstack-nova13:46
*** hamzy has quit IRC13:47
*** moreless_ has quit IRC13:50
*** amotoki has joined #openstack-nova13:54
*** cNilesh has joined #openstack-nova13:56
*** cdent has quit IRC13:57
*** tongli has joined #openstack-nova13:57
sdagueno, but I just pressed approve13:59
*** cNilesh has quit IRC13:59
sdagueso soon13:59
*** zhurong_ has joined #openstack-nova14:00
fricklermriedem: johnthetubaguy: btw. I think https://review.openstack.org/356335 should have had a reno as well. I spent some time searching to find out why our Newton cluster doesn't create libvirt.xml anymore after an update.14:00
*** cNilesh has joined #openstack-nova14:00
openstackgerritJens Rosenboom proposed openstack/nova master: Update network metadata type field for IPv6  https://review.openstack.org/45029714:00
*** zhurong has quit IRC14:01
mriedemfrickler: i'm assuming that was an unintended side effect14:01
mriedemfrickler: we should definitely reno now though14:01
mriedemfor anyone upgrading14:01
*** cdent has joined #openstack-nova14:01
mriedemfrickler: i see it was also backported to newton https://review.openstack.org/#/c/386676/14:01
mriedemfrickler: would you like to work on a reno?14:01
*** iceyao has quit IRC14:02
fricklermriedem: sure, I can do that. I assume it would be added to master and then backported?14:02
*** zz_dimtruck is now known as dimtruck14:02
johnthetubaguyfrickler: +1 good call out14:02
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925714:03
fricklersdague: clarkb: mordred: so https://review.openstack.org/450297 is now just a single patch fixing the ipv6_dhcp type issue14:03
sdaguefrickler: +214:05
fricklerbtw. can anyone give that EMC ScaleIO CI a kick?14:05
sdaguethe fact that it emails everyone about patches all the time14:06
openstackgerritDan Smith proposed openstack/nova master: Fix hypervisors api missing HostMappingNotFound handlers  https://review.openstack.org/45622714:07
fricklersdague: it seems to do multiple NOT_REGISTERED votes per patch for some time now, yes14:08
*** browne has joined #openstack-nova14:08
*** browne has quit IRC14:08
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Placement doing claims  https://review.openstack.org/43742414:08
*** READ10 has joined #openstack-nova14:08
mriedemfrickler: yeah14:08
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback notification  https://review.openstack.org/40212414:08
mriedememc scaleio is xyang1's team i think14:09
sdaguefrickler: yeh, it's just massively misconfigured I think14:09
mriedemwell,14:09
xyang1mriedem: hi14:09
mriedemit also relies on something that isn't merged yet14:09
mriedemxyang1: can we get the emc scaleio ci in nova just restricted to the one patch that it cares about?14:10
*** lucasxu has joined #openstack-nova14:10
mriedemthis one https://review.openstack.org/#/c/407440/14:10
mriedemxyang1: it shows up as NOT_REGISTERED for all nova patches14:10
*** felipemonteiro has joined #openstack-nova14:10
xyang1mriedem: oh, the nova CI.  I think our CI team is still working on it.  It is not working properly yet14:11
xyang1mriedem: I'll ask them to turn it off for now14:11
mriedemok, but it really won't work on anything except https://review.openstack.org/#/c/407440/ yet,14:11
mriedemand probably not even that given the state it is in14:11
mriedemxyang1: great thanks14:11
sdaguexyang1: they should also configure it to not email14:11
sdagueit's sending emails to all patch submissions14:12
mriedemi didn't notice the email, those are all filtered for me14:12
xyang1mriedem: sdague noted.  Will fix it14:12
sdaguexyang1: thank you14:12
xyang1sdague: np14:12
sfinucanfrickler: I assume the other two patches can be abandoned now?14:14
*** david-lyle has quit IRC14:15
fricklersfinucan: I think I still want to move on with them, but I'll wait with the rebase until the other is finished, so i'll -W for now14:16
*** udesale has joined #openstack-nova14:16
sfinucan(y)14:16
*** cNilesh has quit IRC14:17
*** phuongnh has joined #openstack-nova14:18
*** kaisers has quit IRC14:20
*** hamzy has joined #openstack-nova14:21
*** gjayavelu has joined #openstack-nova14:22
*** satyar has joined #openstack-nova14:23
*** garyk has joined #openstack-nova14:24
*** nic has joined #openstack-nova14:26
nkorabliHi there! I am working on this bug: https://bugs.launchpad.net/nova/+bug/167579114:26
openstackLaunchpad bug 1675791 in OpenStack Compute (nova) "Instance created by demo user(non-admin), shelved by admin and unshelved by demo user --> ends up in error state " [Medium,In progress] - Assigned to Nikolai Korablin (nkorabli)14:26
mriedemnkorabli: so you want to write a functional test to recreate https://bugs.launchpad.net/nova/+bug/1675791 ?14:26
mriedemnkorabli: you should be able to do that in the functional test tree, we have an admin and non-admin user14:27
nkorabliAlright! But, as i remember - you said it is better to fix the bug and then to write a test?14:28
*** nic has quit IRC14:28
mriedemnkorabli: well, you could write the test that shows the bug first14:28
mriedemsee nova/tests/functional/regressions14:28
*** armax has quit IRC14:28
*** ekuris has quit IRC14:28
mriedemalthough i'm not totally sure if our glance fixture would handle this14:28
mriedemthe case where it fails is the admin shelves the instance so they own the snapshot right? and the user tries to unshelve the instance but glance fails since they don't have visibility to the image14:29
mriedemit's a 404 or something14:29
*** nic has joined #openstack-nova14:29
mriedemwe use a glance fixture for this in nova functional tests, and i very much doubt it tracks ownership for an image14:29
nkorabliCorrect14:29
mriedemnot that we couldn't update the fixture14:30
mriedemyou could also do this with a tempest test14:30
mriedemtempest is probably easier14:30
nkorabliSo, fixture is kind of abstraction, that not reproduces 100% functionality?14:31
mriedemcorrect14:31
mriedemnova functional tests don't talk to a real live glance service14:31
mriedemthe fixture is basically a stub14:31
mriedemit has some state and logic, but it's not th real thing14:31
nkorabliI see!14:32
*** links has joined #openstack-nova14:33
mriedemnkorabli: if i remember correctly you were going to ask about this scenario in the mailing list too right?14:33
mriedemdid that happen?14:33
nkorabliJust wondering how complicated is it to add this kind of functionality that I need? Is it possible to make in couple of days? Maybe it won't worth it if it's more than that14:34
mriedemit would be relatively easy to write a test for the recreate in tempest14:34
mriedemmore so than nova functional tests14:34
mriedemsee http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/servers/test_server_actions.py#n50414:34
mriedemfor an example14:35
mriedemyou could have to write a test mostly like that, except the shelve happens by the admin client14:35
mriedems/could/would/14:35
nkorablimriedem: Yes, I believe I raised a mailing thread, but nobody responded14:35
mriedemnkorabli: link?14:35
mriedemnkorabli: here is a tempest test that uses both admin and non-admin clients http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_servers.py#n3014:36
mriedemi think the test just needs to define these credentials http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n46714:36
mriedemi could help with the tempest test if you needed14:36
*** links has quit IRC14:36
mriedemor just do the test in http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_servers.py14:37
mriedemthe thing about tempest is it's branchless, so even if we fix the bug on master, the fix has to be backported to the stable branches for the tempest change to ever merge14:37
mriedemunless we make that configurable in tempest14:38
mriedemsorry this is probably a ton of information14:38
nkorablihaha, yeaaaah :) Guess I need some time to get it.. But I really want to go through this14:39
*** efoley_ has joined #openstack-nova14:39
openstackgerritGábor Antal proposed openstack/nova master: Fix the FakeDriver using same hypervisor names  https://review.openstack.org/45623714:39
*** eharney has quit IRC14:40
*** efoley__ has quit IRC14:42
*** liusheng has quit IRC14:42
*** yamahata has joined #openstack-nova14:45
*** eharney has joined #openstack-nova14:45
openstackgerritGábor Antal proposed openstack/nova master: [WIP] Transform instance.live_migration_rollback notification  https://review.openstack.org/40212414:48
*** hongbin has joined #openstack-nova14:50
mriedemmelwitt: a few comments/questions in https://review.openstack.org/#/c/454352/ but overall it looks good14:51
melwittmriedem: thanks, will reply to the outstanding comments soon14:53
*** nic has quit IRC14:53
*** nic1 has joined #openstack-nova14:53
mriedemnkorabli: i don't see anything in the ML about that bug14:55
mriedemare you sure you sent it?14:55
nkorablimriedem: I saved our conversation, and going through the links you sent me. I will ping you after, most likely I will have more questions14:56
nkorablihmm, I am pretty sure I did, but I haven't joined the mailing list before I did it14:56
*** zhurong_ has quit IRC14:57
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Proposed block accepting AZ renames  https://review.openstack.org/44644614:57
*** annegentle has joined #openstack-nova14:57
*** lucasxu has quit IRC14:57
nkorablimriedem: this was a topic: "[openstack-dev] [openstack-operators] [nova] User that shelves an instance becomes an owner of snapshot for that instance"14:57
*** mkoderer has joined #openstack-nova14:58
*** armax has joined #openstack-nova14:59
nkorabliand I sent it to "openstack-dev@lists.openstack.org, openstack-operators@lists.openstack.org" on the 25th of March15:00
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925715:00
*** marst has joined #openstack-nova15:01
*** Jack_Iv has quit IRC15:02
*** jaosorior is now known as jaosorior_away15:02
*** gszasz_ has joined #openstack-nova15:03
*** gszasz has quit IRC15:03
mriedemnkorabli: you have to join the mailing list before you send something to it15:03
mriedemit was probably rejected15:03
nkorabliohh, I see. I will do it again, then15:04
*** rcernin_ has quit IRC15:04
*** voelzmo has quit IRC15:05
*** gszasz_ has quit IRC15:07
*** gszasz has joined #openstack-nova15:09
*** Jack_Iv has joined #openstack-nova15:09
*** gszasz has quit IRC15:09
*** Apoorva has joined #openstack-nova15:10
*** awaugama has quit IRC15:11
*** sridharg has quit IRC15:12
*** nkorabli has quit IRC15:12
*** nkorabli has joined #openstack-nova15:13
*** Apoorva has quit IRC15:13
*** Jack_Iv has quit IRC15:14
*** lucasxu has joined #openstack-nova15:18
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification  https://review.openstack.org/45580115:22
*** karimb has quit IRC15:24
*** fragatina has quit IRC15:27
*** Sukhdev has joined #openstack-nova15:28
*** baoli has quit IRC15:28
*** baoli has joined #openstack-nova15:29
*** mdrabe has joined #openstack-nova15:31
*** belmoreira has quit IRC15:32
*** mlavalle has joined #openstack-nova15:35
mriedemlyarwood: comments in https://review.openstack.org/#/c/437070/ - mostly questions about the upgrade impact15:38
*** mdnadeem_ has quit IRC15:38
openstackgerritJianghua Wang proposed openstack/nova-specs master: Support VGPU  https://review.openstack.org/45012215:39
*** trinaths has left #openstack-nova15:40
mriedemlyarwood: i'm also not sure how you're going to implement a libvirt-specific encryption provider that doesn't live in nova/volume/encryptors, but i think we could make that clean with a stevedore extension15:40
openstackgerritJianghua Wang proposed openstack/nova-specs master: Support VGPU  https://review.openstack.org/45012215:41
mriedemlike we could have 3 encryptors registered, and each encryptor defines a type (so we'd have 1 cryptsetup and 2 luks registered), a weight, and an is_supported() or is_available() method15:41
*** jianghuaw has joined #openstack-nova15:42
mriedemso for this case, we filter down to the 2 luks encryptors, then filter based on is_available() - the cryptsetup one we have always returns True, but the qemu one has to check versions. then depending on the final list, we sort them based on weight15:42
mriedemso the qemu one is always weighed higher than the cryptsetup luks one, if we have to pick15:42
*** Jack_Iv has joined #openstack-nova15:42
mriedemwe don't need those implementation details in the spec though, i just got thinking about it15:43
*** chyka has joined #openstack-nova15:48
lyarwoodmriedem: sorry was afk, reading now.15:51
lyarwoodmriedem: upgrade impact is something I missed out, I'll get something in there for that now.15:52
*** abhishek has quit IRC15:52
lyarwoodmriedem: re stevedore, I'm not sure how that would work tbh, for this to work with libvirt/qemu I assumed we would need to check ahead of connect_volume to know which approach we are going to use.15:54
lyarwoodmriedem: that's well ahead of the encryptor.attach_volume calls we currently have15:54
lyarwoodmriedem: so unless we rework that flow so we are calling the encryptors before connecting the volume I don't see how that's going to work15:55
lyarwoodmriedem: but I'm likely missing something here15:55
*** Sukhdev_ has joined #openstack-nova15:55
*** artom has quit IRC15:56
*** artom has joined #openstack-nova15:56
openstackgerritSimon Dodsley proposed openstack/nova-specs master: Enhanced KVM Storage QoS  https://review.openstack.org/34016815:57
*** tesseract has quit IRC15:58
*** artom has quit IRC15:58
*** artom has joined #openstack-nova15:58
*** kfarr has joined #openstack-nova15:59
*** ralonsoh_ has joined #openstack-nova15:59
*** ralonsoh has quit IRC16:01
mriedemi probably need to go through and look at the flow too16:01
*** mlakat has quit IRC16:03
*** marst_ has joined #openstack-nova16:03
mriedemlyarwood: oh i see, in attach_volume in the driver we call connect_volume before encryptor.attach_volume16:03
*** dharinic has quit IRC16:03
*** marst has quit IRC16:03
mriedemand libvirt driver has it's own _get_volume_encryptor16:03
*** nic1 has quit IRC16:03
mriedemso we could sneak it in there i guess16:03
*** hieulq_ has joined #openstack-nova16:04
mriedemif type == luks and native_luks_available: use the new hotness16:04
mriedemelse call encryptors.get_volume_encryptor as before16:04
lyarwoodyup that's what I had in mind16:04
mriedemok nvm then16:05
openstackgerritStephen Finucane proposed openstack/os-vif master: doc: Add glossary  https://review.openstack.org/45266616:05
mriedemi don't know how we track the upgrade impact for the old style detach16:05
mriedemcan we determine that by looking at the guest config?16:06
*** aarefiev is now known as aarefiev_afk16:06
lyarwoodyeah we should be able to by looking at the disk xml / config object16:06
mriedemcould this should work nicely then16:07
mriedems/could/cool/16:07
*** baoli has quit IRC16:07
*** gjayavelu has quit IRC16:10
*** mnestratov has quit IRC16:13
*** Kevin_Zheng has quit IRC16:13
*** cdent has quit IRC16:14
*** cdent has joined #openstack-nova16:14
openstackgerritSimon Dodsley proposed openstack/nova-specs master: Enhanced KVM Storage QoS  https://review.openstack.org/34016816:15
*** baoli has joined #openstack-nova16:16
mriedemsfinucan: it's odd that 'default_schedule_zone' lives in CONF.compute16:17
mriedemwell, i guess it's in [DEFAULT] right now16:17
mriedembut it's looked up in the api16:17
mriedembut the option is defined in nova/conf/compute.py; seems that should live in nova/conf/availability_zone.py16:17
*** mvk has quit IRC16:18
mriedemi believe i'll move it16:18
*** hieulq_ has quit IRC16:19
mriedemalso, that option is super confusing when we also have default_availability_zone16:19
*** adisky_ has quit IRC16:19
*** READ10 has quit IRC16:19
*** trinaths1 has joined #openstack-nova16:22
*** cdent has quit IRC16:23
openstackgerritMaciej Szankin proposed openstack/nova-specs master: Add API impact to remove-nova-cert spec  https://review.openstack.org/45628416:23
*** baoli has quit IRC16:24
*** dtp has joined #openstack-nova16:24
macszmriedem: i added the API impact for os-cloudpipe in https://review.openstack.org/#/c/456284/16:24
mriedemmacsz: thanks +W16:25
*** dharinic has joined #openstack-nova16:27
*** efried has quit IRC16:28
*** gongysh has quit IRC16:29
*** baoli has joined #openstack-nova16:31
*** Sukhdev_ has quit IRC16:31
*** karimb has joined #openstack-nova16:32
openstackgerritJon Bernard proposed openstack/nova master: Add tempest-dsvm-ceph-rc  https://review.openstack.org/45629216:32
openstackgerritmelanie witt proposed openstack/nova-specs master: Amend spec for placement project/user association  https://review.openstack.org/45435216:33
*** p01nt3r75 has quit IRC16:34
*** lucasagomes is now known as lucas-afk16:36
*** lpetrut has quit IRC16:38
*** jianghuaw_ has joined #openstack-nova16:40
*** david-lyle has joined #openstack-nova16:45
openstackgerritDan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety  https://review.openstack.org/44317416:45
openstackgerritDan Smith proposed openstack/nova master: Clean up ClientRouter debt  https://review.openstack.org/44448716:45
openstackgerritDan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall  https://review.openstack.org/44273616:45
mriedemdansmith: you might want to get this spec update for melwitt https://review.openstack.org/#/c/454352/16:46
*** gyee has joined #openstack-nova16:46
*** dillaman has quit IRC16:47
*** ociuhandu has quit IRC16:47
*** psachin has quit IRC16:49
*** nic has joined #openstack-nova16:51
*** garyk has quit IRC16:51
*** tongli has quit IRC16:52
openstackgerritJianghua Wang proposed openstack/nova-specs master: Support VGPU  https://review.openstack.org/45012216:54
*** fandi has quit IRC16:54
*** mtanino has quit IRC16:57
dansmithmelwitt: mriedem: am I missing where we say what we do if they use an old microversion and don't provide the user/project?16:57
mriedemdansmith: that's the online data migration hand wave16:58
dansmithno16:59
melwittdansmith: no, I didn't mention anything specifically about that. I guess I assumed it would just do what it was doing before. which means there we would have to keep two schema versions, one for the new one16:59
dansmithI mean what happens if I call the api with microversion 1.0 and don't provide it16:59
dansmithokay16:59
dansmithI guess in that case we just can't query those things by project, but it's not a huge problem16:59
mriedemright PUT /allocations/consumer_id at 1.0 doesn't require project_id/user_id16:59
mriedemright16:59
dansmithcells meeting now, but will continue when we're done16:59
mriedemwe will already have existing allocations that don't have the consumers table populated17:00
*** vks1 has joined #openstack-nova17:00
melwittnot after the migration though17:00
jianghuaw_sfinucan, vladikr, jaypipes: Would you please help to have a look at this review: https://review.openstack.org/45012217:00
jianghuaw_thanks.17:00
*** dillaman has joined #openstack-nova17:00
*** fragatina has joined #openstack-nova17:00
*** markus_z has quit IRC17:01
vladikrjianghuaw, sure, thanks17:01
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/45596017:03
*** burt1 has joined #openstack-nova17:03
*** burt has quit IRC17:05
*** psachin has joined #openstack-nova17:06
*** fragatina has quit IRC17:07
*** psachin_ has joined #openstack-nova17:11
dansmithmriedem: melwitt: so I guess I was just thinking about what happens if we continue to accept calls without the tenant info and keep putting things into the db without them17:12
dansmithI guess it's just tough noogies for the quota stuff but otherwise not a huge deal17:12
melwittdansmith: oh, hm. I didn't think of that17:13
openstackgerritJianghua Wang proposed openstack/nova-specs master: Support VGPU  https://review.openstack.org/45012217:14
*** psachin_ has quit IRC17:14
melwittdansmith: since we are the ones putting allocations (resource tracker), old computes would be putting allocations without project/user. and that would have happened after the online data migration? or do we have the operator run that after the upgrade is completed17:15
mriedemdansmith: yeah we can't really do anything about it17:15
mriedemmelwitt: the old computes can still be posting allocations w/o project/user17:15
mriedemso,17:15
melwittyeah, I know, but I mean after all computes are upgraded, is that when the online data migration via nova-manage is supposed to be run?17:16
mriedemwe could pin the counting quotas stuff on compute level, so we can't rely on placement until all computes are upgraded17:16
*** peter-hamilton has quit IRC17:16
melwittbecause that would clean it up17:16
mriedemthe online data migration can be run whenever i think17:16
mriedemit's idempotent17:16
mriedemthis might also be something we need to check in nova-status17:16
melwittyeah, true can't rely on placement until all computes upgraded17:16
*** psachin_ has joined #openstack-nova17:19
*** jianghuaw_ has quit IRC17:20
melwittmriedem: can we do something to check computes version to have a conditional in the code on when to start calling placement maybe?17:21
*** psachin_ has quit IRC17:21
dansmiththe online migration doesn't really need to happen if the compute side will heal like it does today17:21
melwittis that what you meant by pinning it?17:22
*** psachin_ has joined #openstack-nova17:22
*** psachin_ has quit IRC17:23
melwittdansmith: that's an interesting point. the RT is constantly doing a "update allocations" and the upgraded computes would update them with the project/user17:23
melwittso the only allocations that would get missed are ones with a down compute or some such thing17:24
dansmithright17:24
*** garyk has joined #openstack-nova17:24
dansmithand I would think that we could just do away with the migration entirely,17:24
dansmithespecially since that's kindof a placement-side thing17:24
dansmithand being in nova is kinda icky from a separation point of view17:24
melwittyeah ... just thinking, if a compute is down, that means its allocations don't get included in the person's usage. which seems right on the surface, but if it's a temporary thing then when the compute comes back up, they could go over quota17:26
dansmithmelwitt: just like losing a cell, but it's only a window where it matters,17:26
dansmithand if they're over quota they're over quota17:26
*** fragatina has joined #openstack-nova17:26
dansmithwould be an easy reno I think17:26
*** efoley_ has quit IRC17:26
melwittdansmith: what about what I mentioned earlier, do you think there's a way we could do an if/else to decide whether to call to placement for usages based on whether the service version is new enough or something?17:28
dansmithmelwitt: well, kinda have to have some gate for that anyway, even if we had the online migration, since that runs late17:28
dansmithit's unfortunate to have counting code we only use in that window, but.. otherwise we have to wait a cycle, so probably worth it17:28
*** egonzalez has quit IRC17:29
*** peter-hamilton has joined #openstack-nova17:29
melwittyeah17:30
*** udesale has quit IRC17:30
mriedemi'm wholly dissatisfied with the new bed set that just showed up17:32
mriedemle sigh17:32
mriedemmelwitt: yeah i meant a conditional, like we have in the filter scheduler in ocata17:33
*** psachin has quit IRC17:34
*** jdillaman has quit IRC17:35
*** dillaman has quit IRC17:35
*** fragatina has quit IRC17:36
melwittmriedem: k, I'll update the spec. looks like I would have benefited from an "Upgrade impact" template section but maybe I still wouldn't have though of these :P17:36
*** ralonsoh_ has quit IRC17:36
*** fragatina has joined #openstack-nova17:36
mriedemmelwitt: we could/should add one to the template17:36
*** sambetts is now known as sambetts|afk17:36
mriedemto think about stuff like this17:36
mriedemyou can totally take the lead on adding that :)17:37
melwittyeah. can't hurt, could possibly help17:37
melwittyes, I can do that :)17:37
*** esberglu has quit IRC17:42
melwittmriedem: do you have an opinion on whether or not to have the online migration? dansmith pointed out that new computes will heal existing allocation records. the only allocations that wouldn't get updated are ones for say, a down compute17:44
*** awaugama has joined #openstack-nova17:45
*** Sukhdev has quit IRC17:46
*** jdillaman has joined #openstack-nova17:47
*** dillaman has joined #openstack-nova17:49
*** ociuhandu has joined #openstack-nova17:50
*** karimb has quit IRC17:55
*** crushil has quit IRC17:55
*** esberglu has joined #openstack-nova17:56
*** karimb has joined #openstack-nova17:56
*** esberglu has quit IRC17:58
*** esberglu has joined #openstack-nova17:58
*** annegentle has quit IRC18:01
*** annegentle has joined #openstack-nova18:02
*** Apoorva has joined #openstack-nova18:02
*** crushil has joined #openstack-nova18:02
*** Apoorva has quit IRC18:02
*** Apoorva has joined #openstack-nova18:03
*** baoli has quit IRC18:03
mriedemreading back18:03
*** baoli has joined #openstack-nova18:04
mriedemyeah i think we could drop the online migration if it's self healing18:04
mriedemold computes won't self heal for this, but we won't use it if we have old computes18:04
mriedemso that problem solves itself18:04
mriedemi think18:04
*** krtaylor has quit IRC18:05
mriedemmelwitt: dansmith: ^18:05
* mriedem gets lunch18:05
melwittthanks mriedem18:05
*** ociuhandu has quit IRC18:05
*** ociuhandu has joined #openstack-nova18:05
*** MasterOfBugs has joined #openstack-nova18:06
*** pramodrj07 has joined #openstack-nova18:06
*** pramodrj07 has quit IRC18:06
*** MasterOfBugs has quit IRC18:06
*** MasterOfBugs has joined #openstack-nova18:06
*** pramodrj07 has joined #openstack-nova18:06
*** phuongnh has quit IRC18:07
*** gjayavelu has joined #openstack-nova18:08
*** dharinic has quit IRC18:10
*** ociuhandu has quit IRC18:17
*** ociuhandu has joined #openstack-nova18:18
*** Jack_Iv has quit IRC18:20
*** Jack_Iv has joined #openstack-nova18:21
*** satyar has quit IRC18:23
*** tbachman has quit IRC18:26
*** vks1 has quit IRC18:29
*** fragatina has quit IRC18:29
*** fragatina has joined #openstack-nova18:29
*** ekuris has joined #openstack-nova18:32
*** peter-hamilton has quit IRC18:42
*** karimb has quit IRC18:42
*** cdent has joined #openstack-nova18:42
*** whenry has joined #openstack-nova18:43
*** Sukhdev has joined #openstack-nova18:44
*** vks1 has joined #openstack-nova18:44
*** vks1 has quit IRC18:49
*** krtaylor has joined #openstack-nova18:49
*** avolkov has quit IRC18:52
*** baoli has quit IRC18:54
*** baoli has joined #openstack-nova18:54
*** Sukhdev_ has joined #openstack-nova18:56
openstackgerritLee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU  https://review.openstack.org/43707018:57
*** dharinic has joined #openstack-nova18:58
*** abalutoiu has quit IRC18:58
*** Sukhdev_ has quit IRC18:59
*** Sukhdev_ has joined #openstack-nova19:00
*** iceyao has joined #openstack-nova19:01
*** mtanino has joined #openstack-nova19:02
*** tbachman has joined #openstack-nova19:03
mriedemdansmith: ^ lgtm now19:04
mriedempretty straightforward19:04
*** vks1 has joined #openstack-nova19:04
mriedemi'm a gonna run to home depot to get some plywood, bbiab19:05
dansmithokay19:05
*** iceyao has quit IRC19:06
*** mtanino has quit IRC19:09
*** Sukhdev_ has quit IRC19:09
*** voelzmo has joined #openstack-nova19:12
*** Jack_Iv has quit IRC19:16
*** lucasxu has quit IRC19:18
*** efried has joined #openstack-nova19:18
*** lucasxu has joined #openstack-nova19:20
*** baoli has quit IRC19:23
*** baoli has joined #openstack-nova19:23
*** tbachman has quit IRC19:24
*** Sukhdev_ has joined #openstack-nova19:25
*** Sukhdev_ has quit IRC19:26
openstackgerritMathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend  https://review.openstack.org/45327219:27
*** edmondsw_ has joined #openstack-nova19:34
*** trinaths1 has quit IRC19:35
*** karimb has joined #openstack-nova19:35
openstackgerritMathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend  https://review.openstack.org/45327219:37
*** edmondsw_ has quit IRC19:40
*** rajinir has joined #openstack-nova19:53
rajinirhttps://bugs.launchpad.net/nova/+bug/1682222 is causing the ironic CIs to fail19:53
openstackLaunchpad bug 1682222 in OpenStack Compute (nova) "Instance deployment failure due to neutron syntax error" [Undecided,New]19:53
*** tbachman has joined #openstack-nova19:53
*** artom has quit IRC19:54
*** artom has joined #openstack-nova19:54
openstackgerritMario Villaplana proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648719:55
*** artom has quit IRC19:56
*** artom has joined #openstack-nova19:56
*** smatzek has quit IRC20:02
*** artom has quit IRC20:12
*** artom has joined #openstack-nova20:12
*** artom has quit IRC20:14
*** nkorabli has quit IRC20:14
*** nkorabli_ has joined #openstack-nova20:14
*** artom has joined #openstack-nova20:14
*** vks1 has quit IRC20:18
*** mnestratov has joined #openstack-nova20:18
*** jose-phillips has joined #openstack-nova20:19
sdagueanyone else want to +A to fix the dhcp_v6 lie? https://review.openstack.org/#/c/450297/ :)20:19
bauzasin case people wanna bite a spec proposal :) https://review.openstack.org/#/c/446446/20:19
bauzasoh oops PS5 is already commented :)20:20
*** vks1 has joined #openstack-nova20:20
*** dharinic has quit IRC20:26
*** mnestratov has quit IRC20:27
*** vks1 has quit IRC20:29
*** voelzmo has quit IRC20:34
*** dimtruck is now known as zz_dimtruck20:36
*** karimb has quit IRC20:38
openstackgerritJan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support  https://review.openstack.org/45638620:44
*** garyk has quit IRC20:46
*** marst_ has quit IRC20:48
*** marst_ has joined #openstack-nova20:49
*** smatzek has joined #openstack-nova20:49
*** karimb has joined #openstack-nova20:52
*** liverpooler has quit IRC20:53
*** Sukhdev has quit IRC20:56
openstackgerritPeter Hamilton proposed openstack/nova-specs master: Add support for certificate validation  https://review.openstack.org/35715120:56
*** rdo has quit IRC20:59
openstackgerritKaitlin Farr proposed openstack/nova master: Remove backend dependency for key types  https://review.openstack.org/45620420:59
openstackgerritJan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support  https://review.openstack.org/45638621:00
*** crushil has quit IRC21:00
*** rdo has joined #openstack-nova21:01
*** thorst has quit IRC21:01
*** ociuhandu has quit IRC21:01
*** thorst has joined #openstack-nova21:02
*** dharinic has joined #openstack-nova21:04
*** eharney has quit IRC21:05
*** thorst has quit IRC21:06
*** sdague has quit IRC21:07
*** zz_dimtruck is now known as dimtruck21:08
*** smatzek has quit IRC21:08
*** lucasxu has quit IRC21:09
*** Apoorva_ has joined #openstack-nova21:09
*** MarkMielke has quit IRC21:09
*** MarkMielke has joined #openstack-nova21:09
*** peter-hamilton has joined #openstack-nova21:10
*** tbachman has quit IRC21:10
*** Apoorva has quit IRC21:13
*** purplerbot has quit IRC21:13
*** purplerbot has joined #openstack-nova21:13
*** lucasxu has joined #openstack-nova21:13
*** Sukhdev has joined #openstack-nova21:14
openstackgerritJan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support  https://review.openstack.org/45638621:15
*** Jack_Iv has joined #openstack-nova21:16
*** armax has quit IRC21:17
*** eharney has joined #openstack-nova21:18
*** rajathagasthya has joined #openstack-nova21:18
openstackgerritSteven Webster proposed openstack/nova master: Race condition between audit and migrate/resize revert  https://review.openstack.org/40099521:19
*** gouthamr has quit IRC21:21
*** Jack_Iv has quit IRC21:21
rajathagasthyaHello! I just ran a bandit check on nova master and I see a couple of high severity jinja2 autoescape warning: http://paste.openstack.org/show/606352/21:23
rajathagasthyaWould there be any objection to submitting a patch setting autoescapt=True?21:23
openstackgerritSteven Webster proposed openstack/nova master: Fix mitaka online migration for PCI devices  https://review.openstack.org/45639721:23
*** crushil has joined #openstack-nova21:24
-openstackstatus- NOTICE: Restarting Gerrit for our weekly memory leak cleanup.21:26
*** karimb has quit IRC21:28
*** esberglu has quit IRC21:28
*** esberglu has joined #openstack-nova21:29
*** jangutter has joined #openstack-nova21:29
*** xinliang has quit IRC21:29
*** kfarr has quit IRC21:30
*** lucasxu has quit IRC21:31
*** krtaylor has quit IRC21:32
jangutterHi! I'm the person who submitted the netronome-smartnic-enablement spec. Would anyone perhaps have spare cycles to review?21:33
*** edmondsw has quit IRC21:33
*** esberglu has quit IRC21:33
*** edmondsw has joined #openstack-nova21:34
*** cdent has quit IRC21:37
*** crushil has quit IRC21:37
*** marst_ has quit IRC21:38
*** edmondsw has quit IRC21:38
*** fragatina has quit IRC21:40
*** fragatina has joined #openstack-nova21:40
*** Qiming has quit IRC21:41
*** ociuhandu has joined #openstack-nova21:41
*** esberglu has joined #openstack-nova21:42
*** priteau has quit IRC21:42
*** xinliang has joined #openstack-nova21:42
*** xinliang has joined #openstack-nova21:42
*** mnestratov has joined #openstack-nova21:43
*** marst_ has joined #openstack-nova21:44
*** john51 has quit IRC21:45
*** john51 has joined #openstack-nova21:45
*** awaugama has quit IRC21:45
*** Qiming has joined #openstack-nova21:46
*** esberglu has quit IRC21:46
*** john51 has quit IRC21:47
*** annegentle has quit IRC21:48
*** baoli has quit IRC21:50
*** john51 has joined #openstack-nova21:50
*** pcaruana has quit IRC21:51
mriedemwhere is the admin_api policy rule defined, because i don't see it in nova's policy.json and i don't see it in keystone21:51
mriedemdansmith: ew see this? https://review.openstack.org/#/c/456397/21:52
dansmithmriedem: well, I guess we now know how many people are(n't) using those kinds of devices21:56
efriedjangutter Reviewed, for what it's worth :)21:56
mriedem1s of 2s of users21:56
jangutterThanks!21:57
*** efried has quit IRC21:58
openstackgerritmelanie witt proposed openstack/nova-specs master: Amend spec for placement project/user association  https://review.openstack.org/45435221:58
*** haplo37 has quit IRC21:59
*** takashin has joined #openstack-nova22:06
*** MasterOfBugs has quit IRC22:06
*** pramodrj07 has quit IRC22:06
*** pramodrj07 has joined #openstack-nova22:06
*** MasterOfBugs has joined #openstack-nova22:06
*** Apoorva_ has quit IRC22:07
*** Apoorva has joined #openstack-nova22:07
*** mdrabe has quit IRC22:07
*** haplo37 has joined #openstack-nova22:08
*** edmondsw has joined #openstack-nova22:10
*** lyan has quit IRC22:10
*** mlavalle has quit IRC22:14
openstackgerritJan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support  https://review.openstack.org/45638622:14
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: List/show all server migration types  https://review.openstack.org/40723722:14
*** edmondsw has quit IRC22:14
*** jangutter has quit IRC22:17
*** eharney has quit IRC22:18
*** fragatin_ has joined #openstack-nova22:18
*** felipemonteiro has quit IRC22:19
*** tbachman has joined #openstack-nova22:20
*** abalutoiu has joined #openstack-nova22:20
*** esberglu has joined #openstack-nova22:21
*** fragatina has quit IRC22:21
*** fragatin_ has quit IRC22:22
*** fragatina has joined #openstack-nova22:22
*** ianw is now known as ianw_pto22:26
*** suresh has joined #openstack-nova22:34
*** lyan has joined #openstack-nova22:37
sureshhi, I am not able to install nova-api and nova-placement-api together in ubuntu, due to conflict of packages........is it a bug or can someone help me with it.....please22:39
mriedemdansmith: claims in conductor was approved http://forumtopics.openstack.org/cfp/details/6322:40
mriedemeven though it should have probably been renamed now :)22:40
dansmithcool22:40
dansmithmeh, we'll surprise people22:40
mriedemsuresh: check https://docs.openstack.org/ocata/install-guide-ubuntu/22:40
mriedemthe docs team hit some package issues when writing the install guide22:40
mriedemso those are either fixed or documented in there now22:41
mriedemhopefully anyway22:41
sureshthanks mriedem, but I am following the same document, still am getting this error22:43
*** marst_ has quit IRC22:43
mriedemsuresh: i'd report a bug against ubuntu then22:44
mriedemhttps://launchpad.net/ubuntu/+source/nova ?22:44
*** baoli has joined #openstack-nova22:45
sureshnova-placement-api : Conflicts: nova-api but 2:14.0.4-0ubuntu1.2 is to be installed22:46
sureshmriedem, is there any woraround for this22:48
*** catintheroof has quit IRC22:48
sureshmriedem, is there any work around for this22:48
openstackgerritClark Boylan proposed openstack/nova master: Fix libvirt group selection in live migration test  https://review.openstack.org/45640822:48
mriedemsuresh: i haven't used the ubuntu packages so i don't know22:48
clarkbdansmith: that nova change plus my devstack change will fix the live migration job I think22:49
mriedemclarkb: hmm, but we don't run with ceph in the live migration job22:49
dansmithclarkb: rock on22:50
clarkbmriedem: thats what you think :P22:50
*** baoli has quit IRC22:50
clarkbmriedem: the live migration job uses the ceph scripts (I don't know why)22:50
sureshmriedem, thanks, I will try to find any work around :)22:50
mriedemthinking of https://review.openstack.org/#/c/389582/22:50
mriedemclarkb: oh weird, ok i'll have to look22:50
mriedemtdurakov: ^22:50
*** claudiub has quit IRC22:58
*** xyang1 has quit IRC23:02
*** crushil has joined #openstack-nova23:04
*** Apoorva_ has joined #openstack-nova23:04
*** Sukhdev has quit IRC23:04
*** trinaths has joined #openstack-nova23:07
*** Apoorva has quit IRC23:07
*** rajathagasthya has quit IRC23:12
*** thorst has joined #openstack-nova23:20
*** baoli has joined #openstack-nova23:20
*** baoli has quit IRC23:21
*** thorst has quit IRC23:21
*** baoli has joined #openstack-nova23:22
*** armax has joined #openstack-nova23:23
*** nic has quit IRC23:26
*** armax_ has joined #openstack-nova23:26
*** trinaths has left #openstack-nova23:26
*** armax has quit IRC23:28
*** armax_ is now known as armax23:28
*** chyka has quit IRC23:30
*** chyka has joined #openstack-nova23:31
*** erlon has quit IRC23:35
*** chyka has quit IRC23:35
*** eharney has joined #openstack-nova23:35
*** Apoorva_ has quit IRC23:36
clarkbdansmith: mriedem https://review.openstack.org/456408 just passed so getting that stack in soonish would likely be good23:36
*** Apoorva has joined #openstack-nova23:37
*** esberglu has quit IRC23:38
*** esberglu has joined #openstack-nova23:39
mriedemhmm23:40
*** gongysh has joined #openstack-nova23:40
mriedemlet me check the old failopotamus on gate-grenade-dsvm-neutron-multinode-live-migration-nv23:41
*** zeroDivisible has quit IRC23:41
mriedemclarkb: that doesn't show a big spike in failures23:41
openstackgerritClark Boylan proposed openstack/nova master: Remove backend dependency for key types  https://review.openstack.org/45620423:41
clarkbgrenade is unaffected because they will run the old libvirt23:42
mriedemoh crap that's grenade23:42
clarkbsince they start on old branch23:42
mriedemoh huh i didn't think gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial was voting23:42
*** priteau has joined #openstack-nova23:42
mriedemoh shit there i see the spike23:42
mriedemis this because of the UCA change?23:43
*** esberglu has quit IRC23:43
clarkbyes, debian appears to have used libvirt instead of libvirtd, UCA pciked up that change and newer ubuntu as well23:43
clarkbso you can't just blindly use "libvirtd"23:43
clarkbthe switch to UCA tried to address that but didn't realize noav was doing its own hard coding of the value too23:43
mriedemok fast approved23:45
mriedemclarkb: thanks for noticing and actually fixing it23:45
mriedemi didn't even know we were busted all day23:45
*** zeroDivisible has joined #openstack-nova23:45
mriedemthat's what i call service!23:45
clarkbmriedem: can you reassign https://bugs.launchpad.net/nova/+bug/1681946 to kfarr? I helpfully set the depends on there but then got the bug assigned to me :)23:46
openstackLaunchpad bug 1681946 in OpenStack Compute (nova) "py27 unit test failure: ImageMetaProps has a system dependent hash signature." [High,In progress] - Assigned to Clark Boylan (cboylan)23:46
mriedemmtreinish: oomichi: can you poke https://review.openstack.org/#/c/456388/ ?23:46
mriedemyeah it's actually dane-fichters but done23:46
clarkbmriedem: it was jbernard and kfarr that noticed I just did the fixing (I also did the breaking so you know my bad)23:46
*** priteau has quit IRC23:47
*** zeroDivisible has quit IRC23:52
*** dimtruck is now known as zz_dimtruck23:52
*** zeroDivisible has joined #openstack-nova23:54
*** ssurana has joined #openstack-nova23:59

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