Tuesday, 2017-05-23

*** yingjun has joined #openstack-nova00:01
*** salv-orlando has quit IRC00:01
*** yamahata has joined #openstack-nova00:04
*** crushil has joined #openstack-nova00:08
*** crushil has quit IRC00:08
*** crushil has joined #openstack-nova00:08
*** marst_ has quit IRC00:09
*** masber has joined #openstack-nova00:09
*** mtanino has joined #openstack-nova00:09
*** nic has quit IRC00:10
*** felipemonteiro has quit IRC00:13
*** crushil has quit IRC00:14
*** crushil has joined #openstack-nova00:14
*** sree has joined #openstack-nova00:16
jaypipesartom: hi! hey, what parts of http://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/virt-device-tagged-attach-detach.html were fully implemented and what's still left to do?00:19
*** sree has quit IRC00:20
*** gyee has quit IRC00:22
*** hongbin has quit IRC00:22
artomja00:24
artomjaypipes, hey. We have device tagging at boot time - ie, nova boot --nic net-id=foo,tag=foo test00:25
jaypipesartom: but not dynamic, right?00:25
jaypipesartom: ala our ML convo from around a month ago, yeah?00:25
artomjaypipes, yep :)00:25
jaypipesartom: got it. thank you sir!]00:26
jaypipesartom: internal product manager was wondering about it..00:26
*** gcb has quit IRC00:26
artomjaypipes, whoa!00:26
* artom feels useful00:26
jaypipesartom: :)00:27
*** ijw has quit IRC00:28
*** esberglu has quit IRC00:30
*** esberglu has joined #openstack-nova00:30
*** esberglu has quit IRC00:31
*** yamahata has quit IRC00:33
*** sajeesh has joined #openstack-nova00:33
openstackgerritZhaokun Fu proposed openstack/nova master: fix typo  https://review.openstack.org/46694900:37
*** tovin07_ has joined #openstack-nova00:37
*** crushil has quit IRC00:38
*** crushil has joined #openstack-nova00:39
*** itlinux has joined #openstack-nova00:40
*** itlinux has quit IRC00:41
*** itlinux has joined #openstack-nova00:42
*** sajeesh has quit IRC00:47
*** sajeesh has joined #openstack-nova00:49
*** nicolasbock has joined #openstack-nova00:56
*** pwiltsey has joined #openstack-nova00:58
*** sajeesh has quit IRC01:00
*** ijw has joined #openstack-nova01:02
*** phuongnh has joined #openstack-nova01:04
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538501:04
*** kaisers_ has joined #openstack-nova01:07
openstackgerritNgo Quoc Cuong proposed openstack/nova master: Remove translation of log messages  https://review.openstack.org/46695701:08
*** dimtruck is now known as zz_dimtruck01:09
*** trungnv has joined #openstack-nova01:09
*** RuiChen has joined #openstack-nova01:12
*** kaisers_ has quit IRC01:12
*** itlinux has quit IRC01:28
*** gianpietro has joined #openstack-nova01:28
*** catintheroof has joined #openstack-nova01:29
*** thorst_afk has joined #openstack-nova01:29
*** thorst_afk has quit IRC01:30
*** catintheroof has quit IRC01:30
*** yolkfull has joined #openstack-nova01:31
*** gianpietro has quit IRC01:33
*** RuiChen has left #openstack-nova01:33
*** Shunli has joined #openstack-nova01:33
*** cNilesh has joined #openstack-nova01:35
*** dave-mccowan has quit IRC01:36
*** ijw has quit IRC01:41
*** edmondsw has quit IRC01:45
openstackgerritJohn Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter  https://review.openstack.org/46659501:46
*** sree has joined #openstack-nova01:53
*** jamielennox is now known as jamielennox|away01:54
*** zz_dimtruck is now known as dimtruck01:56
*** sree has quit IRC01:57
*** takashin_ has joined #openstack-nova01:57
*** links has joined #openstack-nova01:58
*** takashin has quit IRC01:59
*** mriedem_afk is now known as mriedem01:59
mriedemdansmith: looks like https://review.openstack.org/#/c/464059/ is happy02:00
*** brault has quit IRC02:03
*** gcb has joined #openstack-nova02:04
*** hongbin has joined #openstack-nova02:06
*** rfolco has joined #openstack-nova02:08
*** rfolco has quit IRC02:08
*** mtanino has quit IRC02:15
*** crushil has quit IRC02:16
*** tbachman has quit IRC02:16
mriedemefried: it's starred but i'm shot, ping me about it tomorrow02:26
mriedemefried: and review this https://review.openstack.org/#/c/459753/ since you asked me to review something02:27
mriedemxenserver ci seems to have gone nuts02:28
*** mdrabe has quit IRC02:32
*** tovin07__ has joined #openstack-nova02:34
*** yamahata has joined #openstack-nova02:34
*** tovin07_ has quit IRC02:37
*** diga has joined #openstack-nova02:39
*** xinliang has quit IRC02:40
*** crushil has joined #openstack-nova02:41
*** dikonoor has joined #openstack-nova02:46
*** takashin_ has left #openstack-nova02:48
*** hongbin_ has joined #openstack-nova02:48
*** xinliang has joined #openstack-nova02:52
*** jichen has joined #openstack-nova02:53
*** takashin has joined #openstack-nova02:54
jichenmriedem: hi, sorry, could you please help in guide some 3rd CI related questions?02:54
*** hongbin has quit IRC02:55
*** hongbin_ has quit IRC02:56
*** coreywright has quit IRC02:56
*** edmondsw has joined #openstack-nova02:56
*** hongbin has joined #openstack-nova02:56
*** fragatina has joined #openstack-nova02:58
mriedemjichen: if you're not proposing a driver in-tree then running after patches merge is fine, it's really up to you as it's all external to nova02:59
mriedemjichen: i don't see value in reporting CI results to nova changes for an out of tree driver02:59
*** mingyu has joined #openstack-nova03:00
jichenmriedem: we want to make it in tree just like powervm did in last cycle, we knew CI is the mandatory work that we need to show03:00
*** edmondsw has quit IRC03:01
mriedemjichen: powervm worked on that for about 2 years i think03:01
mriedembtw, which z driver is this for? because there are at least 3 z-related drivers03:01
jichenmriedem: so some of my colleagues thought the only request is to report 'gate' result while my understanding is we need report in 'check' work03:01
mriedemjichen: there is no guide really03:01
jichenmriedem: https://github.com/openstack/nova-zvm-virt-driver is the one we are working on03:01
mriedemhonestly no one upstream is even thinking about adding zvm in tree because it hasn't come up03:02
*** liuyulong has joined #openstack-nova03:02
jichenmriedem: actually we have been work on open source for a few years but our CI is not ready and need more resource, we knew CI is mandatory so we didn't bring it up until now03:03
mriedemi think you might have asked me about this in barcelona? or someone did in atlanta?03:03
mriedemthe zHPM (?) thing also came up03:03
mriedemwhich is a different team03:03
mriedemand zKVM still never started any CI, which they talked about doing since Paris03:03
*** gianpietro has joined #openstack-nova03:04
jichenyes, they are from India team and you can think they zHPM are barematel on z03:04
*** litao__ has joined #openstack-nova03:04
mriedemjichen: well, nothing is going to happen in pike,03:04
jichenthere are 3 different hypervisors  z/VM is old one and last for some big customers like ADP etc, zKVM is a new one and libvirt based , DPM (zHPM) is more like a barematel thing03:04
mriedemi think we've previously told z driver people and huawei fusionsphere people to follow what the powervm team did,03:05
jichenmriedem: yes, I know, so we want to bring this out at next PTG03:05
jichenmriedem: ok, for the CI , we need to be as good as or better to PowerVM, need 'check' for every patch, right?03:05
mriedemi'd need to ask thorst or efried what they were doing, but i thought their ci was pretty solid on their out of tree driver first, including on check queue changes03:06
*** baoli has joined #openstack-nova03:06
mriedembut their driver is also quite different from the zvm driver i believe, but honestly i don't really know anything about the zvm driver architecture,03:07
mriedemso i can't really say03:07
mriedemthis probably needs some advertising on the dev ML first03:07
mriedemif you guys have summit sessions / slides / whatever that might be a good start03:07
jichenok, we do have some thing there, I already drafted a WIP spec there, I will try to collect whatever info needed and send in the ML03:08
jichenthanks a lot for your info and guide03:08
*** gianpietro has quit IRC03:09
*** coreywright has joined #openstack-nova03:09
*** diga has quit IRC03:10
*** baoli has quit IRC03:11
*** ijw has joined #openstack-nova03:12
mriedemjichen: sure, sorry i can't be of more help. i'd talk to thorst and the powervm team about their experience.03:15
jichenmriedem: thanks a lot for your help and we will try to make PTG/summit and show more info there03:16
*** ijw has quit IRC03:17
*** Apoorva has joined #openstack-nova03:18
*** zsli_ has joined #openstack-nova03:20
openstackgerritNgo Quoc Cuong proposed openstack/nova master: Remove translation of log messages  https://review.openstack.org/46699503:21
*** Shunli has quit IRC03:22
*** zsli_ is now known as Shunli03:24
*** vks1 has joined #openstack-nova03:24
*** itlinux has joined #openstack-nova03:29
openstackgerritJohn Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter  https://review.openstack.org/46659503:31
mriedemha ^03:33
mriedemcburgess: ^03:36
mriedemcburgess: i think if you can get enough different cloud providers to open separate specs to pass through the volume type, at some point we just have to give in03:36
mriedemit's in page 45 of the nova core dungeon master's guide03:37
*** liuyulong has quit IRC03:42
*** psachin has joined #openstack-nova03:49
*** mriedem has quit IRC03:50
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060803:50
*** tovin07_ has joined #openstack-nova03:58
*** fragatina has quit IRC04:00
*** tovin07__ has quit IRC04:00
openstackgerritfalseuser proposed openstack/nova master: [BugFix] Change the condition to delete a resizing instance  https://review.openstack.org/46346604:04
*** nicolasbock has quit IRC04:13
*** david-lyle has quit IRC04:14
*** mingyu has quit IRC04:16
*** hongbin has quit IRC04:20
*** Apoorva has quit IRC04:22
*** jamielennox|away is now known as jamielennox04:23
*** luos has quit IRC04:24
*** itlinux has quit IRC04:25
*** luos has joined #openstack-nova04:25
*** gyee has joined #openstack-nova04:31
*** trinaths has joined #openstack-nova04:34
*** sree has joined #openstack-nova04:39
*** armax has quit IRC04:40
*** edmondsw has joined #openstack-nova04:44
*** edmondsw has quit IRC04:49
*** marst_ has joined #openstack-nova04:49
*** sdatko has joined #openstack-nova04:56
*** dpawlik has joined #openstack-nova04:56
*** danpawlik has quit IRC04:56
*** sdatko_ has quit IRC04:57
*** dpawlik1 has joined #openstack-nova05:02
*** sree has quit IRC05:04
*** kaisers_ has joined #openstack-nova05:04
*** dpawlik has quit IRC05:04
*** prateek has joined #openstack-nova05:08
*** jamielennox is now known as jamielennox|away05:11
*** ratailor has joined #openstack-nova05:12
*** dpawlik1 has quit IRC05:15
*** sdatko has quit IRC05:15
*** udesale has joined #openstack-nova05:16
*** claudiub has joined #openstack-nova05:20
*** dpawlik1 has joined #openstack-nova05:21
*** jaosorior_away is now known as jaosorior05:24
*** dpawlik2 has joined #openstack-nova05:26
*** dpawlik1 has quit IRC05:26
*** sdatko has joined #openstack-nova05:28
*** ltomasbo|away is now known as ltomasbo05:32
*** gyee has quit IRC05:32
*** bkopilov__ has quit IRC05:34
*** adisky_ has joined #openstack-nova05:36
*** mtreinish has quit IRC05:38
*** brault has joined #openstack-nova05:47
*** ayogi has joined #openstack-nova05:51
*** jamielennox|away is now known as jamielennox05:51
*** ayogi has quit IRC05:52
*** yogesh_ has joined #openstack-nova05:52
*** crushil has quit IRC05:54
*** mingyu has joined #openstack-nova06:00
*** rcernin has quit IRC06:01
*** mingyu has quit IRC06:16
*** rcernin has joined #openstack-nova06:18
*** Oku_OS-away is now known as Oku_OS06:19
*** RuiChen has joined #openstack-nova06:21
*** ecelik has joined #openstack-nova06:22
*** ecelik has left #openstack-nova06:23
*** jamielennox is now known as jamielennox|away06:23
*** ijw has joined #openstack-nova06:24
*** takashin_ has joined #openstack-nova06:25
*** sridharg has joined #openstack-nova06:26
*** takashin has quit IRC06:26
*** mingyu has joined #openstack-nova06:27
*** zengjf has joined #openstack-nova06:27
*** ijw has quit IRC06:28
*** edmondsw has joined #openstack-nova06:32
*** mtreinish has joined #openstack-nova06:33
*** edmondsw has quit IRC06:37
*** pcaruana has joined #openstack-nova06:39
*** mdnadeem has joined #openstack-nova06:43
*** avolkov has joined #openstack-nova06:46
*** belmoreira has joined #openstack-nova06:47
*** trungnv has quit IRC07:00
openstackgerritNgo Quoc Cuong proposed openstack/nova master: Remove translation of log messages  https://review.openstack.org/46699507:06
*** trungnv has joined #openstack-nova07:07
*** sree has joined #openstack-nova07:09
*** salv-orlando has joined #openstack-nova07:10
*** saphi has joined #openstack-nova07:12
*** psachin has quit IRC07:12
*** rcernin has quit IRC07:16
*** rcernin has joined #openstack-nova07:17
*** salv-orlando has quit IRC07:25
*** andreas_s has joined #openstack-nova07:26
*** priteau has joined #openstack-nova07:28
*** psachin has joined #openstack-nova07:29
openstackgerritjichenjc proposed openstack/nova master: Deprecate unused policy from policy doc  https://review.openstack.org/46550507:34
*** jpena|off is now known as jpena07:40
*** abalutoiu__ has quit IRC07:40
*** dpawlik2 is now known as danpawlik07:42
*** psachin has quit IRC07:43
*** ltomasbo is now known as ltomasbo|away07:43
*** karimb has joined #openstack-nova07:45
*** psachin has joined #openstack-nova07:45
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948307:45
*** prateek_ has joined #openstack-nova07:46
*** prateek has quit IRC07:48
*** ralonsoh has joined #openstack-nova07:53
*** prateek__ has joined #openstack-nova07:57
*** diga has joined #openstack-nova07:57
*** prateek_ has quit IRC07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec  https://review.openstack.org/46084708:03
*** brault has quit IRC08:05
*** saphi_ has joined #openstack-nova08:14
*** saphi has quit IRC08:14
*** lucas-afk is now known as lucasagomes08:16
*** lpetrut has joined #openstack-nova08:17
*** openstackgerrit has quit IRC08:18
*** mvk has quit IRC08:19
*** RuiChen has quit IRC08:19
*** edmondsw has joined #openstack-nova08:20
*** CristinaPauna has joined #openstack-nova08:23
*** edmondsw has quit IRC08:25
*** openstackgerrit has joined #openstack-nova08:26
openstackgerritsahid proposed openstack/nova master: libvirt: configure trust mode for vfs  https://review.openstack.org/45851408:26
openstackgerritsahid proposed openstack/nova master: network: update pci request spec to handle trusted tags  https://review.openstack.org/45882008:26
openstackgerritsahid proposed openstack/nova master: network: add command to configure trusted mode for VFs  https://review.openstack.org/45851308:26
sfinucanjohnthetubaguy, alex_xu: Could you take a look at [1] as it's blocking 4 other patches [1] https://review.openstack.org/#/c/461630/08:32
*** lei-zh has joined #openstack-nova08:32
sfinucanoomichi too ^08:33
openstackgerritHuan Xie proposed openstack/nova master: XenAPI use os-xenapi v2 in nova  https://review.openstack.org/45349308:35
*** derekh has joined #openstack-nova08:36
*** baoli has joined #openstack-nova08:37
*** baoli has quit IRC08:42
*** ltomasbo|away is now known as ltomasbo08:44
*** saphi_ has quit IRC08:47
*** raj_sing- has quit IRC08:48
*** sambetts|afk has quit IRC08:48
*** Kimmo_ has quit IRC08:48
*** amotoki has quit IRC08:49
*** raj_singh has joined #openstack-nova08:50
*** amotoki has joined #openstack-nova08:51
*** sambetts_ has joined #openstack-nova08:52
*** takashin has joined #openstack-nova08:53
*** takashin_ has quit IRC08:53
*** jistr is now known as jistr|call08:56
*** Kevin_Zheng has quit IRC08:56
*** mvk has joined #openstack-nova08:57
*** zengjf has quit IRC09:00
*** fragatina has joined #openstack-nova09:01
*** aarefiev_afk is now known as aarefiev09:04
*** fragatina has quit IRC09:09
*** mingyu has quit IRC09:15
*** CristinaPauna has quit IRC09:17
*** jistr|call is now known as jistr09:19
*** mingyu has joined #openstack-nova09:21
*** mkoderer_ has joined #openstack-nova09:21
*** dgonzalez_ has joined #openstack-nova09:21
*** seife_ has joined #openstack-nova09:21
*** tpatzig_ has joined #openstack-nova09:21
*** mkoderer_ has quit IRC09:23
*** dgonzalez_ has quit IRC09:23
*** seife_ has quit IRC09:23
*** tpatzig_ has quit IRC09:23
*** lei-zh has quit IRC09:23
*** masber has quit IRC09:24
*** sambetts_ is now known as sambetts09:24
*** Shunli has quit IRC09:28
*** lei-zh has joined #openstack-nova09:30
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in os-services.inc  https://review.openstack.org/46709909:32
*** mvk has quit IRC09:32
*** gszasz has joined #openstack-nova09:39
*** mingyu has quit IRC09:39
*** takashin has left #openstack-nova09:41
*** yolkfull has quit IRC09:42
*** yolkfull has joined #openstack-nova09:43
*** lei-zh has quit IRC09:44
*** mvk has joined #openstack-nova09:45
*** dikonoo has joined #openstack-nova09:46
openstackgerritZhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed  https://review.openstack.org/45992309:47
*** dikonoor has quit IRC09:49
openstackgerritFelipe Alfaro Solana proposed openstack/nova-specs master: Implements: blueprint nova-lxd-profiles  https://review.openstack.org/46710409:54
*** tovin07_ has quit IRC09:55
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore  https://review.openstack.org/46162809:57
*** tbachman has joined #openstack-nova09:57
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore  https://review.openstack.org/46162809:59
*** faizy has joined #openstack-nova09:59
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore  https://review.openstack.org/46081810:02
*** brault has joined #openstack-nova10:03
*** sdague has joined #openstack-nova10:03
*** danpawlik has quit IRC10:04
*** dpawlik has joined #openstack-nova10:06
*** faizy has quit IRC10:07
*** rmart04 has joined #openstack-nova10:08
*** edmondsw has joined #openstack-nova10:09
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.resize_prep notification  https://review.openstack.org/46508110:09
*** yingjun has quit IRC10:11
*** yogesh_ has quit IRC10:11
*** yogesh_ has joined #openstack-nova10:11
*** edmondsw has quit IRC10:13
*** dpawlik has quit IRC10:14
*** dpawlik has joined #openstack-nova10:14
openstackgerritVu Cong Tuan proposed openstack/nova master: Repair links in Nova documentation  https://review.openstack.org/46711410:18
*** cNilesh has quit IRC10:20
*** lpetrut has quit IRC10:22
*** lpetrut has joined #openstack-nova10:22
*** mpaolino has joined #openstack-nova10:26
*** sridharg has quit IRC10:34
*** Kimmo_ has joined #openstack-nova10:48
*** salv-orlando has joined #openstack-nova10:49
*** jichen has quit IRC10:50
*** nicolasbock has joined #openstack-nova10:51
*** fragatina has joined #openstack-nova10:52
*** salv-orlando has quit IRC10:53
*** sridharg has joined #openstack-nova10:54
*** brault has quit IRC10:56
*** brault has joined #openstack-nova10:57
*** CristinaPauna has joined #openstack-nova11:01
*** thorst_afk has joined #openstack-nova11:05
*** yolkfull has quit IRC11:05
*** smatzek has joined #openstack-nova11:11
artomo_O11:15
artomhttps://review.openstack.org/#/c/400763/3 merged without unit tests11:16
*** udesale has quit IRC11:16
artomIs that legit? We're allowed to do that? I'd have though making sure the exception is caught would be a good test11:16
*** yamamoto has quit IRC11:17
*** phuongnh has quit IRC11:25
*** cdent has joined #openstack-nova11:26
*** ralonsoh_ has joined #openstack-nova11:32
*** ralonsoh_ has quit IRC11:32
sfinucanartom: Hmm, yeah. It should have tests, yes11:32
*** gouthamr_ has joined #openstack-nova11:36
*** lucasagomes is now known as lucas-afk11:38
*** lucas-afk is now known as lucas-hungry11:38
*** yogesh_ has quit IRC11:39
*** salv-orlando has joined #openstack-nova11:42
*** cdent has quit IRC11:45
*** vladikr has quit IRC11:45
sfinucansdague: If I want to check out a particular commit of nova and run it with Devstack, I simply check it out and reboot all nova services with 'systemctl restart devstack@n-*', right?11:47
sfinucan(First time tinkering with it since the systemd changes went in)11:48
sdagueyep11:48
sfinucancool11:50
*** cdent has joined #openstack-nova11:50
*** salv-orlando has quit IRC11:52
*** pwiltsey has quit IRC11:52
*** pwiltsey has joined #openstack-nova11:53
*** litao__ has quit IRC11:53
*** yamamoto has joined #openstack-nova11:58
*** sree has quit IRC11:58
*** thorst_afk has quit IRC11:59
*** yamamoto_ has joined #openstack-nova12:00
*** diga has quit IRC12:00
*** mingyu has joined #openstack-nova12:01
*** Kevin_Zheng has joined #openstack-nova12:02
*** gouthamr_ has quit IRC12:02
*** liverpooler has joined #openstack-nova12:02
*** yamamoto has quit IRC12:04
*** trinaths has quit IRC12:05
*** rmk has quit IRC12:11
*** dgonzalez has quit IRC12:12
*** MVenesio has joined #openstack-nova12:13
*** rmk has joined #openstack-nova12:14
*** rmk has quit IRC12:14
*** rmk has joined #openstack-nova12:14
*** dgonzalez has joined #openstack-nova12:14
*** dave-mccowan has joined #openstack-nova12:14
*** moshele has joined #openstack-nova12:15
openstackgerritFeodor Tersin proposed openstack/nova master: Implement ScaleIO image backend  https://review.openstack.org/40744012:16
ftersinmdbooth: hi. did you have a chance to look at ^^^^ ?12:17
mdboothftersin: Only very briefly, I'm afraid. I'm stuck on downstream work right now. It's top of my review list, though.12:18
*** ratailor has quit IRC12:20
*** ociuhandu has joined #openstack-nova12:21
ftersinmdbooth: ok, i understand. Probably you might review it by part. E.g. driver.py at the first, or whatever you assume is the most 'suspicious' and requires attention12:22
mdboothftersin: That is indeed where I started :)12:23
ftersinmdbooth: to let us an opportunity to discuss and to rework complex things not being in rush12:23
ftersincool12:24
mdboothIncidentally, if the driver could be confined to imagebackend.py I'd be in favour pretty much on your word that it works.12:24
mdboothIIRC it's a bit differenty in a couple of places, though.12:24
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/46717012:25
*** edmondsw has joined #openstack-nova12:25
*** edmondsw_ has joined #openstack-nova12:27
ftersinmdbooth: i think there are 3 kings of changes in driver.py. the 1st is trivial, i.e. the adding 'sio' into some places where 'rbd', 'lvm', etc already exist12:29
*** edmondsw has quit IRC12:29
ftersinthe 2nd is intended to map/unmap feature. it cannot be removed but might be reworked12:30
ftersinthe 3rd one are some side changes, which might be separated into different patches or patch series12:30
openstackgerritFelipe Alfaro Solana proposed openstack/nova-specs master: Implements: blueprint nova-lxd-profiles  https://review.openstack.org/46710412:31
mdboothftersin: Are the side changes generally applicable, or specific to this driver?12:31
*** edmondsw_ has quit IRC12:31
*** vks1 has quit IRC12:31
*** edmondsw has joined #openstack-nova12:33
ftersinboth. e.g. the fix of swap device is generally applicable, but the filtering of disk_info items (to prevent use them as files) is required for SIO driver12:33
*** liverpooler has quit IRC12:33
*** liverpooler has joined #openstack-nova12:34
ftersinhowever it is required for Cinder driver as well12:34
ftersinit or some other solution of the problem12:34
*** crushil has joined #openstack-nova12:34
mdboothftersin: If you have time, I'd appreciate splitting them out, I think. I'll review without, though.12:35
openstackgerritEric Berglund proposed openstack/nova master: Add strict option to discover_hosts  https://review.openstack.org/46721812:35
*** lucas-hungry is now known as lucasagomes12:36
*** edmondsw has quit IRC12:37
ftersinmdbooth: i proposed a series, which resolve that problem separately: https://review.openstack.org/#/q/topic:disk_info_processing12:38
*** edmondsw has joined #openstack-nova12:39
*** sridhargaddam has joined #openstack-nova12:41
*** catintheroof has joined #openstack-nova12:42
*** baoli has joined #openstack-nova12:43
*** edmondsw has quit IRC12:44
*** sridharg has quit IRC12:44
*** edmondsw has joined #openstack-nova12:47
*** salv-orlando has joined #openstack-nova12:48
*** jamielennox|away is now known as jamielennox12:49
*** jpena is now known as jpena|lunch12:49
openstackgerritZhenyu Zheng proposed openstack/nova master: Exclude deleted service records when calling hyervisor statistics  https://review.openstack.org/46722012:50
*** edmondsw has quit IRC12:51
*** salv-orlando has quit IRC12:53
*** edmondsw has joined #openstack-nova12:53
*** lyan has joined #openstack-nova12:53
*** gcb has quit IRC12:55
*** edmondsw has quit IRC12:55
*** edmondsw has joined #openstack-nova12:56
*** esberglu has joined #openstack-nova12:56
*** crushil has quit IRC12:56
openstackgerritEric Berglund proposed openstack/nova master: Add strict option to discover_hosts  https://review.openstack.org/46721812:56
*** jaosorior has quit IRC12:57
*** crushil has joined #openstack-nova12:57
*** jaosorior has joined #openstack-nova12:57
*** udesale has joined #openstack-nova12:58
openstackgerritsahid proposed openstack/nova master: libvirt: configure trust mode for vfs  https://review.openstack.org/45851413:00
*** crushil has quit IRC13:04
*** crushil has joined #openstack-nova13:04
*** mdrabe has joined #openstack-nova13:04
*** sridhargaddam has quit IRC13:04
*** READ10 has joined #openstack-nova13:05
*** sridhargaddam has joined #openstack-nova13:07
*** thorst_afk has joined #openstack-nova13:09
*** winston-d_ has quit IRC13:09
*** mriedem has joined #openstack-nova13:11
mriedemahoy matey13:11
mriedem(s)13:11
cdentis it talk like a pirate day again?13:11
* asettle comes in 13:12
asettleYar13:12
asettleit is13:12
cdentsince I'm in cornwall, I claim authenticity13:12
asettlePlease, please tell me you're in Penzance13:13
*** salv-orlando has joined #openstack-nova13:15
mriedemdansmith: when you're decent i have some questions about your bottom cells change for target_cell13:15
*** lucasxu has joined #openstack-nova13:17
*** lucasxu has quit IRC13:17
cdentasettle: sadly right this moment I am in Truro13:19
asettleTruro is nice!13:19
asettleBut not Penzance13:20
*** salv-orlando has quit IRC13:20
*** sree_ has joined #openstack-nova13:24
*** kylek3h has joined #openstack-nova13:24
*** sree_ is now known as Guest8530913:25
*** cdent has quit IRC13:26
*** abalutoiu has joined #openstack-nova13:27
*** Guest85309 has quit IRC13:29
*** lucasxu has joined #openstack-nova13:30
*** pabelanger has joined #openstack-nova13:30
dansmithmriedem: in the review or you want to ask them here?13:30
*** pwiltsey has quit IRC13:31
pabelangermorning!13:31
*** udesale has quit IRC13:31
dansmithmriedem: I still need to follow up with a few more of your earlier comments I think, but removing the object targeting seems to not break anything13:31
mriedemdansmith: either13:31
mriedemi just went through PS1313:31
*** pwiltsey has joined #openstack-nova13:31
dansmithmriedem: okay well I'm yet to be caffeinated so give me a few13:31
mriedemok13:31
*** crushil has quit IRC13:32
pabelangerQ: Why is there no PUT /os-keypairs functionality? We have a use case in openstack-infra where we want to update an existing key (to rotate public_keys) but we have to delete / create to achieve this13:32
pabelangerSpecifically, this would be for nodepool13:33
dansmithpabelanger: I imagine it comes from not confusing the user about why they PUT a new keypair and can't suddenly use it to log into all the instances they've booted with that keypair13:34
dansmithseveral other cloud-like platforms I know of have the same rule13:34
*** pwiltsey has quit IRC13:36
pabelangerdansmith: I can understand that, but POST / PUT seem to be well documented on there behaviours. And other resources in nove do support PUT actions /servers and /quotas come to mind13:38
dansmithbut not flavors for the same reason13:38
dansmithservers and quotas update doesn't affect something else, right?13:38
dansmithlike if I'm allowed to make a change to a server, I'm not affecting another server elsewhere13:39
*** dimtruck is now known as zz_dimtruck13:39
dansmithone server does not reference another, nor does quota have such a link13:39
pabelangersure, but the same could be said about an SSH key, if only used for a single server13:39
dansmithbut if you have it for one server and you change that key, it no longer matches what is in the server13:40
mriedem^ is definitely like flavors,13:40
mriedemand we store the keypair in the instance in the db like flavors13:40
dansmithyup13:41
*** smatzek has quit IRC13:41
pabelangerThe issue we are having is, if we need to rotate keys for nodepool (infra-root-keys in nova), we need to do a 2 step approach. This means, when we delete the key, nodepool starts failing to create new servers until we create infra-root-keys again. And since we do this via automation, it is difficult to do this process without causing an outage for nodepool13:41
mriedemwhy not create the new key first and then delete the old key?13:42
mriedemduplicates i guess13:42
mriedemon the name13:42
dansmithhow can you use this for key rotation if it doesn't cause the existing servers to change the key they use?13:42
dansmithmriedem: or just date your keys13:42
mriedemi'm married13:42
dansmithdate *stamp* before you go there :)13:42
dansmithdammit13:42
*** awaugama has joined #openstack-nova13:44
*** awaugama has quit IRC13:44
pabelangerthe SSH key, infra-root-keys is only used when (humans) need to log into a test node and debug it. We are okay with old nodes still using the first key, because up to 2 hours later the server is deleted.13:44
*** gcb has joined #openstack-nova13:45
*** awaugama has joined #openstack-nova13:45
*** prateek__ has quit IRC13:45
*** cfriesen has joined #openstack-nova13:45
dansmithokay so why not just create a new datestamped key?13:46
*** jamesdenton has joined #openstack-nova13:46
dansmithpabelanger: anyway, you asked _why_ and I think that's why. I think it would be confusing to enable changing them at this point, but if you want to propose it we can discuss I suppose13:47
mriedemi'm asking in ops too to see if users are asking for this13:47
*** ralonsoh has quit IRC13:47
mriedemnon-infra users let me say13:47
mriedeminfra are users too!13:47
*** dave-mcc_ has joined #openstack-nova13:47
pabelangerSo, to rotate this out today, we'd need to do the following:13:47
pabelanger1) update clouds_layout.yaml with new key: http://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/clouds_layouts.yml#n3513:48
pabelangerland and merge (30mins)13:48
dansmithmriedem: even if they are, we should decide whether we think it breaks our model or not13:48
*** armax has joined #openstack-nova13:48
*** hamzy has quit IRC13:48
mriedemdansmith: sure, i'd just like to know if this is one of those things everyone has been secretly wanting forever and just no one asked13:48
mriedemlike proxying volume type13:49
pabelanger2) Replace infra-root-keys with new keyname in nodepool.yaml: http://git.openstack.org/cgit/openstack-infra/project-config/tree/nodepool/nodepool.yaml (~78 entries)13:49
dansmithmriedem: aye13:49
pabelangerreview and merge13:49
mriedemdansmith: see my ML thread on volume type from late last night?13:49
*** dave-mccowan has quit IRC13:49
dansmithpabelanger: or build your system to list your keys and choose the one with the highest serial number at the point of each build right?13:49
dansmithmriedem: no13:49
mriedemit's a hoot and a half13:50
pabelangerdansmith: sure, but as an operator I really only want 1 key at 1 time13:50
*** mlavalle has joined #openstack-nova13:50
*** dave-mccowan has joined #openstack-nova13:50
pabelangermanaging more then one is possible, but something not in our current workflows with other cloud settings.13:51
*** hongbin has joined #openstack-nova13:51
openstackgerritSean Dague proposed openstack/nova master: WIP: request_id passing to neutron  https://review.openstack.org/46724213:51
*** thorst_afk has quit IRC13:52
*** vladikr has joined #openstack-nova13:52
*** crushil has joined #openstack-nova13:53
*** eharney has joined #openstack-nova13:53
*** dave-mcc_ has quit IRC13:53
*** jpena|lunch is now known as jpena13:54
*** brault has quit IRC13:54
efrieddansmith Not sure if you saw esberglu's --strict change go by: https://review.openstack.org/#/c/467218/13:54
dansmithefried: I did not13:54
efrieddansmith He's gonna fold it into our CI to prove it works.  Should have that result by EOD or sooner.13:55
*** brault has joined #openstack-nova13:55
*** jwcroppe has quit IRC13:55
efrieddansmith I took the liberty of adding you to the review ;-)13:55
dansmithmriedem: I just replied to your comments but if you want to chat here or in a hangout now, I'm ready13:55
dansmithefried: heh okay13:55
*** jwcroppe has joined #openstack-nova13:55
*** lei-zh has joined #openstack-nova13:56
*** gianpietro has joined #openstack-nova13:56
*** links has quit IRC13:57
mriedemdansmith: i've got to take maya to school, will ping when i'm back13:57
dansmithokay13:57
efriedmriedem That review you asked me for last night - done.13:57
*** vks1 has joined #openstack-nova13:59
*** brault has quit IRC13:59
*** jwcroppe has quit IRC13:59
*** andreas_s has quit IRC14:01
*** ralonsoh has joined #openstack-nova14:01
*** brault has joined #openstack-nova14:01
*** mtanino__ has joined #openstack-nova14:04
*** smatzek has joined #openstack-nova14:04
*** JoseMello has joined #openstack-nova14:05
*** mingyu has quit IRC14:05
*** smatzek_ has joined #openstack-nova14:06
*** brault has quit IRC14:06
*** salv-orlando has joined #openstack-nova14:10
*** smatzek has quit IRC14:10
*** dikonoor has joined #openstack-nova14:10
*** kfarr has joined #openstack-nova14:11
*** jwcroppe has joined #openstack-nova14:12
alex_xusfinucan: thanks, just +w14:12
sfinucanalex_xu: Ta14:12
*** dikonoo has quit IRC14:13
*** salv-orlando has quit IRC14:14
*** marst_ has quit IRC14:17
*** zz_dimtruck is now known as dimtruck14:17
*** jmlowe has quit IRC14:17
*** chyka has joined #openstack-nova14:18
*** chyka has quit IRC14:18
*** chyka has joined #openstack-nova14:18
*** brault has joined #openstack-nova14:18
mriedemdansmith: in your replies, when you say global context, you mean modifying the single context object that's being passed around everywhere, right?14:20
mriedemit's not an actual 'global'14:21
mriedemtarget_cell mutates the thing that's passed everywhere14:21
dansmithmriedem: right14:21
mriedemok14:21
dansmithshared context or something14:21
mriedemright14:21
mriedem"Maybe it's because we targeted the same context that was used in  _lookup_instance and so when we target it here, it's modifying  instance._context by reference?"14:21
mriedemwas my attempt at saying the same14:21
dansmithokay14:21
mriedemdansmith: ok on to https://review.openstack.org/#/c/458634/13/nova/conductor/manager.py@84614:22
mriedemis that only here because it will be a failure later?14:22
*** burt has joined #openstack-nova14:22
mriedemif not done now?14:22
*** thorst_afk has joined #openstack-nova14:22
*** marst_ has joined #openstack-nova14:23
dansmithmriedem: yeah, but that's the same for the rest of the patch14:23
dansmithit's all calling convention changes so I can change the behavior of the target function and have things continue to work14:23
dansmithI could put any of this in the next patch, but I'm just trying to separate things14:23
*** acormier has joined #openstack-nova14:23
mriedemsure, but the difference is the rest of the patch is already using target_cell14:24
mriedemthat's why everyone is thrown off on that particular change14:24
acormierQuick ?:  Is it possible to configure a subnet of compute nodes in Openstack to use Ceph RBD backend?  Or do all nodes have to move over in wholesale?14:24
mriedemsubset14:24
mriedemacormier: answer is yes14:24
*** pwiltsey has joined #openstack-nova14:25
dansmithmriedem: okay I guess it seems like exactly the same to me14:25
dansmithmriedem: but if you really want it broken out then I will14:25
mriedemi'd prefer it lives in the patch that it's going to be needed otherwise it fails14:26
mriedemdansmith: last thing, so you do plan on changing https://review.openstack.org/#/c/458634/13/nova/objects/host_mapping.py to use the yielded cctxt?14:27
mriedemwhen creating the host mappings i mean,14:27
mriedemeven though the api db connection is the same14:27
dansmithI answered that one didn't I?14:27
mriedemyou replied,14:27
mriedembut i'm not sure it answered my question14:27
dansmithokay, yes I've already got it changed locally14:27
mriedemok14:27
mriedemthat wasn't obvious if the patch wasn't WIPed or something14:28
*** cdent has joined #openstack-nova14:28
mriedemanywho, that's it14:28
mriedemdansmith: can we still be friends?14:28
dansmithmriedem: so, moving those conductor changes into the patch that changes context.py will make that match go from "one functional change and the associated test changes" to "one real code change, an ancillary code change and several test changes"14:29
dansmithcan I just move it into another patch and explain in the commit message or something?14:29
*** cleong has joined #openstack-nova14:29
mriedemsure14:29
dansmithI really want that context switch patch to be really tight14:29
*** chyka has quit IRC14:29
dansmithokay, I totally don't understand why that's different than having it in the first patch, but I'll do it anyway :)14:29
mriedemor just leave it and point it out in the commit message in this one14:29
mriedemthe fact 3 people had to ask about it,14:30
mriedemmeans it's not clear14:30
mriedemso just point it out in the commit message14:30
dansmithif that's good enough then I'd like that option14:30
mriedemfine by me14:30
*** pchavva has joined #openstack-nova14:30
mriedemmelwitt: it's 7:30am, i assume you're 2 hours into addressing https://review.openstack.org/#/c/446239/14:30
mriedembecause i want to +2 that one14:30
*** hamzy has joined #openstack-nova14:37
*** yamahata has quit IRC14:38
*** eharney has quit IRC14:38
*** smatzek_ is now known as smatzek14:39
mriedemalex_xu: replied in https://review.openstack.org/#/c/459569/14:41
*** chyka has joined #openstack-nova14:42
*** pchavva has quit IRC14:42
mriedemalex_xu: Kevin_Zheng: jaypipes: why is this a 403 and not a 409? https://review.openstack.org/#/c/464548/5/nova/api/openstack/compute/servers.py14:42
mriedemi thought we normally handled over quota as 409 in the api14:42
*** jmlowe has joined #openstack-nova14:43
openstackgerritDaniel Pawlik proposed openstack/nova master: Add `hide_hypervisor_id` image property  https://review.openstack.org/45975314:43
mriedemnevermind, i guess at some point those were switched to 40314:44
melwittmriedem: yeah, I got most of it done yesterday and I'm stuck on a unit test of all things. hope I'll get past that soon14:44
Kevin_Zhengmriedem: I did a search and I saw it was handled in that way somewhere else14:44
mriedemKevin_Zheng: yeah me too, i didn't realize that was changed14:45
Kevin_ZhengBtw, I tired to cherry pick it to N but seems conflicting14:45
mriedemthen you have to do it manually using git cherry-pick -x14:46
mriedemresolve the merge conflict locally, and leave a note in the commit message about what the conflict was14:46
*** nicolasbock has quit IRC14:47
mriedemlyarwood: when you have a minute i'd like to get this released in ocata since it's an upgrade issue https://review.openstack.org/#/c/466774/14:47
lyarwoodmriedem: looking14:48
lyarwoodmriedem: while we are talking about stable/ocata, should we drop Citrix XenServer CI to non-voting until they get it passing?14:48
Kevin_Zhengmriedem: ok I will try tomorrow14:48
mriedemlyarwood: did that last night, emailed the team and bobball is already aware and disabled the failing tests until fixed14:49
lyarwoodmriedem: cool, thanks :)14:49
*** psachin has quit IRC14:50
*** lei-zh has quit IRC14:51
*** lei-zh has joined #openstack-nova14:51
*** cdent has quit IRC14:51
*** fishbone_ has joined #openstack-nova14:52
*** chyka has quit IRC14:52
*** cdent has joined #openstack-nova14:52
*** chyka has joined #openstack-nova14:52
fishbone_Hello all, I am having a strange issue...After an upgrade from newton to ocata all of my old virtual servers are innaccessible14:52
*** eharney has joined #openstack-nova14:53
mriedemlyarwood: the depends-on here seems wrong https://review.openstack.org/#/c/466480/114:53
fishbone_I am able to see them in the database and when runnin openstack server list, however if i run openstack server show ID it will say the server does not exist14:53
openstackgerritDaniel Pawlik proposed openstack/nova master: Add `hide_hypervisor_id` image property  https://review.openstack.org/45975314:54
lyarwoodmriedem: yeah that's wrong, gerritui just copied it across for the original commit, I'll remove it now14:54
mriedemfishbone_: did you run nova-manage cell_v2 map_instances as part of the upgrade?14:55
mriedemfishbone_: check to see if you have instance_mappings in the nova_api db14:56
*** moshele has quit IRC14:56
fishbone_yes I did, I wonder if the fact that I had servers before doing that is the cause of the issue14:56
fishbone_Any new servers I create register correctly14:56
fishbone_oh wait14:56
*** yamahata has joined #openstack-nova14:56
fishbone_mriedem: I dont think I ran the one for instances14:56
mriedemmap_instances is specifically for mapping existing instances to a cell14:56
mriedemhttps://docs.openstack.org/developer/nova/man/nova-manage.html#nova-cells-v214:56
*** lpetrut has quit IRC14:57
*** itlinux has joined #openstack-nova14:57
mriedemfishbone_: note that nova-manage cell_v2 simple_cell_setup will also do map_instances automatically14:57
fishbone_mriedem: I did not use that command when I performed the upgrade in this case14:58
fishbone_mriedem: I will try to run that command now14:58
mriedemok, if you don't use simple_cell_setup, then you need to run map_instances14:58
mriedemfrom the release note too http://git.openstack.org/cgit/openstack/nova/tree/releasenotes/notes/cells-discover-hosts-06a3079ba687e092.yaml14:59
openstackgerritEric Berglund proposed openstack/nova master: Add strict option to discover_hosts  https://review.openstack.org/46721815:00
*** slaweq has joined #openstack-nova15:00
*** dpawlik has quit IRC15:02
*** dpawlik has joined #openstack-nova15:02
*** dpawlik has quit IRC15:02
*** dpawlik has joined #openstack-nova15:04
*** dpawlik has quit IRC15:05
*** mingyu has joined #openstack-nova15:06
*** dpawlik has joined #openstack-nova15:07
*** dpawlik is now known as danpawlik15:07
*** mingyu_ has joined #openstack-nova15:09
*** danpawlik has quit IRC15:09
*** mingyu has quit IRC15:09
mriedemmelwitt: so can we abandon this? https://review.openstack.org/#/c/462744/15:10
fishbone_mriedem: Thanks a lot, that worked, I didn't use the simple cell setup because when I did it wasn't naming the cell appropriately15:10
*** mlakat has quit IRC15:11
mriedemfishbone_: yup, which i'm resurrecting in newton https://review.openstack.org/#/c/420461/15:11
mriedembut that doesn't help you here15:11
mriedemfishbone_: i'm going to start a faq in the nova devref for cells v2 things, i'll put your case in there15:11
jaypipessdague, mriedem: can you point to docs on how to update the global requirements properly for a new dependency (in this case, os-vif and os-traits)15:12
*** dpawlik has joined #openstack-nova15:12
*** dpawlik is now known as danpawlik15:12
melwittmriedem: yeah. I'm disturbed by the fact that the message seems to vanish when not converted to unicode (since that will happen to other people in the wild). I was trying to decide if I wanted to recheck it again to see if I could locate the unconverted messages in the logs15:12
jaypipessdague, mriedem: reference this patch https://review.openstack.org/#/c/462769/15:12
fishbone_mriedem: Thanks again15:12
*** danpawlik has quit IRC15:13
mriedemjaypipes: so you need a new minimum os-traits in g-r right?15:13
mriedemjaypipes: just bump the version here https://github.com/openstack/requirements/blob/master/global-requirements.txt#L15915:13
jaypipesmriedem: yeah. and AJaeger sent an email to the ML mentioning some problems with os-vif as well.15:13
mriedemwell,15:14
jaypipesmriedem: got it. that's easy enough. ty.15:14
mriedemdo we have merged code that needs newer minimums?15:14
mriedemjaypipes: anyway, for a pending patch that needs a new minimum of a library/client, bump g-r in the openstack/requirements repo and then add a depends-on to your nova change to the reqs change and you should be good15:14
*** thorst_afk has quit IRC15:14
*** gjayavelu has joined #openstack-nova15:14
mriedemyou'd want to also bump the requirements.txt in your nova patch15:14
*** lpetrut has joined #openstack-nova15:14
mriedembut the depends-on will keep the ci happy15:14
jaypipesok, thanks15:15
openstackgerritDan Smith proposed openstack/nova master: De-complicate some of the instance delete path  https://review.openstack.org/45863615:15
openstackgerritDan Smith proposed openstack/nova master: Move to proper target_cell calling convention  https://review.openstack.org/45863415:15
openstackgerritDan Smith proposed openstack/nova master: Make target_cell() yield a new context  https://review.openstack.org/45863515:15
openstackgerritDan Smith proposed openstack/nova master: Fix live migration devstack hook for multicell environment  https://review.openstack.org/45853715:15
openstackgerritDan Smith proposed openstack/nova master: Target cell on local delete  https://review.openstack.org/46405915:15
*** hieulq_ has joined #openstack-nova15:15
*** thorst_afk has joined #openstack-nova15:16
mriedemmelwitt: btw, remember this? https://review.openstack.org/#/c/420461/15:17
mriedemmelwitt: i need that for backporting the patch to handle special characters in the db connection url15:18
mriedemwhich makes me sad15:18
*** danpawlik has joined #openstack-nova15:18
melwittah, right15:18
melwittbummer15:18
mriedemdespite grenade failing, i'm kind of hesitant to put that out in a newton fix, but it would be good for anyone that hasn't rolled up yet15:18
mriedembecause i think fishbone_ ran into the same thing15:18
mriedemthe cell0 db name was wrong on first attempt15:18
mriedemand i think i've seen that in the forum15:18
mriedemaka ask.o.o15:19
melwittugh15:19
*** thorst_afk has quit IRC15:20
*** efried has quit IRC15:21
*** rcernin has quit IRC15:22
*** gianpietro has quit IRC15:23
*** gianpietro has joined #openstack-nova15:24
openstackgerritMatt Riedemann proposed openstack/nova master: doc: add links to summit videos on cells v2  https://review.openstack.org/46728415:24
mriedemalaski: dansmith: ^ etched in stone for all time15:24
dansmithoh my15:24
*** nicolasbock has joined #openstack-nova15:24
alaskinot sure how I feel about that15:25
mriedemyou feel great about it15:25
mriedemyou put in all that hard work preparing15:25
mriedempicked out your best and most professional shorts15:25
mriedemi kid15:25
dansmithheh15:26
alaski:)15:26
alaskiI should get some suit pants made into shorts someday for formal occassions15:26
mriedemdon't worry i'm going to do the same for jaypipes's videos on placement15:26
*** yamamoto_ has quit IRC15:26
mriedemalaski: "get made into"? you have a scissors in the house right?15:26
*** armax has quit IRC15:27
*** yamamoto has joined #openstack-nova15:27
alaskiyep, it's the sewing up after that would be new territory for me15:27
mriedemyou're putting too much work into this15:27
mriedemthe cut off look is part of the style15:28
mriedemit's like a mullet15:28
melwittyou need to choose the hem for the suit shorts15:28
mriedembusiness in the front, party in the back15:28
*** mingyu_ has quit IRC15:28
mriedempleated would be extra nice on cut offs15:28
mriedemyou can go from the easter service at church to the monster truck rally w/o even changing15:29
melwittnow that's versatility15:29
jaypipesalaski: or just give the pant suits to Luke and see what happens? :)15:30
alaskimriedem: I have to admit, I'm half tempted now15:30
mriedemboogers and slobber sweet cut offs they will not make15:30
mriedemmaybe he's over that by now15:31
*** mdrabe_ has joined #openstack-nova15:31
alaskislightly past that, but not yet capable with scissors15:31
alaskieven if he was, I would not want to inflict the result on the world15:32
*** gyee has joined #openstack-nova15:32
*** yamamoto has quit IRC15:32
jaypipesalaski: it would be haute couture. I know you're well into that.15:32
mriedemalaski: alternatively, no scissors, just rip them off with brute strength, now that i know about your over the top past15:33
jaypipesmriedem: I leave this to you to Wallaby: https://review.openstack.org/#/c/458634/15:33
alaskijaypipes: you've described me perfectly :)15:33
mriedemhttps://en.wikipedia.org/wiki/Over_the_Top_(film)15:34
mriedemsomeone needs to put laski's face on that promo15:34
jaypipesindeed15:34
*** mdrabe has quit IRC15:34
mriedemhttps://en.wikipedia.org/wiki/Over_the_Top_(film)#/media/File:Over_the_Top_-_Renato_Casaro_(1987_Film).jpg15:34
mriedemi thought that was a turkey in the background,15:34
mriedembut i guess it's a hawk15:34
mriedemb/c that's his name,15:34
mriedembecause of course it is15:34
*** Oku_OS is now known as Oku_OS-away15:35
*** ociuhandu has quit IRC15:35
mriedemjaypipes: was waiting on tests15:35
*** djohnsto has joined #openstack-nova15:35
*** hongbin has quit IRC15:36
jaypipesalaski: for you: http://us.asos.com/asos/asos-onesie-in-gray-marl/prd/723607415:36
*** belmoreira has quit IRC15:37
alaskithat's a steal at $4815:37
jaypipesindeed. I figure you can find a matching one for the boy.15:37
jaypipesand be like "hey, check out my mini-me."15:38
*** mingyu has joined #openstack-nova15:38
alaskiwe'll need the matching tattoos to really pull it off15:39
*** itlinux has quit IRC15:39
jaypipesalaski: ++15:39
mriedemone piece would be best https://goo.gl/photos/gM3yNwCeS44j14Ns515:40
*** jmlowe has quit IRC15:40
melwittwhaaaaaat15:40
melwittlol15:41
mriedembtw, i still use that when blowing snow15:41
*** jmlowe has joined #openstack-nova15:41
mriedemsans the hat15:41
alaskithat is amazing15:42
*** mlavalle has quit IRC15:43
*** lucasxu has quit IRC15:44
*** sean-k-mooney has joined #openstack-nova15:45
*** pcaruana has quit IRC15:47
dansmithwow, beer in hand too15:49
mriedemsuperbowl in the dorms, 2001?15:49
*** lucasxu has joined #openstack-nova15:49
dansmithI guess that's how a SD party gets crazy, people strip down to their longjohns?15:50
mriedemstrip down?15:50
dansmithsuit up?15:50
mriedemmaintain15:50
*** sridhargaddam has quit IRC15:51
*** slaweq has quit IRC15:52
*** slaweq has joined #openstack-nova15:53
*** abalutoiu has quit IRC15:57
*** vks1 has quit IRC15:57
*** slaweq has quit IRC15:58
*** Apoorva has joined #openstack-nova15:59
*** Apoorva has quit IRC15:59
*** Apoorva has joined #openstack-nova16:00
openstackgerritEric Berglund proposed openstack/nova master: Add strict option to discover_hosts  https://review.openstack.org/46721816:00
*** nic has joined #openstack-nova16:03
*** cristicalin has joined #openstack-nova16:05
cfriesenmriedem: you ever figure out why you weren't seeing the "Refreshing intance network info cache..." logs in the live migration jobs?  (re: https://review.openstack.org/#/c/465787/2/nova/compute/manager.py)   That seems really odd.16:05
mriedemcfriesen: nope, i was going to ask you16:05
cfriesenmriedem: I assume those jobs are using neutron and have at least one vif?16:05
mriedemyes16:06
mriedemwell i know they are neutron,16:06
mriedemi assume they have a single port on the instance16:06
mriedemauto-created by nova when creating the server16:06
cfriesenis the test using get-me-a-network?  or could it be booting with no port?16:08
*** fishbone_ has quit IRC16:09
mriedemno it's not those16:09
cfriesenmriedem: on of our neutron guys says it might happen if nova didn't update the host_id, or if there was an issue with vif-unplug/vif-plug on either side.16:11
*** efried has joined #openstack-nova16:11
*** lei-zh has quit IRC16:12
*** lucasxu has quit IRC16:14
mriedemif that happened, i'd think the job would fail16:14
mriedemor we'd see errors in the logs16:14
mriedemgranted, these tests don't actually try to ssh into the guest once it's migrated to the target host16:14
*** pwiltsey has quit IRC16:15
*** kornica has quit IRC16:16
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce basic framework for security proxying  https://review.openstack.org/34539616:20
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication  https://review.openstack.org/34539716:20
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme  https://review.openstack.org/34539816:20
openstackgerritStephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation  https://review.openstack.org/34539916:20
*** lpetrut has quit IRC16:22
openstackgerritMatt Riedemann proposed openstack/nova master: doc: start a FAQs section for cells v2  https://review.openstack.org/46730916:24
mriedemkfox1111: ^16:24
mriedemi said i'd do it16:24
*** abalutoiu has joined #openstack-nova16:24
*** Sukhdev has joined #openstack-nova16:24
*** mdnadeem has quit IRC16:24
edleafejaypipes: regarding your comment on UUID passing in https://review.openstack.org/#/c/465171/, that was my original approach. However, dansmith preferred to see it as an additional RPC param16:28
jaypipesedleafe: well, based on this ML thread, we may have to rethink this all anyway...16:29
*** yamamoto has joined #openstack-nova16:30
edleafejaypipes: not sure I follow (I'm still catching up from PyCon)16:30
edleafejaypipes: We can claim in the scheduler to make sure we have at least one sufficient host16:30
edleafejaypipes: and then return that host and some 'alternates'16:31
*** salv-orlando has joined #openstack-nova16:31
edleafethat would get passed down to the cell, which could retry an alternate in the event of a local build failure16:31
*** winston-d_ has joined #openstack-nova16:32
edleafeWhere is the problem under discussion?16:32
jaypipesedleafe: I'm referring to this thread http://lists.openstack.org/pipermail/openstack-dev/2017-May/117221.html16:32
jaypipesedleafe: http://lists.openstack.org/pipermail/openstack-operators/2017-May/013464.html16:32
edleafejaypipes: yes, I've read all that thread16:33
jaypipesedleafe: I'd like to get rid of retries entirely.16:33
openstackgerritDan Smith proposed openstack/nova master: De-complicate some of the instance delete path  https://review.openstack.org/45863616:33
openstackgerritDan Smith proposed openstack/nova master: Fix live migration devstack hook for multicell environment  https://review.openstack.org/45853716:33
openstackgerritDan Smith proposed openstack/nova master: Target cell on local delete  https://review.openstack.org/46405916:33
jaypipesedleafe: but ops are crapping on the idea.16:33
openstackgerritEric Berglund proposed openstack/nova master: Add strict option to discover_hosts  https://review.openstack.org/46721816:33
*** mariusv is now known as lunarlamp16:33
edleafejaypipes: so wouldn't adding a list of alternates be an acceptable compromist?16:33
dansmithedleafe: that was the compromise16:34
dansmithedleafe: jaypipes was apparently missing that bit of context from the summit16:34
mriedemyou know, if you're providing a monthly subscription service, you'd think you'd want your website to work such that your customers can update their credit card information to actually pay you, rather than cancel their subscription, even after you've submitted a ticket and called customer service16:34
jaypipesedleafe: I would still like to do retries (from resource allocation failures due to concurrent claims) in the scheduler.16:34
edleafejaypipes: yes, that's what I've done in my series16:35
dansmithjaypipes: you shouldn't call those retries, IMHO, to avoid confusion16:35
jaypipesedleafe: I know :)16:35
jaypipesdansmith: touche16:35
edleafeso what is the problem?16:35
jaypipesedleafe: I'm still having a tantrum about not getting my way with ops.16:35
edleafedansmith: I like to call them 'claims' :)16:35
jaypipesedleafe: gimme ten minutes to come to my senses please.16:36
*** gouthamr has joined #openstack-nova16:36
edleafejaypipes: ok. Grab your blankie and open a juice pack16:36
* cdent gives jaypipes a crunchie bar16:36
jaypipesthanks guys.16:36
*** trinaths has joined #openstack-nova16:36
* jaypipes gurgles and throws toy16:36
*** mdrabe_ has quit IRC16:37
*** armax has joined #openstack-nova16:39
*** yamamoto has quit IRC16:39
openstackgerritMatt Riedemann proposed openstack/nova master: doc: add cells v2 FAQ on refreshing global cells cache  https://review.openstack.org/46731816:39
*** trinaths has quit IRC16:41
*** mvk has quit IRC16:43
*** pchavva has joined #openstack-nova16:44
*** ltomasbo is now known as ltomasbo|away16:44
*** mdrabe_ has joined #openstack-nova16:45
*** aarefiev is now known as aarefiev_afk16:45
*** ihrachys has joined #openstack-nova16:46
ihrachysanyone seen the failure? https://bugs.launchpad.net/neutron/+bug/169298416:46
openstackLaunchpad bug 1692984 in neutron "VM boot fails with "TCG doesn't support requested feature: CPUID.01H:ECX.vmx"" [High,New]16:46
mriedemihrachys: nope, libvirt/qemu fun16:46
mriedemkashyap: ^16:47
*** sree has joined #openstack-nova16:47
ihrachysI am not completely sure this is the culprit. the log suggests it's a 'warning'16:47
ihrachysso *maybe* it just spills it as part of stderr when some other failure happens.16:47
ihrachysI am too dumb to distinguish :)16:47
*** derekh has quit IRC16:48
mriedemplease standby, firefox is struggling to open logs16:49
*** lpetrut has joined #openstack-nova16:49
mriedemihrachys: it definitely killed the server build http://logs.openstack.org/83/466983/1/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/bd69e6b/logs/screen-n-cpu.txt.gz?level=TRACE#_May_23_03_04_29_68898616:50
*** dikonoor has quit IRC16:50
*** sree has quit IRC16:51
ihrachysyeah, sure. I am merely saying I am not sure if the cpu flag thing is a red herring, or the culprit of the failure16:51
jaypipescdent: in a gabbit, can I test for greater or equal to a value?16:52
*** mpaolino has quit IRC16:52
jaypipescdent: for example, $.traits.`len`: 216:52
jaypipescdent: can I change that to test for >= 2?16:53
cdentmaybe, I need to do some checking16:53
mriedemihrachys: hmm, yeah http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22process%20exited%20while%20connecting%20to%20monitor%3A%20warning%3A%20TCG%20doesn't%20support%20requested%20feature%3A%20CPUID.01H%3AECX.vmx%20%5Bbit%205%5D%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d16:53
mriedemihrachys: it's definitely not 100% failure16:53
mriedemnot even close16:53
*** thorst_afk has joined #openstack-nova16:55
ihrachysright. so I was thinking, maybe it's just spilled all the time, but we dump stderr on failures. but other log messages are very generic to my taste, and I needed to capture something in the bug, so...16:56
*** harlowja has joined #openstack-nova16:56
*** lucasagomes is now known as lucas-afk16:57
*** gianpietro has quit IRC16:57
*** thorst_afk has quit IRC16:57
cdentjaypipes: simple answer is "not really" but you could perhaps do a regex to get the results you want?: /[2-9]/16:57
*** arunman has joined #openstack-nova16:57
*** gianpietro has joined #openstack-nova16:57
openstackgerritMatt Riedemann proposed openstack/nova master: doc: add cells v2 FAQ on mapping instances  https://review.openstack.org/46732216:57
*** thorst_afk has joined #openstack-nova16:58
mriedempabelanger: dansmith: fyi, no one replied to my question about doing PUT for keypairs in the ops channel16:58
cdentjaypipes: the limitation at this point is that the RHS is that the assert is either an assertEqual or an assertRegex16:58
*** eharney has quit IRC16:59
*** baoli has quit IRC17:00
*** baoli has joined #openstack-nova17:01
*** ralonsoh has quit IRC17:01
*** fragatin_ has joined #openstack-nova17:01
*** thorst_afk has quit IRC17:02
*** ijw has joined #openstack-nova17:03
*** fragatina has quit IRC17:03
*** salv-orlando has quit IRC17:03
*** thorst_afk has joined #openstack-nova17:03
*** cristicalin has quit IRC17:04
*** Apoorva_ has joined #openstack-nova17:04
*** Apoorva has quit IRC17:08
*** cristicalin has joined #openstack-nova17:10
*** pchavva has quit IRC17:11
jaypipescdent: k, no prob. thx!17:12
*** arunman has quit IRC17:12
*** eharney has joined #openstack-nova17:15
*** baoli has quit IRC17:15
*** gjayavelu has quit IRC17:16
*** rmart04 has quit IRC17:16
*** cristicalin has quit IRC17:17
*** jaosorior is now known as jaosorior_away17:17
*** slaweq has joined #openstack-nova17:17
*** salv-orlando has joined #openstack-nova17:19
*** kfarr has quit IRC17:19
*** baoli has joined #openstack-nova17:19
*** vladikr has quit IRC17:20
jaypipesmriedem, smcginnis: if it makes you feel any better (doubtful), k8s is dealing with similar feature requests and inconsistent UX around volume types. There, it's called dynamicStorageClass but the concept is virtually identical17:21
* jaypipes goes back into hole17:22
cfriesenjaypipes: got a sched race for you to consider for placement/claims:  2 vms in server group with anti-affinity, both vms get migrated at same time, scheduler picks same host for both, both end up on same host.17:22
*** thorst_afk has quit IRC17:23
jaypipescfriesen: two trains leave New York and San Francisco, each going 60mph...17:23
cfriesenjaypipes: make that live-migrated. ^17:23
*** salv-orlando has quit IRC17:23
*** thorst_afk has joined #openstack-nova17:23
cfriesennot sure how to fix this without either atomic schedule/claim or secondary check on compute and possible reschedule17:24
cfriesen(I guess secondary check could be anywhere post-claim)17:24
*** itlinux has joined #openstack-nova17:24
jaypipescfriesen: we can dramatically decrease the length of time that such a decision can be made with claims in the scheduler...17:25
cfriesenjaypipes: agreed,17:26
cfriesenbut it might still make sense to allow reschedule within the cell17:27
*** thorst_afk has quit IRC17:27
*** thorst_afk has joined #openstack-nova17:29
*** thorst_afk has quit IRC17:29
*** jpena is now known as jpena|off17:30
jaypipescfriesen: yes, perhaps.17:30
jaypipescfriesen: trying to avoid that though if possible. (doesn't look like we'll be able to, though)17:31
*** Apoorva_ has quit IRC17:33
pabelangermriedem: dansmith: Ya, for now we are leaning towards updating ansible os_keypair module to support the update functionality17:33
*** claudiub has quit IRC17:33
pabelangerthen delete / create the key with same name17:34
*** claudiub has joined #openstack-nova17:34
*** dimtruck has quit IRC17:36
*** hferenc has quit IRC17:37
*** hferenc has joined #openstack-nova17:37
*** Anticimex has quit IRC17:38
*** dimtruck has joined #openstack-nova17:38
*** Anticimex has joined #openstack-nova17:39
cfriesenjaypipes: something to consider in the above scenario...if we have an anti-affinity instance migrating from host1 to host2, we probably want to exclude *both* hosts from consideration when scheduling other instances in the same group.17:40
*** gjayavelu has joined #openstack-nova17:41
jaypipescfriesen: sure, no disagreement.17:42
*** lucasxu has joined #openstack-nova17:44
*** slaweq has quit IRC17:46
*** slaweq has joined #openstack-nova17:46
*** thorst_afk has joined #openstack-nova17:47
*** thorst_afk has quit IRC17:47
*** lbragstad has joined #openstack-nova17:49
*** gszasz has quit IRC17:50
*** slaweq has quit IRC17:51
*** salv-orlando has joined #openstack-nova17:51
*** hongbin has joined #openstack-nova17:55
*** Apoorva has joined #openstack-nova17:56
*** brault has quit IRC17:57
*** brault has joined #openstack-nova17:58
*** brault has quit IRC17:58
*** brault has joined #openstack-nova17:58
*** brault has quit IRC17:59
*** thorst_afk has joined #openstack-nova17:59
*** baoli has quit IRC17:59
*** brault has joined #openstack-nova17:59
*** baoli has joined #openstack-nova18:00
ildikovlyarwood: sfinucan: hi18:00
ildikovlyarwood: sfinucan: if you have some time there're two patches related to the new Cinder attach/detach flow with one +2 already: https://review.openstack.org/#/c/459111/ https://review.openstack.org/#/c/459113/18:00
ildikovlyarwood: sfinucan: tnx :)18:00
*** itlinux has quit IRC18:02
*** thorst_afk has quit IRC18:03
*** brault has quit IRC18:03
*** lpetrut has quit IRC18:05
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/46734418:06
efriedjaypipes johnthetubaguy mordred Would y'all please have a look at this spec update?  Been +2 for a few weeks.  https://review.openstack.org/#/c/461481/18:10
* jaypipes readies red pen.18:10
* efried readies Adamantium Claymore of Typo Correction +1018:11
efriedBut the pen IS mightier than the sword, so...18:12
jaypipes:)18:12
*** itlinux has joined #openstack-nova18:13
jaypipesefried: done.18:14
efriedjaypipes Thanks dude.18:14
efriedjaypipes Still on my list to ask you for that pointer to the shared resource pool code.  Is that a quick answer?18:16
jaypipesefried: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/shared-resources-pike18:17
*** rmart04 has joined #openstack-nova18:17
*** hieulq_ has quit IRC18:19
*** fragatin_ has quit IRC18:19
efriedjaypipes Okay, looking through.  I know you've slept (in multiple time zones) since then, so to refresh your memory on the conversation: I'm looking to get my compute driver itself to do the registration of its shared storage pool.18:20
jaypipesefried: ah. yeah, sorry.. one sec18:21
jaypipesefried: https://review.openstack.org/#/c/457782/18:21
efriedjaypipes As impressive as your SQL construction is...  ;-)18:21
jaypipesefried: that's more what you're looking for I think?18:21
efried...18:21
jaypipesspeaking of which... dansmith, if you or bauzas have a chance to review https://review.openstack.org/#/c/457782/, that would be great18:22
efriedhmph, I've been watching this change set; but clearly didn't realize what I was looking at...18:22
dansmithoh that fell off, I'll look18:22
*** rmart04 has quit IRC18:22
*** rmart04 has joined #openstack-nova18:23
*** chyka has quit IRC18:24
*** chyka has joined #openstack-nova18:25
*** felipemonteiro has joined #openstack-nova18:27
efriedjaypipes I don't think this is quite what I'm looking for.  I do think I'd like to do it from get_inventory(); but what I'm looking to do involves talking to the placement API and saying something like, "Do ya got a shared storage pool with UUID <uuid>?  No: Okay, here it is.  ==> And this compute host is part of it."18:28
*** felipemonteiro_ has joined #openstack-nova18:28
*** slaweq has joined #openstack-nova18:28
jaypipesefried: yeah, you'll have to wait a bit for a few more patches in that shared-resources-pike series to get to that18:29
efriedjaypipes Roger that.  If it helps, the words I wrote down while we were chatting were: "scheduler client report"18:29
*** baoli has quit IRC18:30
*** vladikr has joined #openstack-nova18:31
*** vladikr has quit IRC18:31
*** baoli has joined #openstack-nova18:32
*** vladikr has joined #openstack-nova18:32
*** ijw has quit IRC18:32
*** slaweq has quit IRC18:32
*** mingyu has quit IRC18:33
*** felipemonteiro has quit IRC18:33
*** mingyu has joined #openstack-nova18:33
*** annegentle has joined #openstack-nova18:33
*** itlinux has quit IRC18:34
*** gjayavelu has quit IRC18:35
*** krtaylor has quit IRC18:35
*** abramley has left #openstack-nova18:37
*** mingyu has quit IRC18:39
*** itlinux has joined #openstack-nova18:42
*** thorst_afk has joined #openstack-nova18:43
mordredmriedem: in my ongoing effort to report 'fun' api interactions ...18:44
*** Sukhdev has quit IRC18:44
mordredefried: (I just opened that link, will look in a sec)18:44
cfriesenweird question....if we start with nova version N, and upgrade everything to N+1, am I correct that Service._check_minimum_version() would prevent us from downgrading even if the RPC is explicitly still set to something suitable for version N?18:45
jaypipesefried: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py18:45
mordredmriedem: today I learned that GET /os-simple-tenant-usage?start=%s&end=%s&detailed=%s ... which takes an ISO 8601 date string as input for start and ed18:45
*** ihrachys is now known as ihrachys|afk18:46
mordredmriedem: fails if your ISO 8601 date string includes a timezone offset18:46
*** salv-orlando has quit IRC18:46
jaypipesefried: and an example of usage from the scheduler: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L17018:46
mriedemmordred: yeah https://developer.openstack.org/api-ref/compute/?expanded=list-tenant-usage-statistics-for-all-tenants-detail#id36518:46
mordredmriedem: so it is incumbent on a consumer to take their ISO 8601 date string, apply the timezone offset expressed in the string and then submit the string without the timezone offset18:47
jaypipesefried: and where the compute resource tracker calls it: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L759-L76318:47
mordredmriedem: indeed - so it is documented, which is nice :)18:47
*** thorst_afk has quit IRC18:48
mordredmriedem: however, also, python-novaclient does this:18:48
mordred        return self._list(18:48
mordred            "/os-simple-tenant-usage?start=%s&end=%s&detailed=%s" %18:48
mordred            (start.isoformat(), end.isoformat(), int(bool(detailed))),18:48
mordred            "tenant_usages")18:48
*** gianpietro has quit IRC18:49
mordredso if the datetime passed to python-novaclient is a datetime that has a tzinfo attached, sadness occurs - because it isn't actually isoformat that is accepted18:49
mordred(this is what results from starting by looking at the novaclient code instead of the docs for how to work an API, btw)18:49
*** thorst_afk has joined #openstack-nova18:50
openstackgerritDan Smith proposed openstack/nova master: re-Allow adding computes with no ComputeNodes to aggregates  https://review.openstack.org/46061618:50
dansmithmriedem: rebased ^18:51
mriedemmordred: it also does this thing https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/usage.py#L3218:51
mriedemwhich is related to this thing in the api https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/simple_tenant_usage.py#L22818:51
mordredfascinating18:52
mordredmriedem: I'm truly curious as to the reasoning behind defining our own format that isn't the ISO standard but close to it - but it's not super important ...  I mostly just learned a new thing today and want to check in and see if it was bug or feature18:53
mriedemchanging it would be a microversion18:53
mriedemwhy it's that way, you'd have to ask vish or someone18:54
sdagueor mordred or jaypipes18:54
mordredI don't think it needs to change -since it's clearly documented and has obviously been implemented on purpose -- I've got the conversion call written on my side now18:54
mriedemyeah no sh*t, *those* guys18:54
mordredjaypipes: oh - you're here ... can you tl;dr me? ^^18:55
*** JoseMello has quit IRC18:55
jaypipesmordred: ya, me here.18:55
mordredmriedem: would I be safe in assuming that anywhere nova takes a date it's gonna be in that format and not full ISO?18:55
mriedemi don't know where else we take a date18:55
*** salv-orlando has joined #openstack-nova18:56
mriedemthe compute api ref has a standard description for fields in the response that are a datetime18:56
mriedeme.g. https://github.com/openstack/nova/blob/master/api-ref/source/parameters.yaml#L164818:57
jaypipessdague: I have no part in the making of os-simple-tenant-usage...18:57
sdaguemordred: the best recollections I've got are there are at least 3 different time formats because people picked things oddly (all predates me)18:58
sdagueI think the oslo utils actually take any of them and digest into the same thing18:59
sdaguethat was an early markmc push from when I first showed up18:59
* edleafe thinks it might be a holdover from the original RAX cloud API19:00
*** itlinux has quit IRC19:01
*** salv-orlando has quit IRC19:02
mordrededleafe: ah - that probably makes sense19:02
*** lucasxu has quit IRC19:02
mordredsdague: nod. hrm - I should ping markmc and ask him what he remembers19:03
mordredI mostly found it weird since we ALSO use the iso8601 library in OpenStack ... so doing work to support a subset of ISO 8601 just seems weird19:03
*** Apoorva has quit IRC19:04
sdaguewhen did iso8601 get added?19:04
sdagueyou have to realize, a lot of this code is 2012 era stuff19:04
*** Apoorva has joined #openstack-nova19:04
*** mvk has joined #openstack-nova19:06
*** thorst_afk has quit IRC19:06
*** adisky_ has quit IRC19:07
*** slaweq has joined #openstack-nova19:08
sdaguemordred: simple tenant usage is 7726b3d763a136347f2324e630f0a3cdc60a045b (Aug 2011)19:08
sdagueiso8601 is 7726b3d763a136347f2324e630f0a3cdc60a045b (Mar 2011)19:08
sdagueit just looks like simple tenant usage was written to python date format and straight to db19:09
mordredsdague: man - 2011 was a long time ago19:09
sdagueyep19:09
sdaguegit log -S for the win19:10
mordred++19:10
*** salv-orlando has joined #openstack-nova19:10
mriedemefried: did you see that nova-specs docs on longer build?19:10
mriedem/home/user/git/nova-specs/doc/source/specs/juno/implemented/pci-passthrough-sriov.rst:412:Citation [VIF_DETA] is not referenced.19:10
*** sree has joined #openstack-nova19:11
mordredsdague: fwiw - today I added a default value to shade for "start" of 2010-07-06 - since it's impossible for an openstack cloud to have usage going back before that :)19:11
*** salv-orlando has quit IRC19:11
*** rmart04 has quit IRC19:12
mriedem*no longer19:12
efriedmriedem "nova-specs docs on longer build"?19:13
efriedoh...19:13
mriedemhttp://logs.openstack.org/81/461481/3/gate/gate-nova-specs-docs-ubuntu-xenial/9e65575/console.html#_2017-05-23_18_19_21_05604719:13
mriedemand local19:13
mriedemstarted in the last 24 hours19:14
mriedemthe citations in that file all look ok to me19:15
efriedjaypipes Thanks, will peruse.19:15
*** cristicalin has joined #openstack-nova19:15
efriedmriedem Sorry, what does "nova-specs docs no longer build" mean?19:15
jaypipesefried: peruse. persue. do with it what you may :)19:15
*** sree has quit IRC19:15
*** lucasxu has joined #openstack-nova19:15
mriedemefried: you asked jaypipes to review your nova-specs patch, right?19:16
mriedemhe approved it,19:16
mriedemit failed in the gate,19:16
*** xyang1 has joined #openstack-nova19:16
mriedemmeaning the docs job for nova-specs is broken19:16
sdaguemordred: nice19:16
efriedmriedem Oh.19:16
mriedemlooking at logstash, something changed in the last 24 hours19:16
mriedemmy guess is some upstream library19:16
efriedNot by design19:16
efriedor because I'm a bad, bad person.19:16
mriedemi recreated locally19:16
mriedemi'm checking to see which packages changed in the last 24 hours19:16
mriedembut it's nearly nap:30 here and i'm having a hard time caring19:17
*** thorst_afk has joined #openstack-nova19:17
*** crushil has quit IRC19:20
*** thorst_afk has quit IRC19:22
mriedemi don't see any recent package releases that look related19:23
*** READ10 has quit IRC19:24
*** slaweq has quit IRC19:24
*** slaweq has joined #openstack-nova19:25
*** jmlowe has quit IRC19:25
mriedemefried: it's sphinx 1.6.119:26
efriedmriedem Cool, guess nova-specs doesn't get auto-updates to requirements from g-r.19:27
mriedemnova-specs doesn't use upper-constraints19:27
efriedmriedem Shall I propose that?19:27
mriedemno19:27
efried(the !1.6.1, that is, not adding the bot)19:27
mriedemwell, it's on purpose for some reason19:27
mriedemi've go the thing local19:28
mriedemjust writing the bug19:28
*** krtaylor has joined #openstack-nova19:28
cdentedleafe: quick! rewrite placement to use etcd while I feed jaypipes crunchie bars19:29
mriedemi don't really know why we don't use upper-constraints for building nova-specs docs really19:29
jaypipesmelwitt: tell James I'm going to hire a choir of angels to sing to him. :)19:29
*** jmlowe has joined #openstack-nova19:29
sdaguemriedem: it just never got there19:30
sdaguethe assumption I think is it was just fast enough to locally fix19:30
melwittjaypipes: lol :)19:30
mriedemsdague: i don't even know that the docs jobs would use it anyway19:30
mriedemsince they don't do tox -e docs19:30
mriedemi think they do build_sphinx directly right?19:31
jaypipescdent: nom nom!19:31
sdaguemriedem: yeh, good point, I don't know19:31
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Blacklist sphinx 1.6.1  https://review.openstack.org/46736719:35
openstackgerritchristopher.uhler proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale  https://review.openstack.org/44395119:38
*** priteau has quit IRC19:39
*** mingyu has joined #openstack-nova19:39
* melwitt needs to find out what crunchie bars are19:40
mriedemUK things19:40
mriedemprobably have toffee in them19:40
*** gianpietro has joined #openstack-nova19:41
*** crushil has joined #openstack-nova19:41
melwittheh19:41
*** penick has joined #openstack-nova19:43
*** mvk has quit IRC19:43
*** mingyu has quit IRC19:44
penick@jaypipes congrats on the fiber rich bug smash19:44
mriedemdansmith: looks like you dropped something in the rebase https://review.openstack.org/#/c/460616/519:45
*** eharney has quit IRC19:46
openstackgerritLucian Petrut proposed openstack/nova master: [wip] Hyper-V: accept Glance vhdx images  https://review.openstack.org/46736919:46
dansmithmriedem: hmm, not sure how that happened, but okay19:46
mriedemalso,19:47
mriedem_find_service was removed in ps4 but not ps519:47
*** clenimar has quit IRC19:47
*** tbachman_ has joined #openstack-nova19:47
dansmithoh, I see, you pushed to my set19:47
dansmithI didn't realize, sorry19:47
*** tbachman has quit IRC19:48
*** itlinux has joined #openstack-nova19:48
*** tbachman has joined #openstack-nova19:48
*** tbachman_ has quit IRC19:49
*** thorst_afk has joined #openstack-nova19:50
openstackgerritDan Smith proposed openstack/nova master: re-Allow adding computes with no ComputeNodes to aggregates  https://review.openstack.org/46061619:51
*** mdrabe_ is now known as mdrabe19:51
dansmithmriedem: how about that?19:51
cdentpenick: you get a gold star19:51
penick;)19:52
mriedemgod damn we should restrict updating an instance until it's active19:53
edleafepenick: was going to respond to the list, but you used all the good puns19:53
melwittmriedem: I thought we did that already. or was that only tags19:54
edleafepenick: couldn't think of a clever way to say that crap doesn't stick to porcelain, but sure seems to stick to nova19:54
*** tbachman_ has joined #openstack-nova19:54
mriedemmelwitt: tags19:54
mriedemlook at this fing mess https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2732-L278219:54
* melwitt kicks dirt19:54
dansmithmriedem: did you mean to remove the line range when referring to the thing that is a fing mess?19:54
melwitthah that19:55
mriedemline range is in there19:55
dansmithmriedem: what I meant was, you could say "just look at this fing mess" with this link and be quite valid:  https://github.com/openstack/nova/blob/master/nova/compute/api.py19:55
mriedemoh19:56
mriedemwell,19:56
*** crushil has quit IRC19:56
penickedleafe: so you're saying that prior to being the porcelain of choice, Nova would require vigorous scrubbing?19:56
mriedemi know some volume type in there would help19:56
*** tbachman has quit IRC19:56
*** tbachman_ is now known as tbachman19:56
penicks/Nova/Nova's codebase/19:56
dansmithmriedem: that'd class it up real nice19:56
mriedemi wonder if we can delete any of that now19:56
mriedemit refers to the before times when we still created the instance in the api19:56
*** crushil has joined #openstack-nova19:56
*** ijw has joined #openstack-nova19:57
mriedempenick: while you're here,19:57
mriedemdo your users use cinder?19:57
mriedemwell, do your users boot from volume?19:57
penickThey do not, yet. I've paused that project. However we're starting it back up soon.19:59
mriedemthat project - a project which has cinder?19:59
penickWe've done the work to migrate instances with local disk to boot from volume, and all of our developer productivity environments will switch to using boot from volume as a default19:59
mriedempenick: ok, well when your users complain that they have to create a volume with a non-default type in cinder and then provide that volume to nova, rather than just do it all through nova first, let us konw20:00
penick"that project" = "a project to move our clusters to boot from volume". Sorry, left out context there20:00
mriedem*know20:00
penickUhm. I'll complain right now. That would suck.20:00
*** eharney has joined #openstack-nova20:01
smcginnispenick: Them asking for that? Or not being able to do it?20:01
mriedempenick: will you have multiple cinder backends?20:01
penickFor developer productivity I only intend to support boot from volume and local disk20:02
dansmithwait, what?20:02
dansmithpenick: you want to move _to_ BFV?20:02
dansmithmay I go ahead and wish luck to your therapist?20:03
melwittpenick: note that boot from volume is different than booting instances with ephemeral storage on cinder20:03
*** vladikr has quit IRC20:03
smcginnismelwitt: +120:04
penickwell, I intend to boot instances to having disks which are situated on network storage20:04
penickLet me try that again, in english this time20:05
dansmithceph ephemeral is not equivalent to BFV if that's what you mean20:05
mriedemso use the ceph image backend in nova, do you need volumes?20:05
penickUgh ceph.20:05
melwittyeah, you might be thinking of backing instance ephemeral storage on cinder (ceph?). boot from volume is creating a volume in cinder from an image and attaching it to an instance, and it (the volume) can live on after the instance is deleted20:05
mriedemfine fine20:05
mriedemuse scaleio!20:05
mriedemmelwitt: or not, delete_on_termination ftw20:06
dansmith8GB swap for all!20:06
penickSo Nova would interact with something like scaleio directly, and not go through cinder?20:06
melwittmriedem: yep, can do that too20:06
dansmithpenick: this is the thing we were talking about -- nova duplicates a fraction of cinder for the drivers we like20:07
dansmithpenick: so you can back your instances on ceph, or local, but not much else20:07
dansmithbecause we like writing things twice and maintaining them once20:07
cdentcan I quote you on that dansmith? that's very quotable20:07
penickBut..why not let Cinder do the word of managing volumes?20:07
dansmithcdent: you may20:08
dansmithpenick: yeah, that20:08
dansmithhas been my suggestion,20:08
dansmithbut it's a ton of work20:08
dansmithand openstack being already dead according to the press, we don't have enough time to finish that before we're done20:08
*** sambetts is now known as sambetts|afk20:08
penickPfft. Dead. Hah20:08
dansmithregardless, I think we all agree that that's how it should work20:08
dansmithbut it doesn't today20:09
dansmithunless you BFV everything, which is also broken for lots of things20:09
dansmithand, nova devs will stop talking to you if you move to that20:09
penickSo if I add a feature my users need, in the only way that it works right now, you'll kick me out of nova hogworts?20:10
mriedempenick: i'm much more interested in your doing cells v2 from trunk than doing BFV20:10
*** dave-mccowan has quit IRC20:10
dansmithpenick: don't worry, you'll hate yourself soon after adopting it anyway20:10
*** boris-42_ has joined #openstack-nova20:11
penickmriedem: i'm much more interested in cellsv2 as well. But to quote a great poet "If you don't eat your meat, you can't have any pudding"20:11
penickNot sure who said that. Might have been Socrates.20:12
dansmithpretty sure it was Waters :P20:12
mriedemdude,20:12
penick:P20:12
mriedemyou don't want to eat boiled meat anyway right?20:12
mriedemwhere is mdbooth when i need to joke about terrible british meat20:12
penickCorrect, but if I picked the things I -wanted- to do, over what the business needs, i'd have a really kickass 10 node openstack cluster running on trunk. But I wouldn't be moving the whole company over to it20:13
*** lucasxu has quit IRC20:17
*** lucasxu has joined #openstack-nova20:21
mriedemwell riddle me this, where is accessIPv4 actually used anywhere in nova anymore?!20:23
*** rcernin has joined #openstack-nova20:24
mriedemi need an onboarding session20:24
cburgessmriedem Thats a dangerous thing to say to me. :P20:24
mriedemcburgess: whatever i said, it was probably a day ago,20:25
mriedemso remind me20:25
cburgessmriedem Getting enough cloud providers to ask for volume type.20:25
mriedemoh right, page 4520:25
cburgesspage 45?20:25
*** cleong has quit IRC20:25
*** gianpietro has quit IRC20:25
mriedempage 45 of the nova core dungeon master's guide states: "once you get enough people asking for the same feature, regardless of policy, it must be accepted"20:25
cburgesslol20:26
penickPretty sure you just made that up20:26
cburgessI didn't know such a guide exists. Must be for PTLs only.20:26
mriedem"nova core dungeon master's guide"20:26
cburgessOh core20:26
cburgessMy bad..20:26
*** crushil has quit IRC20:26
mriedempenick: melwitt's is keeping your dining table steady atm20:26
dansmithI haven't seen this guide20:26
cburgessmriedem You do realize that has to become a think now right?20:26
penickWe have a dining table?20:27
cburgessmelwitt is doing what now?20:27
dansmithisn't that melwitt's workstation now?20:27
melwittwhat is happening20:27
mriedemi know what's not happening20:27
*** crushil has joined #openstack-nova20:27
mriedem... https://review.openstack.org/#/c/446239/20:27
mriedem:)20:27
melwittit's happening, at the subaru dealership even!20:28
mriedemoh snap20:28
*** crushil has quit IRC20:28
*** gianpietro has joined #openstack-nova20:28
melwittabout to upload the quotas bomb again20:28
penickher workstation locality depends on three factors. The season, is the cat being annoying, or is she at the dealership20:28
mriedempage 62 has a subaru coupon coincidentally20:28
mriedemon free granola bar20:28
mriedem*one20:28
penickfree granola bar, that's a sweet deal20:28
*** slaweq has quit IRC20:28
melwittI ate a free Cliff bar here earlier20:28
mriedemhaha20:28
*** slaweq has joined #openstack-nova20:29
mriedemi always feel great when some joke i make which might be offensive turns out to be actual fact20:29
*** smatzek has quit IRC20:29
melwittlol, right?20:29
penickman that nova dungeon masters guide is handy20:29
mriedemwhy do you think we don't add more cores?20:29
mriedemlimited supply20:30
mriedemof guides20:30
*** McNinja has quit IRC20:31
*** McNinja has joined #openstack-nova20:32
penickNot cheap to print. also the "Worlds best nova core" mugs don't exactly grow on trees. Those legbreakers at Cafepress charge a fortune20:32
melwittmerge conflicts in local delete code ... grrrr20:32
mriedemmelwitt: blame dansmith20:32
*** Apoorva_ has joined #openstack-nova20:32
melwitt>:| angry eyebrows20:32
*** harlowja_ has joined #openstack-nova20:32
dansmithgo ahead20:33
dansmithI'll flip this table and leave20:33
*** harlowja has quit IRC20:33
melwittthe workstation table might be hard to flip. but could be toppled over20:34
penickCool cores don't watch tables flip. They just walk away. (-_- )ノ⌒┫ ┻ ┣20:34
dansmithwell, I'm not going to flip _this_ table since it has all my monitors and laptop on, but I will go find a suitable table with minimal collateral damage and flip that mofo20:35
melwitt:D20:35
melwittwe're all on the same page here20:36
*** Apoorva has quit IRC20:36
cburgessdansmith pics or it didn't happen20:36
mriedemyou know what i think we can all agree on?20:36
mriedemremoving the gluster volume driver https://review.openstack.org/#/c/463992/20:36
cburgess+1 LGTM20:36
dansmithheh20:37
*** jose-phillips has quit IRC20:39
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup update_instance cell mapping handling  https://review.openstack.org/46738020:39
mriedemdansmith: i'm also hoping you don't find problems with ^20:39
mriedemi think that stuff all predates _get_instance storing the targeted cell context on the instance object20:40
dansmithmriedem: can we wait and or depends-on that on the multicell devstack patch?20:40
mriedemi can put a change on top with the depends-on to test it out20:40
dansmithokay20:41
dansmithor just dependson it and remove it if it's good20:41
mriedemhttps://review.openstack.org/#/c/436094/ right?20:41
dansmithyar20:41
mriedemi'm going to do another one on top of this for something else in that method20:41
dansmithokay20:41
*** jose-phillips has joined #openstack-nova20:41
mriedemwe have some funny business toward the end20:41
efriedmriedem Thanks for the Sphinx thing; that spec change merged.20:42
mriedem\o/20:42
*** yamahata has quit IRC20:45
*** acormier has quit IRC20:49
*** gjayavelu has joined #openstack-nova20:50
*** annegentle has quit IRC20:50
*** tbachman has quit IRC20:51
*** acormier has joined #openstack-nova20:51
*** fragatina has joined #openstack-nova20:52
*** jmlowe has quit IRC20:53
*** slaweq has quit IRC20:54
*** david-lyle has joined #openstack-nova20:54
*** slaweq has joined #openstack-nova20:55
*** thorst_afk has quit IRC20:56
*** yamahata has joined #openstack-nova20:56
*** fragatina has quit IRC20:57
openstackgerritMatt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance  https://review.openstack.org/46738220:59
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells  https://review.openstack.org/46738320:59
*** itlinux has quit IRC21:00
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells  https://review.openstack.org/46738321:00
openstackgerritMatt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance  https://review.openstack.org/46738221:00
*** thorst_afk has joined #openstack-nova21:02
*** lucasxu has quit IRC21:03
*** fragatina has joined #openstack-nova21:03
*** harlowja_ has quit IRC21:05
mriedemefried: heh https://review.openstack.org/#/c/465135/21:05
mriedemthis was a known issue awhile ago21:05
*** gouthamr has quit IRC21:05
*** fragatina has quit IRC21:06
*** Apoorva_ has quit IRC21:06
efriedmriedem Yes, I'm pretty sure I also proposed blocking 1.6.1 last week in some other project that didn't get updated by the bot.21:06
*** gjayavelu has quit IRC21:06
*** thorst_afk has quit IRC21:06
*** fragatina has joined #openstack-nova21:07
*** Apoorva has joined #openstack-nova21:07
efriedmriedem keystone-specs, it was: https://review.openstack.org/#/c/465543/21:07
efriedmriedem I believe it was to get https://review.openstack.org/#/c/460344/ to pass.21:08
*** jamesdenton has quit IRC21:08
*** harlowja has joined #openstack-nova21:08
*** jmlowe has joined #openstack-nova21:10
*** david-lyle has quit IRC21:10
*** armax has quit IRC21:14
*** annegentle has joined #openstack-nova21:15
*** eharney has quit IRC21:16
*** thorst_afk has joined #openstack-nova21:20
*** david-lyle has joined #openstack-nova21:22
-openstackstatus- NOTICE: The logserver has filled up, so jobs are currently aborting with POST_FAILURE results; remediation is underway.21:22
*** ChanServ changes topic to "The logserver has filled up, so jobs are currently aborting with POST_FAILURE results; remediation is underway."21:22
*** acormier has quit IRC21:26
*** slaweq has quit IRC21:30
*** slaweq has joined #openstack-nova21:30
*** tbachman has joined #openstack-nova21:31
*** jwcroppe has quit IRC21:32
*** dimtruck is now known as zz_dimtruck21:33
*** catintheroof has quit IRC21:33
*** slaweq has quit IRC21:35
*** gjayavelu has joined #openstack-nova21:35
*** tbachman has quit IRC21:35
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service  https://review.openstack.org/46151921:36
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get  https://review.openstack.org/46153221:36
openstackgerritMatt Riedemann proposed openstack/nova master: Use uuid for id in Services API  https://review.openstack.org/46428021:36
*** imacdonn has joined #openstack-nova21:39
*** thorst_afk has quit IRC21:40
*** priteau has joined #openstack-nova21:40
*** gjayavelu has quit IRC21:40
*** krtaylor has quit IRC21:41
*** tbachman has joined #openstack-nova21:41
melwittthe target_cell behavior change tripped me up, sigh21:42
melwittI wish I had realized it much earlier21:42
mriedemthat's why i had to rebase ^21:44
dansmithyou're welcome21:44
*** brault has joined #openstack-nova21:44
melwittI totes forgot I used target_cell in my patches. go me21:45
*** priteau has quit IRC21:45
melwittI was like, why are my counts totally whack?21:46
*** MVenesio has quit IRC21:46
dansmithbecause you forgot to use target_cel and were in 1998?21:47
*** jroll has quit IRC21:47
mriedemi'm assuming pants were backward21:47
mriedem*weren't21:47
*** acormier has joined #openstack-nova21:47
*** jroll has joined #openstack-nova21:47
melwittdansmith: does target_cell magically make objects queried from the db under it have self._context set to the cell db?21:47
*** edmondsw has quit IRC21:48
dansmithmelwitt: if you use the yielded context to initialize them then yes21:48
melwittlike with target_cell(ctxt, mapping): instance = Instance.get()21:48
*** edmondsw has joined #openstack-nova21:48
*** jroll has quit IRC21:49
dansmithtarget_cell(ctxt, mapping) as cctxt21:49
dansmithinstance = Instance.get_by_uuid(cctx)21:49
dansmithwill work21:49
*** brault has quit IRC21:49
*** xyang1 has quit IRC21:49
melwittdansmith: like this, the instances are queried inside of group.count_members_by_user() https://review.openstack.org/#/c/457859/7/nova/quota.py@189021:50
melwittand I'm not sure if the instances will have the right context in that case?21:50
*** yamahata has quit IRC21:50
*** salv-orlando has joined #openstack-nova21:50
dansmithmelwitt: no, see that's the problem.. group already has a context, right?21:51
dansmiththe target won't do anything it that context anymore21:51
melwittokay, I see21:52
dansmithmelwitt: not sure that code would actually work before actually21:52
dansmithwith multiple cells21:52
*** mdrabe has quit IRC21:53
melwittyeah, I don't think so, now that I realize what's going on here21:53
*** edmondsw has quit IRC21:53
dansmithbecause it'll start several greenthreads, but by the time they run they'll all be back to untargeted I think21:53
dansmithunless you copied it first, but that's what you get with the yield now anyway21:53
*** tbachman has quit IRC21:53
*** zz_dimtruck is now known as dimtruck21:53
*** jroll has joined #openstack-nova21:53
penickmriedem did you just make a kris kross reference?21:54
mriedempenick: you need to be around more,21:54
mriedemthose get made nearly weekly21:54
melwittyep, that21:54
penickI felt like I had to say something, so you'd know at least someone got it21:55
cdentdon't encourage him21:55
melwittwe all know kris kross21:55
melwittdansmith: I think things worked by chance bc the already targeted (via set_target or something) got the instance_group from the right cell earlier in the request21:56
penickAh ok. I wont jump (jump) next time I see it then21:56
dansmithmelwitt: sounds right21:57
*** gianpietro has quit IRC21:57
*** esberglu has quit IRC21:59
*** jose-phillips has quit IRC21:59
*** efried has quit IRC21:59
*** thorst_afk has joined #openstack-nova21:59
*** esberglu has joined #openstack-nova22:00
*** david-lyle has quit IRC22:01
mriedemefried is gone?22:01
melwittinstance create does a get for the requested server group to check the number of members quota. it currently assumes the group is in the same cell as the instance but I think that might not be always true. ugh22:01
*** gianpietro has joined #openstack-nova22:03
*** awaugama has quit IRC22:04
*** acormier has quit IRC22:04
*** esberglu has quit IRC22:04
*** burt has quit IRC22:06
melwittI guess that's out of scope for my patches though22:09
*** cdent has quit IRC22:09
*** kylek3h has quit IRC22:10
*** baoli has quit IRC22:13
*** esberglu has joined #openstack-nova22:13
*** jose-phillips has joined #openstack-nova22:14
*** armax has joined #openstack-nova22:16
*** esberglu has quit IRC22:18
*** acormier has joined #openstack-nova22:19
*** gouthamr has joined #openstack-nova22:21
*** felipemonteiro_ has quit IRC22:22
*** acormier has quit IRC22:24
*** sdague has quit IRC22:24
*** karimb has quit IRC22:28
*** penick has quit IRC22:28
*** thorst_afk has quit IRC22:29
*** abalutoiu has quit IRC22:33
*** karlamrhein has quit IRC22:34
*** NobodyCam has quit IRC22:35
*** NobodyCam has joined #openstack-nova22:35
*** lyan has quit IRC22:35
*** baoli has joined #openstack-nova22:36
*** karlamrhein has joined #openstack-nova22:36
*** marst_ has quit IRC22:38
*** baoli has quit IRC22:41
*** dave-mccowan has joined #openstack-nova22:44
*** salv-orlando has quit IRC22:48
*** chyka has quit IRC22:52
*** chyka has joined #openstack-nova22:53
*** yamahata has joined #openstack-nova22:54
*** Swami has joined #openstack-nova22:55
*** annegentle has quit IRC22:58
*** edmondsw has joined #openstack-nova22:59
*** cristicalin has quit IRC23:02
*** edmondsw has quit IRC23:04
*** esberglu has joined #openstack-nova23:07
*** esberglu has quit IRC23:07
*** jose-phillips has quit IRC23:08
*** jose-phillips has joined #openstack-nova23:09
*** slaweq has joined #openstack-nova23:09
*** slaweq has quit IRC23:15
*** tristanC has quit IRC23:17
openstackgerritmelanie witt proposed openstack/nova master: Add FixedIP.get_count_by_project()  https://review.openstack.org/44624623:19
openstackgerritmelanie witt proposed openstack/nova master: Add FloatingIP.get_count_by_project()  https://review.openstack.org/44624723:19
openstackgerritmelanie witt proposed openstack/nova master: Add get_count_by_vm_state() to Instance object  https://review.openstack.org/44624423:20
openstackgerritmelanie witt proposed openstack/nova master: Add SecurityGroup.get_counts()  https://review.openstack.org/44624523:20
openstackgerritmelanie witt proposed openstack/nova master: Remove 'reserved' count from used limits  https://review.openstack.org/44624223:20
openstackgerritmelanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage  https://review.openstack.org/44624323:20
openstackgerritmelanie witt proposed openstack/nova master: Count server groups to check quota  https://review.openstack.org/44624023:20
openstackgerritmelanie witt proposed openstack/nova master: Count networks to check quota  https://review.openstack.org/44624123:20
openstackgerritmelanie witt proposed openstack/nova master: Count instances to check quota  https://review.openstack.org/41652123:20
openstackgerritmelanie witt proposed openstack/nova master: Add InstanceGroup.get_counts()  https://review.openstack.org/45785723:20
openstackgerritmelanie witt proposed openstack/nova master: Add check_deltas() and limit_check_project_and_user() to Quotas  https://review.openstack.org/44623923:20
openstackgerritmelanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db  https://review.openstack.org/45785823:20
openstackgerritmelanie witt proposed openstack/nova master: Count server group members to check quota  https://review.openstack.org/45785923:20
openstackgerritmelanie witt proposed openstack/nova master: Count security groups to check quota  https://review.openstack.org/45786023:20
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094523:20
openstackgerritmelanie witt proposed openstack/nova master: Count fixed ips to check quota  https://review.openstack.org/45786123:20
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094623:20
openstackgerritmelanie witt proposed openstack/nova master: Count floating ips to check quota  https://review.openstack.org/45786223:20
*** rkrum has joined #openstack-nova23:23
*** gjayavelu has joined #openstack-nova23:26
mriedemmelwitt: are the members of the group not created at the same time in the same cell?23:29
mriedemcould be on different hosts if an anti-affinity group, that would be the one to wonder about23:29
*** mtanino__ has quit IRC23:30
*** tbachman has joined #openstack-nova23:30
*** Swami has quit IRC23:34
*** rkrum has quit IRC23:35
*** priteau has joined #openstack-nova23:41
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec  https://review.openstack.org/46084723:41
*** takashin has joined #openstack-nova23:42
*** chyka has quit IRC23:44
*** hongbin has quit IRC23:45
*** priteau has quit IRC23:45
mriedemdansmith: is your fleetify devstack change expected to pass the live migration job?23:47
mriedemi don't think that runs on devstack does it?23:47
mriedemlooks like it doesnt23:47
dansmithmriedem: a nova change that depends on it should pass yeah,23:47
dansmithit was passing all last week23:47
*** dimtruck is now known as zz_dimtruck23:48
mriedemok i hit this http://logs.openstack.org/83/467383/2/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/05a0806/console.html#_2017-05-23_21_53_09_47216523:48
dansmithmriedem: there is a patch on the top of my set that youneed tho23:48
mriedemoh i see23:48
dansmithmriedem: https://review.openstack.org/#/c/458537/23:48
mriedemok, everything else pass on https://review.openstack.org/#/c/467382/ and down so i'll consider that a success for now23:49
mriedem*passed23:49
dansmithcool23:49
*** yamahata has quit IRC23:50
*** zz_dimtruck is now known as dimtruck23:51
cfriesenunder what circumstances would we hit https://bugs.launchpad.net/nova/+bug/1691545 ?  Do you need to have multiple cells?23:54
openstackLaunchpad bug 1691545 in OpenStack Compute (nova) ocata "Significant increase in DB connections with cells" [High,Fix committed] - Assigned to melanie witt (melwitt)23:54
*** yamahata has joined #openstack-nova23:54
*** salv-orlando has joined #openstack-nova23:59
*** fragatina has quit IRC23:59

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