Thursday, 2016-06-30

*** muawiakhan has joined #openstack-nova00:01
mikalHeh00:02
mikalBut I have the lurgy!00:02
*** rfolco_ has joined #openstack-nova00:02
*** rfolco_ has quit IRC00:02
*** rfolco_ has joined #openstack-nova00:03
dansmithmikal: I will be your best distant friend.. beyond the reach of the pox00:03
*** shaohe_feng has quit IRC00:03
*** rfolco_ has quit IRC00:03
*** amotoki has joined #openstack-nova00:03
mikalHeh00:03
*** shaohe_feng has joined #openstack-nova00:03
*** rfolco_ has joined #openstack-nova00:03
mikalI would review if gerrit was working00:03
mikalBut it might be my internets00:03
openstackgerritDan Smith proposed openstack/nova: Device tagging API support  https://review.openstack.org/30451000:04
*** rfolco_ has quit IRC00:04
dansmithit's not just you00:04
dansmithgerrit seems to be constipated from here as well00:04
artomOh, is it Gerrit FF failure time?00:05
*** thorst has joined #openstack-nova00:05
dansmithjust about00:06
*** muawiakhan has quit IRC00:06
dansmithmikal: I just saw you pushed up another patch for my and other's comments on the vendordata thing00:06
*** amotoki has quit IRC00:08
*** rama_y has quit IRC00:08
*** thorst has quit IRC00:12
*** shaohe_feng has quit IRC00:13
*** shaohe_feng has joined #openstack-nova00:14
*** gongysh has joined #openstack-nova00:15
*** thorst has joined #openstack-nova00:16
*** stewie925 has quit IRC00:16
*** thorst has quit IRC00:17
*** rfolco_ has joined #openstack-nova00:23
*** rfolco_ has quit IRC00:23
*** rfolco__ has joined #openstack-nova00:23
*** rfolco__ has quit IRC00:23
*** shaohe_feng has quit IRC00:23
*** shaohe_feng has joined #openstack-nova00:24
artomBoom, Tempest test!00:26
*** Apoorva has joined #openstack-nova00:26
artomdansmith, and it passes locally on your code!00:26
dansmithwoot00:26
* artom gets... supper? Yeah, why not, let's call it supper.00:27
dansmithheh00:28
*** Sukhdev has quit IRC00:30
*** Apoorva has quit IRC00:31
mriedemartom: dansmith: i have a query00:31
mriedemshould the rest api schema for tags be the same as we have for the server tags api?00:32
*** MikeG451 has quit IRC00:32
* mikal goes to look at that review00:32
*** shaohe_feng has quit IRC00:34
*** shaohe_feng has joined #openstack-nova00:34
*** Jeffrey4l has joined #openstack-nova00:36
dansmithmriedem: I think being consistent with those is probably a good idea00:36
*** baoli has joined #openstack-nova00:38
*** baoli has quit IRC00:39
*** yamahata has quit IRC00:41
*** ducttape_ has joined #openstack-nova00:42
*** shaohe_feng has quit IRC00:44
*** shaohe_feng has joined #openstack-nova00:44
*** kevinzh has joined #openstack-nova00:46
*** Sukhdev has joined #openstack-nova00:47
mriedemartom: dansmith: on the api change, -1 is for the schema thing https://review.openstack.org/#/c/304510/00:48
mriedemi have some nits in the unit tests about maybe mocking some things that don't need to be mocked, but those aren't a big deal00:49
mriedembut that one is really close, should be mergable tonight00:49
*** shaohe_feng has quit IRC00:54
*** shaohe_feng has joined #openstack-nova00:55
*** baoli has joined #openstack-nova00:56
*** baoli has quit IRC00:57
*** links has joined #openstack-nova00:57
*** woodster_ has quit IRC00:59
*** shihanzhang has joined #openstack-nova00:59
*** gongysh has quit IRC00:59
*** phuongnh has joined #openstack-nova01:03
*** shaohe_feng has quit IRC01:04
*** shaohe_feng has joined #openstack-nova01:05
*** shaohe_feng has quit IRC01:15
*** shaohe_feng has joined #openstack-nova01:15
*** greghaynes has quit IRC01:16
*** thorst has joined #openstack-nova01:17
*** rfolco__ has joined #openstack-nova01:17
*** rfolco__ has quit IRC01:18
*** ducttape_ has quit IRC01:19
artommriedem, the mocks are because I couldn't get create to finish without them01:19
artomI might be missing something, but I needed all those mocked otherwise they threw errors01:19
eliqiaohi mriedem danpb, evening, thanks for your reviewing on "libvirt perf" patches :)01:22
*** code-R has quit IRC01:22
mriedemeliqiao: np01:23
*** ducttape_ has joined #openstack-nova01:24
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451101:25
*** shaohe_feng has quit IRC01:25
artommriedem, don't panic that's ^^ just a rebase01:25
*** amotoki has joined #openstack-nova01:25
artomBecause *someone* didn't push the entire series01:25
*** thorst has quit IRC01:26
*** shaohe_feng has joined #openstack-nova01:26
*** rrecio_ has joined #openstack-nova01:26
eliqiaomriedem: BTW, we will have Bug smash day next week in China, I wonder if there is any hot bug list that you can pointing to? I would like to help to do something.01:27
*** cheneydc has joined #openstack-nova01:27
mriedemeliqiao: you should ask markus_z01:27
mriedemhe runs the bug meeting01:28
eliqiaomriedem: ya, thanks, good to konw that, will try to catch him my evening time.01:29
*** amotoki has quit IRC01:30
mriedemartom: just for that, -1 on the metadata api change01:30
artomoh noes now it's been made personal01:30
*** baoli has joined #openstack-nova01:30
*** greghaynes has joined #openstack-nova01:30
*** baoli has quit IRC01:31
artomOh crap, there's an actual for serious -1 though :D01:31
mriedemheh yeah01:32
mriedemi'm reviewing more now01:32
*** thorst has joined #openstack-nova01:33
*** thorst has quit IRC01:33
*** thorst has joined #openstack-nova01:33
*** wangqun has joined #openstack-nova01:34
*** rrecio_ has quit IRC01:34
*** ducttape_ has quit IRC01:34
*** Apoorva has joined #openstack-nova01:34
*** shaohe_feng has quit IRC01:35
*** Apoorva has quit IRC01:36
*** shaohe_feng has joined #openstack-nova01:36
*** thorst has quit IRC01:38
*** jichen has joined #openstack-nova01:38
*** browne has quit IRC01:39
*** yuikotakadamori has joined #openstack-nova01:41
*** EinstCrazy has joined #openstack-nova01:41
*** ssurana has quit IRC01:42
*** gongysh has joined #openstack-nova01:44
*** shaohe_feng has quit IRC01:45
*** shaohe_feng has joined #openstack-nova01:46
mriedemartom: ok, done reviewing the metadata api change01:46
*** Sukhdev has quit IRC01:50
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3)  https://review.openstack.org/33576501:51
*** gjayavelu has joined #openstack-nova01:52
*** TravT has quit IRC01:54
*** baoli has joined #openstack-nova01:54
*** sdake has joined #openstack-nova01:54
*** yamahata has joined #openstack-nova01:54
*** baoli has quit IRC01:55
*** shaohe_feng has quit IRC01:56
*** thorst has joined #openstack-nova01:56
*** thorst has quit IRC01:56
*** shaohe_feng has joined #openstack-nova01:57
*** thorst has joined #openstack-nova01:57
mriedemgus: for all intents and purposes, should we just mark this complete? https://blueprints.launchpad.net/nova/+spec/privsep01:58
*** code-R has joined #openstack-nova01:59
*** code-R has quit IRC01:59
*** sacharya has joined #openstack-nova01:59
*** houming has joined #openstack-nova01:59
openstackgerritZhenyu Zheng proposed openstack/nova: Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040602:01
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Use oslo_config new type PortOpt for port options  https://review.openstack.org/23834702:03
artommriedem, https://review.openstack.org/#/c/304511/67/nova/api/metadata/base.py@371 am I understanding correctly that that's more of an improvement for future work?02:04
openstackgerritLi Yingjun proposed openstack/nova: Microversion 2.32 adds pagination support for hypervisors  https://review.openstack.org/32694002:04
*** sdake has quit IRC02:04
mriedemartom: yeah02:04
gusmriedem: We haven't actually shipped anything using privsep, and even if we had, then there's still cleanup work on the obsolete rootwrap bits...02:04
mriedemgus: yeah, but the cleanup is going to be multiple releases02:05
mriedemwhich hasn't started02:05
mriedemonce the grenade issue is sorted out then we'll be using os-brick with privsep, and os-vif with privsep02:05
*** thorst has quit IRC02:05
mriedembut those are going to be handled via the bug i think02:05
gusmriedem: I agree that I don't think the spec is helping anymore, so we could certainly close it if you just want to clean house.02:05
mriedemgus: yup, i'm just cleaning up blueprints for newton02:05
mriedemthanks02:05
*** markvoelker has quit IRC02:06
*** shaohe_feng has quit IRC02:06
*** shaohe_feng has joined #openstack-nova02:07
*** ducttape_ has joined #openstack-nova02:07
*** dingboopt has quit IRC02:08
*** cheneydc has quit IRC02:08
*** Apoorva has joined #openstack-nova02:08
mriedemmelwitt: is https://blueprints.launchpad.net/nova/+spec/cells-mq-connection-switching done ?02:09
*** _ducttape_ has joined #openstack-nova02:09
*** gjayavelu has quit IRC02:11
*** ducttape_ has quit IRC02:11
*** sacharya_ has joined #openstack-nova02:12
*** TravT has joined #openstack-nova02:12
*** rama_y has joined #openstack-nova02:14
*** sacharya has quit IRC02:15
*** shaohe_feng has quit IRC02:16
openstackgerritbailin.zhang proposed openstack/nova: Delete reduplicate code in test_compute_mgr.py  https://review.openstack.org/33110802:16
*** shaohe_feng has joined #openstack-nova02:17
*** kevin_zh has joined #openstack-nova02:18
*** kevinzh has quit IRC02:18
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451102:19
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging API support  https://review.openstack.org/30451002:19
*** tjones has joined #openstack-nova02:21
*** gongysh has quit IRC02:25
*** shaohe_feng has quit IRC02:26
*** shaohe_feng has joined #openstack-nova02:27
*** _ducttape_ has quit IRC02:32
*** ljxiash has joined #openstack-nova02:32
*** ljxiash has quit IRC02:33
*** jbernard has quit IRC02:33
*** ljxiash has joined #openstack-nova02:33
*** ljxiash has quit IRC02:34
*** jraju has joined #openstack-nova02:34
*** ljxiash has joined #openstack-nova02:34
*** ljxiash has quit IRC02:34
*** ljxiash has joined #openstack-nova02:34
*** links has quit IRC02:36
*** rfolco__ has joined #openstack-nova02:36
*** rfolco__ has quit IRC02:36
mriedemmikal: a couple of easy +Ws here on the ones with the +2s https://review.openstack.org/#/q/topic:bp/rm-object-dict-compat-newton+status:open,n,z02:36
mriedemthen i'm going to mark that bp complete for newton02:36
mriedemit's just a cleanup series anyway02:36
*** ducttape_ has joined #openstack-nova02:37
*** shaohe_feng has quit IRC02:37
*** shaohe_feng has joined #openstack-nova02:37
*** ljxiash has quit IRC02:39
mriedemtjones: is the vmware subteam good with this? https://review.openstack.org/#/c/297574/02:39
*** ljxiash has joined #openstack-nova02:40
*** baoli has joined #openstack-nova02:41
*** keivn has joined #openstack-nova02:43
*** kevin_zh has quit IRC02:43
*** keivn is now known as kevin_zh02:43
*** kevinzh has joined #openstack-nova02:43
mriedemtjones: nevermind, -102:44
openstackgerritEli Qiao proposed openstack/nova: Mapping power_state from integer to string  https://review.openstack.org/33577602:44
*** baoli has quit IRC02:45
*** gjayavelu has joined #openstack-nova02:46
mriedemalaski: no tests needed for this? https://review.openstack.org/#/c/332216/02:46
*** Daisy has joined #openstack-nova02:47
*** shaohe_feng has quit IRC02:47
*** amotoki has joined #openstack-nova02:47
*** shaohe_feng has joined #openstack-nova02:47
tjonesmriedem:  lemme look just in case02:47
*** browne has joined #openstack-nova02:48
mriedemBobBall: ^02:48
tjonesmriedem: ok i'll ping rado on it02:49
mriedemtjones: tomorrow is non-priority feature freeze02:51
*** amotoki has quit IRC02:51
mriedemtjones: fyi02:51
tjonesi know :-)  he gets up soon (bulgaria)02:52
openstackgerritDao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console()  https://review.openstack.org/32815702:55
*** shaohe_feng has quit IRC02:57
*** shaohe_feng has joined #openstack-nova02:58
*** ducttape_ has quit IRC02:59
mriedemeliqiao: is there a dev list thread i should put in the nova meeting agenda for the bug smash?02:59
mriedemor some other wiki page or something?02:59
*** harlowja_at_home has joined #openstack-nova03:00
mriedemtristanC: kashyap: is anyone going to update this? https://review.openstack.org/#/c/327624/03:00
*** thorst has joined #openstack-nova03:03
*** jraju has quit IRC03:03
*** markvoelker has joined #openstack-nova03:06
*** shaohe_feng has quit IRC03:07
*** shaohe_feng has joined #openstack-nova03:08
*** Daisy has quit IRC03:09
*** Daisy has joined #openstack-nova03:09
*** thorst has quit IRC03:10
*** ljxiash has quit IRC03:10
*** ljxiash has joined #openstack-nova03:11
*** Daisy has quit IRC03:11
*** Daisy has joined #openstack-nova03:11
*** markvoelker has quit IRC03:11
*** Daisy has quit IRC03:16
mriedemartom: 11:17pm for you?03:16
artommriedem, yeah03:17
mriedemartom: ok, real quick, on https://review.openstack.org/#/c/304510/ the -1 is for the release note having the metadata api thing in it - that should go in the following change03:17
mriedemthat adds the meta api support03:18
artomI'll handle that feedback from you tomorrow, I think I'll post the lyrics to Rick Roll if I attempt to do anything else tonight03:18
*** shaohe_feng has quit IRC03:18
mriedemha03:18
mriedemfair enough03:18
dansmithartom: go to bed03:18
artomdansmith, finishing up the season finale of GoT03:18
artom:P03:18
*** shaohe_feng has joined #openstack-nova03:18
mriedemi could probably just update this myself real quick03:18
mriedembut mroning is fine03:18
dansmithmriedem: I can do it03:18
mriedem*morning03:18
mriedemdansmith: ok03:19
*** sree has joined #openstack-nova03:20
*** harlowja_at_home has quit IRC03:20
mriedemdansmith: also on the following meta api change https://review.openstack.org/#/c/304511/03:21
mriedemeasy -1 to address03:21
mriedemthen i'm +2 on that one03:21
dansmithokay03:21
mriedemgonna watch the daily show quick03:22
dansmithoh, don't miss that important piece of "news"03:22
*** harlowja_at_home has joined #openstack-nova03:22
* artom sleeps03:22
*** Apoorva has quit IRC03:23
*** Daisy has joined #openstack-nova03:24
*** ducttape_ has joined #openstack-nova03:28
*** shaohe_feng has quit IRC03:28
dansmithmriedem: here it comes03:28
openstackgerritDan Smith proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451103:28
*** sheel has joined #openstack-nova03:28
openstackgerritDan Smith proposed openstack/nova: Device tagging API support  https://review.openstack.org/30451003:28
dansmithI haven't touched these two other than these tweaks just now, so if you're still comfortable with me +Wing after you +2 I'll do that03:29
*** shaohe_feng has joined #openstack-nova03:29
openstackgerritRong Han proposed openstack/nova: libvirt: Optimize _get_instance_disk_info  https://review.openstack.org/29759803:30
*** jwcroppe has quit IRC03:30
*** amotoki has joined #openstack-nova03:36
*** ljxiash has quit IRC03:37
*** ljxiash has joined #openstack-nova03:37
*** shaohe_feng has quit IRC03:38
*** shaohe_feng has joined #openstack-nova03:39
mriedemdansmith: not with that daily show sarcasm03:40
dansmithfair enough03:40
*** ljxiash has quit IRC03:42
*** zz_dimtruck is now known as dimtruck03:42
*** zerda2 has joined #openstack-nova03:46
mriedemgmann_: alex_xu: oomichi: can you take a look at this? https://review.openstack.org/#/c/304510/63/nova/api/openstack/compute/schemas/block_device_mapping.py@77 i'm not sure if this is going to work given how the server tags extension does it's tag validation03:46
mriedemdansmith: ^ i'm not sure about that one so gonna leave it until one of the other guys can comment03:47
dansmithmriedem: okay cool03:47
*** shaohe_feng has quit IRC03:48
mriedemsomething slipped into the meta api change too that should have been in the rest api change03:49
mriedemso i'm gonna call it a night03:49
*** mriedem has quit IRC03:50
dansmithoh crap yeah03:50
*** amotoki has quit IRC03:50
*** Apoorva has joined #openstack-nova03:51
*** zhangjn has quit IRC03:51
*** shaohe_feng has joined #openstack-nova03:51
*** Daisy has quit IRC03:52
*** Daisy has joined #openstack-nova03:52
openstackgerritDan Smith proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451103:53
openstackgerritDan Smith proposed openstack/nova: Device tagging API support  https://review.openstack.org/30451003:53
*** Daisy_ has joined #openstack-nova03:55
*** Daisy has quit IRC03:55
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/33318103:55
*** zerda2 has quit IRC03:56
*** Daisy has joined #openstack-nova03:57
*** amotoki has joined #openstack-nova03:58
*** shaohe_feng has quit IRC03:59
*** code-R has joined #openstack-nova03:59
*** shaohe_feng has joined #openstack-nova03:59
*** zhangjn has joined #openstack-nova03:59
*** Daisy has quit IRC04:01
*** Daisy_ has quit IRC04:01
*** Daisy has joined #openstack-nova04:01
*** rama_y has quit IRC04:04
*** zhangjn has quit IRC04:05
*** tbachman has quit IRC04:05
*** tbachman_ has joined #openstack-nova04:05
*** pwnall has quit IRC04:05
*** markvoelker has joined #openstack-nova04:07
*** thorst has joined #openstack-nova04:08
*** shaohe_feng has quit IRC04:09
*** shaohe_feng has joined #openstack-nova04:09
*** tiendc_ has joined #openstack-nova04:09
*** gjayavel_ has joined #openstack-nova04:10
*** zhangjn has joined #openstack-nova04:11
*** markvoelker has quit IRC04:12
tiendc_dansmith: for the bp ironic serial console support --> https://blueprints.launchpad.net/nova/+spec/ironic-serial-console-support04:12
tiendc_dansmith: this bp is depended on another RFE in Ironic04:13
tiendc_and the patch-set take a long time for review https://review.openstack.org/#/c/328157/04:13
tiendc_dansmith: we are currently working on both side nova and ironic04:13
tiendc_dansmith: but today is the deadline of non-priority feature freeze, can you take a look and think about move this bp to exception list04:14
*** gjayavelu has quit IRC04:14
*** thorst has quit IRC04:15
*** ducttape_ has quit IRC04:16
*** amotoki has quit IRC04:18
*** shaohe_feng has quit IRC04:19
*** shaohe_feng has joined #openstack-nova04:22
*** Sukhdev has joined #openstack-nova04:25
*** Apoorva has quit IRC04:26
*** amotoki has joined #openstack-nova04:26
*** jwcroppe has joined #openstack-nova04:28
*** ayoung has quit IRC04:29
*** ljxiash has joined #openstack-nova04:29
*** shaohe_feng has quit IRC04:29
*** shaohe_feng has joined #openstack-nova04:30
*** jwcroppe has quit IRC04:30
*** diga has joined #openstack-nova04:30
*** harlowja_at_home has quit IRC04:30
*** jwcroppe has joined #openstack-nova04:33
*** ljxiash has quit IRC04:36
*** shaohe_feng has quit IRC04:40
*** shaohe_feng has joined #openstack-nova04:40
*** jwcroppe has quit IRC04:44
*** sdake has joined #openstack-nova04:49
*** irenab has quit IRC04:50
*** shaohe_feng has quit IRC04:50
*** shaohe_feng has joined #openstack-nova04:50
*** sdake_ has joined #openstack-nova04:51
*** sdake_ has quit IRC04:53
*** sdake has quit IRC04:54
*** abalutoiu has quit IRC04:54
*** dimtruck is now known as zz_dimtruck04:54
*** xenogear has quit IRC04:55
*** zhs_ss has joined #openstack-nova04:57
openstackgerritmelanie witt proposed openstack/nova: WIP Get cell service version for compute RPC calls  https://review.openstack.org/32690604:57
*** zhs_s has quit IRC04:57
*** salv-orlando has joined #openstack-nova05:00
*** shaohe_feng has quit IRC05:00
*** shaohe_feng has joined #openstack-nova05:00
*** jwcroppe has joined #openstack-nova05:01
*** salv-orl_ has quit IRC05:03
openstackgerritAihua Edward Li proposed openstack/nova: Always clean up network resource when VM creation failed close-bug: #1597596  https://review.openstack.org/33578805:03
openstackbug 1597596 in OpenStack Compute (nova) "network not always cleaned up when spawning VMs" [Undecided,New] https://launchpad.net/bugs/1597596 - Assigned to Aihua Edward Li (aihuaedwardli)05:03
*** sudipto has joined #openstack-nova05:04
*** s1061123 has joined #openstack-nova05:04
*** jwcroppe has quit IRC05:05
*** ratailor has joined #openstack-nova05:06
*** GB21 has joined #openstack-nova05:06
*** moshele has joined #openstack-nova05:06
*** ljxiash has joined #openstack-nova05:10
*** shaohe_feng has quit IRC05:10
*** klkumar has joined #openstack-nova05:11
*** shaohe_feng has joined #openstack-nova05:11
*** gjayavel_ has quit IRC05:12
*** prateek has joined #openstack-nova05:12
*** thorst has joined #openstack-nova05:13
*** abalutoiu has joined #openstack-nova05:13
*** zerda2 has joined #openstack-nova05:14
*** prateek has quit IRC05:17
*** lpetrut has joined #openstack-nova05:18
*** edand has joined #openstack-nova05:18
*** thorst has quit IRC05:21
*** irenab has joined #openstack-nova05:21
*** shaohe_feng has quit IRC05:21
*** shaohe_feng has joined #openstack-nova05:21
*** rama_y has joined #openstack-nova05:22
openstackgerritDao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console()  https://review.openstack.org/32815705:22
*** sdake has joined #openstack-nova05:24
*** rcernin has joined #openstack-nova05:26
*** rama_y has quit IRC05:26
*** rama_y has joined #openstack-nova05:27
*** sacharya_ has quit IRC05:27
*** sacharya has joined #openstack-nova05:27
*** vtech has joined #openstack-nova05:30
*** jwcroppe has joined #openstack-nova05:30
*** shaohe_feng has quit IRC05:31
*** shaohe_feng has joined #openstack-nova05:32
*** code-R has quit IRC05:32
*** code-R has joined #openstack-nova05:32
*** liverpooler has joined #openstack-nova05:33
*** rcernin has quit IRC05:34
*** abalutoiu has quit IRC05:34
*** jichen has quit IRC05:35
*** prateek has joined #openstack-nova05:35
*** jichen has joined #openstack-nova05:35
*** sdake_ has joined #openstack-nova05:36
*** sdake has quit IRC05:37
*** code-R_ has joined #openstack-nova05:39
*** Daisy has quit IRC05:41
*** Daisy has joined #openstack-nova05:41
*** shaohe_feng has quit IRC05:41
*** shaohe_feng has joined #openstack-nova05:42
*** code-R has quit IRC05:42
*** fragatina has joined #openstack-nova05:46
*** ducttape_ has joined #openstack-nova05:46
*** hoangcx has joined #openstack-nova05:47
*** Sukhdev has quit IRC05:47
*** bswartz has quit IRC05:47
*** s1061123 has quit IRC05:47
*** chandankumar has joined #openstack-nova05:48
*** abalutoiu has joined #openstack-nova05:49
*** nkrinner_afk is now known as nkrinner05:50
*** Daisy has quit IRC05:51
*** Daisy has joined #openstack-nova05:51
*** shaohe_feng has quit IRC05:51
*** ducttape_ has quit IRC05:52
*** shaohe_feng has joined #openstack-nova05:52
*** Daisy has quit IRC05:53
*** code-R has joined #openstack-nova05:53
*** Daisy has joined #openstack-nova05:53
*** code-R_ has quit IRC05:54
*** code-R_ has joined #openstack-nova05:56
*** fragatina has quit IRC05:57
*** fragatina has joined #openstack-nova05:57
*** Daisy has quit IRC05:58
*** vtech has quit IRC05:58
*** code-R has quit IRC05:59
*** pbandark has joined #openstack-nova06:01
*** Daisy has joined #openstack-nova06:02
*** shaohe_feng has quit IRC06:02
*** rama_y has quit IRC06:02
*** shaohe_feng has joined #openstack-nova06:03
*** pradk has quit IRC06:03
*** mfedosin has joined #openstack-nova06:04
*** muawiakhan has joined #openstack-nova06:04
*** diga has quit IRC06:05
gmann_johnthetubaguy: alex_xu alaski oomichi dansmith : can you check this (serial console support for Ironic driver) - https://review.openstack.org/#/c/328157/606:06
gmann_johnthetubaguy: alex_xu alaski oomichi dansmith : i feel that should depends on Ironic patch which implement that on ironic side so that we do not publish the feature before ironic implementation gets merged06:07
gmann_need your opinion06:08
*** eantyshev has left #openstack-nova06:08
*** harlowja_at_home has joined #openstack-nova06:08
*** markvoelker has joined #openstack-nova06:09
*** rcernin has joined #openstack-nova06:09
*** paul-carlton2 has joined #openstack-nova06:09
*** shaohe_feng has quit IRC06:12
*** paul-carlton2 has quit IRC06:12
*** paul-carlton2 has joined #openstack-nova06:13
*** markvoelker has quit IRC06:13
*** shaohe_feng has joined #openstack-nova06:15
*** gjayavelu has joined #openstack-nova06:15
*** rook- has quit IRC06:17
*** chhavi has joined #openstack-nova06:19
*** vtech has joined #openstack-nova06:20
*** andreas_s has joined #openstack-nova06:22
*** shaohe_feng has quit IRC06:22
*** shaohe_feng has joined #openstack-nova06:23
*** liverpooler has quit IRC06:24
*** sridharg has joined #openstack-nova06:24
*** liverpooler has joined #openstack-nova06:24
openstackgerritoctopuszhang proposed openstack/nova: Allow irrelevant,self-defined specs in ComputeCapacityFilter  https://review.openstack.org/31730606:26
*** sdake_ has quit IRC06:30
*** chhavi has quit IRC06:31
*** shaohe_feng has quit IRC06:32
*** shaohe_feng has joined #openstack-nova06:33
*** pcaruana has joined #openstack-nova06:37
*** gjayavelu has quit IRC06:37
*** harlowja_at_home has quit IRC06:39
*** belmoreira has joined #openstack-nova06:40
*** shaohe_feng has quit IRC06:43
*** shaohe_feng has joined #openstack-nova06:43
*** jichen has quit IRC06:44
*** ducttape_ has joined #openstack-nova06:48
*** zenoway has joined #openstack-nova06:48
*** luis5tb has joined #openstack-nova06:50
*** sacharya has quit IRC06:51
*** shaohe_feng has quit IRC06:53
*** ducttape_ has quit IRC06:53
*** shaohe_feng has joined #openstack-nova06:54
*** mpavone has joined #openstack-nova06:56
*** Daisy has quit IRC06:57
*** Daisy has joined #openstack-nova06:57
*** jwcroppe has quit IRC06:59
*** Daisy has quit IRC07:00
*** Daisy has joined #openstack-nova07:00
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3)  https://review.openstack.org/33576507:01
*** paul-carlton2 has quit IRC07:01
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (2)  https://review.openstack.org/33541107:01
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (1)  https://review.openstack.org/33440907:01
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (4)  https://review.openstack.org/33582007:01
*** sahid has joined #openstack-nova07:01
*** paul-carlton2 has joined #openstack-nova07:01
*** shaohe_feng has quit IRC07:03
*** diga has joined #openstack-nova07:03
*** shaohe_feng has joined #openstack-nova07:04
*** aloga has quit IRC07:06
*** aloga has joined #openstack-nova07:06
*** markvoelker has joined #openstack-nova07:10
*** mfeoktistov has joined #openstack-nova07:11
*** thorst has joined #openstack-nova07:11
*** shaohe_feng has quit IRC07:13
*** markvoelker has quit IRC07:14
*** shaohe_feng has joined #openstack-nova07:14
*** tesseract- has joined #openstack-nova07:15
*** hoangcx has quit IRC07:15
*** roeyc has joined #openstack-nova07:15
*** pilgrimstack has joined #openstack-nova07:17
gmann_alex_xu: dansmith : regarding this - https://review.openstack.org/#/c/304510/6407:19
gmann_alex_xu: dansmith we are not doing tag on volume-attach and interface-attach API ?07:19
gmann_those were mentioned in spec07:20
*** thorst has quit IRC07:20
gmann_in this - https://specs.openstack.org/openstack/nova-specs/specs/newton/approved/virt-device-role-tagging.html07:20
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: Allow multi-attach in compute api  https://review.openstack.org/27104707:22
openstackgerritIldiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758707:22
*** gszasz has joined #openstack-nova07:22
*** paul-carlton2 has quit IRC07:23
*** shaohe_feng has quit IRC07:24
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051307:25
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365407:25
*** shaohe_feng has joined #openstack-nova07:25
*** GB21 has quit IRC07:25
*** jichen has joined #openstack-nova07:28
*** jlanoux has joined #openstack-nova07:30
*** jlanoux has quit IRC07:30
*** jlanoux has joined #openstack-nova07:31
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051307:33
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365407:33
*** shaohe_feng has quit IRC07:34
*** shaohe_feng has joined #openstack-nova07:34
*** fragatina has quit IRC07:36
*** paul-carlton2 has joined #openstack-nova07:37
*** zz_dimtruck is now known as dimtruck07:37
*** adisky has joined #openstack-nova07:38
*** dedery has joined #openstack-nova07:41
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata  https://review.openstack.org/33199607:41
*** dedery has joined #openstack-nova07:41
*** itamarl has joined #openstack-nova07:41
*** vtech has quit IRC07:41
*** liverpooler has quit IRC07:43
*** liverpooler has joined #openstack-nova07:43
*** penick_ has joined #openstack-nova07:44
*** penick has quit IRC07:44
*** penick_ is now known as penick07:44
*** mfedosin has quit IRC07:44
*** shaohe_feng has quit IRC07:44
*** shaohe_feng has joined #openstack-nova07:45
*** GB21 has joined #openstack-nova07:46
*** jwcroppe has joined #openstack-nova07:48
*** ducttape_ has joined #openstack-nova07:50
*** fragatina has joined #openstack-nova07:50
*** code-R has joined #openstack-nova07:52
*** code-R_ has quit IRC07:52
*** ducttape_ has quit IRC07:54
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051307:54
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365407:54
*** shaohe_feng has quit IRC07:54
*** shaohe_feng has joined #openstack-nova07:55
openstackgerritEvgeny Antyshev proposed openstack/nova: Release note for vzstorage volume driver  https://review.openstack.org/33543907:56
*** browne has quit IRC07:58
*** zzzeek has quit IRC08:00
bauzasgood morning Nova08:00
*** pilgrimstack1 has joined #openstack-nova08:00
*** pilgrimstack has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** jichen has quit IRC08:01
*** jichen has joined #openstack-nova08:02
openstackgerritZhihai Song proposed openstack/nova: Transform instance.restore notifications  https://review.openstack.org/33197208:02
openstackgerritZhihai Song proposed openstack/nova: Transform instance.power_on notifications  https://review.openstack.org/32974608:03
*** derekh has joined #openstack-nova08:04
*** shaohe_feng has quit IRC08:05
*** jlanoux_ has joined #openstack-nova08:05
*** shaohe_feng has joined #openstack-nova08:05
openstackgerritjichenjc proposed openstack/nova: refactory query user_id code for quota-sets  https://review.openstack.org/28016508:06
*** pilgrimstack1 has quit IRC08:06
*** pilgrimstack has joined #openstack-nova08:06
*** jlanoux has quit IRC08:08
*** gmann_ has quit IRC08:08
*** gmann has joined #openstack-nova08:09
*** mjura has quit IRC08:10
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c  https://review.openstack.org/30334508:10
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c  https://review.openstack.org/30334508:10
*** markus_z has joined #openstack-nova08:10
*** mjura has joined #openstack-nova08:10
*** markvoelker has joined #openstack-nova08:11
*** takashin has joined #openstack-nova08:11
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end)  https://review.openstack.org/25028308:12
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (error)  https://review.openstack.org/32805508:13
openstackgerritJohn Hua proposed openstack/nova: POC: WIP: XenAPI: device tagging  https://review.openstack.org/33378108:13
*** shaohe_feng has quit IRC08:15
*** markvoelker has quit IRC08:15
*** zhs_ss has quit IRC08:15
*** zhs_ss has joined #openstack-nova08:16
*** jamielennox is now known as jamielennox|away08:16
*** danpb has joined #openstack-nova08:17
*** shaohe_feng has joined #openstack-nova08:17
*** thorst has joined #openstack-nova08:18
*** shz has joined #openstack-nova08:18
*** shihanzhang has quit IRC08:19
openstackgerritGene Kuo proposed openstack/nova: Fix nova delete with EFI booted VMs  https://review.openstack.org/33551208:20
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Implement stable disk device instance rescues  https://review.openstack.org/27028908:22
*** shaohe_feng has quit IRC08:25
*** thorst has quit IRC08:25
*** shaohe_feng has joined #openstack-nova08:26
*** zhangyufei has joined #openstack-nova08:26
*** Daisy has quit IRC08:27
*** Daisy has joined #openstack-nova08:28
*** tbachman_ has quit IRC08:28
*** lucas-afk is now known as lucasagomes08:30
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051308:31
*** tbachman has joined #openstack-nova08:31
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33554908:31
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365408:31
openstackgerritDao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console()  https://review.openstack.org/32815708:32
*** jwcroppe has quit IRC08:32
*** Daisy has quit IRC08:32
openstackgerritjichenjc proposed openstack/nova: Verify os-aggregates.inc on sample files  https://review.openstack.org/33311408:34
*** shaohe_feng has quit IRC08:35
*** shaohe_feng has joined #openstack-nova08:36
*** pradk has joined #openstack-nova08:37
*** muawiakhan has quit IRC08:37
*** muawiakhan has joined #openstack-nova08:37
openstackgerritJohn Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance  https://review.openstack.org/32697008:38
openstackgerritJohn Hua proposed openstack/nova: POC: WIP: XenAPI: device tagging  https://review.openstack.org/33378108:39
*** danpb has quit IRC08:41
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end)  https://review.openstack.org/25028308:42
*** danpb has joined #openstack-nova08:42
alex_xugmann: good point08:43
markus_zkaisers: Hi, I've seen your bug report https://bugs.launchpad.net/nova/+bug/159764408:43
openstackLaunchpad bug 1597644 in OpenStack Compute (nova) "Permissioned denied on console.log during instance startup" [Undecided,New]08:43
markus_zkaisers: The link you provided doesn't open for me.08:44
markus_zkaisers: I assume you mean this one http://78.46.57.153:8081/refs-changes-30-260930-40/ ?08:44
*** sheel has quit IRC08:45
*** shaohe_feng has quit IRC08:46
*** shaohe_feng has joined #openstack-nova08:46
markus_zkaisers: I was wondering why the domain XML has a "serial type=file" and a "console type=file" at the same time.08:47
*** ihrachys has joined #openstack-nova08:47
*** yassine has joined #openstack-nova08:49
*** yassine is now known as Guest8538308:49
*** ducttape_ has joined #openstack-nova08:50
*** s1061123 has joined #openstack-nova08:53
*** roeyc has quit IRC08:55
*** ducttape_ has quit IRC08:55
*** zenoway has quit IRC08:56
*** shaohe_feng has quit IRC08:56
*** jwcroppe has joined #openstack-nova08:56
*** shaohe_feng has joined #openstack-nova08:56
*** imcsk8_ has joined #openstack-nova08:57
*** bapalm has quit IRC08:57
*** zhs_s has joined #openstack-nova08:57
*** mnestratov|2 has joined #openstack-nova08:59
*** PaulMurray has joined #openstack-nova08:59
*** ildikov_ has joined #openstack-nova08:59
*** bapalm has joined #openstack-nova09:00
*** kfox1111 has joined #openstack-nova09:00
*** fragatina has quit IRC09:00
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end)  https://review.openstack.org/25028309:00
*** jwcroppe has quit IRC09:01
openstackgerritPaul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances  https://review.openstack.org/30819809:01
openstackgerritPaul Carlton proposed openstack/nova: API changes for live migration of rescued instance  https://review.openstack.org/32828009:01
*** mrsoul` has joined #openstack-nova09:02
*** migi_ has joined #openstack-nova09:02
*** bradm_ has joined #openstack-nova09:03
*** zehua- has joined #openstack-nova09:03
*** zenoway has joined #openstack-nova09:03
*** aloga has quit IRC09:04
*** aloga has joined #openstack-nova09:04
*** Dave_ has joined #openstack-nova09:04
*** auggy_ has joined #openstack-nova09:05
*** alex_xu_ has joined #openstack-nova09:06
*** markmc` has joined #openstack-nova09:06
*** shaohe_feng has quit IRC09:06
*** shaohe_feng has joined #openstack-nova09:06
*** mirrorbo1 has joined #openstack-nova09:07
*** sfinucan_ has joined #openstack-nova09:07
*** afazekas_ has joined #openstack-nova09:07
*** zhs_ss has quit IRC09:08
*** sudipto has quit IRC09:08
*** diogogmt has quit IRC09:08
*** mirrorbox has quit IRC09:08
*** imcsk8 has quit IRC09:08
*** ericksonsantos has quit IRC09:08
*** alex_xu has quit IRC09:08
*** mnestratov has quit IRC09:08
*** chenfei has quit IRC09:08
*** jinquan has quit IRC09:08
*** sfinucan has quit IRC09:08
*** FL1SK has quit IRC09:08
*** afazekas has quit IRC09:08
*** kfox1111_ has quit IRC09:08
*** j_king has quit IRC09:08
*** Dave has quit IRC09:08
*** bradm has quit IRC09:08
*** mordred has quit IRC09:08
*** mrsoul has quit IRC09:08
*** zehua has quit IRC09:08
*** auggy has quit IRC09:08
*** migi has quit IRC09:08
*** ildikov has quit IRC09:08
*** markmc has quit IRC09:08
*** rmk has quit IRC09:08
*** konetzed has quit IRC09:08
*** zehua- is now known as zehua09:08
*** tiendc_ has quit IRC09:09
*** rmk has joined #openstack-nova09:09
*** j_king has joined #openstack-nova09:09
*** sudipto has joined #openstack-nova09:09
*** ildikov_ is now known as ildikov09:09
*** edand has quit IRC09:09
*** auggy_ is now known as auggy09:10
*** chenfei has joined #openstack-nova09:10
*** mordred has joined #openstack-nova09:12
*** ericksonsantos has joined #openstack-nova09:14
*** jinquan has joined #openstack-nova09:15
*** Daisy has joined #openstack-nova09:16
*** FL1SK has joined #openstack-nova09:16
*** shaohe_feng has quit IRC09:16
*** shaohe_feng has joined #openstack-nova09:17
*** Daisy has quit IRC09:18
*** Daisy has joined #openstack-nova09:18
*** mrsoul` is now known as mrsoul09:19
*** matrohon has joined #openstack-nova09:20
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (error)  https://review.openstack.org/32805509:20
*** baoli has joined #openstack-nova09:22
*** thorst has joined #openstack-nova09:24
*** mfedosin has joined #openstack-nova09:25
*** paul-carlton has joined #openstack-nova09:25
*** paul-carlton2 has quit IRC09:26
*** konetzed has joined #openstack-nova09:26
*** paul-carlton has quit IRC09:26
*** baoli has quit IRC09:26
*** paul-carlton2 has joined #openstack-nova09:26
*** Daisy has quit IRC09:27
*** shaohe_feng has quit IRC09:27
*** Daisy has joined #openstack-nova09:27
*** shaohe_feng has joined #openstack-nova09:27
*** Dave_ is now known as Dave09:28
*** GB21 has quit IRC09:29
*** jinquan has quit IRC09:29
*** sameo has joined #openstack-nova09:29
*** thorst has quit IRC09:30
markus_zkaisers: I left a possible workaround|hack in the bug report.09:30
*** ljxiash has quit IRC09:32
openstackgerritLee Yarwood proposed openstack/nova: WIP - libvirt: Use the recreated config disk during a rescue  https://review.openstack.org/33588409:34
*** hoangcx has joined #openstack-nova09:34
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications samples  https://review.openstack.org/32886609:34
*** shaohe_feng has quit IRC09:37
*** konetzed has quit IRC09:37
*** shaohe_feng has joined #openstack-nova09:39
openstackgerritPaul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances  https://review.openstack.org/30819809:40
openstackgerritPaul Carlton proposed openstack/nova: API changes for live migration of rescued instance  https://review.openstack.org/32828009:41
bauzasgibi_: I had a question in https://review.openstack.org/#/c/313654/4109:45
bauzasgibi_: oh snap, nvm09:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33554909:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590109:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590209:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590309:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590409:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590509:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590609:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590709:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590809:45
openstackgerritDaniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug.  https://review.openstack.org/33590909:45
* danpb hopes that will speed up reproducing the non-deterministic live migration bug :-)09:46
*** shaohe_feng has quit IRC09:47
*** shaohe_feng has joined #openstack-nova09:47
andrearosadanpb: +1 for the ascii art09:49
PaulMurraydanpb, that's got to be the best commit message I have seen09:49
danpbPaulMurray: which one, they're all different :-)09:49
*** sree has quit IRC09:50
PaulMurraydanpb, I like the tyrannosaurus - probably says something about my character09:51
*** ljxiash has joined #openstack-nova09:51
artomgmann, alex_xu_, answer to your query in https://review.openstack.org/#/c/304510/ - I hope it it satisfactory :)09:52
*** breton has quit IRC09:52
kashyapAh, dan submitted multiple invocations in one go.09:53
alex_xu_artom: it is an agreement with the team?09:53
artomalex_xu_, hrmm, I don't think it's been explicitly described as such09:54
*** sameo has quit IRC09:56
artomalex_xu_, but for example here: https://review.openstack.org/#/c/303335/09:56
artomWe're talking about the changes that need to to happen to the config drive, and I understand it as dansmith and alaski being onboard09:57
artomBut best to wait until they are here :)09:57
*** shaohe_feng has quit IRC09:57
gmannartom: yea, actually spec still have those, that's why got confused09:57
artomgmann, for sure09:58
*** shaohe_feng has joined #openstack-nova09:58
gmannartom: Thanks :)09:58
*** GB21 has joined #openstack-nova09:58
artomI totally get the confusion. There's been specs that have been implemented over multiple cycles before, no?09:58
gmannartom: yea but here we are doing microversion on device tag and its always good to do all related changes in one version bump instead of multiple10:00
artomgmann, ah, yeah, good point. Definitely cleaner to not bump twice for similar things10:01
gmannyea10:01
PaulMurraytdurakov, what's happening with https://review.openstack.org/#/c/314932/ ?10:03
openstackgerritJohn Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance  https://review.openstack.org/32697010:04
artomWell, it's you guys's call :) On our side we'll grovel for an extension if the API thing is blocked and add tagged interface/volume if we get it.10:04
*** shhjwang has joined #openstack-nova10:05
*** wangqun has quit IRC10:06
openstackgerritJohn Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance  https://review.openstack.org/32697010:08
*** shaohe_feng has quit IRC10:08
*** david-lyle has quit IRC10:09
*** shaohe_feng has joined #openstack-nova10:09
*** david-lyle_ has joined #openstack-nova10:09
*** fragatina has joined #openstack-nova10:10
*** pilgrimstack has quit IRC10:12
openstackgerritJohn Garbutt proposed openstack/nova: Create _validate_requested_port_ids  https://review.openstack.org/32960010:16
openstackgerritJohn Garbutt proposed openstack/nova: Extract _validate_requested_network_ids  https://review.openstack.org/32960110:16
openstackgerritJohn Garbutt proposed openstack/nova: Extract _filter_hypervisor_macs  https://review.openstack.org/32956910:16
gibi_bauzas: thanks for the review10:17
*** fragatina has quit IRC10:17
*** roeyc has joined #openstack-nova10:17
*** lchen__ has joined #openstack-nova10:17
*** shaohe_feng has quit IRC10:18
*** lchen_ has quit IRC10:18
*** shaohe_feng has joined #openstack-nova10:19
openstackgerritJohn Garbutt proposed openstack/nova: Reduce complexity in _stub_allocate_for_instance  https://review.openstack.org/33208610:22
openstackgerritjichenjc proposed openstack/nova: Verify os-aggregates.inc on sample files  https://review.openstack.org/33311410:22
openstackgerritJohn Garbutt proposed openstack/nova: Reorder allocate_for_instance preamble  https://review.openstack.org/32985010:22
openstackgerritJohn Garbutt proposed openstack/nova: Make _validate_requested_network_ids return a dict  https://review.openstack.org/32984910:22
*** thorst has joined #openstack-nova10:28
*** shaohe_feng has quit IRC10:28
*** fragatina has joined #openstack-nova10:28
*** shaohe_feng has joined #openstack-nova10:29
*** sridharg has quit IRC10:31
*** hshiina has joined #openstack-nova10:32
*** fragatina has quit IRC10:32
*** fragatina has joined #openstack-nova10:33
*** mdavidson has quit IRC10:33
*** thorst has quit IRC10:35
*** smatzek has joined #openstack-nova10:36
openstackgerritAlexis Lee proposed openstack/nova: Make LM downtime params mutable  https://review.openstack.org/31925410:36
*** jichen has quit IRC10:37
*** hoangcx has quit IRC10:38
*** shaohe_feng has quit IRC10:38
*** jichen has joined #openstack-nova10:39
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (5)  https://review.openstack.org/33592110:40
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.update notification  https://review.openstack.org/32592310:40
openstackgerritJohn Garbutt proposed openstack/nova: Extract port create from allocate_for_instance  https://review.openstack.org/32896610:40
*** shaohe_feng has joined #openstack-nova10:41
gibi_johnthetubaguy, danpb: I had to rebase https://review.openstack.org/#/c/325923/ and fix the test due to a change on master. Could you re-check it please?10:42
*** frickler has quit IRC10:43
*** Guest79590 has quit IRC10:43
*** Daisy has quit IRC10:43
*** _fortis has quit IRC10:43
*** Daisy has joined #openstack-nova10:43
*** toabctl has quit IRC10:44
*** jichen_ has joined #openstack-nova10:44
danpbgibi_: so iiuc the change was instance_type_id being renamed to flavor_id ?10:45
*** jichen has quit IRC10:45
*** frickler has joined #openstack-nova10:45
*** toabctl has joined #openstack-nova10:46
gibi_danpb: that was a fix in the instance.delete patch propagated to the instance.update. But besides that some neutron v2 interaction changed on master that caused that the sample test needed adjustem in the NeutronFixture10:46
*** toabctl has quit IRC10:47
* danpb wishes gerrit could show you a proper diff-of-diffs between two versions10:47
*** jinquan has joined #openstack-nova10:47
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (end)  https://review.openstack.org/33592510:48
BobBalldanpb: +10010:48
gibi_danpb: unfortunately if a rebase has merge conflict then we cannot really separate the code coming in due to the rebase from the code needed to solve the conflict10:48
*** toabctl has joined #openstack-nova10:48
*** Daisy has quit IRC10:48
*** shaohe_feng has quit IRC10:49
openstackgerritJohn Garbutt proposed openstack/nova: Extract port create from allocate_for_instance  https://review.openstack.org/32896610:49
*** shaohe_feng has joined #openstack-nova10:49
*** jichen_ has quit IRC10:49
BobBallBut being able to diff(diff(base1,patch9,diff(base2,patch10)) would be very useful even if the two diffs aren't directly comparible because of a conflict...10:49
*** _fortis has joined #openstack-nova10:50
*** vtech has joined #openstack-nova10:50
sahidahahah danpb you looks to have fun time with that serie https://review.openstack.org/#/q/topic:mig-debug10:53
*** sridharg has joined #openstack-nova10:54
openstackgerritPaul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances  https://review.openstack.org/30819810:55
openstackgerritPaul Carlton proposed openstack/nova: API changes for live migration of rescued instance  https://review.openstack.org/32828010:55
kashyapsahid: Epic commit messages, eh :-)10:55
openstackgerritAlexis Lee proposed openstack/nova: Make LM downtime params mutable  https://review.openstack.org/31925410:55
paul-carlton2johnthetubaguy, would you be able to review the above patches again pls10:56
*** vtech has quit IRC10:56
openstackgerritJohn Garbutt proposed openstack/nova: Extract _update_ports_for_instance  https://review.openstack.org/32985110:56
johnthetubaguypaul-carlton2: I will try get to those again today, probably after lunch10:56
openstackgerritJohn Garbutt proposed openstack/nova: Extract port create from allocate_for_instance  https://review.openstack.org/32896610:56
paul-carlton2ok, thanks10:56
*** shaohe_feng has quit IRC10:59
*** shaohe_feng has joined #openstack-nova10:59
*** sameo has joined #openstack-nova10:59
*** salv-orl_ has joined #openstack-nova11:00
*** phuongnh has quit IRC11:01
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Make allocate_for_instance skip create option  https://review.openstack.org/33271211:02
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Make allocate_for_instance skip create option  https://review.openstack.org/33271211:03
openstackgerritJohn Garbutt proposed openstack/nova: Create _validate_requested_port_ids  https://review.openstack.org/32960011:03
openstackgerritJohn Garbutt proposed openstack/nova: Reduce complexity in _stub_allocate_for_instance  https://review.openstack.org/33208611:03
openstackgerritJohn Garbutt proposed openstack/nova: Extract _validate_requested_network_ids  https://review.openstack.org/32960111:03
openstackgerritJohn Garbutt proposed openstack/nova: Reorder allocate_for_instance preamble  https://review.openstack.org/32985011:03
openstackgerritJohn Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance  https://review.openstack.org/32697011:03
openstackgerritJohn Garbutt proposed openstack/nova: Extract _update_ports_for_instance  https://review.openstack.org/32985111:03
openstackgerritJohn Garbutt proposed openstack/nova: Extract port create from allocate_for_instance  https://review.openstack.org/32896611:03
openstackgerritJohn Garbutt proposed openstack/nova: Make _validate_requested_network_ids return a dict  https://review.openstack.org/32984911:03
openstackgerritJohn Garbutt proposed openstack/nova: Extract _filter_hypervisor_macs  https://review.openstack.org/32956911:03
*** salv-orlando has quit IRC11:03
*** salv-orl_ has quit IRC11:05
openstackgerritStephen Finucane proposed openstack/nova: Fixes compute API unit tests for python3  https://review.openstack.org/31148311:05
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end)  https://review.openstack.org/25028311:06
*** rook- has joined #openstack-nova11:06
*** sree has joined #openstack-nova11:08
*** shaohe_feng has quit IRC11:09
*** shaohe_feng has joined #openstack-nova11:10
*** sree has quit IRC11:12
*** EinstCrazy has quit IRC11:13
openstackgerritAlex Xu proposed openstack/nova: Allow irrelevant,self-defined specs in ComputeCapacityFilter  https://review.openstack.org/31730611:14
*** claudiub has joined #openstack-nova11:15
*** rtheis has joined #openstack-nova11:16
*** mirrorbo1 is now known as mirrorbox11:19
*** mirrorbox has joined #openstack-nova11:19
*** shaohe_feng has quit IRC11:19
*** shaohe_feng has joined #openstack-nova11:20
*** bvanhav has joined #openstack-nova11:22
*** irenab has quit IRC11:26
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (error)  https://review.openstack.org/32805511:28
*** ducttape_ has joined #openstack-nova11:29
*** mvk has quit IRC11:29
*** shaohe_feng has quit IRC11:30
*** smcginnis has quit IRC11:30
*** smcginnis has joined #openstack-nova11:30
*** shaohe_feng has joined #openstack-nova11:30
*** ericksonsantos has quit IRC11:38
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications samples  https://review.openstack.org/32886611:39
*** pilgrimstack has joined #openstack-nova11:39
*** shaohe_feng has quit IRC11:40
*** jinquan has quit IRC11:40
*** jinquan has joined #openstack-nova11:42
openstackgerritMatthew Booth proposed openstack/nova: Fix fake data returned by LibvirtDriverTestCase._disk_info  https://review.openstack.org/32836611:42
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Pass path to Image base class  https://review.openstack.org/33327111:42
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Improve mocking of Image objects in test_driver  https://review.openstack.org/33324211:42
openstackgerritMatthew Booth proposed openstack/nova: libvirt: rewrite test_rescue and test_rescue_config_drive  https://review.openstack.org/33596611:42
mdboothlyarwood: ^^^11:42
mdboothlyarwood: Just going to grab lunch. A quick look at the top of that stack should tell you if it's what you want.11:42
*** shaohe_feng has joined #openstack-nova11:43
*** jed56 has quit IRC11:45
*** sahid has quit IRC11:46
*** rfolco has joined #openstack-nova11:47
*** GB21 has quit IRC11:47
*** links has joined #openstack-nova11:48
*** links has quit IRC11:48
*** shaohe_feng has quit IRC11:50
*** sahid has joined #openstack-nova11:51
*** shaohe_feng has joined #openstack-nova11:51
*** svenkat has joined #openstack-nova11:51
*** thorst has joined #openstack-nova11:52
*** thorst has quit IRC11:53
*** thorst_ has joined #openstack-nova11:53
*** GB21 has joined #openstack-nova11:54
*** diga has quit IRC11:55
*** sdake has joined #openstack-nova11:55
*** ducttape_ has quit IRC11:57
*** sdake_ has joined #openstack-nova11:57
*** mvk has joined #openstack-nova11:57
*** eharney has quit IRC11:58
*** shaohe_feng has quit IRC12:00
*** sdake has quit IRC12:01
*** mgoddard_ has joined #openstack-nova12:01
*** fragatina has quit IRC12:01
*** shaohe_feng has joined #openstack-nova12:01
*** lucasagomes is now known as lucas-hungry12:03
*** jwcroppe has joined #openstack-nova12:03
*** mgoddard has quit IRC12:04
markus_zdanpb: https://review.openstack.org/#/c/334480/ has finally all the CI +1 it needs12:04
*** chenfei has quit IRC12:05
*** pilgrimstack has quit IRC12:05
danpbmarkus_z:  lol, i hate bugs like that where you stare at it for hours missing the obvious12:05
markus_zdanpb: Yeah, I was a little embarrassed. :/12:06
markus_zFunny thing is, I looked at it for a second and thought for myself "yeah, no issue, python uses the last variable as return value automatically" ...12:06
*** salv-orlando has joined #openstack-nova12:06
markus_zbad thing is, it doesn't. Scala does...12:06
markus_zSometimes it's bad to know multiple languages12:07
*** Fdaisuke has quit IRC12:07
*** hshiina has quit IRC12:07
*** hieulq_ has joined #openstack-nova12:07
*** hshiina has joined #openstack-nova12:07
*** Fdaisuke has joined #openstack-nova12:07
danpbmarkus_z: perl does too :-)12:07
*** Qiming has quit IRC12:07
*** qadaemon has joined #openstack-nova12:08
*** nicolasbock_ has quit IRC12:08
mdboothAnd Ruby12:08
*** nicolasbock has joined #openstack-nova12:09
*** qadaemon has quit IRC12:09
markus_zAh, I wasn't sure anymore with Ruby. But as Scala "borrowed" a lot from Ruby, it makes sense.12:09
* mdbooth has never liked that language feature, though.12:09
*** hieulq has quit IRC12:09
* mdbooth likes things to be explicit12:09
*** chenfei has joined #openstack-nova12:09
danpband ruby shamelessly copied perl :-)12:09
mdboothAnd perl copied everything simultaneously, including mutually exclusive design concepts :)12:10
*** Qiming has joined #openstack-nova12:10
markus_zHa! So perl is guilty one!12:10
mdboothperl: Yeah, I guess you could do it that way, too.12:11
*** shaohe_feng has quit IRC12:11
markus_z:D12:11
* mdbooth is fond of perl12:11
*** shaohe_feng has joined #openstack-nova12:11
markus_zyeah? I've looked at perl code samples and I was a little revolted by it.12:12
mdboothIt's very easy to write bad perl, but that doesn't mean you have to :)12:12
mdboothAlso, you have to love regular expressions. Pretty sure that's a rule.12:13
*** pilgrimstack has joined #openstack-nova12:13
*** raildo-afk is now known as raildo12:13
*** markvoelker has joined #openstack-nova12:13
*** daemontool has joined #openstack-nova12:14
*** svenkat has quit IRC12:14
markus_zhm, maybe I put it on my "languages to learn" list12:14
*** salv-orlando has quit IRC12:14
*** svenkat has joined #openstack-nova12:14
*** apanaitiu has joined #openstack-nova12:14
*** markvoelker has quit IRC12:14
*** markvoelker has joined #openstack-nova12:15
mdboothIts time is gone, imho.12:15
*** eharney has joined #openstack-nova12:15
*** zhangyufei has quit IRC12:15
markus_z"go" is the cool kid now, isn't it?12:17
*** vishwanathj has joined #openstack-nova12:17
* mdbooth only knows about the buzz. Haven't really looked at the language. Rust is very interesting, imho, but not for openstack.12:18
alaskiartom: gmann alex_xu_ just catching up on review comments, but I'm okay with splitting the API exposure to expose tags on boot now and then tags on attach later. That feels like a natural split to me, as compared to saying lets do nics now and bdms later.12:18
*** svenkat has quit IRC12:18
alaskias in it feels complete to me to say you can now tag things on boot12:19
* markus_z crossed out "perl" from the list and added "rust"12:19
danpbmdbooth: i'll come back in 10 years and see which of the "cool" new languages are still in use :-)12:19
alex_xu_alaski: cool, i'm also think that is ok12:19
mdboothdanpb: Hehe, pretty much.12:19
*** ociuhandu has joined #openstack-nova12:20
mdboothdanpb: Rust will either take off, or inspire some other language to have all its most useful features.12:20
mdboothIt really does have some great ideas.12:20
*** zerda2 has quit IRC12:21
*** shaohe_feng has quit IRC12:21
markus_zmdbooth: What's your favorite idea?12:21
*** shaohe_feng has joined #openstack-nova12:21
mdboothmarkus_z: The big one is its explicit object ownership model.12:21
mdboothThe language itself doesn't have GC (although it has a library which does GC), but it can still automatically manage lifecycles in many/most cases12:22
mdboothBecause lifespans are explicit in the language12:22
artomalaski, awesome :)12:22
*** ericksonsantos has joined #openstack-nova12:23
markus_zmdbooth: Ah, so the scope defines the lifetime, if I read that correctly.12:24
*** hshiina has quit IRC12:24
*** Fdaisuke has quit IRC12:24
*** baoli_ has joined #openstack-nova12:24
*** Fdaisuke has joined #openstack-nova12:24
*** hshiina has joined #openstack-nova12:24
lyarwoodmdbooth: just back from lunch and looking at the rescue test rewrite now, thanks again12:25
*** edmondsw has joined #openstack-nova12:25
mdboothmarkus_z: I haven't written any in over a year, so it's starting to get fuzzy in my head, but it's way more involved than that. You can pass pointers around and the compilers still knows statically when the object is out of scope.12:25
mdboothAlso, when it is accessible from another thread.12:25
mdboothIn Rust, the compiler knows all things.12:25
*** ericksonsantos has quit IRC12:26
*** bswartz has joined #openstack-nova12:26
*** hieulq has joined #openstack-nova12:26
*** hieulq_ has quit IRC12:26
*** jamie_h has joined #openstack-nova12:27
*** paul-carlton2 has quit IRC12:27
markus_zI see. That's definitely something for cold winter evenings, I'm going to have a look :)12:27
*** rlrossit has joined #openstack-nova12:30
*** adisky has quit IRC12:30
*** paul-carlton2 has joined #openstack-nova12:31
openstackgerritMaciej Szankin proposed openstack/nova: config options: Improve help for SPICE  https://review.openstack.org/31704612:31
*** shaohe_feng has quit IRC12:31
*** mdrabe has joined #openstack-nova12:32
*** shaohe_feng has joined #openstack-nova12:32
*** paul-carlton has joined #openstack-nova12:33
*** fragatina has joined #openstack-nova12:34
*** paul-carlton has quit IRC12:35
*** aloga has quit IRC12:36
*** aloga has joined #openstack-nova12:36
*** ericksonsantos has joined #openstack-nova12:37
*** mgoddard_ has quit IRC12:38
*** mgoddard has joined #openstack-nova12:38
mdboothartom vladikr: Another regression in I931421ea688641e2ceb212c6dc099639c53433f2, btw. For uml, it now tries to chown the root disk before it has been created.12:39
mdboothI'm assuming that would actually prevent uml from working. Does anybody use that?12:39
*** pilgrimstack has quit IRC12:41
*** shaohe_feng has quit IRC12:41
markus_zdanpb: During the virtlogd tests I've seen that the boot message won't get printed into the "console.log" file *unless* I'm already connected to the terminal (executing a 'reboot' for example).12:42
markus_zdanpb: Folks from my Linux department looked into it and they guess that this could be an issue in qemu itself.12:42
*** shaohe_feng has joined #openstack-nova12:42
*** ratailor has quit IRC12:42
artommdbooth, thanks for catching that12:43
mdboothartom: Just writing a bug for it12:43
*** pilgrimstack has joined #openstack-nova12:43
*** pradk has quit IRC12:43
markus_zdanpb: They said it looks like the qemu logd code gets only executed *after* a terminal connection, which prevents it to log the boot messages.12:43
mdboothCaught it rebasing my patch series on yours :)12:43
artomTo be honest I'm not sure what uml is, but it's definitely something we'll look into12:43
*** salv-orlando has joined #openstack-nova12:43
mdboothuml == user mode linux12:43
artommdbooth, btw, did you see vladikr's fix for the first bug?12:44
artomhttps://review.openstack.org/#/c/335676/12:44
markus_zdanpb: Maybe you can have a look at it? I tried to understand your change http://git.qemu.org/?p=qemu.git;a=blobdiff;f=qemu-char.c;h=11caa5648de99c9e0ee158f280fbc02ab05915d3;hp=d7be1851e5e9d268aa924a05958da292b048839c;hb=d0d7708ba29cbcc343364a46bff981e0ff88366f;hpb=f1c17521e79df863a5771d96974fab0d07f02be0 but I have no experience in qemu itself.12:44
*** GB21 has quit IRC12:44
mdboothartom: Will look, thanks12:44
*** dtantsur has joined #openstack-nova12:44
dtantsurhi everyone! does this ring any bells? http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=ERROR12:44
dtantsurironic grenade jobs started failing due to it several hours ago12:45
artommdbooth, tag me on the bug or something if you're filing it now please :)12:45
mdboothartom: Ok12:45
mdboothI actually have a patch which fixes it, but as it's patch 19 in my series, I don't mind if you fix it first :)12:45
artommdbooth, haha - as long as it gets fixed by someone somewhere ;)12:46
*** chandankumar has quit IRC12:46
*** xyang1 has joined #openstack-nova12:47
markus_zdtantsur: Looks like the exception is raised here: https://github.com/openstack/nova/blob/0871f4953d99bc6e5b35dbe8ccf8644a282167d3/nova/db/sqlalchemy/api.py#L1524-L152412:48
danpbmarkus_z: hmm, its possible - will have a look12:48
*** apanaitiu has quit IRC12:48
danpbdata will only be written to the log file when it is also written to the primary channel12:48
markus_zdtantsur: Without having the DB error it's hard to say what's wrong. Can you add a log and repeat it?12:48
*** mdavidson has joined #openstack-nova12:49
mdboothartom: Consider yourself spammed :)12:49
danpbbut i though that when no client is connected, the data would just be written to null effectively, so the logs would still be output12:49
artommdbooth, as long as its now enlargement pills12:49
artomErr, not12:49
* artom gulps more coffee12:49
dtantsurmarkus_z, hmm, so it requires patching nova... what about NotImplementedError above?12:50
*** tbachman has quit IRC12:50
alaskidtantsur: artom might be related to https://review.openstack.org/#/c/264017/12:50
artomalaski, it is?12:51
artomI'm seeing the virt driver raise a NotImplemented error on detach_interface, maybe I'm not reading deep enough12:52
markus_zdanpb: In my setup it didn't. I did it on a system z machine, if that's important. I wasn't able to get a setup running on x86 with the needed libvirt and qemu versions.12:52
*** shaohe_feng has quit IRC12:52
alaskiartom: yeah, not sure on that yet. But I'm seeing http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=ERROR#_2016-06-30_12_37_11_285 as wekk12:52
alaskiwell12:52
*** shaohe_feng has joined #openstack-nova12:52
openstackgerritAlexis Lee proposed openstack/nova: Notifications: single source schema  https://review.openstack.org/31201012:52
artomalaski, ok, that last one we need to know what's happening at the db level12:53
artomalaski, althought, the db changes for our stuff merged ages ago, comparatively12:53
artomIf this was a schema thing we'd have caught it earlier12:53
dtantsurI wonder why it only happens in grenade12:54
*** tbachman has joined #openstack-nova12:54
markus_zdtantsur: That's with the ironic driver, isn't it?12:55
dtantsuryep12:55
markus_zthe ironic driver doesn't override this https://github.com/openstack/nova/blob/b466c382eaecb95566357b473e599f749baeaf6f/nova/virt/driver.py#L528-L52812:55
alaskiif we throw up a logging patch in Nova and then a patch in Ironic that depends on it I think we can get logs12:55
dtantsurmarkus_z, should we? it was not required for quite some time12:56
markus_znote: I'm no expert with ironic, I'm just reading code here12:56
dtantsuralaski, ++12:56
*** mdrabe_ has joined #openstack-nova12:56
*** mdrabe has quit IRC12:56
*** mfeoktistov has quit IRC12:57
dtantsurI wonder if we hit some hidden upgrade problem12:57
openstackgerritAndrea Rosa proposed openstack/nova: Enabling detach root volume for shelved_offloaded  https://review.openstack.org/32689912:58
*** ducttape_ has joined #openstack-nova12:58
openstackgerritAndrea Rosa proposed openstack/nova: Attach/detach root volume for shelved_offloaded  https://review.openstack.org/31744612:58
openstackgerritAndrea Rosa proposed openstack/nova: Adding device_present field to bdm table  https://review.openstack.org/26805312:58
*** rodrigods has quit IRC12:58
*** rodrigods has joined #openstack-nova12:58
openstackgerritRyan Moore proposed openstack/nova: Add better help to rdp options  https://review.openstack.org/25355612:58
openstackgerritAndrew Laski proposed openstack/nova: DNM Log extra stuff for debugging  https://review.openstack.org/33599512:59
alaskidtantsur: ^12:59
*** jed56 has joined #openstack-nova12:59
alaskican you throw up the ironic patch?12:59
dtantsurthnx, sure12:59
*** tbachman has quit IRC13:00
*** andymaier has joined #openstack-nova13:00
dtantsuralaski, https://review.openstack.org/33599813:01
*** amotoki has quit IRC13:01
alaskidtantsur: thansk13:01
dtantsuralaski, also FWIW I think this logging should be there13:01
alaski*thanks13:01
alaskiyeah, I agree13:02
*** shaohe_feng has quit IRC13:02
*** ducttape_ has quit IRC13:02
*** dedery has quit IRC13:02
*** ljxiash has quit IRC13:03
*** shaohe_feng has joined #openstack-nova13:03
markus_zdtantsur: IIUC, change https://review.openstack.org/#/c/328168/ was the first one which logged the NotImplementedError.13:03
gibi_alaski: Hi! I fixed your comment in https://review.openstack.org/#/c/313654/13:04
markus_zaccording to logstash13:04
*** tblakeslee has joined #openstack-nova13:04
dtantsurmarkus_z, good catch. so this NotImplementedError is probably not the cause13:04
*** lucas-hungry is now known as lucasagomes13:06
markus_zdtantsur: nah. I guess, that the NotImplementedError gets raised because it tries to rollback the failed VIF creation.13:06
*** code-R_ has joined #openstack-nova13:06
johnthetubaguyalaski: are you not tempted to merge this one? https://review.openstack.org/#/c/335995/1/nova/db/sqlalchemy/api.py13:07
*** amotoki has joined #openstack-nova13:07
*** muawiakhan has quit IRC13:07
markus_zwhich means: "unknown DB error" -> raise VirtualInterfaceCreateException -> rollback VIF creation -> detach_interface -> raise NotImplementedError13:08
alaskiI suppose we could, I just didn't put much thought into the message there13:08
*** zenoway has quit IRC13:08
*** code-R_ has quit IRC13:09
*** code-R_ has joined #openstack-nova13:09
*** code-R has quit IRC13:09
dtantsuryeah, probably13:09
dtantsurso seems like if we fix the DB error, we can get back to being green13:09
*** muawiakhan has joined #openstack-nova13:09
markus_zI assume that, yes13:09
*** sdake has joined #openstack-nova13:10
markus_zdtantsur: rethinking the rollback behavior for the ironic driver could be another task.13:10
dtantsuryep13:11
*** houming has quit IRC13:11
*** muawiakhan has quit IRC13:11
openstackgerritAlexis Lee proposed openstack/nova: Catch errors during VIF plug/unplug  https://review.openstack.org/26414613:11
openstackgerritJohn Garbutt proposed openstack/nova: Improve help text for resource tracker options  https://review.openstack.org/32014713:11
*** pilgrimstack has quit IRC13:11
*** shaohe_feng has quit IRC13:12
openstackgerritAndrew Laski proposed openstack/nova: Log DB exception if VIF creation fails  https://review.openstack.org/33599513:12
alaskijohnthetubaguy: should be mergeable now ^13:12
*** sdake_ has quit IRC13:13
alaskithat set us back about 11 minutes on getting logs13:13
*** shaohe_feng has joined #openstack-nova13:13
*** cleong has joined #openstack-nova13:14
*** rlrossit has quit IRC13:14
openstackgerritClaudiu Belu proposed openstack/nova: WIP: policy: clean-up  https://review.openstack.org/33392513:17
*** takashin has left #openstack-nova13:17
*** prateek has quit IRC13:17
*** erlon has joined #openstack-nova13:18
*** woodster_ has joined #openstack-nova13:19
*** pilgrimstack has joined #openstack-nova13:20
*** tbachman has joined #openstack-nova13:20
bauzasp!spy13:20
*** mriedem has joined #openstack-nova13:21
*** ametts has joined #openstack-nova13:21
edleafeOK, so I have a reno formatting failure for a patch. How can I test that locally to avoid the wrath of Gerrit?13:22
*** Administrator_ has joined #openstack-nova13:22
*** shaohe_feng has quit IRC13:22
mriedemedleafe: tox -e releasenotes13:23
*** shaohe_feng has joined #openstack-nova13:23
edleafemriedem: thx13:23
markus_zedleafe: IIRC the reno file needs to be committed locally to check it with tox.13:23
mriedemyeah that too13:24
mriedemreno only works on commits13:24
mriedemnot staged files13:24
edleafeI'm not afraid of commitment13:24
markus_z;)13:24
*** pwnall has joined #openstack-nova13:24
mriedemmarkus_z: i learned about walpurgis night last night13:25
mriedemfun stuff13:25
markus_zyeah, we're known for that :)13:26
*** takashin has joined #openstack-nova13:26
mriedemdo the witches get together with satan, or get burned in a bonfire, or both?13:26
kashyapmriedem: Noticed your ping from yesterday about that stable/liberty CVE fix, just uploaded this change, does this look reasonable?  (Comment on the review): https://review.openstack.org/#/c/327624/13:26
mriedemor depends on country?13:26
markus_zsince a few years we don't use people anymore13:26
mriedemha, ok13:26
markus_zAFAIK you could do fingerpointing and yell "witch" and the rest will be resolved in fire...13:27
markus_zyeah, good old times...13:27
kashyapmarkus_z: I _now_ learnt Walpurgis Night!  Crazy...13:28
markus_zI'm not super into history, so don't rely solely on my information :D13:29
dansmithalex_xu_: yeah, tags in attach commands will have to come later13:29
dansmithalex_xu_: we have to rework some configdrive code in order to support it13:29
*** kevinzs has joined #openstack-nova13:29
*** jbernard has joined #openstack-nova13:30
*** Administrator_ has quit IRC13:31
*** jamesdenton has joined #openstack-nova13:31
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Raise an exception in v2.client for direct_use  https://review.openstack.org/33282913:31
*** rlrossit has joined #openstack-nova13:32
*** kevinzh has quit IRC13:32
*** kevin_zh has quit IRC13:33
*** shaohe_feng has quit IRC13:33
mriedemdansmith: would you agree here? https://review.openstack.org/#/c/313654/41/nova/notifications/objects/instance.py@12913:33
*** shaohe_feng has joined #openstack-nova13:33
*** READ10 has joined #openstack-nova13:33
*** kevin_zh has joined #openstack-nova13:33
dansmithmriedem: yups, commented13:34
*** jaypipes has joined #openstack-nova13:34
mriedemdid the tags thing get sorted out?13:34
*** sacharya has joined #openstack-nova13:34
jaypipesmorning supernovas.13:35
mriedemah, jsonschema can only validate the tag in the body13:35
*** s1061123 has quit IRC13:35
mriedemnot the path13:35
mriedemcool13:35
*** tiendc has joined #openstack-nova13:35
mriedemjaypipes: o/13:35
*** karimb has joined #openstack-nova13:35
mriedemjaypipes: not to be confused with https://github.com/major/supernova13:36
jaypipesmriedem: heh, true nuf.13:36
*** moshele has left #openstack-nova13:37
*** moshele has quit IRC13:37
openstackgerritAndrew Laski proposed openstack/nova: Bump service version for BuildRequest deletion  https://review.openstack.org/33247813:37
openstackgerritAndrew Laski proposed openstack/nova: Stop instance build if BuildRequest deleted  https://review.openstack.org/33247913:37
openstackgerritAndrew Laski proposed openstack/nova: Refactor block_device_mapping handling during boot  https://review.openstack.org/33409413:37
openstackgerritAndrew Laski proposed openstack/nova: Add block_device_mappings to BuildRequest  https://review.openstack.org/33409513:37
openstackgerritAndrew Laski proposed openstack/nova: Cells: Handle delete with BuildRequest  https://review.openstack.org/32598513:37
dtantsuralaski, created a bug to track the issue: https://bugs.launchpad.net/nova/+bug/159777013:38
openstackLaunchpad bug 1597770 in Ironic "Ironic grenade gate fauls due to VirtualInterfaceCreateException in nova-compute" [Critical,Confirmed]13:38
*** xenogear has joined #openstack-nova13:39
*** sacharya has quit IRC13:39
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: remove dead code path in "_get_console_output_pty"  https://review.openstack.org/33602113:39
mriedemdansmith: do it to it https://review.openstack.org/#/c/304510/13:39
dansmithwoot13:40
dtantsuralaski, btw we've also found http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-q-dhcp.txt.gz?level=ERROR I wonder if it's related as well13:40
openstackgerritPaul Carlton proposed openstack/python-novaclient: [microversion] Bump to 2.32  https://review.openstack.org/31602613:40
openstackgerritPaul Carlton proposed openstack/python-novaclient: Add support for microversion 2.31  https://review.openstack.org/33439013:40
mriedemdtantsur: http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=TRACE13:41
mriedemactually that shouldn't break you...13:41
dtantsurinteresting...13:41
dtantsurETOOMANYSTACKTRACES13:41
*** code-R has joined #openstack-nova13:42
mriedemit fails here13:42
mriedemhttp://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=TRACE#_2016-06-30_12_37_14_85613:42
*** andymaier has quit IRC13:43
mriedemdtantsur: this is the change yesterday https://github.com/openstack/nova/commit/e2eb6659a426bae05a54b93ebf31ff324712525813:43
*** karimb has quit IRC13:43
mriedemi don't know why that would break ironic13:43
mriedemi guess ironic is using it's own conductor?13:43
*** shaohe_feng has quit IRC13:43
*** shaohe_feng has joined #openstack-nova13:43
dtantsurwdym by "own conductor"? I don't think we mess with nova-conductor13:43
*** sheel has joined #openstack-nova13:44
*** edmondsw has quit IRC13:44
mriedemnvm13:45
mriedemi don't know why this would fail though http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz#_2016-06-30_12_37_14_85613:45
*** code-R_ has quit IRC13:45
mriedemprobably something with the mac address on the vif?13:45
*** tbachman has quit IRC13:46
mriedemironic is the only virt driver that provides macs to allocate_for_instance13:46
alaskimriedem: we've got https://review.openstack.org/#/c/335995/ up to try to find out13:46
johnthetubaguydansmith: I refresh the my refactor stuff after the tagging merge, if you get chance today13:46
*** bvanhav has quit IRC13:46
*** tbachman has joined #openstack-nova13:46
alaskimriedem: and https://review.openstack.org/#/c/335998/13:46
*** bvanhav has joined #openstack-nova13:46
dansmithjohnthetubaguy: cool, one of your patches screwed me up yesterday, so glad we returned the favor :P13:47
johnthetubaguydansmith: :p13:47
mriedemalaski: we should log something here https://review.openstack.org/#/c/264017/75/nova/network/neutronv2/api.py@70113:47
dansmithjohnthetubaguy: one of those rare times where jenkins (or whatever) merged without a failure, but the code was subtly wrong..13:47
mriedemalaski: but the db exception tracing is good too13:47
johnthetubaguydansmith: oh, nice13:47
*** ayoung has joined #openstack-nova13:48
mriedemalaski: dtantsur: my money is on duplicate macs13:48
mriedemschema.UniqueConstraint("address", "deleted",13:48
mriedem                        name="uniq_virtual_interfaces0address0deleted"),13:48
dtantsurthis is likely, except for why didn't it fail previously?13:49
mriedemdtantsur: we never created virtualinterfaces in nova's db when using neutron before yesterday13:49
dtantsurhmmm13:50
dansmithmriedem: just to be clear, johnthetubaguy's refactor is covered under at least one priority realm, right?13:50
dtantsurthen it's possible. we definitely do not change MACs between old and new runs13:50
*** tbachman_ has joined #openstack-nova13:50
*** rlrossit has quit IRC13:50
mriedemdansmith: i wasn't tracking that as a priority13:50
mriedemit was a dependency for neutron routed networks13:50
mriedemwhich is also dependent on generic resource pools13:51
dansmithmriedem: hmm13:51
*** chandankumar has joined #openstack-nova13:51
dansmithmriedem: it's kinda critical for cells and general sanity13:51
johnthetubaguyalthough routed networks kinda required to drop cells v113:51
dansmithooh yeah13:51
dtantsurmriedem, so if it's really duplicated MACs, how do we fix it?13:51
*** tbachman has quit IRC13:51
*** tbachman_ is now known as tbachman13:51
*** pradk has joined #openstack-nova13:52
mriedemdtantsur: well let's figure out first if that's what it is13:52
dtantsurokie13:52
mriedemhttps://specs.openstack.org/openstack/nova-specs/priorities/newton-priorities.html13:52
*** krtaylor has joined #openstack-nova13:52
mriedemwe already have a lot of priority stuff still to work on13:52
mriedemand while that doc doesn't mention api proxy deprecations as priority, i'm guessing sdague will want it to be so we can get that out as early as possible13:53
mriedemsame with api extension removal13:53
dansmithit's also debt cleanup until a point13:53
mriedemi'm not even sure if libvirt storage pools has started13:53
*** shaohe_feng has quit IRC13:53
dansmithmriedem: what? of course it has :)13:53
*** shaohe_feng has joined #openstack-nova13:53
mriedemthe imagebackend stuff was a separate bp13:53
mriedemstorage pools built o nthat13:54
*** diogogmt has joined #openstack-nova13:54
mriedembut i thought there were still specs up for storage pools13:54
dansmithmriedem: thought there are code patches too13:54
mriedemPaulMurray: what's the status on the libvirt storage pools bp?13:54
johnthetubaguyanyways, patches are up that I think would unblock the first phase of routed networks (the non-scheduler bits)13:54
mriedemis there code?13:54
*** irenab has joined #openstack-nova13:54
dansmithhttps://review.openstack.org/#/c/314134/13:55
mriedemPaulMurray: and i mean after mdbooth and diana_clarke's imagebackend storage series13:55
PaulMurraymriedem, its dependant on the stuff mdbooth and diana_clarke are doing13:55
PaulMurraywhat do you need to know ?13:55
mriedemPaulMurray: is it just that one patch dansmith linked above?13:55
*** karimb has joined #openstack-nova13:55
mriedemdansmith: the spec isn't approved yet either https://review.openstack.org/#/c/310505/13:55
johnthetubaguyhmm, fun, its not13:56
danpbmriedem: about https://blueprints.launchpad.net/nova/+spec/websocket-proxy-to-host-security13:56
danpbmriedem: "Doesn't look like anything was pushed up for code for this so I'm deferring for Newton. -- mriedem 20160629"13:56
danpbmriedem: i've pushed 10 new versions of the patches during this cycle :-(13:56
PaulMurraymriedem, it just didn't get approved yet.....13:56
*** rlrossit has joined #openstack-nova13:56
mriedemdanpb: https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security,n,z13:56
PaulMurraymriedem, I actually forgot that - is there a deadline for the specs for priority ?13:57
*** akshai has joined #openstack-nova13:57
mriedemdanpb: what were those changes tracked under?13:57
danpbmriedem: they're listed right there on the blueprint whiteboard13:57
danpbthey've been on review since aug 2014 now13:58
mriedemdanpb: i only see specs in the whiteboard13:58
mriedemand abandoned changes from solly13:58
*** kylek3h has joined #openstack-nova13:58
mriedemdanpb: please link me to the series in gerrit13:58
mriedemnova meeting starting in 1 minute13:59
*** ducttape_ has joined #openstack-nova13:59
danpbmriedem: those changes by solly aren't abandoned - they're the ones i'm frequently updating13:59
*** ctrath has joined #openstack-nova13:59
mriedemah i see13:59
mdboothdanpb: ?13:59
mriedemso it is https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security13:59
mriedemi saw the merge conflicts14:00
mdboothAh, websocket.14:00
mriedemdanpb: sorry, i'll update the bp14:00
mdboothFor a moment I thought you were talking about libvirt storage pools14:00
danpbmriedem: its the god damn nova/exception.py file whicih is merge conflict hell14:00
openstackgerritDaniel Berrange proposed openstack/nova: Introduce VNC Security Proxy Framework  https://review.openstack.org/11548314:00
openstackgerritDaniel Berrange proposed openstack/nova: Add VeNCrypt (TLS/x509) Security Proxy Driver  https://review.openstack.org/11548414:00
* danpb should just add new exceptions randomly in the middle of the file instead of add the end to avoid conflicts14:00
*** kevin_zh has quit IRC14:00
mdboothdanpb: I was just about to advise that :)14:00
PaulMurraymriedem, dansmith I was going to give an update on storage pools in nova meeting14:01
lxslidanpb: https://review.openstack.org/#/c/319203/ if you'd be so kind, Alex +2d14:01
*** kevin_zh has joined #openstack-nova14:01
lxsliI have also removed a lot of complexity from https://review.openstack.org/31925414:01
andreykurilinsdague mriedem: hi! Could you look at small backport to novaclient's stable/mitaka? https://review.openstack.org/#/c/332038/14:01
*** gjayavelu has joined #openstack-nova14:02
mriedemnova meeting14:02
markus_zdanpb: yeah, I use that strategy too14:02
mdboothsahid: https://review.openstack.org/#/c/333271/ That -1 is kinda harsh. I think mine is more readable, so there isn't really anywhere to go from there. It's subjective.14:02
*** mpavone has quit IRC14:03
openstackgerritJohn L. Villalovos proposed openstack/nova: WIP: Revert "Compute manager device tagging support"  https://review.openstack.org/33604114:03
*** ducttape_ has quit IRC14:03
*** zenoway has joined #openstack-nova14:03
*** shaohe_feng has quit IRC14:03
danpblxsli: ok i'll look14:04
mdboothsahid: I appreciate the note, but -1 means: the code base is better off without this.14:04
*** diga has joined #openstack-nova14:04
*** shaohe_feng has joined #openstack-nova14:04
*** s1061123 has joined #openstack-nova14:07
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices  https://review.openstack.org/32376514:07
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: simplify "get_console_output" interface  https://review.openstack.org/33448014:07
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: remove dead code path in "_get_console_output_pty"  https://review.openstack.org/33602114:07
*** roeyc has quit IRC14:07
*** sdake_ has joined #openstack-nova14:07
markus_zdang it...14:08
sahidmdbooth: well i gave to this change several comments so I don't understand which one you are talking about14:08
*** smatzek has quit IRC14:08
mdbooth https://review.openstack.org/#/c/333271/14:08
mdboothAll comments are about the same thing, which is a minor stylistic thing14:09
*** EinstCrazy has joined #openstack-nova14:09
sahidmdbooth: not really14:09
mdboothAnd I prefer mine, because it requires less immediate context to read14:09
mdboothI don't think yours is bad, just mine is better14:09
mdboothAnd I already wrote mine, and don't want to change it :)14:10
mdboothAnd it's directly relevant to the patch it's in.14:10
sahidmdbooth: feel free to ignore my comments14:10
danpbmarkus_z: given the issue with console.log not being written unless connected to the serial port, i'm wondering whether we should hold off on the console patch, or have it temporarily disabled by default  ?14:10
mdboothWell I would, but they're accompanied by an ugly -1.14:10
mdbooth-1 isn't really for that.14:10
*** syjulian has joined #openstack-nova14:10
*** gongysh_ has quit IRC14:11
*** sdake has quit IRC14:11
*** aloga_ has joined #openstack-nova14:11
sahidit's not like it's a minus 2 so i guess if your code is good and my comments silly, you will get the 2 necessaries +214:11
*** paul-carlton2 has quit IRC14:11
*** paul-carlton2 has joined #openstack-nova14:12
mdboothsahid: -1 means "This patch needs further work before it can be merged"14:12
mdboothThat's literally what it means, we discussed it a while back and changed the wording.14:13
mdboothIt means that the codebase is better off without the change.14:13
*** amotoki has quit IRC14:13
mdboothYou're asserting with your -1 that the codebase is worse off with the change than without it.14:13
*** gagehugo has joined #openstack-nova14:13
*** GB21 has joined #openstack-nova14:13
mdboothBecause of a minor stylistic difference.14:13
markus_zdanpb: yeah, I can add a "AND False" conditional with a TODO note, would that be ok? Easy to remove after we figured out a solution.14:14
mdboothIt isn't what it's for.14:14
*** shaohe_feng has quit IRC14:14
*** diogogmt has quit IRC14:14
*** shaohe_feng has joined #openstack-nova14:14
danpbmarkus_z: or could add a config option to the workaround group to disable it and default to disabled14:14
mdboothI know that it gets used that way a fair bit anyway, but it really shouldn't be.14:14
danpbso people can at least try it out14:14
*** rcernin has quit IRC14:14
markus_zdanpb: yeah, the workaround section sounds good, I'll push it in the next hour14:15
* mdbooth is quite careful to mark nits as nits, and I only -1 if there's at least 1 thing which isn't a nit.14:15
*** tbachman has quit IRC14:17
rlrossitdansmith: I have an objects question for you after the nova meeting is over (so don't go running away when it's done)14:18
*** sdake_ is now known as sdake14:20
*** kevin_zh has quit IRC14:23
jaypipesmriedem, dansmith: here is the line that introduces the issue: https://github.com/openstack/nova/commit/e2eb6659a426bae05a54b93ebf31ff3247125258#diff-da7e16a0727fa850ace01ae175cc2363R67514:23
*** mjura has quit IRC14:23
lxslithanks Dan14:23
*** nkrinner has quit IRC14:23
mriedemjaypipes: that's nova-net14:23
jaypipesmriedem, dansmith, artom: when a request for devices has >1 tag, it attempts to insert one record in virtual_interfaces for each tag.14:23
*** kevin_zh has joined #openstack-nova14:23
mriedemironic is blowing up with neutron14:24
*** itamarl has quit IRC14:24
*** shaohe_feng has quit IRC14:24
artomjaypipes, you mean one device with multiple tags?14:24
dansmithyeah and that line is just a lookup table for us14:24
artomThat... shouldn't be possible?14:24
dansmithartom: right14:24
dansmithartom: and the bug is not in n-net anyway, so not that I think :)14:24
*** tbachman has joined #openstack-nova14:25
*** shaohe_feng has joined #openstack-nova14:25
alaskithis telnet interface to zuul(or whatever I'm actually connected to) is pretty nifty14:26
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051314:26
*** edmondsw has joined #openstack-nova14:26
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365414:26
gibi_mriedem, dansmith: fixed your comments in https://review.openstack.org/313654 ^^14:27
*** aloga_ has quit IRC14:28
*** belmoreira has quit IRC14:28
openstackgerritLee Yarwood proposed openstack/nova: WIP - libvirt: Use the recreated config disk during a rescue  https://review.openstack.org/33588414:29
*** kevin_zh has quit IRC14:29
*** rcernin has joined #openstack-nova14:29
*** kevinzs has quit IRC14:30
*** ctrath has quit IRC14:31
*** jistr is now known as jistr|mtg14:31
edleafemdbooth: I looked at that patch, and agree that it isn't just "style". That line can be using the wrong value.14:32
mdboothedleafe: You mean if the super class alters the path it was given to store?14:33
mdboothOk, I'll update the contract to assert that it doesn't :) That would be confusing.14:33
*** ctrath has joined #openstack-nova14:33
edleafemdbooth: sure - just like you altered the path that was passed14:34
*** krtaylor has quit IRC14:34
*** shaohe_feng has quit IRC14:34
mdboothedleafe: My patch doesn't alter the path...14:34
mdboothThat would be bug14:34
edleafemdbooth: if the superclass is modified to handle some other path-related thing, this class would never see it (at least in the __init__)14:34
mdboothI would be happy to address that.14:34
jaypipesmriedem, dansmith, artom: yeah, sorry, I was wrong :( looks like now that in the neutronv2 code we're creating an objects.VirtualInterface object for each requested network we're running into that MAC address unique constraint.14:34
dansmithjaypipes: yeah, I know what the problem is, no question about that14:35
mdboothedleafe: I'll update the doctext if you like to state that it doesn't do that.14:35
dansmithjaypipes: just not sure why we're getting dupes from ironic14:35
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end)  https://review.openstack.org/25028314:35
*** shaohe_feng has joined #openstack-nova14:35
edleafemdbooth: sure it does: on line 44614:35
*** sahid has quit IRC14:35
dansmithjaypipes: but I suspect it may be because we're not deleting the VIFs in the neutron case so we fail when we've used one a second time14:35
mdboothedleafe: That's *before* passing it to the super class14:35
edleafemdbooth: right14:36
*** vsaienko1 has joined #openstack-nova14:36
mdboothEvery base class now follows the same pattern: it creates a path, which it gives to the super class14:36
edleafepath gets passed to this class. This class potentially changes it, and then passes it to the superclass14:36
jaypipesdansmith: even so, I don't believe Neutron has a similar constraint about MAC uniqueness does it?14:36
edleafemdbooth: there's no contract that says that the superclass might manipulate it further14:36
mdboothActually, if the super class did something with it, we would still be correct to use the unmodified one14:36
dansmithjaypipes: it does not, no14:36
mdboothBecause that's the context of the user14:36
*** ducttape_ has joined #openstack-nova14:38
mdboothedleafe: Look at it this way, disk_info_path is explicitly created from the local path we created locally14:38
mdboothIf the super class altered the path variable to do something else14:38
vsaienko1hello nova team, does anybody seen VirtualInterfaceCreateException_Remote: Virtual Interface creation failed Error?14:38
mdboothWell, that would be subversive and silly14:38
*** gcb has quit IRC14:38
mdboothHowever, we would *still* want to use the local path we created locally14:39
*** MikeG451 has joined #openstack-nova14:39
*** ducttape_ has quit IRC14:39
mdboothSo, in fact, using the super class path instance variable is more fragile14:39
mdboothBut it's completely irrelevant in either case14:39
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.update notification  https://review.openstack.org/32592314:39
mdboothBecause I wrote it this way, and it's not worth a -1, even if you disagree with me14:39
*** smatzek has joined #openstack-nova14:39
vsaienko1full trace is http://paste.openstack.org/show/524274/14:39
paul-carlton2johnthetubaguy, did you get chance to look at https://review.openstack.org/#/q/topic:bp/live-migrate-rescued-instances14:40
*** phuongnh has joined #openstack-nova14:40
edleafemdbooth: I am not as familiar with the entire libvirt codebase. But if you have an instance variable, it should be respected throughout the class hierarchy14:40
*** sacharya has joined #openstack-nova14:40
edleafemdbooth: otherwise, it should be local14:40
*** Guest85383 is now known as yass14:40
mdboothIt is local14:40
mdboothThe patch made it local14:40
edleafemdbooth: are you saying that you no longer need 'self.path' anywhere else in the codebase?14:40
* mdbooth goes back to his 30 substantive patches :)14:41
mdboothThis is the worst kind of bikeshedding14:41
alaskivsaienko1: that bug is being tracked at https://bugs.launchpad.net/nova/+bug/1597770, it was brought up this morning14:41
openstackLaunchpad bug 1597770 in OpenStack Compute (nova) "Ironic grenade gate fails: Failed to allocate the network(s)" [Critical,Confirmed]14:41
*** gcb has joined #openstack-nova14:41
*** aunnam_ has quit IRC14:41
*** dharinic__ has joined #openstack-nova14:41
edleafemdbooth: no, it's getting a separate POV14:41
edleafemdbooth: again, I'm not a libvirt expert, so you may be perfectly safe14:42
dtantsuralaski, mriedem, the logs: http://logs.openstack.org/98/335998/1/check/gate-grenade-dsvm-ironic/e8cbaac/logs/new/screen-n-cpu.txt.gz?level=TRACE I'm having hard time finding the relevant bit though14:42
dtantsurthe initial failure caused too many failures itself...14:42
edleafemdbooth: my issue is strictly python class interactions14:42
johnthetubaguypaul-carlton2: not yet, will do14:43
*** aunnam_ has joined #openstack-nova14:43
paul-carlton2thanks14:43
alaskidtantsur: it's on conductor because computes defer db queries there http://logs.openstack.org/98/335998/1/check/gate-grenade-dsvm-ironic/e8cbaac/logs/new/screen-n-cond.txt.gz#_2016-06-30_14_32_25_36014:43
*** cdelatte has joined #openstack-nova14:43
alaskiartom: dansmith ^14:43
artomdtantsur, why not just modify that review 335998 to not catch anything in the db API and just barf up the raw sqlalchemy error?14:43
dtantsurartom, it's not a question for me, rather for nova folks :)14:44
dansmithI have a hack to propose14:44
dansmithgive me a few14:44
dtantsuralaski, so mriedem was right, that's the NIC duplicate14:44
dtantsurs/NIC/MAC/14:44
*** shaohe_feng has quit IRC14:44
danpbPaulMurray: success, in so much as we got a failure in the live-migraiton CI job !14:44
*** sacharya has quit IRC14:45
*** ljxiash has joined #openstack-nova14:45
danpbPaulMurray: the non-deterministic failure is caused by QEMU getting an I/O error when flushing data on the source qemu :-(14:45
artomdtantsur, it's apparently already been done by someone else anyways14:45
*** tonytan4ever has joined #openstack-nova14:45
PaulMurraygood job danpb14:45
*** rcernin has quit IRC14:46
PaulMurraydanpb, I seem to remember that there was an IO error - but didn't know why14:46
PaulMurraycan you tell ?14:46
*** sudipto has quit IRC14:46
*** pcaruana has quit IRC14:46
*** sudipto has joined #openstack-nova14:47
*** pushkaru has joined #openstack-nova14:47
*** shaohe_feng has joined #openstack-nova14:48
paul-carlton2what is the best room for tempest dev questions?14:48
markus_zmriedem: I don't anymore if I RSVP'ed to the midcycle, but I won't make it this time. Company's money is tight again.14:48
*** jlanoux_ has quit IRC14:48
dansmithdtantsur: is this blocking ironic code? meaning, is this a voting job on ironic?14:48
dtantsurdansmith, yes14:49
dansmithokay14:49
*** dimtruck is now known as zz_dimtruck14:49
PaulMurraymdbooth, are you going to go to mid cycle?14:51
PaulMurrayor diana_clarke ?14:51
mdboothPaulMurray: No. Is that a problem?14:51
mriedemmarkus_z: you didn't rsvp, but yeah, thanks14:51
*** david-lyle_ is now known as david-lyle14:51
*** takashin has quit IRC14:51
PaulMurraymdbooth, probably not14:51
danpbPaulMurray: yeah, syslog is showing a kernel iscsi error at the same time as qemu gets EIO14:51
mdboothPaulMurray: Cool.14:51
diana_clarkePaulMurray: I think we're only sending cores to mid cycle.14:52
PaulMurraymdbooth, paul-carlton2 will be there with me14:52
mriedempaul-carlton2: #openstack-qa14:52
openstackgerritBob Ball proposed openstack/nova: XenAPI:  Add UT for independent compute option  https://review.openstack.org/33606514:52
mdboothdiana_clarke: You're not going, are you?14:52
diana_clarkemdbooth: nope14:52
*** jaugustine has joined #openstack-nova14:52
mdboothDidn't think so.14:52
danpbmriedem: do you have any idea how cinder deals with iscsi during migration14:53
mriedemBobBall: thanks14:53
*** diga has quit IRC14:53
edleafediana_clarke: you're not using that "baby" excuse again, are you? :)14:53
danpbmriedem: in particular it seems that we must have multiple qemu instances connected to the same cinder iscsi lun at the same time14:53
mriedemdanpb: i'd have to look14:53
mriedemdanpb: hemnavacation would know, but he's on vacation this week14:53
*** diga has joined #openstack-nova14:53
danpbi'm wondering if connecting the 2nd qemu could cause cinder to kick off the first qemu ?14:53
PaulMurraydanpb, both ends are connected before the migration14:53
diana_clarkeedleafe: Baby didn't have a passport in time for PyCon, but she has one now. I might head to PyGotham next week with her.14:53
PaulMurraydanpb, but we rely on destinatino not doing anything14:54
*** muawiakhan has joined #openstack-nova14:54
*** raj_singh has quit IRC14:54
johnthetubaguyoh, good question, normally you do have two things pointing at the same lun, but only ever have one "active" at once, in theory14:54
*** shaohe_feng has quit IRC14:55
alex_xu_diga: if you are interesting to help on the api-ref, there is mail from sdague explain how we fix the doc http://lists.openstack.org/pipermail/openstack-dev/2016-April/092936.html, and there is wiki recorded some rules for api-ref https://wiki.openstack.org/wiki/NovaAPIRef14:55
danpband that's true here - source qemu is trying to flush out all outstanding writes before the target qemu starts running vCPUs14:55
johnthetubaguybut not sure if the domain is active in the live-migrate destination node, before the old domain stops, well active in the send of ISCSI being connected14:55
*** hshiina has quit IRC14:55
danpbso we should only have one writer here14:55
* bauzas f***cked, missed the nova meeting14:55
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051314:55
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.update notification  https://review.openstack.org/32592314:55
danpbbut it seems like the source qemu iscsi connection is getting interrupted14:55
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365414:55
paul-carlton2mriedem, ta14:55
digaalex_xu_: okay14:55
*** shaohe_feng has joined #openstack-nova14:56
*** sdake_ has joined #openstack-nova14:56
rlrossitdansmith: I have been discussing the HostManager with doffm, and we're wondering: does the ComputeNodeList.get_all() call at https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L574 go through conductor, or is there magic in place that allows conductor to go directly to the database to get the hosts?14:56
mriedembauzas: don't worry we cancelled since you weren't there14:56
alex_xu_diga: and there are lot of patch for reference https://review.openstack.org/#/q/status:merged+topic:bp/api-ref-in-rst, you can start with some review first, then you will know how to start quickly14:56
openstackgerritDan Smith proposed openstack/nova: HACK: Test making VIF.address unique with port id for neutron  https://review.openstack.org/33606914:56
bauzasmriedem: meh, untrue :p14:56
PaulMurraydanpb, I really don't know enough about iscsi, but I thought it could handle multiple writers14:56
dansmithdtantsur: can you get a run on top of https://review.openstack.org/#/c/336069/1 ?14:56
alex_xu_diga: also free to ping me if you need help :)14:56
digasure alex_xu_14:56
dansmithrlrossit: depends on who is callingit14:56
dtantsurdansmith, sure14:56
sudiptomriedem, are there going to be any extensions for non prio features via extension applications beyond today?14:57
PaulMurraydanpb, it should just mess up the data on disk if you don't coordinate right14:57
openstackgerritRyan Moore proposed openstack/nova: Add better help to rdp options  https://review.openstack.org/25355614:57
rlrossitdansmith: in this case, it would be the host manager within the scheduler14:57
digaalex_xu_: sure14:57
dansmithmriedem: the above is a pretty silly hack, but it's easy to hide and it avoids just changing the constraint which could have some unintended n-net impacts14:57
lxslialex_xu_: thanks for your +2 earlier, this one is similar: https://review.openstack.org/#/c/319254/14:57
danpbPaulMurray: i think we'll need to get some cinder people involved here14:57
dansmithmriedem: also happy to just change the constraint in some way if you'd prefer14:57
PaulMurraydanpb, yes14:57
dtantsurdansmith, https://review.openstack.org/#/c/335998/14:57
edleafeHmmm... must be doing something wrong. Pulled a fresh master and ran `tox -r -e releasenotes` on it, and it failed.14:58
*** raj_singh has joined #openstack-nova14:58
dansmithrlrossit: then no, not through conductor14:58
edleafeWhat's the next step for debugging this?14:58
markus_zedleafe: What does the error say?14:58
alex_xu_lxsli: np, let me check14:58
rlrossitdansmith: where is this magic that lets it pass through?14:58
PaulMurrayscottda, you around ?14:58
danpbPaulMurray: at this point it is looking like kernel / iscsi client / iscsi server problem, not qemu14:58
dansmithdtantsur: thanks, poke me when it finishes if you would14:58
openstackgerritNicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd  https://review.openstack.org/20375114:58
scottdaPaulMurray: Hi14:59
danpband some kind of race condition in the mix14:59
dansmithrlrossit: it's always direct unless your service declares indirection_api, which compute does, but scheduler, api (and of course conductor) don't14:59
*** sdake has quit IRC14:59
edleafemarkus_z: http://paste.openstack.org/show/524279/14:59
*** sacharya has joined #openstack-nova14:59
*** rcernin has joined #openstack-nova14:59
PaulMurrayHi scottda who would be best to talk to about iscsi issues ?14:59
dtantsurjlvillal, I'm leaving soon, could you please watch https://review.openstack.org/#/c/335998/ and poke dansmith when it finishes?14:59
edleafemarkus_z: do you want the full log?14:59
mriedemdansmith: ah tricky, hacking the uniqueness scoped to the port uuid14:59
openstackgerritSujitha proposed openstack/nova: Transform instance.pause notifications  https://review.openstack.org/32914114:59
rlrossitdansmith: so where is the indirection_api defined for the scheduler? (or for a service in general)14:59
*** akshai has quit IRC14:59
johnthetubaguyedleafe: have you checked out mitaka stable?14:59
PaulMurrayscottda, danpb has tracked a live migration bug down to an iscsi kernel error14:59
markus_zedleafe: yeah, maybe the content of /tmp/sphinx-err-yYpjnS.log14:59
scottdaPaulMurray: hemanthm would be better, but he's on vacation I think. I can help. What's the issue?14:59
dansmithmriedem: there's plenty of space in the address field for it, and our facade lets us hide it really easily.. it's so easy I'm tempted to say this is a good workaround for a bit, but it's definitely a hack14:59
edleafejohnthetubaguy: nope, just a fresh master15:00
scottdaPaulMurray: Bug?15:00
mriedemdansmith: sure15:00
danpbscottda: https://bugs.launchpad.net/nova/+bug/152489815:00
scottdaPaulMurray: As in Launchpad bug #?15:00
openstackLaunchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed]15:00
dansmithrlrossit: it's not that's why it's direct, but look in cmd/compute for an example15:00
mriedemdansmith: some small comments for if this is the actual thing15:00
johnthetubaguyedleafe: so it seems to say this is failing: ['git', 'log', '--simplify-by-decoration', '--pretty="%d"', 'origin/stable/mitaka']15:00
danpbscottda: lemme recap though, its easier than reading the bug history15:00
*** efried has joined #openstack-nova15:00
dansmithmriedem: sure, just a bluesky hail mary at this point :)15:00
*** kevin_zh has joined #openstack-nova15:00
scottdadanpb: OK15:00
*** kevinzs has joined #openstack-nova15:00
johnthetubaguyedleafe: see if that works on your command line I guess15:00
*** akshai has joined #openstack-nova15:00
bauzasrlrossit: there is no indirection api for the scheduler15:00
danpbscottda: we're basically seeing failures in the nova live migration CI jobs15:00
jlvillaldtantsur, I will watch it15:00
bauzasrlrossit: it goes read directly the DB15:00
johnthetubaguyedleafe: maybe you don't use "origin" as the name of your upstream remote?15:01
rlrossitdansmith: ah: https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L65-L6615:01
danpbin this case we have 2 QEMU running on different hosts, both connected to cinder volumes via iscsi15:01
alaskidansmith: should be vifobj not vif15:01
dtantsurjlvillal, thanks!15:01
rlrossitdoffm: ^^15:01
danpband are migrating between them, so the source QEMU is still writing to the iSCSI volume, and target QEMU just has it open readonly15:01
bauzasrlrossit: compared to https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L63-L6615:01
edleafejohnthetubaguy: hmmm... didn15:01
bauzasargh, jinxed15:01
edleafedidn't notice the 'mitaka' reference15:01
dansmithalaski: eff, thanks15:01
danpbscottda: randomly though, we're seeing the source hosts's connection to the iscsi server break15:01
*** jaypipes has quit IRC15:02
danpbscottda: whicih causes qemu to fail its migration15:02
edleafejohnthetubaguy: but yes, 'origin' is the upstream remote name15:02
openstackgerritDan Smith proposed openstack/nova: HACK: Test making VIF.address unique with port id for neutron  https://review.openstack.org/33606915:02
danpbscottda: you can see an example syslog iscsi error here  http://logs.openstack.org/03/335903/1/check/gate-tempest-dsvm-multinode-live-migration/5bae103/logs/subnode-2/syslog.txt.gz#_Jun_30_14_28_1215:02
mriedemgsilvis: are you going to be at the nova midcycle?15:02
*** krtaylor has joined #openstack-nova15:02
mriedemmnestratov|2: ^ same question15:02
danpbscottda: wondering if there's anything in cinder which could be getting upset by us having 2 qemus connected to the same cinder volume ?15:03
mriedempkoniszewski: ^15:03
*** mat128 is now known as mat128|afk15:03
pkoniszewskimriedem: yes, i will be there15:03
*** BjoernT has joined #openstack-nova15:03
danpbeg, could connecting the 2nd qemu cause a temporary interruption in the iscsi connection of the original QEMU for some reason15:03
rlrossitdansmith: bauzas: so out of curiosity, how would I do a rolling upgrade with multiple schedulers if it isn't using the indirection of conductor?15:03
mriedempkoniszewski: ack15:03
edleafejohnthetubaguy: that 'git log' command didn't work.15:03
dansmithrlrossit: you don't15:03
johnthetubaguyedleafe: you mean git log --simplify-by-decoration --pretty="%d" origin/stable/mitaka15:03
scottdadanpb: Not that I can think of offhand. Cinder just provides an iscsi target and you can do whatever you want with it...15:04
bauzasrlrossit: what dansmith said15:04
dansmithrlrossit: you just (a) don't run two schedulers and (b) have to reset scheduler as part of the api and conductor group15:04
edleafejohnthetubaguy: Do I have to explicitly check out stable mitaka locally?15:04
johnthetubaguyedleafe: have you done "git remote origin update" ?15:04
markus_zedleafe: maybe your git version doesn't know yet the flag "--simplify-by-decoration"?15:04
bauzasit's not yet part of the contract we honor15:04
scottdadanpb: i.e. you can attach 2 volumes to the target, or 10.15:04
*** kevinzh has joined #openstack-nova15:04
johnthetubaguymarkus_z: oh, good point15:04
edleafemarkus_z: no, I get "fatal: ambiguous argument 'origin/stable/mitaka': unknown revision or path not in the working tree."15:04
rlrossitdansmith: so if I do have multiple schedulers, I need to take them all down when doing an upgrade15:04
rlrossit?15:04
johnthetubaguyedleafe: yeah, try that remote origin update15:05
*** moorryan has joined #openstack-nova15:05
*** ducttape_ has joined #openstack-nova15:05
scottdadanpb: But there is plenty I don't know, so I might be missing something. I'll point people in #cinder to this, in case there's other insight.15:05
dansmithrlrossit: they're just like API nodes.. they have no protection... it may work on one release depending on what has changed, but otherwise, yeah15:05
*** shaohe_feng has quit IRC15:05
*** tbachman_ has joined #openstack-nova15:05
dansmithrlrossit: does't matter that you have one or two, they're both the same15:05
*** tbachman has quit IRC15:05
*** tbachman_ is now known as tbachman15:05
rlrossitdansmith: I'm not following, why doesn't it matter if I have one or two?15:05
*** shaohe_feng has joined #openstack-nova15:06
edleafejohnthetubaguy: doesn't seem to be a legal command15:06
edleafejohnthetubaguy: "error: Unknown subcommand: origin"15:06
johnthetubaguyedleafe: sorry, git remote update origin15:06
edleafejohnthetubaguy: doh!15:06
*** paul-carlton2 has quit IRC15:06
dansmithrlrossit: it's just no different to have N vs. 1 scheduler.. neither calls through conductor nor has any sort of upgrade protection15:06
gsilvismriedem: I am, as is knikolla15:06
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end)  https://review.openstack.org/25028315:07
*** paul-carlton2 has joined #openstack-nova15:07
mriedemgsilvis: ack15:07
rlrossitdansmith: oh I thought you were saying there is literally nothing gained by horizontally scaling the scheduler15:07
rlrossitthat's the part that confused me15:07
edleafejohnthetubaguy: ok, that pulled stable/liberty and stable/mitaka15:07
*** kevinzs has quit IRC15:07
*** kevin_zh has quit IRC15:07
dansmithno15:07
openstackgerritCharles Volzka proposed openstack/nova: Add get_host_memory_stats virt driver API  https://review.openstack.org/28883815:07
openstackgerritCharles Volzka proposed openstack/nova: Add memory metric monitor  https://review.openstack.org/28884815:07
*** kevin_zh has joined #openstack-nova15:07
bauzassudipto: well, not sure we should import some 3rd-party library this way https://review.openstack.org/#/c/203163/15:07
*** code-R has quit IRC15:07
*** armax_ has joined #openstack-nova15:08
*** dtantsur is now known as dtantsur|afk15:08
johnthetubaguyedleafe: wow, I guess you don't run that one much, I usually do that to update my origin/master, although its a little wasteful15:08
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices  https://review.openstack.org/32376515:08
*** paul-carlton2 has quit IRC15:08
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: simplify "get_console_output" interface  https://review.openstack.org/33448015:08
johnthetubaguyedleafe: the gate does a fresh clone I guess, which would do the same thing15:08
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: [functional] Fix wrong message in server description test  https://review.openstack.org/33607915:08
markus_zdanpb: ^ uses the workaround config15:08
*** paul-carlton2 has joined #openstack-nova15:08
*** armax has quit IRC15:09
*** armax_ is now known as armax15:09
*** mgould has joined #openstack-nova15:09
*** ducttape_ has quit IRC15:09
edleafejohnthetubaguy: ok, I'm just trying to figure out why it didn't like the reno in https://review.openstack.org/33564815:09
mriedemsudipto: bauzas: libvirt-python is optional and managed through g-r15:09
BobBallmriedem: No probs.  You're right, could have added in the first version.15:09
mriedemas is oslo.vmware15:09
rlrossitdansmith: so if I have multiple APIs, it's kinda the same thing, you better figure it out yourself?15:10
mriedemBobBall: i even saw you said 'obviously need to add tests but just poc'15:10
mriedem:)15:10
markus_zedleafe: could be the # char15:10
andreykurilinFolks, does anyone know about changes in error messages for `nova update` call?15:10
*** sdake has joined #openstack-nova15:10
rlrossitis there documentation anywhere telling me how I should do an upgrade like that?15:10
*** jaypipes has joined #openstack-nova15:10
dansmithrlrossit: you just have to reset them at the same time.. total outage window like 10ms15:10
bauzasmriedem: yeah, that's my point15:10
edleafemarkus_z: that's what I was thinking, but I wanted to test locally first15:10
mriedemsudipto: i guess this is the lib? https://pypi.python.org/pypi/pcp/0.115:10
rlrossitdansmith: you're making the assumption they all come back up properly ;)15:10
bauzasmriedem: just importing a pypi libary without explaining why at least is bad15:10
dansmithrlrossit: is this line of questioning related to FF today?15:11
rlrossitdansmith: it is not15:11
*** coreywright has quit IRC15:11
BobBallmriedem: oops :)15:11
mriedemrlrossit: we have several fires right now15:11
dansmithrlrossit: I will totally discuss this with you at length after tomorrow, if that's okay :)15:11
rlrossitdansmith: fine with me15:11
*** ducttape_ has joined #openstack-nova15:11
dansmithrlrossit: sorry, but thanks15:12
mriedemrlrossit: there are upgrade docs in the ops guide i believe15:12
sudiptomriedem, yeah that's right...15:12
mriedemrlrossit: or probably ask around in #openstack-operators15:12
*** coreywright has joined #openstack-nova15:12
mriedemsudipto: pre-alpha 0.1 :(15:12
bauzasrlrossit: TBH, that just means to me that we can't live upgrade the schedulers, that's it15:12
*** diogogmt has joined #openstack-nova15:12
sudiptomriedem, asked in the pcp channel...just give me a few mins...15:13
*** sdake_ has quit IRC15:13
sudiptotypically, i don't install it via pip -15:13
rlrossitbauzas: which, for most people is fine, but now I need to start thinking in mega scale, where things like that start breaking down :)15:13
bauzasrlrossit: but there are other ways for circling around upgrading schedulers :)15:13
*** isq_ has quit IRC15:13
sudiptomriedem, like i thought, this is the response " they just get distributed alongside pcp core, built at the same time as the c code"15:13
bauzassudipto: well, not sure I love some library being installed by other way than pip honestly15:13
sudiptobauzas, ok.15:14
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (error)  https://review.openstack.org/32805515:14
mriedemi guess most distros will probably also package their python/perl/ruby/java/c clients along with the main service package15:14
*** muawiakhan has quit IRC15:15
*** tbachman has quit IRC15:15
danpbPaulMurray: cinder is running on the target host. so another possibility could be that when qemu is started on the target, and neutron configures its nics, perhaps something is causing a temporary network interruption which breaks the iscsi connection to the source host15:15
openstackgerritVictor Stinner proposed openstack/nova: Port test_fakelibvirt to Python 3  https://review.openstack.org/33325115:15
openstackgerritVictor Stinner proposed openstack/nova: Port ironic test_driver to Python 3  https://review.openstack.org/33325015:15
openstackgerritVictor Stinner proposed openstack/nova: Port libvirt test_driver to Python 3  https://review.openstack.org/33324915:15
openstackgerritVictor Stinner proposed openstack/nova: Port test_virt_drivers to Python 3  https://review.openstack.org/33324815:15
openstackgerritVictor Stinner proposed openstack/nova: Port virt test_hardware to Python 3  https://review.openstack.org/33324715:15
openstackgerritVictor Stinner proposed openstack/nova: Port pci unit tests to Python 3  https://review.openstack.org/33279115:15
openstackgerritVictor Stinner proposed openstack/nova: Port test_metadata to Python 3  https://review.openstack.org/33273815:15
sudiptomriedem, i guess that's the case here...15:15
openstackgerritVictor Stinner proposed openstack/nova: Port test_hacking to Python 3  https://review.openstack.org/33273715:15
bauzasmriedem: but that's another story, not the upstream one :)15:15
*** shaohe_feng has quit IRC15:15
mriedembauzas: sure, you still have to get the pcpd from the distro15:15
mriedemwell, not have to i guess15:15
mriedemyou could build from source yourself15:15
bauzasmriedem: yup, I mean I'm fine with distros providing their own versions15:16
mriedemwhat do we do with osinfo?15:16
bauzasmriedem: but that's something we at least need to import from pypi upstream IMO15:16
*** shaohe_feng has joined #openstack-nova15:16
*** rrecio has joined #openstack-nova15:16
mriedem                libosinfo = importutils.import_module(15:16
mriedem                                             'gi.repository.Libosinfo')15:16
*** jlanoux has joined #openstack-nova15:17
*** Daisy has joined #openstack-nova15:17
bauzashuh15:17
mriedembauzas: there is a py-osinfo in pypi15:17
bauzasI thought we added it to g-r15:17
mriedemnope15:18
bauzasso, say I want to use a big 3rd-party lib, I could just import the python API in Nova and not add it as a dep ?15:18
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices  https://review.openstack.org/32376515:19
danpbmriedem: bauzas  libosinfo doesn't have any python bindings at all15:19
*** rcernin has quit IRC15:19
danpbthe magic of gobject introspection means the python interface just magically appears as a FFI wrapper around the c library15:20
mriedemhttps://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=1&ved=0ahUKEwiriuXohdDNAhXJOSYKHdjlD6MQFggcMAA&url=https%3A%2F%2Fpypi.python.org%2Fpypi%2Fpy-osinfo&usg=AFQjCNFcQwhU9mzzXPJTJguh_gfB3Kaufg&sig2=PRXpHASMqrtHL_bY9xgIsA&bvm=bv.125801520,d.eWE&cad=rja15:20
mriedemoops15:20
mriedem# TODO(vladikr) The current implementation will serve only as a temporary15:20
mriedem# solution, due to it's dependency on the libosinfo gobject library.15:20
mriedem# In the future it will be replaced by a pure python library or by a direct15:20
mriedem# parsing of the libosinfo XML files. However, it will be possible only when15:20
mriedem# libosinfo project will declare the XML structure to be a stable ABI.15:20
mriedemyeah15:20
bauzasmriedem: google troller15:20
bauzasdanpb: ack15:20
mriedemdanpb: that's how libvirt-python works right?15:20
PaulMurraydanpb, does all this happen before the migration is considered complete ?15:20
mriedemwhch always confuses me15:20
danpbno, libvirt python is a hand written python C binding15:20
openstackgerritVictor Stinner proposed openstack/nova: Port test_fakelibvirt to Python 3  https://review.openstack.org/33325115:21
openstackgerritVictor Stinner proposed openstack/nova: Port ironic test_driver to Python 3  https://review.openstack.org/33325015:21
openstackgerritVictor Stinner proposed openstack/nova: Port libvirt test_driver to Python 3  https://review.openstack.org/33324915:21
openstackgerritVictor Stinner proposed openstack/nova: Port test_virt_drivers to Python 3  https://review.openstack.org/33324815:21
openstackgerritVictor Stinner proposed openstack/nova: Port virt test_hardware to Python 3  https://review.openstack.org/33324715:21
PaulMurraydanpb, the networking gets configured after the migration completes15:21
bauzasthe thing is, I'm fine with us commenting in the relnotes that "this" feature now uses a 3rd party lib15:21
danpbmriedem: nb   py-osinfo is nothing at all todo with libosinfo - just a name collision15:21
PaulMurraydanpb, when the ports are bound on the destination15:21
*** Daisy has quit IRC15:21
danpbPaulMurray: yeah, we get the fail before we switch to running on the target15:22
*** armax has quit IRC15:22
sudiptobauzas, i think i mentioned that in the release notes...15:22
danpbPaulMurray: it just wonder if there's some background task in neutron - as we've created the ports by then, just not started using them15:22
bauzassudipto: orly? in that change?15:22
PaulMurraydanpb, that could be right15:22
PaulMurraydanpb, I'm guessing we don't use DVR in CI15:23
sudiptohttps://review.openstack.org/#/c/329785/11/releasenotes/notes/bp-memory-bw-c423266d4e4ae663.yaml bauzas - it's got a minor space issue, but you can comment if i need to add more info.15:23
danpband i know neutron does alot of async work after api requests finish15:23
*** burt has joined #openstack-nova15:23
bauzassudipto: oh that's in a follow-up change15:23
*** jistr|mtg is now known as jistr15:23
sudiptobauzas, yeah...on the final one that enables the monitor.15:23
bauzassudipto: well, I was about to say that relnotes are good, but we loose track on what we add when we're on the next cycle15:24
danpbmriedem: sudipto bauzas looking at this pcp change, i can't help thinking this functionality should be exposed directly by libvirt15:24
mgouldhi everyone15:24
danpbrather than having it go out to this pcp api15:24
openstackgerritSarafraj Singh proposed openstack/nova: Improve help context of ironic options  https://review.openstack.org/33243315:24
*** gb21_ has joined #openstack-nova15:24
mgouldover in #openstack-ironic, we're trying to understand the extra_specs syntax15:24
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications samples  https://review.openstack.org/32886615:24
danpbwe're already doing similar stuff in libvirt with the code we just added to libvirt + nova to query performance counters for cpu cache related stuff15:24
sudiptodanpb, yeah, when i started working on this about 1 year back - there was no libvirt exposure to perf data...i see that there are a few patches submitted now.15:24
bauzasdanpb: oh good to know, is libvirt exposing that ?15:25
mgouldis the value being matched a string, or can it be something else?15:25
*** GB21 has quit IRC15:25
danpbso it feels like it would be better to do memory bandwidth in this same way15:25
*** gb21_ is now known as GB2115:25
dansmithdanpb: if available, that sounds like a much better plan to me15:25
*** browne has joined #openstack-nova15:25
*** gagehugo has quit IRC15:25
mriedemone knock against that will be,15:25
sudiptodanpb, while libvirt is definitely a good route from dependency management per say - i guess pcp gives a lot of insight into perf data, that libvirt has to still build into?15:25
danpbdansmith: nb the API structure is available - we'd have to actually make libvirt be able to report the memory bandwidth stuff before we could use it15:25
mriedemthat's libvirt specific15:25
*** shaohe_feng has quit IRC15:25
danpbso you'd obviously end up waiting another release cycle15:25
mriedembut having said that, we only track cpu metrics for libvirt i think15:26
*** shaohe_feng has joined #openstack-nova15:26
bauzasdanpb: that would be a far better option actually15:26
danpbmriedem: this new proposed code is libvirt specific already iiuc15:26
mriedemwell maybe not, but at least for this bw monitor thing15:26
dansmithmriedem: this is dependent on the libvirt driver anyway15:26
mriedemyeah15:26
mriedemthe cpu monitor just uses host stats right?15:26
bauzasmriedem: yup, what dansmith said15:26
mriedemwhich comes from all virt drivers15:26
dansmithmriedem: you can still have the monitor use a generic blob like cpu,15:26
dansmithyou just should get that data from libvirt in the driver15:27
dansmithsame structure as the cpu one15:27
danpbcommit 71f24dfeeeb6713f530b260a0e2b4bb23c00effa15:27
danpbAuthor: Eli Qiao <liyong.qiao@intel.com>15:27
danpbDate:   Wed May 25 14:48:08 2016 +080015:27
danpb    libvirt: add perf event support when create instance15:27
danpbthat's the change i'm refering to in nova15:27
*** sacharya_ has joined #openstack-nova15:27
*** tesseract- has quit IRC15:27
mriedemthat's per instance though right?15:27
mriedemsudipto's is per host15:27
openstackgerritSarafraj Singh proposed openstack/nova: Improve help context of ironic options  https://review.openstack.org/33243315:28
sudiptomriedem, yeah that's per instance and only x86 specific at the moment.15:28
bauzasmriedem: nope, monitors are per host15:28
danpbmriedem: oh, hmm, yes, good point15:28
bauzasunless I'm heavily wrong15:28
mriedembauzas: so s/nope/yup/15:28
danpbi still wonder if we should add a api to libvirt to report these per-host  perf data15:28
mriedemdanpb: that's not yet in upstream libvirt?15:29
*** kaisers has quit IRC15:29
bauzasmriedem: I strongly don't disagree with people not expressing their disagreements with me15:29
mriedembauzas: your double negative mastery amazes me15:29
danpbmriedem: indeed not15:29
mriedemdanpb: ok, so that would push actually testing this out or being in a distro quite a bit15:30
danpbyeah, as i say, it'd certainly delay things for 1 nova cycle15:30
*** sacharya has quit IRC15:30
sudiptodanpb, i read through the libvirt patches that were submitted for the L3-cache and memory bandwidth (which i think will be done next term), however personally i don't like the way perf is being handled by libvirt with the current design.15:30
* mriedem steps away for a bit15:30
markus_zedleafe: It's a mixture of the # plus :15:30
sudiptohowever, that's a different forum discussion maybe...15:30
bauzasmriedem: but monitors are entrypoints, right?15:30
markus_zedleafe: I don't know why, but I could solve it locally by removing those two chars15:31
bauzasso, people can ship their own pcp monitors out-of-tree if they wish15:31
mriedembauzas: i was told those shouldn't be pluggable15:31
mriedembut i think they are yeah15:31
mriedemand sudipto's series changed one of the ABC methods in the base class which will break any plugins out of tree15:31
mriedemwhich was a comment i brought up a few times15:31
*** code-R_ has joined #openstack-nova15:31
bauzasmriedem: yup, but that's a good move15:31
mriedembreaking stuff?15:32
tjonesmriedem: there are just 4 vmware patches ready for core.  rado replied to your -1 last night.  they are https://review.openstack.org/#/c/281134 https://review.openstack.org/#/c/196171/ https://review.openstack.org/#/c/204219/ https://review.openstack.org/#/c/297574/  thanks if you guys can give them a look15:32
bauzasmriedem: if the breakage leads to a better interface15:32
bauzasmriedem: that's what we did with filters using the RequestSpec for example :)15:32
sudiptothe reason i said i didn't like the libvirt implementation of perf is because it hardcodes the value of the platform it supports into libvirt itself...while pcp depends a external library - to load it for various processor types.15:32
sudiptowith that said, the memory bandwidth implementation is currently only there for x86.15:33
mgouldbauzas: I'm trying to understand the semantics of extra_specs, can you help?15:33
mriedemmdbooth: can you see https://bugs.launchpad.net/nova/+bug/1524274/comments/68 ?15:33
openstackLaunchpad bug 1524274 in OpenStack Security Advisory "[OSSA 2016-001] Unprivileged api user can access host data using instance snapshot (CVE-2015-7548)" [Critical,Fix released] - Assigned to Tristan Cacqueray (tristan-cacqueray)15:33
mdboothmriedem: Looking15:33
bauzasmgould: well, okay... will try :)15:33
*** thumpba has joined #openstack-nova15:33
bauzasflavor extra specs are a beast15:33
mgouldin particular, does <all-in> apply to strings or to collections?15:33
mriedemsudipto: which is important for you because you care about ppc15:33
mgouldbauzas: thanks!15:33
PaulMurrayjaypipes, I added you to https://review.openstack.org/#/c/253556/ because you reviewed https://review.openstack.org/#/c/31897415:34
PaulMurrayjaypipes, I hope you don't mind - they were both doing the same thing at the same time15:34
jaypipesPaulMurray: will try to get to that. still waiting on gertty to sync up 980+ patchsets...15:34
mgouldbackground: we need something similar in Ironic, and are trying to work out if the semantics are a good fit for us15:34
sudiptomriedem, i just care about perf to be honest :) and want it to be in a way that different archs can use it...15:34
PaulMurrayjaypipes, np - just letting you know15:34
openstackgerritAnusha Unnam proposed openstack/nova: Transform instance.resize notifications  https://review.openstack.org/32925515:34
bauzasmgould: you mean the filter semantics ?15:35
mgouldyes15:35
*** shaohe_feng has quit IRC15:36
mgouldsorry, my grasp of nova is still pretty weak15:36
mdboothshhjwang: Looking at https://bugs.launchpad.net/nova/+bug/1524274/comments/6815:36
openstackLaunchpad bug 1524274 in OpenStack Security Advisory "[OSSA 2016-001] Unprivileged api user can access host data using instance snapshot (CVE-2015-7548)" [Critical,Fix released] - Assigned to Tristan Cacqueray (tristan-cacqueray)15:36
mdboothFirstly, are you sure you have a vulnerable version of Nova?15:37
*** shaohe_feng has joined #openstack-nova15:37
mdboothThis was fixed quite a while ago15:37
mgouldbauzas: further context is we're looking at https://review.openstack.org/#/c/308398 and trying to decide what tests it needs15:37
bauzasmgould: well, the UTs are good examples, wsec15:37
*** fche has joined #openstack-nova15:37
shhjwangmdbooth: Yeah, my platform has the unpatched nova.15:37
bauzasmgould: https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/filters/test_extra_specs_ops.py#L202-L23515:38
mdboothshhjwang: Why is the passwd file a binary file?15:39
sudiptomriedem, danpb i am not sure if that's an convincing answer, but can you tell me what i should do next?15:39
*** lberk has joined #openstack-nova15:39
*** pushkaru has quit IRC15:39
mgouldbauzas: right, so it's matching against a list15:40
mgouldthanks!15:40
*** sridharg has quit IRC15:40
shhjwangmdbooth: You mean it should not be a binary file? I use " glance image-download xxx --file passwd" as mentioned. So the generated passwd is binary.15:41
*** jaugustine has quit IRC15:41
bauzasmgould: indeed15:41
bauzasbecause we .split()15:41
mdboothshhjwang: Run file against it. What does it say?15:41
mdboothshhjwang: $ file passwd15:41
mdboothWhat does it say?15:41
bauzasmgould: https://github.com/openstack/nova/blob/master/nova/scheduler/filters/extra_specs_ops.py#L3915:42
*** rlrossit has quit IRC15:42
bauzasmgould: so we contract on a string, but we analyse it as a list15:42
openstackgerritChris Friesen proposed openstack/nova: Allow logging detailed info only on sched failure  https://review.openstack.org/33130615:44
*** kaisers has joined #openstack-nova15:44
mgouldbauzas: I see the tests match str(values)15:44
shhjwangmdbooth: It shows "passwd: Qemu Image, Format: Qcow , Version: 2"15:44
mgouldwhich means specs_matcher.match(str(['foo', 'bar']), '<all-in> f') returns True15:44
mgouldis that expected behaviour?15:44
bauzasmgould: see my above link15:45
mdboothshhjwang: Ok, run $ qemu-img info passwd15:45
mdboothWhat backing file does it have?15:45
mgouldbauzas: the link to the tests?15:45
mgouldyes, that's what I'm looking at15:45
*** shaohe_feng has quit IRC15:46
*** rlrossit has joined #openstack-nova15:46
gibi_bauzas, johnthetubaguy: before you call it a day could you check (hopefully last time) the notification transformation patches?15:46
*** shaohe_feng has joined #openstack-nova15:47
mgouldbauzas: let me push up a patch so you can see what I'm talking about, one moment...15:47
bauzasmgould: no, the fact that the match() method takes a string as a req, but splits it15:47
shhjwangmdbooth: No backing file was shown.15:47
mgouldbauzas: you mean it splits the matcher spec?15:47
mgouldI'm talking about the value-being-matched15:48
shhjwangmdbooth: So this is the behavior after the patches?15:48
mdboothshhjwang: The behaviour after the patches should be that you get the contents of bad.qcow2 in passwd15:48
mdboothBefore the patches you get the contents of /etc/passwd15:48
*** harlowja_at_home has joined #openstack-nova15:48
openstackgerritClaudiu Belu proposed openstack/nova: WIP: policy: clean-up  https://review.openstack.org/33392515:49
*** harlowja_at_home has quit IRC15:49
mriedemjlvillal: is there an ironic patch running with dansmith's workaround?15:49
jlvillalmriedem, Yes15:49
bauzasmgould: it splits each value of the dict that is instance_type.extra_specs15:50
mriedemah yeah https://review.openstack.org/#/c/335998/15:50
jlvillalmriedem, https://review.openstack.org/#/c/335998/15:50
jlvillalmriedem, Darn, you are faster than me :)15:50
*** salv-orlando has quit IRC15:50
openstackgerritSTEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk  https://review.openstack.org/33307515:51
jlvillalmriedem, I updated the bug to say a test with reverting that one patch did fix the issue.15:51
mriedemjlvillal: yeah but we're not going to revert it15:51
mriedemjlvillal: it's part of the rest api now15:51
mriedem*required for the rest api now15:51
jlvillalOkay15:51
mriedemdam15:51
shhjwangmdbooth: But why the backing file is not there. I have set use_cow_images to false15:51
mriedem*dan's change should workaround it15:51
dansmithartom: question in here: https://review.openstack.org/#/c/304511/7015:51
mriedemas it will make the mac unique per port uuid15:52
shhjwangmdbooth: And qemu-img info bad.qcow2 shows the backing file.15:52
mriedemvif.address i mean15:52
dansmithmriedem: yeah, been watching it15:52
mriedemthe telnet thing doesn't work for me15:52
mdboothshhjwang: What's images_type in nova.conf?15:52
edleafemarkus_z: I don't think it was the # or the : - I fixed it by adding a | and newline15:53
edleafemarkus_z: like this: http://paste.openstack.org/show/524296/15:53
*** ctrath has quit IRC15:53
shhjwangmdbooth: It's default15:53
mgouldbauzas: so if instance_type.extra_specs is {k: 'foo bar'} then '<all-in> f a' should not match it, but '<all-in> foo bar' should?15:53
shhjwangimages_type=default15:53
*** gagehugo has joined #openstack-nova15:53
mriedemedleafe: have you looked at http://docs.openstack.org/developer/reno/usage.html ?15:54
mriedemhttp://docs.openstack.org/developer/reno/usage.html#formatting15:54
edleafemriedem: yes, but never got that far down :)15:54
mriedem- | is required for ReST15:54
*** fragatina has quit IRC15:54
*** EinstCrazy has quit IRC15:55
mdboothshhjwang: What do you get if you do: $ qemu-img convert -f qcow2 -O raw passwd passwd.raw?15:55
*** fragatina has joined #openstack-nova15:55
*** pbandark has quit IRC15:55
mriedemoh nvm the telnet thing does work, nice15:55
mdboothshhjwang: I wonder if it's the snapshot image conversion converting the file to raw15:55
mdbooths/raw/qcow2/15:55
*** ctrath has joined #openstack-nova15:55
bauzasmgould: yup, like what the test says :)15:55
* mdbooth vaguely recalls something about that15:55
markus_zedleafe: Ha, interesting.15:55
*** andreas_s has quit IRC15:56
*** shaohe_feng has quit IRC15:56
edleafemarkus_z: mriedem: Funny thing is that the template you get when creating a reno doesn't follow the correct formatting15:56
shhjwangmdbooth: I get the file passwd.raw without any console output.15:57
edleafeI just followed the template's format15:57
mdboothshhjwang: What does passwd.raw contain?15:57
*** shaohe_feng has joined #openstack-nova15:57
mgouldbauzas: OK, I think there's a bug in the tests then15:57
openstackgerritMiles Gould proposed openstack/nova: DO NOT MERGE! Test all-in semantics  https://review.openstack.org/33609415:58
mgouldit should say `value=values` rather than `value=str(values)`15:58
mriedemdansmith: artom: i had commented on the same version thing last night15:58
mriedemin the meta api15:58
dansmithmriedem: oh sorry I missed that15:58
mriedemit might have been rebased out, not sure15:58
markus_zedleafe: I've seen "|" sometimes in other reno files, but was never sure why it was there. Maybe an update on https://github.com/openstack/reno/blob/master/reno/create.py is appropriate?15:59
shhjwangmdbooth: OK. Now "qemu-img info" shows backing file /etc/passwd15:59
mriedemanyway, easy fix15:59
edleafemarkus_z: I was just looking at that :)15:59
*** mgoddard has quit IRC15:59
mdboothshhjwang: Ok, sounds like you got a qcow2 whose contents is your bad.qcow215:59
mdboothWhich means you haven't hit the vulnerability16:00
dansmithjlvillal: are the other failures on that patch expected?16:00
mdboothI can't promise that's because you're not vulnerable, though16:00
*** dims has quit IRC16:00
shhjwangmdbooth: so is there any limitation on the type of glance image or any other things?16:01
jlvillaldansmith, Let me look16:01
shhjwangmdbooth: I have a glance images with raw disk format. Is that right?16:02
mdboothshhjwang: I'm not going to be able to help debug this this evening, unfortunately. Can you ping me tomorrow and I'll try to help?16:03
mdboothIn in the UK TZ (GMT + 1)16:03
dansmithjohnthetubaguy: amazing job breaking these changes into move, refactor, move, refactor16:04
*** DaiDV has joined #openstack-nova16:04
shhjwangmdbooth: Yeah. Thank you very much for you help.16:04
*** DaiDV has quit IRC16:05
*** gjayavelu has quit IRC16:05
*** matrohon has quit IRC16:05
jlvillaldansmith, Regarding https://review.openstack.org/#/c/335998/  ?  I don't see it finished running yet. I didn't check zuul though16:05
dansmithjlvillal: yeah, in zuul16:06
*** tonytan4ever has quit IRC16:06
mriedemrunning smoke tests16:06
*** dims has joined #openstack-nova16:06
dansmithjlvillal: a bunch of tests I would have expected to pass failed16:06
danpbanyone tell me which openstack git repo has the code to determine which logfiles to capture for CI runs ?16:06
danpbi need to edit it to capture log files from the iscsi server too16:06
jlvillaldansmith, Let me check zuul16:06
mriedemdanpb: devstack-gate16:06
dansmithhttp://logs.openstack.org/98/335998/2/check/gate-tempest-dsvm-ironic-agent_ssh/f34ee43/16:06
dansmithjlvillal: ^16:06
danpbthanks16:06
*** shaohe_feng has quit IRC16:06
mriedemdansmith: that's a non-voting job16:07
dansmithjlvillal: ah crap it is my patch16:07
mriedemwhich always times out16:07
*** shaohe_feng has joined #openstack-nova16:07
dansmiththe vif object does one thing strangely and screwed me yp16:07
mriedemah16:07
mnestratov|2mriedem: yeah I am16:07
mriedemmnestratov|2: ack16:08
mriedemdid that rescue test run in the vz ci yet?16:08
jlvillaldansmith, Okay and :(16:08
dansmithcrap and now a rebase16:08
edleafemarkus_z: https://review.openstack.org/#/c/336098/16:08
*** Jeffrey4l has quit IRC16:08
mnestratov|2mriedem: running devstack right now16:09
mriedemNameError: global name \'self\' is not defined16:09
mriedemyikes16:09
*** mnestratov|2 is now known as mnestratov16:09
mriedemah db_vif['address'].split16:10
dansmithjohnthetubaguy's code makes it easier16:10
mnestratovyesterday we ran it with correct regexp but forgot to change tempest.conf accordinly, thus all rescue tests were skipped16:11
openstackgerritDan Smith proposed openstack/nova: HACK: Test making VIF.address unique with port id for neutron  https://review.openstack.org/33606916:12
mriedemdanpb: log collection happens somewhere in here https://github.com/openstack-infra/devstack-gate/blob/master/functions.sh#L74016:12
dansmithjlvillal: recheck ^16:12
*** baoli_ has quit IRC16:12
jlvillaldansmith, will do16:12
danpbmriedem: yep, thanks16:12
*** baoli has joined #openstack-nova16:13
dansmithjlvillal: I'll work on local tests to catch any other issues sooner in case something else needs changing16:13
*** ljxiash has quit IRC16:13
*** thorst_ has quit IRC16:13
jlvillaldansmith, Thanks. And recheck started16:13
*** thorst_ has joined #openstack-nova16:14
*** fragatina has quit IRC16:15
*** thorst__ has joined #openstack-nova16:15
*** gongysh has joined #openstack-nova16:15
*** kevinzs has joined #openstack-nova16:15
*** sdake_ has joined #openstack-nova16:15
*** cdelatte has quit IRC16:16
*** rama_y has joined #openstack-nova16:16
*** shaohe_feng has quit IRC16:17
*** migi_ is now known as migi16:17
*** shaohe_feng has joined #openstack-nova16:17
*** browne has quit IRC16:17
*** thorst_ has quit IRC16:18
*** sdake has quit IRC16:18
openstackgerritMatt Riedemann proposed openstack/nova: trivial: remove unnecessary mock from servers API test  https://review.openstack.org/33610516:19
dansmithjlvillal: tests say it's good16:19
*** kevinzs has quit IRC16:20
*** thorst__ has quit IRC16:20
*** kevin_zh has quit IRC16:20
*** kevinzh has quit IRC16:20
*** zhangjn has quit IRC16:20
jlvillaldansmith, Great :)  Hopefully Grenade too16:20
*** aloga_ has joined #openstack-nova16:20
mriedemtjones: nack on https://review.openstack.org/#/c/297574/16:21
mriedemthe spec was approved in mitaka and newton, and if there was a dependency in glanceclient, that shouldn't have been a surprise by now16:22
*** pwnall has quit IRC16:22
*** pwnall has joined #openstack-nova16:22
*** sdake_ is now known as sdake16:23
*** zhangjn has joined #openstack-nova16:25
gibi_mriedem, dansmith: I'm off for about two hours but I can still fix issues with the notification transformation patches if you have any16:27
openstackgerritMatthew Booth proposed openstack/nova: Remove max_size argument to images.fetch and fetch_to_raw  https://review.openstack.org/32694716:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Rename the import of nova.virt.disk.api in driver  https://review.openstack.org/33352116:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Add DiskFromImage and DiskFromFunc  https://review.openstack.org/33352216:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Pass is_block_dev explicitly from create_from_func  https://review.openstack.org/33326316:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Move swap disk fetch into a separate function  https://review.openstack.org/33398416:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Add _create_external_boot_disks  https://review.openstack.org/33398016:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Move local root disk fetch into a separate function  https://review.openstack.org/33398216:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Move enumeration of ephemerals into a separate function  https://review.openstack.org/33398316:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Replace some uses of invalid uuids in test_driver  https://review.openstack.org/33397816:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Don't re-resize disks in finish_migration()  https://review.openstack.org/33111816:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Remove some unnecessary mocking in test_driver  https://review.openstack.org/33397916:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend  https://review.openstack.org/33324416:27
openstackgerritMatthew Booth proposed openstack/nova: Resize an existing disk  https://review.openstack.org/33241616:27
*** shaohe_feng has quit IRC16:27
openstackgerritMatthew Booth proposed openstack/nova: Introduce ImageCacheLocalDir  https://review.openstack.org/27966916:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Rename Backend snapshot and image  https://review.openstack.org/33327216:27
openstackgerritMatthew Booth proposed openstack/nova: Add check_backing_from_func & check_backing_from_image  https://review.openstack.org/32228516:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Use valid uuids and fnames in test_create_image_initrd  https://review.openstack.org/33324116:27
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Pass instance object to _create_images_and_backing in test  https://review.openstack.org/33324316:27
mriedemgibi_: i'm pretty sure mdbooth just killed gerrit16:27
mriedembut ok16:27
mriedemgerrit/zuul16:27
* mdbooth hates those things anyway16:28
*** shaohe_feng has joined #openstack-nova16:28
gibi_mriedem: ohh, good move on ff day :)16:28
*** armax has joined #openstack-nova16:28
andreykurilinmriedem: could you look at +1;-1 change to unblock novaclient's gates?)16:28
mdboothlyarwood: If you look around here: https://review.openstack.org/#/c/333984/16:28
mriedemandreykurilin: which thing?16:29
mdboothlyarwood: _create_image is now a really short function16:29
andreykurilinmriedem: https://review.openstack.org/#/c/336079/16:29
mdboothWell, comparatively short16:29
openstackgerritAihua Edward Li proposed openstack/nova: Always clean up network resource when VM creation failed close-bug: #1597596  https://review.openstack.org/33578816:29
openstackbug 1597596 in OpenStack Compute (nova) "network not always cleaned up when spawning VMs" [Undecided,Fix committed] https://launchpad.net/bugs/1597596 - Assigned to Aihua Edward Li (aihuaedwardli)16:29
mdboothlyarwood: So, rescue and spawn() both want to do: external boot, config disk, root disk, injection16:30
mdboothThat's all rescue wants to do16:30
mdboothspawn want to additionally do ephemeral and swap16:30
* mdbooth was going to create a function from the above bag of bits to do the common bits16:31
danpbmdbooth: rescue should do  ephemeral and swap too16:31
lyarwoodmdbooth: that's all it wants to do for now, stable rescues would just tag on an additional boot disk at the end16:31
mdboothdanpb: This is for *creation*16:31
lyarwoodmdbooth: ah right16:32
mriedemandreykurilin: when did that start failing?16:32
mdboothdanpb: Those are the only things it wants to create, but it should add everything for stable rescue16:32
mdboothlyarwood: However, given the current behaviour of cache(), calling it on an existing ephemeral/swap disk is only a wart16:33
mdboothBecause it won't actually do anything16:33
andreykurilinmriedem: I don't know. I found that tempest release is changed in gates, but quick look of tempest changes did not show me the root of issue16:33
lyarwoodthat's a useful wart for now16:33
mdboothRight16:33
mdboothYou can safely rely on that, in fact. Other code does.16:33
mdboothThis series is about removing that assumption, so I'm happy to clean up stable rescue.16:34
mdboothI was planning to do it anyway16:34
mdboothWell, unstable rescue16:34
*** baoli has quit IRC16:35
lyarwoodyeah but without this _create_image rewrite I'd have to go in and hack around the eph creation logic at present16:35
openstackgerritPaul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances  https://review.openstack.org/30819816:35
mdboothlyarwood: Ahh... you said it's actually creating disk.eph0.rescue?16:35
mdboothYou can hack round that16:36
*** ctrath has quit IRC16:36
mdboothSee _create_image() has 2 functions up the top, image() and raw()16:36
mdboothwhich are used to create image objects16:36
lyarwoodmdbooth: disk.local.rescue sorry no eph16:36
lyarwoodnot*16:36
mdboothRight, same thing16:36
lyarwoodtrue16:36
mdboothUpdate _create_image to not use image() for ephemeral and swap16:37
mdboothIt's not required for rescue anyway16:37
*** vtech has joined #openstack-nova16:37
*** shaohe_feng has quit IRC16:37
mdboothAnd it's a much smaller fix than trying to take my series16:37
*** kaisers has quit IRC16:37
mdboothAnyway, I must dash16:37
lyarwoodmdbooth: np, thanks again16:38
*** shaohe_feng has joined #openstack-nova16:38
*** baoli has joined #openstack-nova16:38
tiendcdansmith: Hi, could you take a look at the gmann's comment in  https://review.openstack.org/#/c/32815716:39
tiendcdansmith: today is the deadline for this patch, I would appreciate if you can give your opinion about that16:40
*** ctrath has joined #openstack-nova16:40
tiendcdansmith: I'm trying to convince the core team for adding the patch to exception list16:40
dansmithtiendc: what? deadline today? I had no idea!16:40
tiendcdansmith: I means non-priority feature freeze16:41
*** abalutoiu has quit IRC16:41
dansmithtiendc: I know, I'm being sarcastic.. :)16:41
markus_ztiendc: you get used to it someday ;)16:41
*** cdelatte has joined #openstack-nova16:41
dansmithtiendc: I haven't looked at that code before so there are soooo many things pending today I'm not sure I'll get to it16:42
dansmithtiendc: it would be good to at least get an ack on it from someone like sahid who is familiar with that code16:42
*** baoli has quit IRC16:43
*** ssurana has joined #openstack-nova16:43
mriedemandreykurilin: https://review.openstack.org/#/c/333795/16:43
mriedem?16:43
tiendcdansmith: thanks, I will try to ask him16:44
andreykurilinmriedem: trivial fix. it didn't change nothing for py27 env16:44
*** aloga_ has quit IRC16:45
*** paul-carlton has joined #openstack-nova16:46
andreykurilinmriedem: python2.7 -c "import string;print(string.letters == string.ascii_letters)" => True16:46
*** danpb has quit IRC16:47
*** paul-carlton2 has quit IRC16:47
*** apanaitiu has joined #openstack-nova16:47
*** shaohe_feng has quit IRC16:47
*** shaohe_feng has joined #openstack-nova16:48
*** zz_dimtruck is now known as dimtruck16:49
*** mdrabe_ has quit IRC16:50
*** diga has quit IRC16:50
*** lucasagomes is now known as lucas-afk16:50
openstackgerritVladik Romanovsky proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451116:51
openstackgerritEd Leafe proposed openstack/nova: Return HTTP 400 on list for invalid status  https://review.openstack.org/31596416:51
openstackgerritEd Leafe proposed openstack/nova: Return HTTP 200 on list for invalid status  https://review.openstack.org/33564816:51
*** markus_z has quit IRC16:51
*** salv-orlando has joined #openstack-nova16:51
*** abalutoiu has joined #openstack-nova16:51
mnestratovmriedem: dansmith: our CI finally did what we begged it to do in https://review.openstack.org/#/c/217679 :)16:52
* mriedem looks for "if change xyz pass rescue tests" :)16:52
dansmithhaha16:52
*** abalutoiu_ has joined #openstack-nova16:53
artomdansmith, hey, back now, went for a run, needed it badly - looks like vladikr already addressed the _os_version stuff in https://review.openstack.org/#/c/304511/, right?16:53
*** baoli has joined #openstack-nova16:54
dansmithartom: ah, hadn't noticed, but yeah16:54
dansmithartom: mdbooth was DoSing gerrit anyway16:54
artomdansmith, can you wag at him like you did at me last time?16:55
dansmithartom: mdbooth is on permanent wag status16:56
*** abalutoiu has quit IRC16:56
*** RuiChen has quit IRC16:56
*** salv-orl_ has joined #openstack-nova16:57
mriedemthar she blar 15:22:27 + [[ 217679,21 =~ 217679 ]]16:57
mgouldbauzas: https://review.openstack.org/#/c/336094/ passed CI16:57
*** diga has joined #openstack-nova16:57
mgouldwhich I think means the extra_specs tests aren't testing what you think16:58
*** shaohe_feng has quit IRC16:58
*** apanaitiu has quit IRC16:58
*** salv-orlando has quit IRC16:58
*** _ducttape_ has joined #openstack-nova16:58
*** shaohe_feng has joined #openstack-nova16:58
*** dimtruck is now known as zz_dimtruck16:59
mriedemdansmith: +2 on https://review.openstack.org/#/c/217679/ now17:00
openstackgerritLudovic Beliveau proposed openstack/nova: Fix revert on migration with SR-IOV  https://review.openstack.org/32617417:00
openstackgerritLudovic Beliveau proposed openstack/nova: Raise exception if PCI device spec is not found  https://review.openstack.org/33183017:00
openstackgerritLudovic Beliveau proposed openstack/nova: Allocate PCI devices on migration  https://review.openstack.org/32898317:00
openstackgerritLudovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports  https://review.openstack.org/24257317:00
*** markmc` is now known as markmc17:00
*** daemontool_ has joined #openstack-nova17:01
*** sree has joined #openstack-nova17:01
dansmithmriedem: cool17:01
dansmithmriedem: I'm just going through the tests of that metadata api patch and then I'm good on it too17:01
*** abalutoiu_ has quit IRC17:02
*** salv-orl_ has quit IRC17:02
*** derekh has quit IRC17:02
*** ducttape_ has quit IRC17:03
mriedemthe newton version check should come after the liberty version check, but that's a nit17:03
*** browne has joined #openstack-nova17:03
*** _ducttape_ has quit IRC17:03
*** vtech has quit IRC17:04
dansmithmriedem: I thought they were going for alphabetical order17:04
mnestratovdansmith: mriedem: thanks!17:04
*** daemontool has quit IRC17:04
dansmithI guess not17:04
dansmithheh, r comes after p17:05
*** thorst_ has joined #openstack-nova17:05
mriedemsmall nit in the reno too17:05
mriedemthis isn't modifying the userdata17:05
mriedemi think i introduced that confusion with a previous comment17:05
*** sree has quit IRC17:05
mriedemother than that i'm good17:06
dansmithokay17:06
*** sudipto has quit IRC17:06
*** mvk has quit IRC17:06
*** chandankumar has quit IRC17:06
dansmithjlvillal: the normal runs are passing now, which is good :)17:06
*** thorst_ has quit IRC17:07
*** svenkat has joined #openstack-nova17:07
*** thorst_ has joined #openstack-nova17:07
jlvillaldansmith, Yep only grenade left. But grenade was the only thing that was failing.17:07
mriedemvladikr: artom: want to address the nits quick? https://review.openstack.org/#/c/304511/17:08
dansmithjlvillal: right, I just mean, the extra problem in my patch is fixed at least17:08
jlvillalSo the patch isn't breaking anything it seems :)17:08
mriedemi'm going to grab lunch17:08
*** shaohe_feng has quit IRC17:08
*** shaohe_feng has joined #openstack-nova17:08
openstackgerritPaul Carlton proposed openstack/nova: API changes for live migration of rescued instance  https://review.openstack.org/32828017:09
*** jlanoux has quit IRC17:09
*** PaulMurray has quit IRC17:10
artommriedem, I'm on it, bon appetit!17:10
dansmithman, I need another monitor17:10
artomdansmith, or a tiling window manager :)17:11
dansmithartom: already tiling17:11
artomdansmith, you need less, err, work then :P17:11
*** ducttape_ has joined #openstack-nova17:13
cfriesen_I think a curved 40+ inch 4K monitor would be nice...17:13
*** ducttape_ has quit IRC17:13
artomOr just develop in VR17:13
dansmithI have three 27s and a 24 already.. not sure a single 4k would be an upgrade17:13
cfriesen_I'm jealous...I've only got two17:14
alaskia VR headset is the answer17:14
*** ducttape_ has joined #openstack-nova17:15
alaskiahh, artom already said that17:15
*** gjayavelu has joined #openstack-nova17:15
*** gjayavelu has quit IRC17:15
*** kaisers has joined #openstack-nova17:17
dansmithjlvillal: it's upgrading now17:17
dansmithjlvillal: it failed before in the post-upgrade tempest run I assume?17:18
*** ihrachys has quit IRC17:18
artomalaski, jinx!17:18
*** shaohe_feng has quit IRC17:18
*** shaohe_feng has joined #openstack-nova17:19
jlvillaldansmith, Are you watching the telnet stream like me? :)17:19
dansmithjlvillal: sorry :)17:19
jlvillalI need to look at the previous runs17:19
*** lucas__ has joined #openstack-nova17:19
*** lucas__ has quit IRC17:20
jlvillaldansmith, Looking at a previous failure it died farther down the line17:21
dansmithokay17:21
dansmithfigured17:21
*** zenoway has quit IRC17:22
jaypipesjohnthetubaguy: my hat is off to you for your patience untangling this unholy mess of unit test feces in neutronv2.py17:22
openstackgerritClaudiu Belu proposed openstack/nova: WIP: policy: clean-up  https://review.openstack.org/33392517:22
*** zenoway has joined #openstack-nova17:23
*** tblakeslee has quit IRC17:23
jaypipesjohnthetubaguy: I know the pain, having just gone through a lot of that with the existing resource tracker unit tests.17:23
*** tbachman has joined #openstack-nova17:24
*** _ducttape_ has joined #openstack-nova17:24
*** yamahata has quit IRC17:24
*** ducttape_ has quit IRC17:25
*** zzelle_ has joined #openstack-nova17:25
*** mgould is now known as mgould|afk17:27
*** kaisers has quit IRC17:27
*** chandankumar has joined #openstack-nova17:27
*** zenoway has quit IRC17:27
*** cdelatte has quit IRC17:27
dansmithjaypipes: feces doesn't really cut it for me17:28
dansmithit's shit.17:28
mriedemtjones: on https://review.openstack.org/#/c/196171/17:28
jaypipesdansmith: heh17:28
mriedemtjones: that's only supported for attaching an interface to an existing server? we can't apply those limits when booting the server?17:28
*** shaohe_feng has quit IRC17:28
jaypipesdansmith: I believe I call it poo-pile in the review.17:29
*** shaohe_feng has joined #openstack-nova17:29
*** salv-orlando has joined #openstack-nova17:29
*** imcsk8_ is now known as imcsk8|doctor17:30
dansmithjaypipes: you said feces above17:30
jaypipesdansmith: oh, I know... wordsmithing, you know, makes me happy.17:30
*** abalutoiu has joined #openstack-nova17:30
dansmithheh17:30
*** tonytan4ever has joined #openstack-nova17:31
*** klkumar has quit IRC17:32
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451117:32
dansmithartom: you don't need to remove the pluralness in the test17:32
dansmithI was just making a comment17:32
dansmithmakes me feel important17:32
artomdansmith, well, I was respining anyways, might as well17:33
artomI agree it's weird to have fakes not correspond to reality17:33
*** gjayavelu has joined #openstack-nova17:33
artommriedem, ^^^ when you're done pheeding17:33
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Use the recreated config disk during a rescue  https://review.openstack.org/33588417:33
openstackgerritMoshe Levi proposed openstack/nova: Update available resources before confirm stage  https://review.openstack.org/32735617:33
*** salv-orlando has quit IRC17:34
mriedemtjones: also on https://review.openstack.org/#/c/196171/ the nsx ci results from 6/7 are gone17:35
mriedemshould be able to keep ci logs around for longer than 3 weeks17:35
*** penick_ has joined #openstack-nova17:35
dansmithjlvillal: got a bunch of server tests that passed17:36
*** penick has quit IRC17:36
*** penick_ is now known as penick17:36
*** TravT has quit IRC17:39
jlvillaldansmith, Yep. I'm still watching. I gotta have high hopes17:39
*** shaohe_feng has quit IRC17:39
dansmithjlvillal: mriedem: yeah passed one we failed before this fix17:39
dansmith2016-06-30 17:32:57.617360 | {0} tempest.api.compute.servers.test_server_addresses.ServerAddressesTestJSON.test_list_server_addresses [0.085498s] ... ok17:39
*** shaohe_feng has joined #openstack-nova17:39
*** sudipto has joined #openstack-nova17:40
dansmithI have the cleaned up patch ready to push when this finishes17:40
lbeliveaumriedem: looks like there is no tempest test for migration with sriov wrt network connectivity (looking at https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_advanced_server_ops.py)17:40
lbeliveaumriedem: I'm surprised, am I missing something ?17:40
*** tonytan4ever has quit IRC17:41
gjayavelumriedem: tjones: the results were probably gone due to the change in jenkins server. let me check if we can pull those results back17:41
mriedemlbeliveau: hmm, i thought something happened in tempest with configuring the port17:41
mriedemfor like direct or macvtap17:41
*** tonytan4ever has joined #openstack-nova17:41
mriedemlbeliveau: yeah port_vnic_type17:42
jlvillaldansmith, I assume you saw the py27 failure in the nova side17:42
mriedemlbeliveau: but port_vnic_type isn't used in test_network_advanced_server_ops.py17:42
dansmithjlvillal: yeah, hadn't done any test work on that patch when I submitted that17:42
mriedemgjayavelu: can you answer the question of why we don't apply vif limits when creating the server?17:42
dansmithjlvillal: got that all fixed locally, just can't push or it will reset your test17:42
jlvillaldansmith, Thanks17:42
*** kaisers has joined #openstack-nova17:43
lbeliveaumriedem: I see tests for resize, reboot, etc, but nothing for migration there, I guess we'll need to add it17:43
mriedemgjayavelu: i rechecked the change otherwise17:43
mriedemlbeliveau: there are no scenario tests in tempest for migration with 2 nodes17:43
mriedemi guess resize would be fine if you had 2 nodes17:43
lbeliveaumriedem: thanks for confirming17:44
*** paul-carlton has quit IRC17:44
mriedemlbeliveau: but i think that port_vnic_type is the thing that tempest uses for sriov testing17:44
*** sdake has quit IRC17:45
gjayavelumriedem: we do apply the limits when creating the vifs https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/vm_util.py#L24817:45
gjayavelumriedem: the current patch is when adding new interfaces17:46
mriedemlbeliveau: actually the base class's create_server method uses vnic_type17:46
*** tbachman has quit IRC17:46
lbeliveaumriedem: yeah https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L18017:47
mriedemgjayavelu: ok so that made it into mitaka17:47
gjayavelumriedem: that's right17:47
mriedemlbeliveau: so do we have any ci jobs that run with vnic type direct or macvtap that runs the resize test in test_network_advanced_server_ops.py ?17:48
lbeliveaumriedem: yes the mellanox CI is executing test_server_connectivity_resize()17:48
mriedemlbeliveau: so it does http://13.69.151.247/73/242573/36/check-nova/Nova-ML2-Sriov/67c190c/testr_results.html.gz17:49
*** shaohe_feng has quit IRC17:49
mriedemhttp://13.69.151.247/73/242573/36/check-nova/Nova-ML2-Sriov/67c190c/tempest.conf.gz17:49
*** karimb has quit IRC17:49
mriedemport_vnic_type = direct17:49
*** zz_dimtruck is now known as dimtruck17:49
*** tbachman has joined #openstack-nova17:49
lbeliveaumridem: I just submitted a new patchset (it wasn't updating neutron ports if the host hasn't changed), I tested it in my env, we'll know soon from CI17:50
mriedemand macvtap here http://13.69.151.247/73/242573/34/check-nova/Nova-MACVTAP-ML2-Sriov/3b695d8/testr_results.html.gz17:50
*** shaohe_feng has joined #openstack-nova17:50
*** yamahata has joined #openstack-nova17:53
dansmithmriedem: jlvillal  Zuul] Job complete, result: SUCCESS17:54
openstackgerritDan Smith proposed openstack/nova: Make VIF.address unique with port id for neutron  https://review.openstack.org/33606917:55
*** paul-carlton has joined #openstack-nova17:55
*** tblakeslee has joined #openstack-nova17:55
jlvillaldansmith, Woo hoo! :)17:56
dansmithyeah17:57
*** paul-carlton has quit IRC17:57
*** paul-carlton2 has joined #openstack-nova17:57
*** salv-orlando has joined #openstack-nova17:57
dansmithhaving an ironic person in town adds a level of in-person violence potential that helps to motivate :)17:58
mriedemartom: vladikr: dansmith: congrats https://review.openstack.org/#/c/304511/17:58
dansmithmriedem: thanks a lot for all your work on it17:59
*** dimtruck is now known as zz_dimtruck17:59
artommriedem, indeed, many thanks17:59
*** shaohe_feng has quit IRC17:59
vladikrdansmith, mriedem, Thanks :)17:59
*** shaohe_feng has joined #openstack-nova18:00
mriedemare there other virt drivers for device tagging that we need to get on quick?18:01
artommriedem, claudio has his stuff here: https://review.openstack.org/#/c/331889/18:01
dansmithmriedem: yeah, the hyper-v one had pending review comments last I checked18:01
artommriedem, it looks fine to me, but if vladikr and dansmith took a look I'd be more confident18:01
artomThe xenapi is here: https://review.openstack.org/#/c/333781/ but I don't think that's mergeable18:02
artomAlso, can we make our Tempest test run against a hyper-v virt driver?18:02
mriedemhyperv doesn't support tagging for block devices?18:02
artomOr are we too pressed for time for this18:02
mriedemartom: the hyper-v ci would have to do that18:02
jlvillaldansmith, Heh.18:03
*** zz_dimtruck is now known as dimtruck18:03
mriedemoh nvm18:03
*** sameo has quit IRC18:03
dansmithmriedem: you mean vifs?18:03
mriedemi found my problem18:04
*** salv-orlando has quit IRC18:05
*** salv-orlando has joined #openstack-nova18:05
*** tiendc has quit IRC18:05
sudiptomriedem, wondering why pypi is preferred over a distro package? Just for my own understanding...18:06
mriedemsudipto: so that people don't have to rely on a distro and can pip install the code18:06
mriedembut you'll still need the pcpd package18:06
sudiptomriedem, yeah pcp package would still have to be distro specific...18:07
mriedemthere might be other reasons18:07
dansmithartom: I guess next step is getting the tempest test in tree18:07
artomdansmith, yep, and the client18:07
dansmithyeah18:08
artomWait, the FF doesn't also apply to Tempest, does it?18:08
fchemriedem, fwiw relying on a distro is a good thing if the distro is tracking and pypi is abandoned18:08
dansmithartom: nor the client, AFAIK18:08
mriedemclient freeze is like 9/218:08
*** chandankumar has quit IRC18:08
mriedemhttps://wiki.openstack.org/wiki/Nova/Newton_Release_Schedule18:09
mriedemfinal client release18:09
artomdansmith, good, otherwise my assumptions and time management would be seriously upended.18:09
*** mvk has joined #openstack-nova18:09
*** shaohe_feng has quit IRC18:09
*** shaohe_feng has joined #openstack-nova18:10
*** lpetrut has quit IRC18:12
*** harshad has joined #openstack-nova18:12
*** Sukhdev has joined #openstack-nova18:15
*** thumpba has quit IRC18:18
*** harshad has quit IRC18:18
*** shaohe_feng has quit IRC18:20
*** shaohe_feng has joined #openstack-nova18:20
*** mdrabe has joined #openstack-nova18:21
mriedemclaudiub: the hyper-v ci runs against nova-network right?18:21
dansmithdoes it/18:22
dansmithI thought they required neutron18:22
*** mfedosin has quit IRC18:24
mriedemlooks like it is neutron http://64.119.130.115/nova/331889/6/devstack_logs/18:25
mriedemyeah that would make sense given their hyperv neutron agent18:25
mriedem:)18:25
dansmithyeah18:25
mriedemalright, so on https://review.openstack.org/#/c/331889/ i think i want to hold that up until artom's tempest change lands and the hyper-v ci can test with it18:25
mriedemunless they can pull that tempest test in as a dependency to their ci for testing this patch18:26
mriedemlike the vz guys did18:26
dansmithmriedem: but okay with it landing after FF?18:26
mriedemmaybe18:26
dansmithI really wanted to see that they could implement it, which they did, so I'm good, but would be nice to show up with more than one driver actually supporting it18:27
mriedemyeah i agree18:27
*** dharinic__ has quit IRC18:27
mriedemmaybe we make that decision at the midcycle18:27
mriedemif it's ready by then18:27
mriedemclaudiub: will be there18:28
*** Sukhdev has quit IRC18:28
artommriedem, who's in charge of the hyper-v CI? claudiub, maybe you know?18:28
artomI can always poke them with my diplomacy stick18:28
dansmithartom: getting the tempest test in shape ASAP makes it easier18:29
ociuhanduartom: hello, we're in charge of the hyper-v ci18:29
*** thumpba has joined #openstack-nova18:29
artomociuhandu, hello! We have the device tagging feature for hyper-v here: https://review.openstack.org/#/c/331889/18:30
openstackgerritSujitha proposed openstack/nova: Transform instance.pause notifications  https://review.openstack.org/32914118:30
*** shaohe_feng has quit IRC18:30
artomociuhandu, and the Tempest test for it here: https://review.openstack.org/#/c/305120/18:30
artomociuhandu, the problem is the Tempest test hasn't merged yet18:30
artomociuhandu, is it at all feasible to run it against that change?18:30
claudiubmriedem: hi, hyper-v ci runs via neutron-hyperv-agent.18:31
*** shaohe_feng has joined #openstack-nova18:31
claudiubartom: I suppose the test will also require all the other patches in the stack, right?18:32
ociuhanduartom: the easiest thing we could to is launch a "debug run" on the patch and fetch the expected tempest and test against that. The only thing is that this will be done "manually" and the logs will be placed in a debug location, but I don't think it should be an issue18:32
artomclaudiub, they've all merged at this point18:32
dansmithociuhandu: I think that would be okay, if it's not too much trouble18:33
*** sridharg has joined #openstack-nova18:33
*** GB21 has quit IRC18:33
artomclaudiub, I think the last one is going through the gate, but assuming that doens't explode it's all in-tree18:33
mriedemociuhandu: claudiub: the other thing to note is the ci job that's testing this right now is gate-tempest-dsvm-neutron-full-ssh18:33
mriedemwhich is a non-voting job18:33
ociuhanduwe can launch debug runs that stop just before running tempest so we can fetch at that time the correct tempest18:33
mriedemi'm not sure what configuration differences you might have, but it's got ssh validation enabled in tempest.conf18:33
*** thumpba has quit IRC18:34
artom(ssh validation which is absolutely required for this test)18:34
mriedemyeah, the skip checks are here https://review.openstack.org/#/c/305120/11/tempest/api/compute/servers/test_device_tagging.py@3918:34
dansmithartom: last one is still in check, but headed to the gate in a few minutes18:34
mriedemheaded for a heartbreak18:35
*** thumpba has joined #openstack-nova18:35
*** thumpba has quit IRC18:35
*** thumpba has joined #openstack-nova18:35
mriedemlooking at http://64.119.130.115/nova/331889/6/devstack_config/tempest.conf.gz18:36
claudiubmriedem: yeah, that's not a problem. we use a cirros image anyways, so ssh won't be a problem.18:36
mriedemrun_validation=False18:37
mriedemok18:37
mriedemthat would have to be updated18:37
mriedemit doesn't disable config drive or the metadata service, and those default to True in tempest.conf, so assuming they should work in the hyperv ci18:37
mriedemenabled_apis = osapi_compute,metadata18:37
mriedemooo yeah18:37
artomAnd I think worst case checking only one of those two is better than nothing, and still gives us some confidence18:37
mriedemso change run_validation=True and you should be good to go18:38
artomMakes me think I should split those into two different tests maybe?18:38
mriedemartom: the test won't run if they aren't both enabled18:38
mriedemartom: it would be nice to test at least one of them18:38
mriedemsince some jobs in the gate don't run the meta service18:38
artommriedem, so maybe I can post a WIP of the Tempest test that splits it into two18:38
artomOne for config drive, one for API18:38
mriedemone thought is you could run the validation per each that's enabled, and if neither are enabled in skip_checks then you skip18:38
mriedembut that way you wouldn't have to duplicate the setup18:39
artomAh, yeah, smarter18:39
openstackgerritRyan Rossiter proposed openstack/nova: WIP: Shared state host manager  https://review.openstack.org/33246318:39
mriedemmtreinish: how do you feel about this https://review.openstack.org/#/c/305120/11/tempest/api/compute/servers/test_device_tagging.py@46 ?18:40
*** shaohe_feng has quit IRC18:40
*** shaohe_feng has joined #openstack-nova18:40
*** paul-carlton2 has quit IRC18:42
*** tbachman has quit IRC18:43
*** Sukhdev has joined #openstack-nova18:47
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/33318118:47
*** ociuhandu has quit IRC18:48
mriedemdansmith: unit test for the vif object change in https://review.openstack.org/#/c/336069/ ? for the actual conversion from the db record18:49
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/33623518:49
dansmithmriedem: ah yeah sorry18:49
*** shaohe_feng has quit IRC18:50
*** Sukhdev has quit IRC18:51
*** mdrabe has quit IRC18:51
*** mdrabe_ has joined #openstack-nova18:51
*** shaohe_feng has joined #openstack-nova18:51
*** TravT has joined #openstack-nova18:52
oomichigmann: about https://review.openstack.org/#/c/328157  yeah, I have the same opinion which requires Depends-On to ironic side patch18:52
*** mdrabe_ has quit IRC18:53
*** mdrabe_ has joined #openstack-nova18:54
*** TravT has quit IRC18:54
*** mdrabe_ has quit IRC18:54
*** mdrabe_ has joined #openstack-nova18:55
*** TravT has joined #openstack-nova18:55
openstackgerritDan Smith proposed openstack/nova: Make VIF.address unique with port id for neutron  https://review.openstack.org/33606918:56
dansmithmriedem: ^18:56
mriedemstyyyyle not professional18:56
dansmithcome on, after that commit message I get this one.18:57
mriedemnamespaced mac addresses, you want'em18:57
*** mdrabe_ has quit IRC18:57
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation  https://review.openstack.org/20421918:58
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations  https://review.openstack.org/19617118:58
*** mdrabe_ has joined #openstack-nova18:58
oomichimriedem: about https://review.openstack.org/#/c/304510 That is already merged, and I cannot find any problems about the schema validation.18:59
oomichimriedem: btw, the schema is hard to read for humen, due to copy.deepcopy and changing original schema.18:59
mriedemdansmith: comment in that change18:59
*** mdrabe_ has quit IRC19:00
oomichimriedem: this thing happens on tempest side and gmann will expand these schemas for the readability, how about doing that on nova side?19:00
*** mdrabe__ has joined #openstack-nova19:00
*** shaohe_feng has quit IRC19:01
mriedemoomichi: i'd probably need to see an example19:01
mriedembut i agree the copying gets weird19:01
mriedemthe nice thing about the copy is we make sure we don't lose anything from the prevoius microversion19:01
*** shaohe_feng has joined #openstack-nova19:01
oomichimriedem: ok, maybe tempest changes will be good examples before doing that on nova side19:02
oomichimriedem: yeah, I can see the point also19:02
openstackgerritDan Smith proposed openstack/nova: Make VIF.address unique with port id for neutron  https://review.openstack.org/33606919:04
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/33318119:07
openstackgerritRyan Rossiter proposed openstack/nova: Use ovo in test_obj_make_compatible()  https://review.openstack.org/27229919:09
*** dimtruck is now known as zz_dimtruck19:11
*** shaohe_feng has quit IRC19:11
*** shaohe_feng has joined #openstack-nova19:12
*** sridharg has quit IRC19:13
mriedemalaski: got a lonely +2 party of one going on here https://review.openstack.org/#/c/336069/19:14
*** amotoki has joined #openstack-nova19:14
*** tbachman has joined #openstack-nova19:15
*** jed56 has quit IRC19:15
* alaski joins the party19:15
mriedemmelwitt: johnthetubaguy: replied to https://review.openstack.org/#/c/331236/19:15
mriedemmelwitt: johnthetubaguy: they are different error conditions19:15
*** lpetrut has joined #openstack-nova19:16
pumaranikarhi, can anyone confirm if we are still using os-volumes_boot API as a part of V2.119:17
mriedempumaranikar: we are19:17
melwittmriedem: I meant why not make each string a message for the exception object, so it gets propagated up that way. or you're saying there isn't any logging of the exception message later on?19:17
mriedemmelwitt: oh that's what you're saying...19:18
mriedemi'd have to check on if anything logs it later19:18
mriedempumaranikar: it's an extension on the servers API19:18
mriedemso it's a bit weird19:18
mriedempumaranikar: are you asking about the bug for no functional api sample test for that api?19:19
pumaranikarmriedem, thanks :). yes19:19
mriedempumaranikar: are you new?19:19
mriedemlike, new to nova19:20
pumaranikarmriedem, yes. I can not find that extension in API extensions doc19:20
mriedempumaranikar: i would recommend avoiding that one19:20
mriedemsdague or alex_xu_ or gmann might be able to help with it, but it's probably something terrible19:20
*** shaohe_feng has quit IRC19:21
pumaranikarmriedem, ok.19:21
*** shaohe_feng has joined #openstack-nova19:22
sudiptois it necessary that whatever is specified in g-r has to be pip installed? I am reading through this : https://github.com/openstack/requirements and i am not sure if it's a necessity?19:22
sudiptomriedem, ^19:22
mriedemsudipto: is there a specific section you're looking at19:24
mriedem?19:24
sudiptomriedem, "For new requirements" that one19:24
*** mdrabe__ has quit IRC19:25
*** sacharya_ has quit IRC19:26
*** sacharya has joined #openstack-nova19:26
mriedemsudipto: i'm not makinga connection between that and your question about being pip installed19:29
*** dharinic_ has joined #openstack-nova19:29
fcheI believe the question is whether being listed in the requirements.txt file means that each python package must be installed via pip19:29
fcheas opposed to being found on the distro19:29
sudiptomriedem, basically pcp satisfies the "new requirements" criteria - however, i wanted to know if the g-r means it has to be pip installed.19:29
artommriedem, https://review.openstack.org/305120 hail mary respin, I think my local devstack borked, running the test can't reach the booted instance19:31
*** shaohe_feng has quit IRC19:31
mriedemsudipto: if it were in nova's requirements.txt then yes it would have to be in g-r and pip installable, but since it's an optional dependency it won't be in requirements.txt. now it could be in test-requirements.txt, like oslo.vmware19:32
*** mdrabe has joined #openstack-nova19:32
mriedemso we could actually unit test with the python library19:32
*** shaohe_feng has joined #openstack-nova19:32
mriedemgiven the 0.1 and pre-alpha status of https://pypi.python.org/pypi/pcp/0.1 i'm guessing that would be a hard sell to get it into g-r19:32
sudiptomriedem, yeah i am not looking to sell the pypi package at all...19:33
mriedemheh i know :)19:33
*** bvanhav has quit IRC19:33
mriedemhmm https://github.com/performancecopilot/pcp/blob/master/src/python/setup.py#L4119:35
*** sheel has quit IRC19:35
mriedemthe python bindings in pcp itself are marked as stable19:35
mtreinishmriedem: that seems kinda weird. Do all the tests in the class require both config drive and the metadata service?19:35
mriedemmtreinish: no19:36
mriedemmtreinish: there are 2 validation calls, one for metadata and one for config drive19:36
mriedemi was suggesitng we run those conditionally19:36
mriedemmtreinish: but skip early if neither of those are enabled19:36
mtreinishthen yeah, that's wrong19:36
mriedemi thought i'd seen other scenario tests like that19:36
mtreinishwhy not just use skip decorators with the conf value19:37
mriedemmtreinish: or that19:37
mriedemsudipto: any idea if this supports python 3?19:37
sudiptomriedem, that shows it's stable no?19:37
*** tonytan4ever has quit IRC19:37
sudiptomriedem, yeah it does. fche ?19:37
mriedemah now i know who fche is19:38
fchejust some guy :)19:38
mriedem#3 contributor is pretty good19:39
fcheanyway it's 'stable' and changing (growing) periodically too.   designed to be used.19:39
mriedemfche: ok, i guess pypi doesn't represent that19:39
mriedembut it didn't for libvirt-python for a long time either19:39
*** aloga_ has joined #openstack-nova19:39
fcheyeah.  the pypi copy was posted long long ago, and that author is NORDO as they say19:39
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation  https://review.openstack.org/20421919:40
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations  https://review.openstack.org/19617119:40
*** hoonetorg has quit IRC19:40
mriedemsudipto: ok so i'm going to drop the pypi / g-r thing19:41
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for vmwareapi_opts  https://review.openstack.org/33250319:41
sudiptomriedem, ok thanks! However i think i have already missed the bus for this release...19:41
*** shaohe_feng has quit IRC19:42
*** shaohe_feng has joined #openstack-nova19:42
mriedemi'd like to get danpb's thoughts on libvirt providing this info for the host, but that's definitely something way out and not available for awhile, so depending on that isn't great19:43
mriedemi'm not sure i want to rush this in today either19:43
fchelibvirt info is great, but there is a big set of other stats pcp can offer, re. containers, rest-of-the-system, etc.   two separate sweet spots, two separate tools19:45
dansmithfche: the bar for depending on another external thing is pretty high for me19:45
dansmithfche: like, the data we get has to be really compelling and even what is proposed now does not seem to meet that bar, IMHO19:46
fcheyeah, understood.19:46
fcheI suspect sudipto et al. are going to look around for a more interesting sampling of data from pcpland19:46
sudiptodansmith, agreed...mem bw is probably not compelling enough atm19:46
*** daemontool_ has quit IRC19:46
*** jaugustine has joined #openstack-nova19:47
fcheat least sudipto's pcp_util.py infrastructure is common in the sense that any of the thousands of other metrics can be processed through that bit of code.19:47
sudiptohowever, this is one of the vehicles i chose to get into more interesting metrics in the future...19:47
*** aloga_ has quit IRC19:48
fchedansmith, mriedem, I assume you haven't had a chance to look at the full smorgasbord - but are there system perf areas you think openstack needs more numbers about?19:48
fchegenerally speaking?19:49
mriedemidk, jaypipes might have an opinion - this is really specific for scheduling19:49
mriedemi honestly don't know how many deployments of nova are relying on the metrics weigher19:49
mriedemto get that fine-grained level of scheduling19:50
*** tonytan4ever has joined #openstack-nova19:50
*** abc has joined #openstack-nova19:50
mriedemthis is also just another thing that will be special and we won't test19:50
*** hoonetorg has joined #openstack-nova19:51
fcheceilometer doesn't need a boost either?19:52
*** shaohe_feng has quit IRC19:52
*** shaohe_feng has joined #openstack-nova19:53
sudiptoi would imagine it's beyond just scheduling... however i am not sure if that means it's not of importance to nova that much...19:53
mriedemdo these metrics get into anything that ceilometer consumes?19:54
dansmithfche: no, I don't really19:54
mriedemwe don't have these going into events or notifications that i'm aware of19:55
mriedemdo we emit anything for compute node stats?19:56
mriedemjaypipes: ^?19:56
dansmithnot that I know of19:56
openstackgerritClaudiu Belu proposed openstack/nova: policy: clean-up  https://review.openstack.org/33392519:56
mriedemdansmith: ok i didn't think so19:56
dansmithceilometer could, of course, scrape them from pcp or whatever itself19:56
dansmithwhich would make much more sense,19:56
dansmiththan having nova proxy all that gorp for it19:56
mriedemceilometer already goes directly to libvirt for some stuff19:57
mriedemanyway, that's sort of tangential19:57
fcheit could work to start small & very optional, then see19:57
gibi_mriedem: I don't think we have any notification about these stats either19:57
*** yass has quit IRC19:57
dansmithfche: we hate optional stuff19:58
mriedemheh :)19:58
fcheok mandate it then :-)19:58
dansmithmriedem: well, it's tre19:58
mriedemi know19:58
mriedemfche: we're doing that19:58
mriedemhttp://lists.openstack.org/pipermail/openstack-dev/2016-June/098556.html19:59
mriedemhttp://specs.openstack.org/openstack/nova-specs/specs/newton/approved/api-no-more-extensions.html19:59
mriedemunicorns are slowly being weeded out19:59
*** dansmith is now known as unicorn_slayer19:59
fcheunderstood; it must be a difficult balance identifying unicorns vs. promising seedlings19:59
unicorn_slayernot really, it's pretty easy actually :)20:00
fchehaha20:00
mriedemit usually comes down to at a minimum what we test in our CI20:00
mriedemand having more than one consumer20:01
*** vtech has joined #openstack-nova20:01
*** shaohe_feng has quit IRC20:02
*** shaohe_feng has joined #openstack-nova20:03
*** fragatina has joined #openstack-nova20:03
sudiptohmm and then you could get system wide utilization data of memory/cpu/disk etc which are not VM specific...20:04
*** smatzek has quit IRC20:04
sudiptoeven network information...20:05
*** tonytan4ever has quit IRC20:05
*** tonytan4ever has joined #openstack-nova20:06
*** ihrachys has joined #openstack-nova20:07
*** _ducttape_ has quit IRC20:07
*** ducttape_ has joined #openstack-nova20:09
openstackgerritClenimar Filemon proposed openstack/nova: Support Identity v3 when connecting to Ironic  https://review.openstack.org/30015420:10
*** tonytan4ever has quit IRC20:12
*** jaugustine has quit IRC20:12
*** shaohe_feng has quit IRC20:12
*** Daisy has joined #openstack-nova20:13
*** tonytan4ever has joined #openstack-nova20:13
*** shaohe_feng has joined #openstack-nova20:13
*** unicorn_slayer is now known as dansmith20:15
*** Daisy has quit IRC20:17
*** tonytan4ever has quit IRC20:17
*** tonytan4ever has joined #openstack-nova20:18
*** shaohe_feng has quit IRC20:23
*** shaohe_feng has joined #openstack-nova20:24
*** gyee has joined #openstack-nova20:26
*** stewie925 has joined #openstack-nova20:28
*** Daisy has joined #openstack-nova20:29
*** cdelatte has joined #openstack-nova20:30
mriedemleast scary ascii croc ever https://review.openstack.org/#/c/335906/120:30
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for cloudpipe  https://review.openstack.org/33128520:32
* auggy is speechless20:33
*** shaohe_feng has quit IRC20:33
mriedemflying cat? https://review.openstack.org/#/c/335907/120:33
*** shaohe_feng has joined #openstack-nova20:33
mriedemand idk wtf this is https://review.openstack.org/#/c/335903/120:33
*** Daisy has quit IRC20:34
melwittomg that's amazing20:34
dansmithmriedem: faceplant20:34
dansmithfeet in the air20:34
dansmithI think20:34
*** cdelatte has quit IRC20:34
mriedemheh maybe20:35
mriedemthis is like fine art20:35
openstackgerritAndy Hsiang proposed openstack/nova: Transform instance.power_off notifications  https://review.openstack.org/33076120:35
mriedemeveryone interprets it differently20:35
*** jaugustine has joined #openstack-nova20:35
*** sudipto has quit IRC20:36
*** aloga_ has joined #openstack-nova20:36
stewie925hello, I am working on the api-ref testing some nova commands - I was able to create a DNS domain by executing 'PUT /os-floating-ip-dns/{domains}' successfully20:36
stewie925but when I tried to list the dns domain, it is returning empty domain_entries [ ]20:36
melwittdanpb, you're a genius20:37
openstackgerritChris Friesen proposed openstack/nova: Allow logging detailed info only on sched failure  https://review.openstack.org/33130620:37
*** imcsk8|doctor is now known as imcsk820:39
dansmithmelwitt: your bar is pretty low20:39
*** cdelatte has joined #openstack-nova20:39
melwitthmph!20:39
*** thumpba has quit IRC20:40
dansmithI can copy and paste too :)20:40
cfriesen_someone should try one of those picture-to-ascii-art conversion sites20:41
*** rcernin has joined #openstack-nova20:41
*** shaohe_feng has quit IRC20:43
dansmithgimp used to have ascii-art export built in20:43
*** shaohe_feng has joined #openstack-nova20:44
openstackgerritPeter Hamilton proposed openstack/nova: Add certificate signature verification  https://review.openstack.org/33182320:46
*** TravT has quit IRC20:47
*** dharinic_ has quit IRC20:48
*** tblakeslee has quit IRC20:48
*** sree has joined #openstack-nova20:50
openstackgerritJulian proposed openstack/nova: Security issue refactor for wrap_exception notification  https://review.openstack.org/33628420:51
*** tblakeslee has joined #openstack-nova20:51
*** shaohe_feng has quit IRC20:53
*** jaugustine has quit IRC20:54
*** shaohe_feng has joined #openstack-nova20:54
*** sree has quit IRC20:55
mriedemmelwitt: replied again in https://review.openstack.org/#/c/331236/ - basically i think the existing exception message is fine to go back up the stack20:55
mriedemw/o the network dict - which is really what i wanted to log20:56
*** stewie925 has quit IRC20:56
mriedemmelwitt: but you got me looking at that error and we don't handle it as a BuildAbortException in the compute manager, and we probably should20:56
mriedemat least for neutron20:56
mriedembecause rescheduling isn't going ot fix it20:56
mriedem*to20:56
melwittmriedem: gotcha20:57
*** syjulian has quit IRC20:58
*** ssurana has quit IRC21:00
*** rfolco has quit IRC21:01
*** eharney has quit IRC21:01
*** ssurana has joined #openstack-nova21:01
*** lpetrut has quit IRC21:02
*** raildo is now known as raildo-afk21:02
mriedemdansmith: so with the libvirt and api changes done, i'd like to mark https://blueprints.launchpad.net/nova/+spec/virt-device-role-tagging as complete - any issues with that? the scheduler filter is a later optimization, and the hyper-v thing would be feature parity which may be ok to get a FFE for21:03
*** shaohe_feng has quit IRC21:04
*** shaohe_feng has joined #openstack-nova21:04
mriedemtdurakov: alaski: i'm guessing https://review.openstack.org/#/q/topic:bp/remove-compute-compute-communication+status:open isn't going to make it today right?21:08
*** cdelatte has quit IRC21:09
openstackgerritAndrew Laski proposed openstack/nova: Add nova-manage commands for policy helpers  https://review.openstack.org/33566721:09
mriedemalaski: is ^ the same as https://blueprints.launchpad.net/nova/+spec/discoverable-policy-cli ?21:09
*** gagehugo has quit IRC21:10
alaskimriedem: tdurakov yeah, that's not going to make it today21:10
claudiubmriedem: ye... still working on that..21:10
alaskimriedem: no, it's more along the lines of https://blueprints.launchpad.net/nova/+spec/policy-in-code21:10
mriedemalaski: ok21:10
*** tblakeslee has quit IRC21:10
claudiubmriedem: I've been working on the policy-in-code, and was planning to finish the cli stuff after finishing the policy-in-code stuff.21:11
mriedemok21:11
alaskimriedem: that last patch is just a nice to have, not essential to the main point of the spec21:11
*** jaugustine has joined #openstack-nova21:11
claudiubspeaking of which, this is amoung the last patches for policy-in-code: https://review.openstack.org/#/c/333925/21:11
*** tblakeslee has joined #openstack-nova21:12
mriedemclaudiub: ok, that gets a pass for FF21:12
*** jaugustine has quit IRC21:13
*** ametts has quit IRC21:13
*** TravT has joined #openstack-nova21:14
*** shaohe_feng has quit IRC21:14
*** TravT has quit IRC21:14
*** shaohe_feng has joined #openstack-nova21:15
*** TravT has joined #openstack-nova21:15
diana_clarkemdbooth: I'm sure you're gone, but here are my notes about ephemeral disks not resizing (before our patch series): https://notehub.org/o3mjz21:15
diana_clarkemdbooth: Had you already entered a bug for this?21:15
*** akshai has quit IRC21:16
*** TravT has quit IRC21:16
*** TravT has joined #openstack-nova21:17
diana_clarkemdbooth: Anyhoo, now that I'm done fighting with devstack, I'm finally going to start on that tempest test (but I'm off tomorrow).21:18
openstackgerritKris Lindgren proposed openstack/nova: create network metadata information as static even with dhcp  https://review.openstack.org/33629321:21
*** tonytan4ever has quit IRC21:21
*** vishwanathj has quit IRC21:22
*** kylek3h has quit IRC21:24
*** edmondsw has quit IRC21:24
*** shaohe_feng has quit IRC21:24
*** shaohe_feng has joined #openstack-nova21:25
*** svenkat has quit IRC21:26
*** lucas-afk has quit IRC21:28
*** pkoniszewski_ has joined #openstack-nova21:28
*** lucasagomes has joined #openstack-nova21:28
*** rtheis has quit IRC21:29
*** tblakeslee has quit IRC21:29
*** pkoniszewski has quit IRC21:29
gjayavelumriedem: updated the unit tests for https://review.openstack.org/#/c/196171. please let me know if this is ok. Also, I heard the issue with posting the right link for vmware ci is fixed. waiting for the results21:30
mriedemgjayavelu: ok21:30
mriedemgjayavelu: looks ok, just waiting on those ci results21:32
gjayavelumriedem: sure. thanks.21:33
*** ctrath has quit IRC21:33
*** Shalmazar has joined #openstack-nova21:33
*** ayoung has quit IRC21:34
*** yamahata has quit IRC21:34
*** Madasi has quit IRC21:34
*** Shalmazar is now known as Madasi21:34
*** shaohe_feng has quit IRC21:34
*** shaohe_feng has joined #openstack-nova21:35
*** diogogmt has quit IRC21:36
*** jamesdenton has quit IRC21:37
*** rlrossit has quit IRC21:39
*** HenryG_ has joined #openstack-nova21:41
*** thorst_ has quit IRC21:42
*** jroll|dupe has joined #openstack-nova21:44
*** jroll|dupe has quit IRC21:44
*** jroll|dupe has joined #openstack-nova21:44
*** shaohe_feng has quit IRC21:45
*** hoonetorg has quit IRC21:45
*** BjoernT has quit IRC21:45
*** cleong has quit IRC21:45
*** hieulq has quit IRC21:45
*** jinquan has quit IRC21:45
*** mordred has quit IRC21:45
*** mirrorbox has quit IRC21:45
*** bradm_ has quit IRC21:45
*** zhs_s has quit IRC21:45
*** HenryG has quit IRC21:45
*** dulek has quit IRC21:45
*** mkoderer__ has quit IRC21:45
*** john51 has quit IRC21:45
*** Wenzhi has quit IRC21:45
*** _gryf has quit IRC21:45
*** jroll has quit IRC21:45
*** ianw has quit IRC21:45
*** hogepodge has quit IRC21:45
*** hamzy has quit IRC21:45
*** tpatzig has quit IRC21:45
*** mkoderer has quit IRC21:45
*** gothicmindfood has quit IRC21:45
*** jroll|dupe is now known as jroll21:45
*** shaohe_feng has joined #openstack-nova21:45
*** yamahata has joined #openstack-nova21:47
*** jamie_h has quit IRC21:47
*** fragatina has quit IRC21:47
*** mdrabe has quit IRC21:48
*** cdelatte has joined #openstack-nova21:51
openstackgerritClenimar Filemon proposed openstack/nova: Support Identity v3 when connecting to Ironic  https://review.openstack.org/30015421:52
*** HenryG_ is now known as HenryG21:52
*** sdake_ has joined #openstack-nova21:53
*** sdake_ has quit IRC21:53
*** sdake_ has joined #openstack-nova21:53
*** stewie925 has joined #openstack-nova21:54
*** shaohe_feng has quit IRC21:55
*** irenab has quit IRC21:55
*** shaohe_feng has joined #openstack-nova21:55
mriedemgibi_: +2 on https://review.openstack.org/#/c/313654/22:02
mriedemi don't see a change for instance.create events22:02
mriedemi see the one for instance.update is higher up the stack with some others in between22:02
mriedemlike pause and power off22:02
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/33318122:02
mriedemmaybe those would be easy to get through though since the bulk of those changes are just the samples22:03
*** catintheroof has joined #openstack-nova22:03
mriedembauzas: want to take a look at this and the one below it again? https://review.openstack.org/#/c/330513/22:03
mriedembauzas: there are 2 +2s on ^22:03
mriedemand a +2 on the base patch22:03
mriedemit's only like 10pm for you22:04
bauzasmriedem: heh, you lucky becauce it's midnight now :p22:04
*** gszasz has quit IRC22:04
bauzasmriedem: but I was looking at an Euro2016 soccer match22:04
mriedemi saw UK lost to iceland22:05
mriedema big deal22:05
*** shaohe_feng has quit IRC22:05
mriedemanyway, i have to head home22:05
mriedemback online later22:05
*** shaohe_feng has joined #openstack-nova22:06
bauzasmriedem: yeah, now Iceland vs. France on Sunday !22:06
mriedembetter watch out22:06
bauzasanyway, is the above an non-prio one ?22:06
mriedembauzas: yeah, but would be good to get the bottom one in22:07
mriedemmight have an FFE for the update one22:07
bauzask22:07
*** dharinic_ has joined #openstack-nova22:07
bauzasyeah, the first one is nice22:07
*** mriedem has quit IRC22:07
*** cdelatte has quit IRC22:09
*** TravT has quit IRC22:12
*** irenab has joined #openstack-nova22:14
*** TravT has joined #openstack-nova22:14
*** shaohe_feng has quit IRC22:15
*** thorst_ has joined #openstack-nova22:16
*** shaohe_feng has joined #openstack-nova22:16
openstackgerritRawlin Peters proposed openstack/nova: [DO NOT REVIEW - FOR TESTING PURPOSES ONLY]  https://review.openstack.org/33247422:17
*** burt has quit IRC22:20
*** thorst_ has quit IRC22:20
*** thorst_ has joined #openstack-nova22:21
*** derekh has joined #openstack-nova22:22
*** mordred has joined #openstack-nova22:22
*** shaohe_feng has quit IRC22:26
*** shaohe_feng has joined #openstack-nova22:26
openstackgerritKris Lindgren proposed openstack/nova: Option to always create network metadata information as static  https://review.openstack.org/33629322:27
*** cdelatte has joined #openstack-nova22:27
*** thorst_ has quit IRC22:29
*** sdake_ has quit IRC22:33
*** pradk has quit IRC22:34
*** salv-orl_ has joined #openstack-nova22:34
*** salv-orlando has quit IRC22:35
*** shaohe_feng has quit IRC22:36
*** jamielennox|away is now known as jamielennox22:37
*** shaohe_feng has joined #openstack-nova22:37
*** shaohe_feng has quit IRC22:46
*** shaohe_feng has joined #openstack-nova22:47
*** aloga_ has quit IRC22:49
*** ducttape_ has quit IRC22:49
*** cdelatte has quit IRC22:52
openstackgerritEd Leafe proposed openstack/nova: Return HTTP 400 on list for invalid status  https://review.openstack.org/31596422:53
*** shaohe_feng has quit IRC22:56
*** shaohe_feng has joined #openstack-nova22:57
*** geekinutah has joined #openstack-nova22:58
*** shaohe_feng has quit IRC23:07
*** shaohe_feng has joined #openstack-nova23:07
*** thorst_ has joined #openstack-nova23:08
*** amotoki has quit IRC23:08
*** thorst_ has quit IRC23:08
*** zzelle_ has quit IRC23:09
*** thorst_ has joined #openstack-nova23:09
*** ayoung has joined #openstack-nova23:09
*** abc has quit IRC23:09
*** amotoki has joined #openstack-nova23:10
*** sdake has joined #openstack-nova23:11
*** sdake has quit IRC23:11
*** sacharya has quit IRC23:11
*** shaohe_feng has quit IRC23:17
*** shaohe_feng has joined #openstack-nova23:17
*** thorst_ has quit IRC23:18
*** catintheroof has quit IRC23:18
*** mriedem has joined #openstack-nova23:18
*** Prophidys has quit IRC23:20
*** READ10 has quit IRC23:21
*** markvoelker has quit IRC23:21
*** vtech has quit IRC23:27
*** shaohe_feng has quit IRC23:27
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation  https://review.openstack.org/20421923:28
*** Prophidys has joined #openstack-nova23:28
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations  https://review.openstack.org/19617123:28
*** EinstCrazy has joined #openstack-nova23:28
*** ihrachys has quit IRC23:29
openstackgerritTracy Jones proposed openstack/nova: remove mox from vmwareapi/test_driver_api.py  https://review.openstack.org/27030323:31
openstackgerritTracy Jones proposed openstack/nova: Stop using mox stubs in test_configdrive  https://review.openstack.org/26547623:31
openstackgerritTracy Jones proposed openstack/nova: remove mox from vmwareapi/test_vif.py  https://review.openstack.org/27106923:31
openstackgerritTracy Jones proposed openstack/nova: VMware - Favor shared datastores  https://review.openstack.org/28963123:32
*** EinstCrazy has quit IRC23:33
*** MikeG451 has quit IRC23:34
dansmithmriedem: looks like ceph job failures have crept up a bit, but I _think_ it's related to cinder.. I sent a mail23:36
*** yamahata has quit IRC23:36
mriedemdansmith: the no valid host one?23:36
mriedemi noticed that a couple weeks ago23:36
dansmithmriedem: yeah23:36
mriedemsomething with the weight23:36
dansmithokay, the fail percentages weren't as striking until recently, so I didn't go looking23:36
*** dharinic_ has quit IRC23:37
*** shaohe_feng has joined #openstack-nova23:38
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end)  https://review.openstack.org/25028323:41
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications (error)  https://review.openstack.org/32805523:42
*** baoli has quit IRC23:42
openstackgerritTakashi NATSUME proposed openstack/nova: Add swap volume notifications samples  https://review.openstack.org/32886623:42
*** cdelatte has joined #openstack-nova23:44
*** thorst_ has joined #openstack-nova23:44
*** BjoernT has joined #openstack-nova23:44
*** amotoki has quit IRC23:44
*** READ10 has joined #openstack-nova23:45
*** pwnall has quit IRC23:45
*** erlon has quit IRC23:46
*** derekh has quit IRC23:47
*** shaohe_feng has quit IRC23:48
*** shaohe_feng has joined #openstack-nova23:48
*** BjoernT has quit IRC23:50
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: device tagging  https://review.openstack.org/33188923:50
*** thorst_ has quit IRC23:54
*** thorst_ has joined #openstack-nova23:55
*** phuongnh has quit IRC23:57
*** shaohe_feng has quit IRC23:58
*** shaohe_feng has joined #openstack-nova23:59
*** hogepodge has joined #openstack-nova23:59

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