Monday, 2017-01-30

*** thorst_ has joined #openstack-nova00:00
*** raunak has quit IRC00:02
*** tobasco has quit IRC00:02
*** thorst_ has quit IRC00:04
*** baoli has joined #openstack-nova00:05
*** kaisers has quit IRC00:06
*** Fdaisuke has quit IRC00:09
*** thorst_ has joined #openstack-nova00:13
*** thorst_ has quit IRC00:13
*** acabot has quit IRC00:13
*** tobasco has joined #openstack-nova00:14
*** jogo has joined #openstack-nova00:31
*** jogo has quit IRC00:32
*** jogo has joined #openstack-nova00:35
*** jogo has quit IRC00:36
*** jog0 has quit IRC00:37
*** jogo has joined #openstack-nova00:37
*** armax has quit IRC00:51
*** takashin has joined #openstack-nova00:52
*** kaisers has joined #openstack-nova01:02
*** ducttape_ has quit IRC01:09
*** dimtruck is now known as zz_dimtruck01:14
*** ducttape_ has joined #openstack-nova01:17
*** liangy has quit IRC01:19
*** raunak has joined #openstack-nova01:31
*** foexle has quit IRC01:35
*** ducttape_ has quit IRC01:39
*** ducttape_ has joined #openstack-nova01:39
*** tbachman has joined #openstack-nova01:42
*** thorst_ has joined #openstack-nova01:44
*** thorst_ has quit IRC01:49
*** amotoki has joined #openstack-nova01:53
*** ducttape_ has quit IRC02:02
*** ducttape_ has joined #openstack-nova02:03
*** ducttape_ has quit IRC02:06
*** _ducttape_ has joined #openstack-nova02:06
*** thorst_ has joined #openstack-nova02:09
*** unicell has quit IRC02:10
*** thorst_ has quit IRC02:12
*** amotoki has quit IRC02:18
*** yamahata has joined #openstack-nova02:23
*** _ducttape_ has quit IRC02:24
*** amotoki has joined #openstack-nova02:26
*** tanee_away is now known as tanee02:31
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Enable cold migration with target host  https://review.openstack.org/42663602:34
*** kaisers has quit IRC02:35
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2)  https://review.openstack.org/40895502:38
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2)  https://review.openstack.org/40895502:39
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2)  https://review.openstack.org/40896402:40
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2)  https://review.openstack.org/40896402:40
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Enable cold migration with target host  https://review.openstack.org/42663602:41
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004202:44
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004202:44
*** hongbin has joined #openstack-nova02:44
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host  https://review.openstack.org/40670702:44
*** amotoki has quit IRC02:45
openstackgerritTakashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host  https://review.openstack.org/41492602:45
openstackgerritTakashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host  https://review.openstack.org/41492602:46
*** thorst_ has joined #openstack-nova02:57
*** thorst_ has quit IRC02:57
*** baoli has quit IRC02:58
*** kaisers has joined #openstack-nova03:00
*** Sukhdev has joined #openstack-nova03:01
*** thorst_ has joined #openstack-nova03:07
*** thorst_ has quit IRC03:07
*** Oku_OS-away has quit IRC03:08
*** Oku_OS-away has joined #openstack-nova03:08
*** rcernin has joined #openstack-nova03:11
*** kaisers has quit IRC03:13
*** rcernin has quit IRC03:13
*** rcernin has joined #openstack-nova03:14
*** tbachman has quit IRC03:23
*** rcernin has quit IRC03:25
*** guchihiro has joined #openstack-nova03:27
*** hfu has joined #openstack-nova03:31
*** nicolasbock has quit IRC03:34
*** tbachman has joined #openstack-nova03:40
*** hoangcx has joined #openstack-nova03:41
*** amotoki has joined #openstack-nova03:45
*** hoangcx has quit IRC03:47
openstackgerritDavid Medberry proposed openstack/nova: Explain how allow_resize_to_same_host is useful  https://review.openstack.org/42653103:54
*** links has joined #openstack-nova03:56
*** links has quit IRC03:57
*** baoli has joined #openstack-nova03:57
*** hoangcx has joined #openstack-nova04:00
*** hoangcx has quit IRC04:00
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc  https://review.openstack.org/41644404:03
*** baoli has quit IRC04:04
*** raunak has quit IRC04:04
*** gouthamr has quit IRC04:09
*** baoli has joined #openstack-nova04:10
*** kaisers has joined #openstack-nova04:11
*** baoli has quit IRC04:12
*** baoli has joined #openstack-nova04:18
*** sree has joined #openstack-nova04:19
*** ducttape_ has joined #openstack-nova04:25
*** mdnadeem has joined #openstack-nova04:29
*** ducttape_ has quit IRC04:30
*** maestropandy has joined #openstack-nova04:31
*** baoli has quit IRC04:32
*** tbachman has quit IRC04:37
*** kristian__ has joined #openstack-nova04:38
*** kristian__ has quit IRC04:42
*** kaisers has quit IRC04:46
*** dikonoo has joined #openstack-nova04:51
*** amotoki has quit IRC04:56
*** ayogi has joined #openstack-nova04:58
*** sudipto has joined #openstack-nova04:59
*** sudipto_ has joined #openstack-nova04:59
*** dikonoor has joined #openstack-nova05:05
*** dikonoo has quit IRC05:05
openstackgerritTakashi NATSUME proposed openstack/nova: Fix doc generation warnings  https://review.openstack.org/42554905:07
openstackgerritTakashi NATSUME proposed openstack/nova: Use ListOfIntegersField in oslo.versionedobjects  https://review.openstack.org/42391205:07
openstackgerritTakashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases  https://review.openstack.org/39473805:07
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters  https://review.openstack.org/41688205:08
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters  https://review.openstack.org/42062605:08
openstackgerritTakashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases  https://review.openstack.org/39543605:08
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix parameters and examples in aggregate API  https://review.openstack.org/42551205:08
openstackgerritTakashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls  https://review.openstack.org/41387605:08
*** hongbin has quit IRC05:10
*** ratailor has joined #openstack-nova05:10
*** diga has joined #openstack-nova05:17
*** mtanino has quit IRC05:19
*** kaisers has joined #openstack-nova05:21
*** amotoki has joined #openstack-nova05:25
*** dikonoor has quit IRC05:27
*** psachin has joined #openstack-nova05:28
*** maestropandy has left #openstack-nova05:33
*** dikonoor has joined #openstack-nova05:38
*** prateek has joined #openstack-nova05:41
*** swamireddy has joined #openstack-nova05:43
*** unicell has joined #openstack-nova05:44
*** Dinesh_Bhor has joined #openstack-nova05:47
*** thorst_ has joined #openstack-nova05:49
*** satyar has joined #openstack-nova05:50
*** udesale has joined #openstack-nova05:51
*** thorst_ has quit IRC05:55
*** adisky_ has joined #openstack-nova05:59
*** swamireddy has left #openstack-nova06:00
*** swamireddy_ has joined #openstack-nova06:01
*** kristian__ has joined #openstack-nova06:01
*** unicell1 has joined #openstack-nova06:01
*** unicell has quit IRC06:02
*** Jack_I has joined #openstack-nova06:02
*** ecelik has joined #openstack-nova06:04
*** ecelik has left #openstack-nova06:04
*** claudiub|2 has joined #openstack-nova06:05
*** kristian__ has quit IRC06:05
*** mtanino has joined #openstack-nova06:13
*** lpetrut has joined #openstack-nova06:14
*** lpetrut has quit IRC06:17
*** nkrinner_afk is now known as nkrinner06:17
*** lpetrut has joined #openstack-nova06:19
openstackgerritTakashi NATSUME proposed openstack/nova-specs: xAbort Cold Migration  https://review.openstack.org/33473206:19
*** mtanino has quit IRC06:19
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Abort Cold Migration  https://review.openstack.org/33473206:19
*** ducttape_ has joined #openstack-nova06:26
*** ducttape_ has quit IRC06:31
*** bhagyashris has joined #openstack-nova06:35
*** toabctl has joined #openstack-nova06:54
*** kristian__ has joined #openstack-nova06:56
*** lpetrut has quit IRC06:58
*** sridharg has joined #openstack-nova07:04
*** unicell has joined #openstack-nova07:07
*** kristian__ has quit IRC07:08
*** unicell1 has quit IRC07:08
*** rcernin has joined #openstack-nova07:11
*** rcernin has quit IRC07:12
*** rcernin has joined #openstack-nova07:12
*** Oku_OS-away is now known as Oku_OS07:18
*** artom_ has joined #openstack-nova07:22
*** andreas_s has joined #openstack-nova07:25
*** mkoderer has joined #openstack-nova07:26
*** artom has quit IRC07:27
*** karthiks has joined #openstack-nova07:29
*** markus_z has joined #openstack-nova07:33
*** vsaienko has joined #openstack-nova07:35
*** lennyb has joined #openstack-nova07:37
*** lennyb_ has joined #openstack-nova07:37
*** lennyb_ has quit IRC07:37
openstackgerritAndrey Volkov proposed openstack/nova: Update pci device fields from deleted object  https://review.openstack.org/42624307:44
*** vsaienko has quit IRC07:49
*** tesseract has joined #openstack-nova07:50
*** thorst_ has joined #openstack-nova07:51
*** abalutoiu has joined #openstack-nova07:51
*** ekuris__ has joined #openstack-nova07:54
*** thorst_ has quit IRC07:56
*** edand has joined #openstack-nova07:58
*** Sukhdev has quit IRC08:02
*** vsaienko has joined #openstack-nova08:03
*** avolkov has quit IRC08:13
*** takashin has left #openstack-nova08:15
*** pcaruana has joined #openstack-nova08:15
*** haukebruno has quit IRC08:20
*** zer0c00l has joined #openstack-nova08:24
zer0c00lI don't know if i have to get my eyes checked, but it looks like _heal_instance_info_cache just going to refresh cache only once for the instance on top of the list08:25
*** haukebruno has joined #openstack-nova08:25
zer0c00lhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L588108:25
zer0c00lWe are getting the first instance that is 'okay' to healed and breaking off that loop08:26
zer0c00lam i missing something that is obvious ?08:26
*** avolkov has joined #openstack-nova08:26
*** ducttape_ has joined #openstack-nova08:27
*** ducttape_ has quit IRC08:32
openstackgerritOleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration  https://review.openstack.org/24691008:33
*** jpena|off is now known as jpena08:35
*** CristinaPauna has quit IRC08:38
*** mnestratov has joined #openstack-nova08:41
*** CristinaPauna has joined #openstack-nova08:47
*** jpena is now known as jpena|off08:48
*** lpetrut has joined #openstack-nova08:50
*** jpena|off is now known as jpena08:55
*** vsaienko has quit IRC08:57
*** amotoki has quit IRC08:59
*** ralonsoh has joined #openstack-nova09:00
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:00
*** amotoki has joined #openstack-nova09:00
*** claudiub has joined #openstack-nova09:02
*** ayogi has quit IRC09:04
*** claudiub|2 has quit IRC09:05
*** ayogi has joined #openstack-nova09:06
*** guchihiro has quit IRC09:09
*** sudipto_ has quit IRC09:10
*** sudipto has quit IRC09:11
*** sudipto_ has joined #openstack-nova09:11
*** yamahata has quit IRC09:11
*** sudswas__ has joined #openstack-nova09:13
*** ababich has quit IRC09:17
*** slaweq has quit IRC09:21
*** yassine has joined #openstack-nova09:27
*** jschwarz is now known as jschwarz|lunch09:27
*** yassine is now known as Guest4885209:28
*** JoseMello has joined #openstack-nova09:28
*** lucas-afk is now known as lucasagomes09:28
*** rcernin has quit IRC09:31
*** rcernin has joined #openstack-nova09:33
*** efoley__ has joined #openstack-nova09:33
* johnthetubaguy makes monday morning noises09:34
*** derekh has joined #openstack-nova09:40
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Change condition of root disk snapshot handling  https://review.openstack.org/38203009:45
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Remove backward compatibility checks for disk snapshots  https://review.openstack.org/38203109:45
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Call snapshot operations for not file based backends only  https://review.openstack.org/41471009:45
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Correct ploop imagebackend attribute  https://review.openstack.org/41470909:46
*** jistr has quit IRC09:49
*** jistr has joined #openstack-nova09:51
*** thorst_ has joined #openstack-nova09:52
*** maestropandy has joined #openstack-nova09:56
*** thorst_ has quit IRC09:57
openstackgerritRikimaru Honjo proposed openstack/nova: Add checks for preventing unexpected detaching volume  https://review.openstack.org/35461709:59
*** maestropandy has quit IRC10:00
*** maestropandy has joined #openstack-nova10:05
*** maestropandy has left #openstack-nova10:05
*** amotoki has quit IRC10:05
sfinucanlyarwood: Morning. Interesting question on https://review.openstack.org/#/c/42508710:07
sfinucanIt appears the test I've added has highlighted a latent Python 3 issue, which has since been fixed on master10:07
sfinucanDo I (a) modify the change to fix said Python issue, (b) backport the Python 3 issue fix first, or (c) remove the test from the patch?10:08
lyarwoodsfinucan: I'd go with a and document it as a conflict in the change10:10
sfinucanlyarwood: and that is what I'll do. Cheers :)10:10
lyarwoodsfinucan: assuming the fix on master also fixes other py3 issues10:10
sfinucanYeah, a lot of them10:10
lyarwoodsfinucan: yeah cool, then just fix this test up and document, that should be enough IMHO10:11
*** amotoki has joined #openstack-nova10:15
*** mkoderer has quit IRC10:16
*** dikonoor has quit IRC10:18
*** ayogi has quit IRC10:22
*** ayogi has joined #openstack-nova10:22
sfinucanlyarwood: Think that's done now10:23
lyarwoodsfinucan: thanks, I'll take a look once CI comes backs.10:24
lyarwoodsfinucan: would you mind taking a look at https://review.openstack.org/#/c/399735/ this morning if you have time10:24
sfinucansure10:24
*** gszasz has joined #openstack-nova10:25
*** ducttape_ has joined #openstack-nova10:28
*** mvk has quit IRC10:28
*** sambetts|afk is now known as sambetts10:28
*** ducttape_ has quit IRC10:33
*** dikonoor has joined #openstack-nova10:33
*** amotoki has quit IRC10:34
*** cdent has joined #openstack-nova10:35
*** satyar has quit IRC10:36
*** sree has quit IRC10:37
mdboothMorning! Now we're out of feature crunch, looking for another +2 on this bugfix which has been sitting a while: https://review.openstack.org/#/c/383859/1310:42
cdentjohnthetubaguy, bauzas : I'm a little lost on the state of things. Any salient updates to share?10:50
bauzascdent: waiting for +2 on the grenade fix10:50
johnthetubaguycdent: I think bauzas's change is waiting on grenade10:50
johnthetubaguyyeah, that10:50
johnthetubaguythe ironic one isn't read yet: https://review.openstack.org/#/c/40447210:51
johnthetubaguyI was thinking about tidying that up right now actually10:51
johnthetubaguybut I suspect its a bit late to get that in now10:51
cdentjohnthetubaguy: cool, that was the one I was wondering about most.10:51
*** eglynn has joined #openstack-nova10:52
bauzasjohnthetubaguy: cdent: well, that one seems at risk10:52
cdentI guess jay will be back today and will have some input. It all remains a bit fuzzy about what the actual intent was.10:52
bauzascdent: question, how can we know it works with the ironic driver ?10:53
bauzasI can only see one ironic job10:53
bauzaswhich tells me it's green http://logs.openstack.org/72/404472/25/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/0b27bd3/10:53
bauzasbut I'm not sure we're covering the patch with that job10:53
cdentbauzas: That's pretty much the source of my question. Without knowing the overall arc of intent, it is hard to discuss coverage.10:54
cdentToo many people saying "I'm not sure".10:54
bauzascdent: I'm just trying to explain that I discovered most of the corner cases when I investigated the grenade issue10:54
bauzaswrt the scheduler change10:54
johnthetubaguybauzas: the follow up patch was trying to get coverage10:55
bauzasI'd be far less concerned if I was somehow seeing that some job is running an ironic cloud that does reporting to nova10:55
bauzasI know the tripleo folks kinda use ironic for their undercloud10:55
bauzasso we could perhaps see if their jobs are happy10:55
cdentPersonally, in a perfect universe, I don't think of grenade or tempest as coverage at all. They are smoke. If there aren't functional tests in tree, we're faking it and causing us the problems we encountered at the end of last week. So the _next_ time we come round to this, we should start from tests that do something (not unit tests).10:56
cdentBut, given that we are not in a perfect universe, and now is not the next time, it's good you did that digging, and it's what we've got...10:56
*** mvk has joined #openstack-nova10:57
cdentI do know that jroll created a dnm devstack change to exercise the ironic-related change. I'm not sure of the current state of that.10:57
johnthetubaguyhttps://review.openstack.org/#/c/42629610:58
jrollcdent: https://review.openstack.org/#/c/426296/10:58
cdentthanks10:58
* jroll just walked in, assumes that needs a recheck on the latest PS10:58
jroller, rebase10:58
bauzasjroll: go to bed :p10:58
jrollI did that once already10:59
bauzasah, nah, it's just early for you :)10:59
jrollindeed :P10:59
*** rfolco has joined #openstack-nova10:59
jrollok so my DNM patch looks happy on the latest code11:00
*** sree has joined #openstack-nova11:00
jroller no, I can't read11:01
cdentjroll: because you been readin' the wrong stuff11:01
jrollhttp://logs.openstack.org/96/426296/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/fd3d15e/logs/screen-n-cpu.txt.gz?level=WARNING11:01
bauzasjroll: what does this job ?11:01
* jroll bets that's the same error as the pep8 job11:01
jrollbauzas: nova+ironic+boot a thing, mostly11:02
cdent(this is a story from a friend's teacher who got upset in some depths of kentucky school: "You know why Johnny can't read? Because he's been reading the wrong stuff!"11:02
johnthetubaguyyeah, I am fixing up that bug right now11:02
*** tbachman has joined #openstack-nova11:02
cdentUnless there's something I should be doing? I'm going to go back to evaluating the number of requests the placement api is receiving.11:03
jroll"Created resource class record via placement API for resource class CUSTOM_LOL." <- so we're halfway there :P11:03
bauzascool11:03
bauzasjroll: good to know, so it's non-voting on check ?11:03
jrollbauzas: yes, we've had a job like this non-voting on check for years :)11:04
* bauzas just resuming his review backlog :)11:04
bauzasjroll: that I know11:04
bauzasjroll: :)11:04
jroll:P11:04
johnthetubaguyFWIW, this is the failure: http://logs.openstack.org/96/426296/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/fd3d15e/logs/screen-n-cpu.txt.gz#_2017-01-28_00_25_20_18711:04
*** dikonoor has quit IRC11:08
*** ayogi has quit IRC11:09
jrollbauzas: cdent: btw, we've an experimental job on nova/ironic that uses resource classes (triggering this code), I just have a bit of devstack plugin code left to make that work, should finish that early this week11:11
bauzascool11:12
*** ayogi has joined #openstack-nova11:13
*** sree has quit IRC11:15
*** sree has joined #openstack-nova11:17
*** dikonoor has joined #openstack-nova11:21
*** udesale has quit IRC11:25
*** dims has joined #openstack-nova11:26
*** sree has quit IRC11:29
*** tbachman has quit IRC11:30
*** psachin has quit IRC11:32
*** ducttape_ has joined #openstack-nova11:35
*** tbachman has joined #openstack-nova11:39
*** tbachman has quit IRC11:44
*** vsaienko has joined #openstack-nova11:50
*** CristinaPauna has quit IRC11:52
*** ducttape_ has quit IRC11:52
*** nicolasbock has joined #openstack-nova11:52
*** thorst_ has joined #openstack-nova11:53
*** markmc has quit IRC11:54
*** markmc has joined #openstack-nova11:54
artom_johnthetubaguy, can I ask you to take a look at https://review.openstack.org/#/c/426030/7 when you have a moment?11:55
artom_It's the device tag 'tag' attribute disappearing issue - you mentioned it needs a microversion in https://review.openstack.org/#/c/423952/, so a spec has been whipped up :)11:55
*** thorst_ has quit IRC11:57
*** diga has quit IRC11:59
johnthetubaguyartom_: ah, its that bug fix that needs a microversion, gotcha11:59
openstackgerritJohn Garbutt proposed openstack/nova: placement: RT now adds proper Ironic inventory  https://review.openstack.org/40447211:59
johnthetubaguyartom_: I will try, honestly it feels like that should be in pike now11:59
johnthetubaguyas RC1 us being cut on thursday12:00
johnthetubaguybut not 100% sure12:00
artom_johnthetubaguy, I know we're short for time :( But if we can avoid a whole release cycle without device tagging...12:00
*** weshay is now known as weshay_errand12:01
artom_(Yeah, I know the client can be version-pinned)12:01
artom_johnthetubaguy, anyways, much thanks in advance :)12:02
*** bkopilov has quit IRC12:02
*** artom_ is now known as artom12:03
artomjohnthetubaguy, and for what it's worth, the patch is already up and Alex and gmann have given it one look over each: https://review.openstack.org/#/c/424759/12:04
tdurakovhi folks12:07
tdurakovdo we have any issues with devstack and cells?12:07
tdurakovfresh multinode  env: {"message": "Host 'node2' is not mapped to any cell", "code": 400, "created": "2017-01-30T12:04:26Z"}12:08
sfinucanjohnthetubaguy: Could I get some eyes on https://review.openstack.org/#/c/392643/ - we're seeing the referenced bug pop up in the real world12:12
*** vsaienko has quit IRC12:13
sfinucanartom: Would appreciate it if you could take a look at ^ too, please :)12:13
*** hshiina has joined #openstack-nova12:15
*** Guest48852 has quit IRC12:16
*** ratailor has quit IRC12:17
*** foexle has joined #openstack-nova12:19
amrithandreykurilin, thank you12:23
*** catintheroof has joined #openstack-nova12:25
*** Guest48852 has joined #openstack-nova12:28
*** alexpilotti has quit IRC12:30
*** alexpilotti has joined #openstack-nova12:30
*** CristinaPauna has joined #openstack-nova12:31
artomsfinucan, sure, I'll get to it this afternoon :)12:32
sfinucanartom: Thanking you kindly, sir12:32
*** alexpilotti has quit IRC12:35
*** bvanhav_ has joined #openstack-nova12:38
*** sdague has joined #openstack-nova12:40
*** thorst_ has joined #openstack-nova12:41
*** bkopilov has joined #openstack-nova12:41
*** lucasagomes is now known as lucas-hungry12:45
*** tbachman has joined #openstack-nova12:46
*** ayogi has quit IRC12:47
*** tbachman has quit IRC12:51
*** tbachman has joined #openstack-nova12:51
*** ekuris__ has quit IRC12:57
*** dave-mccowan has joined #openstack-nova13:00
*** wasmum has quit IRC13:03
*** ducttape_ has joined #openstack-nova13:04
*** jaypipes has joined #openstack-nova13:04
*** prateek has quit IRC13:06
*** edmondsw has joined #openstack-nova13:07
openstackgerritJohn Garbutt proposed openstack/nova: DNM: hack ironic with resource providers  https://review.openstack.org/42629613:09
*** eharney has quit IRC13:17
*** mdnadeem has quit IRC13:21
*** weshay_errand is now known as weshay13:25
*** hshiina has quit IRC13:27
*** cdent has quit IRC13:30
*** tblakes has joined #openstack-nova13:31
edleafejohnthetubaguy: I see that bug and can fix it, but was waiting until after the scheduler meeting discussion for overall direction13:32
*** JoseMello has quit IRC13:35
*** mdrabe has joined #openstack-nova13:36
*** diga has joined #openstack-nova13:37
*** mkoderer has joined #openstack-nova13:38
*** ducttape_ has quit IRC13:41
johnthetubaguyedleafe: which bug?13:41
edleafejohnthetubaguy: sorry, was reading scrollback13:42
edleafejohnthetubaguy: AttributeError: 'SchedulerReportClient' object has no attribute '_compute_node_to_inventory_dict'13:42
*** lucas-hungry is now known as lucasagomes13:42
johnthetubaguyedleafe: ah, gotcha, pushed a fix for that, to see if the ironic tests will pass, just rebased that patch with those tests on13:43
edleafejohnthetubaguy: ok, thanks. Still catching up13:43
johnthetubaguynp13:44
johnthetubaguyedleafe: I think what you have should work, although not 100% sure it gets us enough to push it into ocata13:45
edleafejohnthetubaguy: me neither13:45
*** owalsh has joined #openstack-nova13:45
johnthetubaguyedleafe: if we claimed the new resources we are reporting, then I think that would be good to have13:45
edleafejohnthetubaguy: was really hoping to discuss with jaypipes before we move forward13:45
edleafejohnthetubaguy: I'm not sure that we claim all of them - just the VCPU-rleated ones13:46
johnthetubaguyedleafe : yup, makes sense13:46
jaypipesedleafe: heyo13:46
*** pradk has joined #openstack-nova13:47
edleafehi yerself13:47
*** acabot has joined #openstack-nova13:49
*** baoli has joined #openstack-nova13:50
*** tlian has joined #openstack-nova13:52
*** cleong has joined #openstack-nova13:53
*** jdurgin has quit IRC13:54
edleafeNova Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt13:55
openstackgerritJordan Pittier proposed openstack/nova: Fix unspecified bahavior on GET /servers/detail?tenant_id=X as admin  https://review.openstack.org/42625913:57
*** jpena is now known as jpena|lunch13:57
*** bvanhav_ has quit IRC13:58
openstackgerritMaciej Szankin proposed openstack/nova: config options: remove libvirt imagecache  https://review.openstack.org/37849713:58
*** bvanhav_ has joined #openstack-nova13:58
johnthetubaguyedleafe: oh, I should clarify, we don't claim the new custom one right now, which seems problematic to me.13:58
edleafejohnthetubaguy: agreed13:59
*** weshay is now known as weshay_mtg13:59
*** karthiks has quit IRC13:59
*** jschwarz|lunch is now known as jschwarz14:01
*** cdent has joined #openstack-nova14:04
*** xyang1 has joined #openstack-nova14:08
*** jdurgin has joined #openstack-nova14:12
*** clenimar has quit IRC14:12
*** gcb has joined #openstack-nova14:13
*** clenimar has joined #openstack-nova14:16
*** alexpilotti has joined #openstack-nova14:16
*** dikonoor has quit IRC14:17
*** satyar has joined #openstack-nova14:18
*** gouthamr has joined #openstack-nova14:19
*** ducttape_ has joined #openstack-nova14:19
*** jheroux has joined #openstack-nova14:19
*** alexpilotti has quit IRC14:20
*** jamesdenton has joined #openstack-nova14:21
*** liverpooler has joined #openstack-nova14:22
*** READ10 has joined #openstack-nova14:22
*** alexpilotti has joined #openstack-nova14:26
*** ducttape_ has quit IRC14:26
*** alexpilotti has quit IRC14:37
*** vladikr has joined #openstack-nova14:43
*** hongbin has joined #openstack-nova14:45
*** karthiks has joined #openstack-nova14:46
*** rcernin has quit IRC14:48
*** alexpilotti has joined #openstack-nova14:51
*** ducttape_ has joined #openstack-nova14:52
openstackgerritMaciej Szankin proposed openstack/nova: conf: resolved TODOs in compute configuration  https://review.openstack.org/42679314:55
*** udesale has joined #openstack-nova14:57
johnthetubaguysfinucan: mdbooth: is there always a disk.mirror attribute? re https://review.openstack.org/#/c/39973514:58
mdboothkashyap lyarwood: ^^^14:58
*** diga has quit IRC14:58
mdboothjohnthetubaguy: I don't recall the details now, but I do recall investigating that thoroughly and being satisfied at the time14:58
johnthetubaguyoops, I meant to ask lyarwood as well14:59
mdboothjohnthetubaguy: I suspect the answer is in the libvirt domain xml docs for that element14:59
kashyapjohnthetubaguy: mdbooth: Trying to remember...14:59
* mdbooth looks them up14:59
sfinucanjohnthetubaguy: Would it matter?15:00
johnthetubaguysfinucan: if its not there, we will just assume every job hasn't ever finished15:00
johnthetubaguyI think15:00
*** rmart04 has joined #openstack-nova15:00
*** rcernin has joined #openstack-nova15:00
sfinucanjohnthetubaguy: aha - I read that incorrectly https://review.openstack.org/#/c/399735/19/nova/virt/libvirt/guest.py15:00
mdboothjohnthetubaguy: Since at least libvirt 0.9.12, possibly longer: http://libvirt.org/formatdomain.html15:01
mdboothSearch for 'mirror'15:01
* sfinucan thought the mirror check was an alternative and it would fallback to the existing check if absent15:01
mdboothsfinucan: Nah, it's required as a bug fix in fact15:01
mdboothjohnthetubaguy: MIN_LIBVIRT_VERSION = (1, 2, 1)15:02
*** mvk has quit IRC15:02
mdboothIOW, it's definitely always present15:02
* lyarwood reads 15:03
kashyapmdbooth: Yes, it's been there for the duration at least Matt notes above15:03
kashyapErr, johnthetubaguy ^15:03
johnthetubaguycool, thanks folks15:03
*** jpena|lunch is now known as jpena15:03
johnthetubaguy"mirror" seemed odd to me, but the docs clear that up nicely, thanks15:03
lyarwoodjohnthetubaguy, sfinucan ; I've not had time to improve the commit msg as yet btw so feel free to hold fire on this and I can make that point a little better there15:03
dansmithsdague: did you see the activity on this? https://review.openstack.org/#/c/424730/15:04
*** burt has joined #openstack-nova15:04
johnthetubaguylyarwood: oh boy, I totally didn't see the notes in the commit message somehow, that seems good15:04
lyarwoodjohnthetubaguy: sfinucan did ask for a little more in his review that I'm happy to add if it's still required15:05
johnthetubaguylyarwood: there good point, but the fix seems more important15:05
sfinucanlyarwood, johnthetubaguy Yeah, I'm happy to let it in as is15:05
*** bkopilov has quit IRC15:05
sfinucan(hence +2 - more of a nice-to-have)15:06
kashyapjohnthetubaguy: Yeah, the word 'mirror' comes from the QEMU QMP primitive 'drive-mirror' (even _that_ is not really an accurate word, as it synchronizes as it sees fit, but not "mirrors" immediately).15:06
*** burgerk has joined #openstack-nova15:07
*** nkrinner is now known as nkrinner_afk15:08
*** bkopilov has joined #openstack-nova15:11
*** bvanhav_ is now known as bvanhav15:12
*** abalutoiu has quit IRC15:13
sdaguedansmith: I hadn't yet15:13
dansmithsdague: okay everything else is +A behind that15:13
sdaguewhat was the thing that failed when it was in a dedicated section?15:13
dansmithsdague: we start a newton-based placement15:13
dansmithbecause we do it before nova has upgraded15:13
sdagueoh... right15:14
dansmith...yeah :)15:14
*** owalsh is now known as owalsh-brb15:14
sdagueok, well, that's fine for now then. We can separate later and figure out about earlier installing code15:15
sdague+A15:15
dansmithyeah, thanks15:16
jaypipeshmm, it's always unpleasantly surprising when you run tests expecting them to fail, but they pass...15:17
*** armax has joined #openstack-nova15:17
*** mvk has joined #openstack-nova15:18
*** baoli has quit IRC15:18
*** owalsh-brb has quit IRC15:18
*** baoli has joined #openstack-nova15:19
*** mriedem has joined #openstack-nova15:19
*** alexpilotti has quit IRC15:20
*** kfarr has joined #openstack-nova15:21
mriedemblarg15:22
*** stvnoyes has quit IRC15:22
*** jaosorior has joined #openstack-nova15:23
*** stvnoyes has joined #openstack-nova15:23
*** owalsh-brb has joined #openstack-nova15:23
*** jdurgin has quit IRC15:23
*** udesale has quit IRC15:24
*** owalsh-brb is now known as owalsh15:24
mriedemoh sweet https://review.openstack.org/#/c/424730/15:25
johnthetubaguymriedem: jaypipes is looking at something for the iroinc patch15:25
*** zz_dimtruck is now known as dimtruck15:26
mriedemjohnthetubaguy: cool15:26
jaypipesjohnthetubaguy: yeah, I'm on it.15:26
jaypipesmriedem: trying to get it done before dentist appt at 11am.15:26
mriedemjohnthetubaguy: did we want to try and get https://review.openstack.org/#/c/335358/ in before rc1?15:26
mriedemhate to put it off all release and then say it's too risky at the end15:27
mriedembut i'm kind of feeling like it's too risky at this point15:27
*** jaosorior has quit IRC15:27
johnthetubaguymriedem: too risky might be right :( I should take another peak at that15:28
mriedemi know a certain someone in hungary will be asking about it15:28
johnthetubaguymriedem: it will need rebasing on top of bauzas's change, due to service version bump15:28
mriedemyeah15:28
johnthetubaguymriedem: dang, yeah, its the error handling that scares me in that patch15:29
johnthetubaguyfeels like we land it in master as soon as we open, to let it cook for the whole cycle15:29
mriedemjohnthetubaguy: i agree15:31
mriedemi'll fall on ildikov's sword15:31
*** jaosorior has joined #openstack-nova15:32
*** kfarr has quit IRC15:33
openstackgerritBalazs Gibizer proposed openstack/nova: reno for notification-transformation-ocata  https://review.openstack.org/42681015:33
mriedemi need to go through the tag microversion bug fix thing, starting with the spec https://review.openstack.org/#/c/426030/15:34
*** jordanP has joined #openstack-nova15:35
jordanPdims, mriedem here's a patch that would make the gate-tempest-dsvm-py35-ubuntu-xenial-nv job more reliable: https://review.openstack.org/#/c/426259 It only a one line change (excluding unit tests)15:36
johnthetubaguymriedem: yeah, I am curious on sdague' take on that, I could be over thinking it asking for the microversion15:37
mriedemjohnthetubaguy: this isn't the all_tenants policy thing15:37
dimsjordanP : yep. thanks for that! i had reviewed it earlier but forgot to +115:37
mriedemjohnthetubaguy: this is the bug where we broke device tags after 2.3715:37
johnthetubaguymriedem: yeah, thats the one15:37
mriedemjohnthetubaguy: on the policy thing for all_tenants_visible, i think i'd like to discuss that one at the PTG15:37
johnthetubaguymriedem: yeah, I think that makes sense15:38
*** eharney has joined #openstack-nova15:38
bauzassdague: mriedem: wooot https://review.openstack.org/#/c/424730/15:39
*** mvk has quit IRC15:40
mriedempkoniszewski: did you want to see if you could recreate your reported bug and fix with this patch? https://review.openstack.org/#/c/424428/15:41
mriedemit's huge for an rc1 blocker15:41
*** abalutoiu has joined #openstack-nova15:41
*** avolkov has quit IRC15:44
jordanPmriedem, the patch I linked earlier is not related to "the policy thing for all_tenants_visible". It's just to enforce the same behavior on py27 and py35. Whether this behavior needs to change it another subject15:44
mriedemBobBall: was this a regression in ocata? or a latent bug? https://bugs.launchpad.net/nova/+bug/165887715:45
openstackLaunchpad bug 1658877 in OpenStack Compute (nova) "live migration failed with XenServer as hypervisor" [High,In progress] - Assigned to huan (huan-xie)15:45
*** avolkov has joined #openstack-nova15:45
mriedemjordanP: i know15:46
jordanPok :)15:46
mriedembauzas: in https://etherpad.openstack.org/p/nova-ocata-rc1-todos for placement release notes i think we're covered unless you can think of something besides a prelude section mention15:47
*** abalutoiu has quit IRC15:48
mriedembauzas: i.e. i think https://review.openstack.org/#/c/417961/41/releasenotes/notes/resource_providers_scheduler_db_filters-16b2ed3da00c51dd.yaml is pretty solid15:48
mriedemwe just need prelude15:48
mriedemwe might also want some things in the placement devref about the scheduler integration and config required for comptues in ocata?15:48
mriedemcdent: ^15:48
cdentmriedem: you mean placement.rst or placement_dev.rst?15:49
cdent(or both)15:49
bauzasmriedem: yup, I was thinking of a prelude section15:49
mriedemcdent: the former15:49
bauzasmriedem: that's something I can write15:49
mriedemcdent: more like notes for operators, i.e. nova-compute won't start with ocata code unless placement is configured in nova.conf,15:50
mriedemand maybe just a general note that the filter scheduler will not use placement in ocata while there are <ocata computes, but that's going to be a hard fail in pike15:51
mdboothmriedem: I just sent this to the operators ML: http://lists.openstack.org/pipermail/openstack-operators/2017-January/012540.html15:52
mdboothWould that require a BP or similar? Or would a link to the ML post be sufficient?15:52
*** abalutoiu has joined #openstack-nova15:53
*** mvk has joined #openstack-nova15:53
mriedemmdbooth: bp to remove a broken feature?15:54
mriedemmdbooth: i think we can just bug fix it in pike15:54
mdboothmriedem: That's what I hoped you'd say :)15:55
mriedemmdbooth: i think cfriesen and the windriver people are using lvm15:55
mriedemso might want to ask them about it15:55
mdboothcfriesen: Sparse lvm?15:55
mdboothmriedem: Thanks15:55
mriedemcdent: L17 here if you want to sign up https://etherpad.openstack.org/p/nova-ocata-rc1-todos15:56
cdentI thought bauzas said he'd take it: [t 387] ?15:57
purplerbot<bauzas> mriedem: that's something I can write [2017-01-30 15:49:49] [n 387]15:57
mdboothIncidentally, the issue with it seems to be that it relies on a thinpool being created implicitly, which is created as 64MB on a default setup. This means that as soon as you write more than 64MB of data to your instance you start getting io errors. Which is nice.15:57
mdboothSo a cirros instance works on initial boot, but not if you leave it for a bit.15:57
cdent(happy to do it, just don't want to duplicate the effort)15:57
ildikovjohnthetubaguy: mriedem TBH I don't think that patch is that risky15:57
mriedemcdent: i thought bauzas was going to do the release note prelude section15:59
mriedembut either way15:59
cdentmriedem: if it's not clear when I finish what I'm doing right now, I'll take a stab16:00
*** _ducttape_ has joined #openstack-nova16:00
*** ducttape_ has quit IRC16:01
mdboothmriedem: Is this an rc1 potential bug as in your etherpad: https://review.openstack.org/#/c/383859/1316:01
bauzasmriedem: cdent: I can do both, I don't care that much16:01
bauzasmriedem: cdent: I was mentioning the prelude section but anyway16:01
* cdent bows to bauzas 16:01
* mdbooth would really like to see it land.16:01
mriedemmdbooth: it's not a regression right?16:01
mdboothmriedem: Nope, but it's a bug.16:02
*** jdillaman has joined #openstack-nova16:02
mriedemmdbooth: sure, but we have lots of bugs16:02
mdboothmriedem: Right, but this one has a well-tested fix.16:02
mriedemmdbooth: having said all of that, i was planning on trying to get that one in yet16:02
mdboothThanks16:02
*** annegentle has joined #openstack-nova16:05
*** liverpooler has quit IRC16:06
*** liverpooler has joined #openstack-nova16:07
ildikovjohnthetubaguy: mriedem: so just to ensure we're on the same page, the plan now is to get that remove check_attach patch landed, when master opens for Pike?16:07
mriedemildikov: that's what i would prefer16:08
mriedemsince we're 4 days from rc116:08
*** Oku_OS is now known as Oku_OS-away16:09
ildikovI guess as the new Cinder API changes will be tied to a version etc. we don't necessarily have to land it a cycle earlier16:09
ildikovas it makes those changes easier to do and follow16:09
*** annegentle has quit IRC16:10
ildikovmriedem: so I can annoy you to hell with this, when master opens then?16:10
ildikovmriedem: as it's been two cycles now... :/16:10
ildikovmriedem: and it's really just a bugfix on its own16:10
mriedemyeah, but to be fair, it's also super latent behavior16:11
*** sridharg has quit IRC16:11
ildikovmriedem: I see this as one more reason to remove :)16:12
artommriedem, so I wrote the spec for the device role tagging bug: https://review.openstack.org/#/c/426030/16:13
artomI poked johnthetubaguy to review it, dunno how much bandwidth he has to spare - it'd be awesome if we can fix this for ocata to avoid a cycle without device tagging (yeah, the client the can version pinned, I know)16:13
artomdansmith ^^ same to you :)16:13
artomMany thanks in advance, gentlemen :)16:13
openstackgerritDan Smith proposed openstack/nova: Allow scheduler to run cell host discovery periodically  https://review.openstack.org/42682616:14
*** esberglu has joined #openstack-nova16:14
mriedemartom: i know, i have it open in a tab16:14
dansmithdo we really have to add another microversion to bring that back in?16:16
*** armax_ has joined #openstack-nova16:16
mriedemjordanP: left a question in https://review.openstack.org/#/c/426259/ - i don't really see how that doesn't break the case where the admin is listing instances for a specific tenant16:16
*** armax has quit IRC16:17
*** armax_ is now known as armax16:17
mriedemdansmith: i wrestled with that16:17
mriedemit is an api change though16:18
*** unicell has quit IRC16:18
mriedemi.e. trying to tag bdms on server boot with 2.33 in newton wouldn't work but if we bug fix it in ocata, then it will work16:18
dansmithit's pretty much the suck though16:19
*** baoli has quit IRC16:19
*** rmart04 has quit IRC16:20
jordanPmriedem, I replied. Not sure I understood the question though. Sorry if I misunderstood16:20
mriedemyes i know16:20
mriedemhttp://lists.openstack.org/pipermail/openstack-dev/2017-January/110798.html16:20
mriedemjordanP: before your change, an admin is listing instances for a specific tenant, right?16:20
jordanPonly if all_tenants=1 is passed16:20
jordanPin addition to tenant_id=XXX16:20
mriedembut in your patch, you're removing tenant_id from search_opts if all_tenants is False16:21
mriedemso as an admin, i want to specifically list instances for a single tenant, not all tenants16:22
*** rmart04 has joined #openstack-nova16:22
jordanPyeah.. I know it sucks, but that's how the current api behaves16:22
*** yamahata has joined #openstack-nova16:23
jordanPit's "documented" here: https://bugs.launchpad.net/nova/+bug/118529016:23
openstackLaunchpad bug 1185290 in OpenStack Compute (nova) "nova list's --tenant flag also requires --all-tenants" [Undecided,Fix released] - Assigned to Ghanshyam Mann (ghanshyammann)16:23
*** annegentle has joined #openstack-nova16:24
*** rcernin has quit IRC16:24
mriedemshould be documented here http://developer.openstack.org/api-ref/compute/?expanded=list-servers-detail#id416:25
*** markus_z has quit IRC16:25
mriedemjordanP: your reply in the patch still doesn't make it clear to me *why* all_tenants has to be true16:26
*** rmart04 has quit IRC16:26
mriedemis it because the if all_tenants block elevates the context?16:26
mriedembecause if an admin is making the request, we shouldn't need to elevate the contexzt16:27
*** weshay_mtg is now known as weshay16:27
mriedemis it something in compute_api.get_all()? or something in the DB API?16:27
jordanPI don't understand the question. All I am enforcing is that tenant_id option is dropped if all_tenants doesn"t evaluate to "true"16:29
jordanPwhy, the API was designed this way, I don"t know16:30
mriedemjordanP: i'd like to understand why filtering by tenant_id does not work unless all_tenants=116:31
*** bvanhav has quit IRC16:31
jordanPah ok16:31
*** slaweq has joined #openstack-nova16:31
jordanPI think it should work but then in the get_all() method16:32
jordanPwe have this filter_mapping dict that maps tenant_id to project_id16:33
mriedemright, which could overwrite project_id in filters to point at the tenant_id, rather than context.project_id16:33
jordanPand if we pass both tenant_id and project_id (project_id is automatically added (search_opts['project_id'] = context.project_id), then it becomes unpredictable16:34
jordanPyeah exactly16:34
jordanPmaybe there's a better fix for the issue16:35
*** rmart04 has joined #openstack-nova16:36
mriedemwhy not just sort search_opts.items() in get_all() if it's a random order issue?16:36
*** baoli has joined #openstack-nova16:36
mriedemi don't really enjoy hacking stuff in the REST API because of some behavior in the compute API code,16:36
mriedemmaking it all totally tightly coupled16:36
*** armax has quit IRC16:36
*** baoli has quit IRC16:37
*** rmart04 has quit IRC16:37
jordanPso the get_all() method would have these search_opts :  {'tenant_id': XX, 'project_id': YY}. This is conflicting16:38
*** abalutoiu has quit IRC16:38
jordanPsorting would work because we would be able to control, for sure, whether tenant_id takes precedence over project_id, but the overwritting would still be here and unexpected16:39
*** _ducttape_ has quit IRC16:40
mriedemso if tenant_id is handled first, it maps filters['project_id'] to XX16:40
mriedemand then handles project_id and sets filters['project_id'] to YY16:40
*** ducttape_ has joined #openstack-nova16:40
jordanPsomething like this, the logic inside the "for opt, value in search_opts.items():" is not trivial to say the least16:41
mriedemso if both tenant_id and project_id are in search_opts, you want to handle project_id first16:41
mriedemso that the final value is whatever was specified for tenant_id in the request, XX in your example16:41
*** cdent has quit IRC16:42
mriedemsorting search_opts.items() would fix that right?16:42
jordanPyes16:42
jordanPbut that's more than a hack to me, no one will understand that the order on which items are iterated upon is important16:43
*** jdurgin has joined #openstack-nova16:43
jordanPwe shouldn't be able to call get_all() with both tenant_id and project_id to begin with16:43
*** mlavalle has joined #openstack-nova16:44
BobBallmriedem: Sorry for the delay - I believe it was a regression in ocata; see comment on the bug.16:44
*** andreas_s has quit IRC16:45
BobBallSpeaking of which, I'm very confused - anyone got a thought why Gerrit shows https://review.openstack.org/#/c/424428/2..3/nova/tests/unit/virt/xenapi/test_driver.py as a change in the comment at the top of the file?16:45
*** rmart04 has joined #openstack-nova16:46
jordanPBobBall, did you edit this file on a Windows OS ?16:46
mriedemjordanP: if filtering by tenant_id is only randomly not working when all_tenants=0, then your change would definitely make it never work16:47
jordanPlooks like a \n vs \r\n issue16:47
BobBalloh dear... I hope that's not the casue... Let me check... (It wasn't me)16:47
*** baoli has joined #openstack-nova16:47
jordanPmriedem, that's the point, it should never work16:47
mriedemjordanP: i don't see why16:47
mriedemjordanP: we don't say anything about that in our api docs16:48
mriedemjordanP: and if it randomly DOES work today, then breaking it is worse16:48
jordanPit randomly work today only on Py35 and it never works on Py2716:48
jordanPthe goal of this patch is to make it never work, also on Py3516:48
*** bvanhav has joined #openstack-nova16:49
jordanPthe doc doesn't say anything about it, I agree. So I can make it work on both py27 and py35 but that would require a Tempest change too. Because Tempest has a test to check that filtering by tenant when all_tenant is 0 doesn't work16:49
*** ekhugen- has joined #openstack-nova16:50
*** abalutoiu has joined #openstack-nova16:51
BobBalljordanP: Indeed, you are right.16:51
openstackgerritBob Ball proposed openstack/nova: Fix live migrate with XenServer  https://review.openstack.org/42442816:52
mriedemjordanP: what's different about py27?16:53
mriedemthe dict ordering is random on py27 too isn't it?16:53
jordanPthe order on which the items in the search_opts dict are iterated upon16:53
BobBallmriedem: There was a CRLF / LF issue causing https://review.openstack.org/424428 to look huge.  I'm not arguing that it's small, but the diff is really 600 lines shorter than patchset 3...16:54
mriedemBobBall: ha16:54
mriedemjordanP: again, i thought dict.items() was random in py27 too isn't it?16:54
jordanPI can't confirm. Maybe it is, maybe someone/something set PYTHONSEED (whatever its name) to °16:55
jordanP016:55
*** hshiina has joined #openstack-nova16:56
edleafemriedem: not random, but not guaranteed to stay in order after a change16:56
*** nic has joined #openstack-nova16:56
*** abalutoiu has quit IRC16:56
*** bnemec has quit IRC16:56
*** chyka has joined #openstack-nova16:57
mriedemso dict.items() in py27 will return the tuples in the order that they were added?16:57
mriedembecause in the REST API code we're adding tenant_id first and then project_id16:57
edleafemriedem: no, not in the order they were added. But if you call items() once, and then call it again without changing the dict, the order will be the same16:58
*** tesseract has quit IRC16:58
openstackgerritMaciej Szankin proposed openstack/nova: conf: resolved TODOs in compute configuration  https://review.openstack.org/42679316:58
mriedemedleafe: we only call items() once in this case16:58
mriedemas far as i can tell16:58
edleafemriedem: yeah, so no guarantee of ordering16:58
jordanPwe need to distinguish guarantee and what actually happens16:59
mriedemok, so i'm not really buying the 'this never worked on py27' argument16:59
*** baoli has quit IRC16:59
mriedemjordanP: i agree we need consistency16:59
bauzasshit, the bottom change for the scheduler upcall will be -216:59
bauzasbecause of a cells job16:59
mriedembauzas: so recheck16:59
jordanPthis never works on py27 and the proof is that the Tempest job for that behavior passes on py27 but sometimes fails on py3517:00
bauzasmriedem: it will stop the gate jobs ?17:00
bauzasI thought not17:00
bauzasunless rebasing my change17:00
bauzasI'll actually wait for the grenade change to land17:01
bauzasand then recheck17:01
openstackgerritGábor Antal proposed openstack/nova: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622517:01
*** bnemec has joined #openstack-nova17:01
*** abalutoiu has joined #openstack-nova17:02
*** rmart04 has quit IRC17:02
bauzasmriedem: FYI, I'll also write a bit of explanation about cellsv2 in the prelude section17:02
mriedembauzas: don't do the cells v2 prelude17:02
mriedembauzas: unless you plan on updating this https://review.openstack.org/#/c/424737/17:03
mriedem^ will be the reno for any cells v2 stuff that's missing17:03
bauzasmriedem: the prelude section works differently from the other sections17:03
bauzasmriedem: so I'd like to only have one change for the prelude section17:03
bauzasbecause that's not bullets17:04
bauzasI mean bullet points17:04
*** kfarr has joined #openstack-nova17:04
*** takedakn has joined #openstack-nova17:04
bauzasmriedem: but I can do the prelude change that would be a follow-up of https://review.openstack.org/#/c/424737/8/releasenotes/notes/ocata-cellsv2-support-4b3b5e70e76bc756.yaml17:04
*** takedakn has quit IRC17:05
bauzasso we would be sure that we don't miss them17:05
bauzasmriedem: WFY ?17:05
mriedemsure17:05
*** abalutoiu has quit IRC17:06
*** Swami has joined #openstack-nova17:06
*** vladikr has quit IRC17:06
*** gcb has quit IRC17:07
*** chyka has quit IRC17:12
openstackgerritJordan Pittier proposed openstack/nova: Fix unspecified bahavior on GET /servers/detail?tenant_id=X as admin  https://review.openstack.org/42625917:14
*** mtanino has joined #openstack-nova17:14
*** baoli has joined #openstack-nova17:15
*** oanson has quit IRC17:15
jordanPthe question remains: why do we ignore tenant_id if all_tenants=0, I can't answer to this, it's probably unspecified, undefined but it's been like this for a long long time.17:16
jordanPI have to go17:16
*** jordanP has quit IRC17:16
*** Guest48852 has quit IRC17:17
*** cdent has joined #openstack-nova17:17
*** oanson has joined #openstack-nova17:17
*** jose-phillips has joined #openstack-nova17:19
mriedemartom: comments inline https://review.openstack.org/#/c/426030/17:19
*** yamahata has quit IRC17:20
artommriedem, woot!17:20
mriedemartom: mostly ok, just some nits17:20
artommriedem, sweet, lemme fix those and respin17:21
*** chyka has joined #openstack-nova17:23
*** mvk has quit IRC17:24
*** Apoorva has joined #openstack-nova17:24
*** edand has quit IRC17:28
*** hshiina has quit IRC17:30
openstackgerritArtom Lifshitz proposed openstack/nova-specs: Fix tag attribute disappearing  https://review.openstack.org/42603017:33
*** Guest48852 has joined #openstack-nova17:34
*** crushil has joined #openstack-nova17:34
mriedemartom: +217:37
openstackgerritStephen Finucane proposed openstack/nova: console: Move proxies to 'console/proxy'  https://review.openstack.org/40819217:38
artommriedem, phast17:38
*** vladikr has joined #openstack-nova17:39
mdboothI didn't notice this one missed -3: https://review.openstack.org/#/c/423392/17:40
mdboothIt's trivial, but it needs to go in to Ocata because it's upgrade related.17:40
mdboothmriedem: Is there anywhere I can add stuff like ^^^ in your etherpad?17:41
mdboothmriedem: Or maybe somewhere else?17:41
*** crushil has quit IRC17:43
*** kfarr has quit IRC17:43
* mdbooth shoved it in a new section at the bottom just in case.17:44
jaypipesmriedem: k, doing final run of tests now for the Ironic inventory patch.17:45
jaypipesmriedem: hopefully will be pushed in <30 minutes17:45
dansmithartom: link me to the actual fix?17:45
*** rmart04 has joined #openstack-nova17:45
artomdansmith, https://review.openstack.org/#/c/424759/17:45
*** lucasagomes is now known as lucas-afk17:46
openstackgerritSylvain Bauza proposed openstack/nova: Add prelude section for Ocata  https://review.openstack.org/42686417:51
bauzasshit, forgot the dep17:51
openstackgerritDavid Medberry proposed openstack/nova: Explain how allow_resize_to_same_host is useful  https://review.openstack.org/42653117:52
*** liverpooler has quit IRC17:52
*** liverpooler has joined #openstack-nova17:53
dansmithartom: okay, no +1s on that from the API folks17:54
mriedemdansmith: api folks are pretty much all out right now17:54
mriedemeither holiday or tz17:54
artomdansmith, Chinese new year I think17:54
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Collected release notes for Ocata CellsV2  https://review.openstack.org/42473717:54
openstackgerritSylvain Bauza proposed openstack/nova: Add prelude section for Ocata  https://review.openstack.org/42686417:54
mriedemmdbooth: how would you feel about writing up something in the nova devref about the image cache?17:54
mriedemmdbooth: b/c as far as i can tell, that's not really documented anywhere17:55
dansmithmriedem: ugh, okay17:55
artomdansmith, though gmann should be around, and he said he'd look at the new patchset17:55
dansmithmriedem: are you gonna review/+2 that? I usually get those guys to look before I hit stuff like that17:55
dansmithartom: okay17:55
mriedemit should be pretty straightforward though, i just haven't gotten to it yet since i was reviewing the spec first17:55
dansmithshould we wait for him to +1?17:55
dansmithokay17:55
mriedemdansmith: yes it's on the list17:55
mriedemgmann won't be around until tonight17:55
dansmithmriedem: well, I'll wait for you, but I've gone through it17:56
artomAt least they weeded out the obvious stuff in a first pass, hopefully it's been adresses like they wanted :S17:56
artom*addressed17:56
*** bvanhav has quit IRC17:57
artomAnd at this point, what's another microversion if we mess up again :D17:58
*** baoli has quit IRC17:58
*** baoli has joined #openstack-nova17:59
*** rmart04 has quit IRC17:59
*** armax has joined #openstack-nova18:02
*** harlowja has joined #openstack-nova18:02
*** ralonsoh has quit IRC18:02
*** derekh has quit IRC18:03
*** efoley_ has joined #openstack-nova18:04
*** mvk has joined #openstack-nova18:07
ekhugen-question on the cells patch that just went in https://review.openstack.org/#/c/319379/84 the test in api.py of cells.enable sounds like it's a check between cells v1 and cells v2, but isn't that option for cells enabled at all?  If cells is disabled, it goes down the schedule_and_build_instances path and eventually gets caught in that there's no host-to-cell mapping, right?18:07
*** efoley__ has quit IRC18:08
dansmithekhugen-: cells.enabled is cellsv1 only18:09
*** yamahata has joined #openstack-nova18:10
dansmithekhugen-: cellsv2 has no "enable" flag as it's always "on"18:10
ekhugen-dansmith so there's no choice for "no cells (v1 or v2) at all" anymore, right?18:11
dansmithekhugen-: it's complicated (tm)18:11
dansmithekhugen-: there is no turning cellsv2 on or off, and if you have cellsv1 you still have cellsv2 enabled, but inside your existing cellsv1 cells18:11
dansmithfor now18:11
dansmitheventually there will be a migration path out for cellsv1 users to remove the extra layer18:11
mriedemdansmith: reviewed https://review.openstack.org/#/c/426826/18:12
*** rmart04 has joined #openstack-nova18:12
*** efoley_ has quit IRC18:12
*** rmart04 has quit IRC18:12
mriedemwe only test n-1 computes but do we have anything anywhere that says we don't support n-2?18:13
mriedemi.e. the assertion here https://review.openstack.org/#/c/423392/2/nova/virt/libvirt/imagecache.py@a13718:13
*** bvanhav has joined #openstack-nova18:14
*** jpena is now known as jpena|off18:15
*** sudipto_ has quit IRC18:15
*** sudswas__ has quit IRC18:15
dansmithmriedem: it's always been the rule18:15
*** unicell has joined #openstack-nova18:15
mriedemdansmith: is that documenteD?18:16
mriedemsans the big D18:16
dansmithprobably in the upgrades devref somewhere, I can go look in a sec18:16
sdaguemriedem: the upgrades tag for the TC defines things that way18:17
*** ijw has joined #openstack-nova18:17
sdaguethat's probably the most definitive answer from an OpenStack wide perspective - https://governance.openstack.org/tc/reference/tags/assert_supports-upgrade.html18:17
ekhugen-thanks dansmith18:17
*** alexpilotti has joined #openstack-nova18:18
dansmithmriedem: thanks for those notes, I'll hit it again in a sec18:18
openstackgerritJay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory  https://review.openstack.org/40447218:18
mriedemhttp://docs.openstack.org/developer/nova/upgrade.html just mentions n and n+1, nothing about no n-218:18
dansmithmriedem: I'm refreshing the cells reno now18:18
jaypipesjroll, JayF, mriedem, bauzas, dansmith, edleafe, johnthetubaguy, cdent: pls see latest rev above on Ironic inventory.18:18
mriedemsdague: yeah that says config from n-1 works in n18:19
cdentjaypipes: noted18:19
jrolljaypipes: thanks18:19
*** ijw has quit IRC18:19
mriedemsdague: not sure it's really the same as what i'm thinking, but maybe it implies the same restriction18:19
sdaguemriedem: so maybe https://governance.openstack.org/tc/reference/tags/assert_supports-rolling-upgrade.html should be explicit about n-118:19
*** ijw has joined #openstack-nova18:19
mriedemsdague: was mainly asking because of the assertion made here https://review.openstack.org/#/c/423392/2/nova/virt/libvirt/imagecache.py18:19
sdaguemriedem: well, I agree with service versions the code could probably enforce the n-1 instead of just hoping that's all that exists18:20
openstackgerritDan Smith proposed openstack/nova: Collected release notes for Ocata CellsV2  https://review.openstack.org/42473718:21
dansmithsdague: no, we can't use service version for that18:21
sdaguedansmith: ok18:22
*** alexpilotti has quit IRC18:22
dansmithsdague: unless we start recording which version goes with which release and just do a static check that is always having to get updated or something18:22
dansmithit doesn't make sense for CD people either18:22
*** htruta` is now known as htruta18:23
mriedemi'm not really following. https://review.openstack.org/#/c/423392/ mentions a commit in newton. couldn't the ocata change check the service version based on when that newton commit happened to know if it's in there or not?18:23
mriedemi.e. if you're min compute is past https://review.openstack.org/#/c/320910/18:23
dansmithmriedem: you can use the service version to check for specific things, sure18:24
*** crushil has joined #openstack-nova18:24
dansmithmriedem: using it to tell that you're adding a mitaka compute to an ocata-only deployment isn't easy18:24
mriedemadding?18:25
dansmithchecks in ocata to see if there are mitaka-level services could be done, but with a static version that would have to be updated over time18:25
dansmithI haven't looked at the actual patch, I'm just commenting on using it to detect N-218:25
mriedemok my point was, at the time that 2c389ccc8c266175a71a29358bec7fe219e64fe0 (the newton change referenced in the commit message) was merged, the service version was 11. so in the ocata change that depends on that newton change, i'd think you could only do the ocata thing if the minimum nova-compute service version in the deployment was > 1118:27
dansmithsure, that's not N-2 though right?18:28
dansmiththat's what service_version is for18:28
mriedemright18:28
openstackgerritDan Smith proposed openstack/nova: Allow scheduler to run cell host discovery periodically  https://review.openstack.org/42682618:28
mriedemi'm confusing things. the n-2 question was really about an assumption made in that code that if we land this ocata patch, it means nothing is running n-2 mitaka code, which i'm saying we can't assume, and can figure out using service versions18:29
*** owalsh is now known as owalsh-afk18:29
*** kfarr has joined #openstack-nova18:29
openstackgerritChris Dent proposed openstack/nova: Add placement request id to log when GET aggregates  https://review.openstack.org/42687418:29
*** adisky_ has quit IRC18:29
* mriedem makes lunch18:29
dansmithokay18:30
* cdent has dinner18:30
*** cdent has quit IRC18:30
*** crushil has quit IRC18:30
*** crushil has joined #openstack-nova18:31
*** owalsh has joined #openstack-nova18:32
*** owalsh-afk has quit IRC18:33
*** Sukhdev has joined #openstack-nova18:36
*** crushil has quit IRC18:38
mriedempkoniszewski: ignore the earlier ping about https://bugs.launchpad.net/nova/+bug/1658877 - you didn't report that18:40
openstackLaunchpad bug 1658877 in OpenStack Compute (nova) "live migration failed with XenServer as hypervisor" [High,In progress] - Assigned to Bob Ball (bob-ball)18:40
edleafejaypipes: reviewing now18:40
mriedemBobBall: on ^ it's not a specific nova change that regressed this right?18:40
mriedemBobBall: it's just that nova doesn't support xenserver 7?18:41
mriedemi'm not sure what "1) the change in network name" means18:41
*** tyrola has quit IRC18:46
*** lpetrut has quit IRC18:46
*** foexle has quit IRC18:46
*** tyrola has joined #openstack-nova18:47
mriedemdiana_clarke: suggestion on wording in https://review.openstack.org/#/c/422171/18:52
diana_clarkemriedem: looking. I'm over here struggling with english wording too.18:53
*** browne has joined #openstack-nova18:54
*** sree has joined #openstack-nova18:56
diana_clarkemriedem: oh that... yeah. I remember trying to come up with something better but was really mostly just concerned with not lying about return values.18:58
*** sree has quit IRC19:01
openstackgerritDan Smith proposed openstack/nova: Allow scheduler to run cell host discovery periodically  https://review.openstack.org/42682619:08
zer0c00lThe _heal_instance_info_cache is just going to heal the first instance on the list, https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L588119:10
zer0c00lIs that the right behaviour of this method?19:10
zer0c00lIt loops through instance_uuids, picks up the healable instance and breaks out of that loop19:11
zer0c00lwhich means only first healable instance will get its info cache refreshed19:11
zer0c00lAm i missing something?19:11
dansmithzer0c00l: the next time it runs it uses the list it made, until the list is empty, then gets a new list19:12
dansmithi.e. one heal per cycle19:12
zer0c00ldansmith: i see, so one heal per cycle is by design19:12
dansmithyes19:13
zer0c00ldansmith: we have the interval set to 1800 in our production, so for us it takes forever19:13
zer0c00li guess i will have to reduce that number to something sane19:13
mriedemzer0c00l: so change your interval19:13
zer0c00lyes19:13
zer0c00lthanks19:13
dansmithzer0c00l: the healing really shouldn't need to happen anyway, but yeah19:13
mriedemdefault is 60 seocnds19:13
dansmithit's supposed to be slow19:13
mriedem*seconds19:13
zer0c00lokay19:14
mriedemoh wait maybe it's different for that one19:14
mriedemi guess it's 60 http://docs.openstack.org/newton/config-reference/compute/config-options.html19:14
*** stewie925 has joined #openstack-nova19:16
*** liangy has joined #openstack-nova19:16
*** Cristina_ has joined #openstack-nova19:17
mriedemartom: the fact that the DeviceTaggingTest test in tempest is setup with a max of 2.latest is really confusing19:18
mriedemin that it still works19:18
artommriedem, indeed...19:19
artomI can only assume Tempest doesn't use the max microversion when actually sending the request19:20
*** CristinaPauna has quit IRC19:20
artom(But does use to check what tests to run)19:20
mriedemyeah it uses 2.32 http://logs.openstack.org/51/421251/13/check/gate-tempest-dsvm-neutron-full-ssh/60bb925/logs/tempest.txt.gz#_2017-01-30_15_19_16_67219:20
mriedemin newton, max_microversion is capped for compute to 2.38 http://logs.openstack.org/51/421251/13/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton/7071652/logs/tempest_conf.txt.gz19:21
mriedembut it still looks like tempest just uses the min19:22
mriedeme.g. 2.37 here http://logs.openstack.org/51/421251/13/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton/7071652/logs/tempest.txt.gz#_2017-01-30_15_29_23_67019:22
artomThat can be something we propose to Tempest - to use the max instead of the min19:23
openstackgerritDan Smith proposed openstack/nova: Allow scheduler to run cell host discovery periodically  https://review.openstack.org/42682619:26
mriedem"which re-adds the tag in all the right places." heh19:27
mriedemthat makes me think dirty thoughts for some reason19:27
*** lpetrut has joined #openstack-nova19:28
*** raunak has joined #openstack-nova19:28
*** jaosorior is now known as jaosorior_lunch19:28
artommriedem, gooood...?19:28
artomAt least I didn't word it about reinserting the missing (dict) member19:29
mriedemactually it makes me think of this https://www.youtube.com/watch?v=FAyDmJvjxbg19:29
mriedemis there anyway we can talk about a bag of dicts?19:30
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell_and_hosts  https://review.openstack.org/42142019:30
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance  https://review.openstack.org/42143619:30
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 delete_cell  https://review.openstack.org/42145919:30
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 discover_hosts  https://review.openstack.org/42144219:30
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 list_cells  https://review.openstack.org/42145819:30
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 create_cell  https://review.openstack.org/42144419:30
*** burgerk has quit IRC19:30
artomWe could conceivable describe the request body as a bad of dicts19:30
*** burgerk has joined #openstack-nova19:32
*** owalsh is now known as owalsh-brb19:38
mriedemooo i see abug19:42
mriedemartom: comments in https://review.openstack.org/#/c/424759/19:46
*** mkoderer has quit IRC19:46
mriedemartom: the -1 is for https://review.openstack.org/#/c/424759/9/nova/api/openstack/compute/schemas/servers.py@31 for sure19:46
artommriedem, "If we need further clarification we have this I guess..." oy with the sarcasm ;)19:47
mriedemwell,19:47
mriedemnot trying to be sarcastic,19:47
mriedemif you look at the api-ref19:48
artomHeh, you are right though19:48
mriedemlook at the description for networks.tag here http://developer.openstack.org/api-ref/compute/?expanded=create-server-detail#id719:48
artomI think I was trying to be generic, because... reasons, I guess?19:48
mriedemand block_device_mapping_v2.tag19:48
mriedemgeneric is fine, and what you have in the api-ref is good19:48
mriedemi'm on the fence about over documenting this to death19:49
artomSurely more docs is better?19:49
mriedemprobably19:50
mriedemyou might want to make those specific parameter types then though19:50
artomWhat would we gain by mentioning it less? Someone's bound to run into this, we might as well make the explanation of wtf happened as accessible as possible19:50
mriedemso for device_network_tag you just make the comment about 2.32-2.36, and for device_bdm_tag you make the comment about 2.3219:50
mriedemknow what i mean?19:51
artomAha, and then set the correct min and max version on each19:51
mriedemyeah19:51
artomYep, I agree19:51
mriedemsince device_tag is used for both right now19:51
artomWell19:51
artommin, max, and min again19:51
artomIf I can manage that19:51
mriedemfwiw i don't know if the sphinx plugin supports a max_version19:52
mriedemsdague: do you know if there is a max_version for https://review.openstack.org/#/c/424759/9/api-ref/source/parameters.yaml@1506 ?19:52
*** owalsh-brb is now known as owalsh19:52
mriedemoh there is19:52
mriedemnvm19:52
mriedemartom: i don't think you can do min and max and then min again19:52
mriedemso probably just min-version: 2.32, and then notes in the description about the snafu between 2.32 and 2.4219:53
*** tbachman has quit IRC19:54
artommriedem, maybe if I split it again into nic_tag_2.32-2.37 and another one...19:54
artomWorth it?19:54
mriedemthen we have the same parameter listed in the table twice19:54
artomOr will it just be more confusing then writing it in English words19:54
artom*than19:54
mriedemi think english be gooder19:54
mriedemsorry, *me think*19:55
*** jheroux has quit IRC20:00
*** haplo37 has quit IRC20:01
artomThe schema deepcopy thing is a minefield :(20:04
artomYou caught the network uuid checking, presumably because you wrote 2.37 yourself20:04
mriedemartom: exactly20:04
mriedemartom: the tempest guys stopped doing the deepcopy i think20:05
mriedemfor this very reason20:05
mriedemoomichi: right? ^20:05
artomMaybe I'll remove the network_type refactoring then and stick to copy-pasting20:06
*** crushil has joined #openstack-nova20:07
oomichimriedem: yeah, you are right. gmann takes care of it on Tempest side20:08
oomichimriedem: that requrest came from sdague and mtreinish because the deepcopy of schema seems unreadable to know whole schema on each microversion20:09
*** READ10 has quit IRC20:09
*** haplo37 has joined #openstack-nova20:10
*** stewie925 has quit IRC20:11
*** vsaienko has joined #openstack-nova20:14
*** ijw has quit IRC20:16
*** ijw has joined #openstack-nova20:16
mriedemhmm, what's a better place for the nova-status CLI in the release notes, features or upgrades?20:17
mriedemi assume the upgrades section gets more reads20:17
dansmithyeah, probably20:18
dansmithhopefully anyway20:18
dansmiththe placement patches got reset so they're still in the running20:19
dansmithI'm going to wander off for a bit and will come back to check on them20:19
openstackgerritArtom Lifshitz proposed openstack/nova: Pass APIVersionRequest to extensions  https://review.openstack.org/42587620:20
openstackgerritArtom Lifshitz proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37  https://review.openstack.org/42475920:20
*** dtp has joined #openstack-nova20:21
*** alexpilotti has joined #openstack-nova20:21
*** AAR has joined #openstack-nova20:23
*** AAR has left #openstack-nova20:23
*** jose-phillips has quit IRC20:23
edleafejaypipes: Found a test failure in https://review.openstack.org/#/c/404472/ due to an incomplete test I added. Do you want me to remove it, or are you working on anything else with that patch?20:24
*** baoli has quit IRC20:25
*** sambetts is now known as sambetts|afk20:25
*** sdague_ has joined #openstack-nova20:25
*** alexpilotti has quit IRC20:26
*** baoli has joined #openstack-nova20:26
*** baoli has quit IRC20:27
*** jose-phillips has joined #openstack-nova20:28
bauzasmriedem: I wrote about the nova-status in the prelude20:29
*** dimtruck is now known as zz_dimtruck20:29
bauzasmriedem: https://review.openstack.org/#/c/426864/20:30
mriedemwas coincidentally reading that now20:30
jaypipesedleafe: please go ahead!20:30
*** _ducttape_ has joined #openstack-nova20:31
bauzasmriedem: about the nova-status CLI, I think we should provide a 'feature' section20:31
bauzasmriedem: "upgrade" section is just for discussing about what to need to modify before upgrading20:31
bauzasmriedem: but you could still add a separate section if you want too20:32
mriedembauzas: i assume operators are more likely to read the upgrades release notes first,20:32
mriedemand we want them using nova-status prior to upgrading20:32
*** xinli has joined #openstack-nova20:32
mriedemso seems best to just have it in the upgrade section20:32
bauzasmriedem: sure, but the upgrades section will be huge20:32
mriedemi didn't really want to mention the exact same thing in both ugprades and features20:32
bauzasmriedem: so that's why I provided the prelude20:32
mriedemit already is...20:32
bauzasmriedem: so I think operators would see that by the prelude20:33
dtpanyone need anything?  cup of coffee, etc20:33
*** ducttape_ has quit IRC20:34
mriedemjust had one thanks20:34
mriedembauzas: ok i guess i can rebase my reno on top of yours20:34
*** bvanhav has quit IRC20:34
mriedembauzas: and your reno needs a rebase too because your change and dan's change below it are both adding something to the prelude about cells v220:34
bauzasmriedem: fine by me, which rebase?20:35
mriedemyour change isn't on top of the latest version of https://review.openstack.org/#/c/424737/20:35
*** baoli has joined #openstack-nova20:36
bauzasmriedem: okay20:36
dtpok. let me know if there's anything i can do20:38
*** pradk has left #openstack-nova20:38
openstackgerritMatt Riedemann proposed openstack/nova: Add release note for nova-status upgrade check CLI  https://review.openstack.org/42691120:38
*** kfarr_ has joined #openstack-nova20:39
*** kfarr has quit IRC20:39
*** kfarr_ has quit IRC20:40
*** kfarr has joined #openstack-nova20:40
*** diana_clarke has quit IRC20:42
mriedembauzas: i suppose we still want to get something into nova-status for warning if CoreFilter/RamFilter/DiskFilter aren't in filter_scheduler.enabled_filters...20:43
*** marst has joined #openstack-nova20:44
bauzasmriedem: mmm, it could be nice, agreed20:44
bauzasmriedem: I can write that if you want20:44
bauzasmriedem: you mean, adverting to modify the ratios then ?20:45
mriedemyeah something along those lines,20:45
mriedem(1) is the filter_scheduler being used,20:45
mriedem(2) are any of those things NOT in enabled_filters20:45
mriedem(3) for each filter that's not enabled, check it's related allocation ratio and see if it's the default value and if so, issue a warning20:46
mriedemsomething likethat20:46
*** owalsh_ has joined #openstack-nova20:46
bauzasyeah, like I wrote in the note20:46
*** owalsh has quit IRC20:46
bauzasvery good point20:46
bauzasmriedem: just one point, where operators are running the CLI ?20:46
mriedemwe don't know yet20:47
bauzasmriedem: they need to do that in not in a computer, right? rather close to the other services, I guess20:47
mriedemas we haven't integrated nova-status into the CI system yet20:47
mriedemwell, it's really, give me your full config20:47
*** owalsh_ is now known as owalsh20:47
mriedemso api db, main db, compute config, scheduler config, all of it20:47
mriedemand probably run it in a venv or container20:48
mriedemso you can install and run it in isolation from the newton deployment code (which doesn't have the CLI)20:48
bauzasok20:49
mriedemi'm thinking about writing a wip patch to grenade to run nova-status in the from-netwon script, after we'd done our necessary setup for ocata20:49
dansmithmriedem: did you abandon that devstack patch that sets the db host to the primary?20:50
mriedemyeah20:50
mriedemi can find it20:50
mriedemdansmith: this? https://review.openstack.org/#/c/421399/20:51
dansmithmriedem: yeah, can you un-abandon? I'll need that to start up a second cell on the subnode20:51
*** eharney has quit IRC20:52
mriedemdone20:52
dansmiththanks20:53
mriedemthis was my patch for creating a 2nd cell https://review.openstack.org/#/c/420976/20:53
mriedemwhich is out of date now20:53
mriedemwhich also needed https://review.openstack.org/#/c/420973/20:53
openstackgerritEd Leafe proposed openstack/nova: placement: RT now adds proper Ironic inventory  https://review.openstack.org/40447220:53
dansmithah, okay, I hadn't seen that one20:53
edleafejaypipes: cdent: johnthetubaguy: ^^ fixed the functional test20:54
jaypipesedleafe: danke20:55
bauzasmriedem: dansmith: FYI, merging notes with both prelude sections are most of the time very difficult because they could be misgenerated20:55
bauzasthat's why we add the ">" section header20:55
mriedembauzas: so i guess take the prelude out of dan's and move it into yours20:55
dansmithbauzas: okay, I added prelude text at mriedem's request20:56
dansmithbut yeah, feel free to take it20:56
bauzasokay, just wanted to ask you folks before20:56
bauzasand explain why reno is giving you -1 on its job20:56
bauzas(well, guessing why)20:56
bauzasyeah that http://logs.openstack.org/37/424737/10/check/gate-nova-releasenotes/20a3d9a/console.html#_2017-01-30_18_28_37_03139020:57
dansmithack20:57
bauzasdansmith: I'll push a new rev by deleting your prelude section and putting it in my patch20:59
*** diana_clarke has joined #openstack-nova20:59
*** owalsh_ has joined #openstack-nova20:59
dansmithbauzas: s/deleting/stealing/20:59
dansmithsure20:59
*** owalsh has quit IRC21:00
dansmithbauzas: you keep stealing american work and trump will put france on "the list"21:00
bauzasdansmith: yeah, I'm stealing a job21:00
*** owalsh_ is now known as owalsh21:00
bauzasbut21:00
bauzasour French beauty queen won a Trump meetup, so...21:01
*** liverpooler has quit IRC21:01
dansmithand it's after 1700 in france, so you're a criminal too!21:01
bauzaspoor me21:01
*** _ducttape_ has quit IRC21:02
cburgesslol21:02
*** ducttape_ has joined #openstack-nova21:02
*** eglynn has quit IRC21:05
*** vsaienko has quit IRC21:05
diana_clarkemriedem: Sorry, I meant linking to http://docs.openstack.org/developer/nova/cells.html for more information. And my internet keeps dropping on me, so I may have missed your response.21:06
*** eharney has joined #openstack-nova21:06
*** gouthamr has quit IRC21:07
*** gszasz has quit IRC21:09
*** vsaienko has joined #openstack-nova21:10
*** zz_dimtruck is now known as dimtruck21:11
openstackgerritSylvain Bauza proposed openstack/nova: Collected release notes for Ocata CellsV2  https://review.openstack.org/42473721:11
openstackgerritSylvain Bauza proposed openstack/nova: Add prelude section for Ocata  https://review.openstack.org/42686421:11
bauzasthere21:12
mriedemdiana_clarke: sorry i'm not following21:12
bauzasI added a Co-Authored-By tag, so I won't be in jail when I am in ATL21:12
*** crushil has quit IRC21:12
* bauzas whispers21:12
diana_clarkemriedem: No doubt; that wasn't very clear of me ;)21:13
diana_clarkemriedem: I'm wondering if I should add a few opening paragraphs to the cells v2 commands. Making it clearer which commands should be run when and in what order.21:15
*** slaweq has quit IRC21:15
diana_clarkemriedem: Perhaps adding a high level bucket overview: http://paste.openstack.org/show/596936/21:16
diana_clarkelinking to http://docs.openstack.org/developer/nova/cells.html etc21:16
*** xinli has quit IRC21:16
*** slaweq has joined #openstack-nova21:17
*** jaosorior_lunch is now known as jaosorior21:21
*** crushil has joined #openstack-nova21:21
*** sdague_ has quit IRC21:22
mriedemdiana_clarke: oh,21:25
mriedemdiana_clarke: from the man page or from http://docs.openstack.org/developer/nova/cells.html#setup-of-cells-v2 ?21:25
mriedemi think we definitely want ^ pointing at the man pages21:26
*** xinli has joined #openstack-nova21:26
*** kaisers has quit IRC21:27
*** cleong has quit IRC21:27
diana_clarkemriedem: I was talking about a few overview paragraphs at the top of the cells v2 sections here: doc/source/man/nova-manage.rst21:28
diana_clarkesection21:28
*** dtp has quit IRC21:28
mriedemdiana_clarke: i'm a bit torn on that, not wanting to duplicate something there with what we have in http://docs.openstack.org/developer/nova/cells.html21:28
mriedemdiana_clarke: i think i'd prefer the more detailed big picture stuff in http://docs.openstack.org/developer/nova/cells.html and then link to the man pages for details on the individual commands21:29
mriedemi've noticed the see also in http://docs.openstack.org/developer/nova/man/nova-manage.html#see-also - that, or something like it, should probably refer to http://docs.openstack.org/developer/nova/cells.html21:30
diana_clarkemriedem: Cool, perhaps I'll work on updates to developer/nova/cells.html instead. Thanks!21:30
mriedemsure, np. i'm not docs writer so i won't claim to know the right answer21:31
mriedems/not/no/21:31
openstackgerritMatt Riedemann proposed openstack/nova: Add release note for nova-status upgrade check CLI  https://review.openstack.org/42691121:31
*** slaweq has quit IRC21:32
*** slaweq has joined #openstack-nova21:32
openstackgerritSarafraj Singh proposed openstack/nova-specs: Send service token along with user tokens  https://review.openstack.org/42693321:32
*** kfarr has quit IRC21:34
*** Jeffrey4l__ has joined #openstack-nova21:34
*** rhagarty has joined #openstack-nova21:34
*** Jeffrey4l_ has quit IRC21:35
*** gouthamr has joined #openstack-nova21:35
bauzasmriedem: dansmith: mmm, I had a thought related to cellsV221:36
bauzasmriedem: dansmith: given we get the list of compute nodes in the scheduler when not having all the computes being Ocata, it could be a problem if we have more than 1 cell, right?21:37
bauzasmriedem: dansmith: I mean, a general ComputeNodeList.get_all()21:38
*** sdague has quit IRC21:38
*** catintheroof has quit IRC21:40
*** jose-phillips has quit IRC21:41
mriedemfirst thing that comes to mind is we aren't supporting multiple cells really in ocata21:41
mriedemwe don't test it21:41
mriedemthe scheduler will pull from whichever main database is setup in the nova.conf for the scheduler node21:41
*** catintheroof has joined #openstack-nova21:42
bauzasmriedem: right, that I know21:42
mriedemdansmith: scheduler is global in cells v2 right?21:42
bauzasmriedem: I just wanted to explain that we could needing to work on that for Pike21:42
bauzasmriedem: that's something we agreed21:43
*** rtheis has joined #openstack-nova21:43
*** rtheis has quit IRC21:43
bauzasmriedem: but for example, I'm writing the nova-status patch, and I need to lookup at the compute nodes to see their ratios21:43
mriedemhaving a list of cellsv2-related things to work on in pike would be good for the ptg etherpad21:43
bauzasmriedem: so in a cellsV2 world, I need to lookup all cells21:44
mriedembauzas: then you do it via the cell mappings21:44
mriedemthere is already a check doing that same type of thing in nova-status21:44
mriedemthe RPs check21:44
bauzasmriedem: yup, that's how we do currently for nova-status21:44
bauzasmriedem: I saw that, hence my point above21:44
bauzasmriedem: because the scheduler isn't doing that21:44
mriedemyeah21:45
*** satyar has quit IRC21:45
bauzasI mean, we possibly have a lot of things that do ComputeNodeList.get_sth()21:45
bauzasin the API at least21:45
bauzasfor hypervisors21:45
mriedemwell, this is why we have a caveat in the release notes about things just not working with multiple cells right now21:45
bauzasokay, just wanted to make sure we keep that in mind for Pike21:45
mriedemput it in the ptg etherpad21:45
bauzasack21:46
mriedemhttps://etherpad.openstack.org/p/nova-ptg-pike21:46
bauzasalready in my tabs21:46
bauzas50+21:46
*** baoli has quit IRC21:46
mriedemL55 and L57 are about cells v2 things - we should just throw those into their own cells v2 topics section21:46
dansmithmriedem: yup21:46
*** timello has quit IRC21:46
*** jose-phillips has joined #openstack-nova21:47
mriedembtw, kablammo http://logs.openstack.org/26/426926/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/04aad64/logs/grenade.sh.txt.gz#_2017-01-30_21_41_51_41721:48
mriedemah crap21:48
mriedemcan we start placement before the nova services in grenade?21:49
mriedemwe should be able to right?21:49
*** samueldmq has joined #openstack-nova21:49
mriedemi.e. https://review.openstack.org/#/c/424730/18/projects/60_nova/upgrade.sh@86 before start_nova_api21:49
dansmithshould be able to21:49
* mriedem tinkers21:50
bauzaswhere is the gate for that change ? /me looking21:50
*** abalutoiu has joined #openstack-nova21:51
bauzasarf, still there after 6 hours, wow21:51
mriedemok https://review.openstack.org/#/c/426926/2/projects/60_nova/upgrade.sh21:52
*** catinthe_ has joined #openstack-nova21:52
mriedemshould be fine since that's what the 55_placement was for21:52
*** tbachman has joined #openstack-nova21:52
*** vsaienko has quit IRC21:53
*** liangy has quit IRC21:53
*** catintheroof has quit IRC21:53
*** kaisers has joined #openstack-nova21:57
*** armax has quit IRC21:57
*** baoli has joined #openstack-nova21:58
openstackgerritEric Fried proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif)  https://review.openstack.org/42251221:59
*** armax has joined #openstack-nova21:59
*** lpetrut has quit IRC22:02
*** breitz has quit IRC22:03
*** breitz has joined #openstack-nova22:03
*** catintheroof has joined #openstack-nova22:04
openstackgerritEmilien Macchi proposed openstack/nova: DNM - tripleo CI test  https://review.openstack.org/42694922:05
mriedemsean-k-mooney: do you know of anything that we need to get into os-vif before creating the stable/ocata branch? http://lists.openstack.org/pipermail/openstack-dev/2017-January/111135.html22:05
*** eharney has quit IRC22:05
*** toure has quit IRC22:07
*** catinthe_ has quit IRC22:07
openstackgerritMatt Riedemann proposed openstack/nova: Collected release notes for Ocata CellsV2  https://review.openstack.org/42473722:08
openstackgerritMatt Riedemann proposed openstack/nova: Add release note for nova-status upgrade check CLI  https://review.openstack.org/42691122:08
openstackgerritMatt Riedemann proposed openstack/nova: Add prelude section for Ocata  https://review.openstack.org/42686422:08
*** eglynn has joined #openstack-nova22:09
*** foexle has joined #openstack-nova22:09
bauzasmriedem: so, I'm about to leave, but I'll provide the nova-status check tomorrow morning22:09
mriedemok22:09
bauzasmriedem: just a quick question, do you think I should use the current _check_placement() method or write a new one?22:10
mriedemdansmith: question in https://review.openstack.org/#/c/424737/12/releasenotes/notes/ocata-cellsv2-support-4b3b5e70e76bc756.yaml22:10
mriedembauzas: new one22:10
bauzask22:10
mriedem_check_enabled_filters or something22:10
diana_clarke"barber-pole sorting", that's new to me22:10
dansmithmriedem: I guess it doesn't seem important to me, given that none of the cells stuff is relevant before the upgrade is finished, but I'll say something like "once fully upgraded" or something22:12
mriedemwfm, just don't forget to pull the latest version since both myself and bauzas have sullied your original change22:13
openstackgerritEric Fried proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif)  https://review.openstack.org/42251222:14
*** toure has joined #openstack-nova22:15
*** Guest48852 has quit IRC22:15
*** jose-phillips has quit IRC22:17
*** catinthe_ has joined #openstack-nova22:17
*** eglynn has quit IRC22:18
*** catinth__ has joined #openstack-nova22:18
*** catintheroof has quit IRC22:18
*** jose-phillips has joined #openstack-nova22:19
*** crushil has quit IRC22:20
*** catinthe_ has quit IRC22:22
mriedemmelwitt: you might have thoughts on https://review.openstack.org/#/c/421420/722:22
melwittmriedem: thanks, will look through22:23
*** crushil has joined #openstack-nova22:24
*** tblakes has quit IRC22:24
*** tyrola has quit IRC22:24
*** tyrola has joined #openstack-nova22:25
*** unicell has quit IRC22:27
*** foexle has quit IRC22:27
*** catinth__ has quit IRC22:29
*** Jack_I has quit IRC22:30
openstackgerritSarafraj Singh proposed openstack/nova-specs: Send service token along with user tokens  https://review.openstack.org/42693322:30
*** crushil has quit IRC22:31
mriedemdiana_clarke: were you going to address these comments? https://review.openstack.org/#/c/421444/6/doc/source/man/nova-manage.rst22:33
mriedemthe rest of the series lgtm22:33
*** edmondsw has quit IRC22:34
*** thorst_ has quit IRC22:36
*** annegentle has quit IRC22:37
*** unicell has joined #openstack-nova22:39
*** baoli has quit IRC22:40
diana_clarkemriedem: sorry, I missed that one. I'll circle back tonight and address the feedback from melwitt too.22:41
mriedemWHY GOD http://logs.openstack.org/26/426926/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/8b536f3/logs/grenade.sh.txt.gz#_2017-01-30_22_20_29_46622:44
mriedemoh sorry god, i know why22:45
*** armax has quit IRC22:49
openstackgerritEric Fried proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif)  https://review.openstack.org/42251222:51
*** armax has joined #openstack-nova22:51
*** jamesdenton has quit IRC22:51
*** marst has quit IRC22:53
mriedemartom: you want to address the nits in https://review.openstack.org/#/c/424759/ or shall i?22:55
*** slaweq has quit IRC22:56
*** Swami_ has joined #openstack-nova22:56
*** Swami has quit IRC22:56
*** slaweq has joined #openstack-nova22:58
*** Swami_ has quit IRC23:00
*** slaweq has quit IRC23:02
*** slaweq has joined #openstack-nova23:04
*** unicell has quit IRC23:05
*** thorst_ has joined #openstack-nova23:06
*** tbachman has quit IRC23:07
*** ircuser-1 has quit IRC23:08
*** mriedem has quit IRC23:10
*** thorst_ has quit IRC23:10
*** mdrabe has quit IRC23:11
*** kuzko has quit IRC23:13
*** kuzko has joined #openstack-nova23:14
*** unicell has joined #openstack-nova23:14
*** ircuser-1 has joined #openstack-nova23:20
*** ircuser-1 has quit IRC23:21
*** ducttape_ has quit IRC23:21
*** dtp has joined #openstack-nova23:23
*** armax has quit IRC23:29
*** armax has joined #openstack-nova23:30
*** armax has quit IRC23:31
*** claudiub has quit IRC23:32
*** Swami_ has joined #openstack-nova23:33
*** Swami has joined #openstack-nova23:33
*** Swami__ has joined #openstack-nova23:33
*** jaypipes has quit IRC23:34
*** sileht has quit IRC23:37
*** armax has joined #openstack-nova23:39
*** tbachman has joined #openstack-nova23:41
*** sileht has joined #openstack-nova23:41
*** xinli has quit IRC23:41
openstackgerritmelanie witt proposed openstack/nova: Detach volume after deleting instance with no host  https://review.openstack.org/34061423:42
*** kuzko has quit IRC23:44
*** xyang1 has quit IRC23:44
*** jamesdenton has joined #openstack-nova23:45
*** kuzko has joined #openstack-nova23:45
*** gouthamr has quit IRC23:49
*** catintheroof has joined #openstack-nova23:50
*** Swami_ has quit IRC23:52
*** Swami has quit IRC23:52
*** Swami has joined #openstack-nova23:53
*** baoli has joined #openstack-nova23:55
*** dimtruck is now known as zz_dimtruck23:57
*** jamielennox is now known as jamielennox|away23:58
*** jamielennox|away is now known as jamielennox23:59
*** tbachman has quit IRC23:59

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