Thursday, 2017-07-13

mriedemyup00:00
melwittk.00:00
melwittthink I gonna try to go outdoors now00:00
*** jamesdenton has joined #openstack-nova00:00
mriedemi'm going to go berate my family for awhile00:01
melwittheh00:02
*** thorst has joined #openstack-nova00:02
*** xyang1 has quit IRC00:05
*** penick has quit IRC00:06
mriedemalex_xu: when you're around, i've got the novaclient change up for the 2.50 microversion but a todo question in there about how we want to handle the python api if at all https://review.openstack.org/#/c/483060/3/novaclient/v2/quota_classes.py00:06
mriedemotherwise the functional tests i added for the cli are all passing, so i'd just need to remove the WIP from the commit message00:06
*** thorst has quit IRC00:07
*** dave-mccowan has joined #openstack-nova00:09
*** gmann has quit IRC00:11
*** gmann has joined #openstack-nova00:11
*** acormier has joined #openstack-nova00:17
*** acormier has quit IRC00:17
*** Apoorva has quit IRC00:17
*** acormier has joined #openstack-nova00:18
*** Apoorva has joined #openstack-nova00:21
*** yingjun has joined #openstack-nova00:21
*** acormier has quit IRC00:22
*** acormier has joined #openstack-nova00:23
*** jamesdenton has quit IRC00:24
*** slaweq_ has joined #openstack-nova00:26
*** acormier has quit IRC00:27
*** jamesdenton has joined #openstack-nova00:27
*** acormier has joined #openstack-nova00:29
*** slaweq_ has quit IRC00:30
*** acormier has quit IRC00:31
*** moshele has quit IRC00:31
*** markvoelker has quit IRC00:35
*** rajathagasthya has quit IRC00:36
*** jamesdenton has quit IRC00:37
*** zhurong has joined #openstack-nova00:38
*** acormier has joined #openstack-nova00:41
*** huanxie has joined #openstack-nova00:42
*** acormier has quit IRC00:45
*** acormier has joined #openstack-nova00:46
*** baoli has joined #openstack-nova00:46
*** kiennt has joined #openstack-nova00:47
*** markvoelker has joined #openstack-nova00:47
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895500:48
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896400:49
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004200:49
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492600:50
*** baoli has quit IRC00:51
*** litao__ has joined #openstack-nova00:53
openstackgerritMerged openstack/nova master: Trivial: Remove unnecessary format specifier  https://review.openstack.org/48157400:54
openstackgerritMerged openstack/nova master: api-ref: Fix parameters in server-security-groups  https://review.openstack.org/47643400:58
openstackgerritMerged openstack/nova master: api-ref: Add X-Openstack-Request-Id description  https://review.openstack.org/47484700:59
*** Shunli has joined #openstack-nova01:01
*** raunak has quit IRC01:02
*** chyka has joined #openstack-nova01:03
*** phuongnh has joined #openstack-nova01:04
*** chyka has quit IRC01:04
*** gouthamr has quit IRC01:05
*** liangy has quit IRC01:06
*** chyka has joined #openstack-nova01:07
*** gjayavelu has quit IRC01:08
openstackgerritMerged openstack/nova master: Remove nova.api.extensions.server.extensions usage  https://review.openstack.org/48149401:11
openstackgerritMerged openstack/nova master: Stop using deprecated 'message' attribute in Exception  https://review.openstack.org/48201401:11
openstackgerritMerged openstack/nova master: Fix a missing classifier  https://review.openstack.org/47873301:12
openstackgerritMerged openstack/nova master: Query deleted instance records during _destroy_evacuated_instances  https://review.openstack.org/46777401:12
openstackgerritMerged openstack/nova master: Fix parameters and description for os-volume_attachments  https://review.openstack.org/48083201:13
openstackgerritMerged openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432201:14
openstackgerritMerged openstack/nova master: Use PCIAddressField in oslo.versionedobjects  https://review.openstack.org/46620101:14
*** prateek has quit IRC01:19
*** amotoki_away is now known as amotoki01:19
*** fragatina has quit IRC01:22
*** owalsh has quit IRC01:22
openstackgerritMerged openstack/os-vif master: fix read the representor phys_port_name  https://review.openstack.org/48021301:22
*** fragatina has joined #openstack-nova01:22
*** rajathagasthya has joined #openstack-nova01:22
*** fragatin_ has joined #openstack-nova01:26
*** kristian__ has joined #openstack-nova01:26
*** fragatin_ has quit IRC01:26
*** slaweq_ has joined #openstack-nova01:26
*** liujiong has joined #openstack-nova01:26
*** fragatina has quit IRC01:27
*** fragatina has joined #openstack-nova01:27
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Add support for the 2.51 microversion  https://review.openstack.org/48314901:31
*** slaweq_ has quit IRC01:31
*** kristian__ has quit IRC01:31
*** rajathagasthya has quit IRC01:33
*** yufei has joined #openstack-nova01:36
*** kaisers_ has quit IRC01:37
*** Apoorva_ has joined #openstack-nova01:40
*** baoli has joined #openstack-nova01:40
*** zsli_ has joined #openstack-nova01:41
*** yufei has quit IRC01:43
*** Apoorva has quit IRC01:44
*** Shunli has quit IRC01:44
*** Apoorva_ has quit IRC01:44
*** gcb has joined #openstack-nova01:44
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980001:46
*** yushb has joined #openstack-nova01:48
*** yufei has joined #openstack-nova01:50
*** raunak has joined #openstack-nova01:51
*** raunak has quit IRC01:51
*** acormier_ has joined #openstack-nova01:51
*** raunak has joined #openstack-nova01:52
*** raunak has quit IRC01:52
*** jwcroppe has joined #openstack-nova01:54
mriedemmelwitt: i hope the outdoors was enjoyable. i've finished all of the non-test parts of https://review.openstack.org/#/c/416521/ now - found another thing in the compute manager01:54
mriedemeasy thing01:55
*** acormier has quit IRC01:55
mriedemi also found 2 more places in the compute manager code that was cleaning up deleted instances but wasn't accounting for that video ram thing with adjust ram quota :)01:55
*** hongbin has joined #openstack-nova01:59
*** weshay is now known as weshay_pto02:01
*** gyee has quit IRC02:02
*** amotoki is now known as amotoki_away02:08
*** thorst has joined #openstack-nova02:08
*** itlinux has joined #openstack-nova02:10
alex_xumriedem: I guess your concern about strict parameters is that it may break the backward-compatible?02:10
openstackgerritDebo Zhang proposed openstack/nova master: Fix typo.  https://review.openstack.org/48315902:11
alex_xuit sounds ok we strict parameters in the new microversion. So when the developer upgrade their client code to new microversion, they should check whether there is any interface change in the sdk interface02:11
*** bkopilov has quit IRC02:12
*** thorst has quit IRC02:13
alex_xumriedem: and we have parameter change in the some microversion https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L7602:13
mriedemalex_xu: yeah we'd leave **kwargs for microversion 2.0 to 2.49 and then have strict kwargs for 2.5102:13
mriedem*2.5002:13
alex_xugmann: fyi, mriedem already have a patch for the 2.50 in python-novaclient02:13
mriedemi also identified some issues in the 2.50 api change02:14
mriedemmostly docs02:14
mriedemhttps://review.openstack.org/#/c/467999/02:14
alex_xumriedem: thanks, I can fix that02:15
*** fragatin_ has joined #openstack-nova02:15
*** fragatin_ has quit IRC02:16
*** fragatin_ has joined #openstack-nova02:16
*** fragatin_ has quit IRC02:16
*** baoli has quit IRC02:18
*** fragatina has quit IRC02:19
gmannalex_xu, mriedem thanks for client patch02:22
*** baoli has joined #openstack-nova02:22
gmannalex_xu, you are fixing api comments? or should i02:22
mriedemmy eyes are going to dry up and fall out so i'm going to be afk02:22
*** mriedem is now known as mriedem_away02:23
alex_xugmann: I can handle that02:23
gmannalex_xu, thanks02:23
*** acormier_ has quit IRC02:25
*** slaweq_ has joined #openstack-nova02:27
*** tjones has left #openstack-nova02:28
openstackgerritDebo Zhang proposed openstack/nova master: Fix typo.  https://review.openstack.org/48316202:32
*** slaweq_ has quit IRC02:33
*** tuanluong has joined #openstack-nova02:35
*** hongbin_ has joined #openstack-nova02:37
*** iceyao has joined #openstack-nova02:38
*** hongbin has quit IRC02:39
*** fragatina has joined #openstack-nova02:40
*** fragatina has quit IRC02:44
*** baoli has quit IRC02:46
*** mriedem_away has quit IRC02:47
*** baoli has joined #openstack-nova02:49
*** rajathagasthya has joined #openstack-nova02:51
*** yamahata has quit IRC02:52
*** baoli has quit IRC02:52
*** liangy has joined #openstack-nova02:53
*** fragatina has joined #openstack-nova02:57
*** esberglu has joined #openstack-nova03:00
*** esberglu has quit IRC03:00
*** fragatina has quit IRC03:01
*** gjayavelu has joined #openstack-nova03:05
openstackgerritDebo Zhang proposed openstack/nova master: Fix typo  https://review.openstack.org/48315903:10
*** hongbin_ has quit IRC03:16
*** yamahata has joined #openstack-nova03:17
*** fragatina has joined #openstack-nova03:19
*** liangy has quit IRC03:20
*** yushb has quit IRC03:21
openstackgerritDebo Zhang proposed openstack/nova master: Fix typo  https://review.openstack.org/48316203:21
*** dave-mccowan has quit IRC03:24
*** amotoki_away is now known as amotoki03:25
*** slaweq_ has joined #openstack-nova03:29
*** rajathagasthya has quit IRC03:32
*** hongbin has joined #openstack-nova03:32
*** slaweq_ has quit IRC03:33
openstackgerritAlex Xu proposed openstack/nova master: Fix the releasenote and api-ref for quota-class API  https://review.openstack.org/48317003:34
alex_xugmann: mriedem ^03:34
*** udesale has joined #openstack-nova03:35
*** prateek has joined #openstack-nova03:35
*** bkopilov has joined #openstack-nova03:37
*** mdnadeem has joined #openstack-nova03:39
*** hongbin has quit IRC03:41
*** gjayavelu has quit IRC03:45
*** gbarros has quit IRC03:49
*** esberglu has joined #openstack-nova03:53
openstackgerritDebo Zhang proposed openstack/nova master: Fix typo  https://review.openstack.org/48315903:54
*** esberglu has quit IRC03:58
*** rajathagasthya has joined #openstack-nova03:59
*** links has joined #openstack-nova04:02
*** yufei has quit IRC04:02
*** fragatina has quit IRC04:03
*** yufei has joined #openstack-nova04:03
*** fragatina has joined #openstack-nova04:04
openstackgerritMerged openstack/nova master: improve notification short-circuit  https://review.openstack.org/48249704:09
*** fragatina has quit IRC04:11
*** iceyao has quit IRC04:17
*** zsli__ has joined #openstack-nova04:19
*** zsli_ has quit IRC04:22
*** fragatina has joined #openstack-nova04:26
*** zsli_ has joined #openstack-nova04:26
*** felipemonteiro has joined #openstack-nova04:27
*** rajathagasthya has quit IRC04:28
*** yushb has joined #openstack-nova04:29
*** zsli__ has quit IRC04:29
*** slaweq_ has joined #openstack-nova04:29
*** huanxie has quit IRC04:31
*** psachin has joined #openstack-nova04:31
*** slaweq_ has quit IRC04:35
*** claudiub has joined #openstack-nova04:36
*** sree has joined #openstack-nova04:36
*** hoonetorg has quit IRC04:40
*** mdnadeem has quit IRC04:42
*** iceyao has joined #openstack-nova04:42
*** baoli has joined #openstack-nova04:53
*** mdnadeem has joined #openstack-nova04:54
*** hoonetorg has joined #openstack-nova04:54
*** baoli has quit IRC04:58
*** yufei has quit IRC04:59
*** yufei has joined #openstack-nova04:59
*** felipemonteiro has quit IRC05:01
*** armax has quit IRC05:01
*** armax has joined #openstack-nova05:01
*** armax has quit IRC05:02
*** huanxie has joined #openstack-nova05:02
*** fragatina has quit IRC05:02
*** armax has joined #openstack-nova05:02
*** armax has quit IRC05:03
*** fragatina has joined #openstack-nova05:03
*** armax has joined #openstack-nova05:03
*** armax has quit IRC05:03
*** armax has joined #openstack-nova05:04
*** armax has quit IRC05:04
*** trinaths has joined #openstack-nova05:07
*** fragatina has quit IRC05:08
*** trinaths1 has joined #openstack-nova05:08
*** slaweq_ has joined #openstack-nova05:09
*** slaweq_ has quit IRC05:11
*** trinaths has quit IRC05:12
*** psachin has quit IRC05:12
*** psachin has joined #openstack-nova05:15
*** ratailor has joined #openstack-nova05:17
*** gouthamr has joined #openstack-nova05:34
*** rajathagasthya has joined #openstack-nova05:35
*** marst has joined #openstack-nova05:36
*** rcernin has joined #openstack-nova05:40
*** ekuris_ has joined #openstack-nova05:42
*** kristian__ has joined #openstack-nova05:43
*** hieulq has quit IRC05:45
*** tuanluong has quit IRC05:45
*** tuanluong has joined #openstack-nova05:46
*** hieulq has joined #openstack-nova05:46
*** kristian__ has quit IRC05:47
*** kiennt has quit IRC05:50
*** yufei has quit IRC05:56
*** thorst has joined #openstack-nova05:57
*** kiennt has joined #openstack-nova05:58
openstackgerritjichenjc proposed openstack/nova master: add description for supports_migrate_to_same_host  https://review.openstack.org/48148306:00
*** thorst has quit IRC06:03
*** sridharg has joined #openstack-nova06:03
*** lpetrut has joined #openstack-nova06:06
*** zsli_ has quit IRC06:09
*** yufei has joined #openstack-nova06:09
*** zsli_ has joined #openstack-nova06:10
*** Oku_OS-away is now known as Oku_OS06:10
*** slaweq_ has joined #openstack-nova06:11
*** vvargaszte has joined #openstack-nova06:14
*** slaweq_ has quit IRC06:16
*** slaweq_ has joined #openstack-nova06:19
*** tylerderosagrund has joined #openstack-nova06:19
*** slaweq_ has quit IRC06:24
*** tylerderosagrund has quit IRC06:24
*** yushb has quit IRC06:27
*** huanxie has quit IRC06:31
*** zsli__ has joined #openstack-nova06:32
*** huanxie has joined #openstack-nova06:34
*** zsli_ has quit IRC06:35
*** esberglu has joined #openstack-nova06:37
*** esberglu has quit IRC06:38
*** esberglu has joined #openstack-nova06:38
*** esberglu has quit IRC06:38
*** chyka has quit IRC06:44
*** gjayavelu has joined #openstack-nova06:46
*** crushil has joined #openstack-nova06:51
*** pcaruana has joined #openstack-nova06:52
*** kristian__ has joined #openstack-nova06:53
*** yufei has quit IRC06:53
*** FL1SK has quit IRC06:53
*** crushil has quit IRC06:55
*** tesseract has joined #openstack-nova06:56
*** yufei has joined #openstack-nova06:57
*** sshwarts has joined #openstack-nova07:01
*** sree_ has joined #openstack-nova07:01
*** sree_ is now known as Guest9399007:02
openstackgerritAlex Xu proposed openstack/nova master: Fix the releasenote and api-ref for quota-class API  https://review.openstack.org/48317007:05
*** sree has quit IRC07:05
*** lpetrut has quit IRC07:07
*** rajathagasthya has quit IRC07:08
*** roeyc has joined #openstack-nova07:12
openstackgerritZhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980007:19
*** slaweq_ has joined #openstack-nova07:20
*** gouthamr has quit IRC07:22
*** nicolasbock has joined #openstack-nova07:24
*** slaweq_ has quit IRC07:24
*** moshele has joined #openstack-nova07:25
*** alexchadin has joined #openstack-nova07:25
*** yufei has quit IRC07:32
*** yufei has joined #openstack-nova07:32
*** lpetrut has joined #openstack-nova07:34
*** andreas_s has joined #openstack-nova07:36
*** iceyao has quit IRC07:36
*** tuanluong has quit IRC07:37
*** phuongnh has quit IRC07:39
*** vvargaszte1 has joined #openstack-nova07:41
*** lpetrut has quit IRC07:43
*** vvargaszte has quit IRC07:45
*** vvargaszte1 is now known as vvargaszte07:45
*** mirrorbox has left #openstack-nova07:46
*** ralonsoh has joined #openstack-nova07:46
*** damien_r has joined #openstack-nova07:47
*** damien_r has quit IRC07:47
*** damien_r has joined #openstack-nova07:47
*** itlinux has quit IRC07:47
openstackgerritAlex Szarka proposed openstack/nova master: Raise a real Exception instead of direct Exception call  https://review.openstack.org/48220007:48
*** damien_r has left #openstack-nova07:50
*** nicolasbock has quit IRC07:50
*** kristian__ has quit IRC07:54
*** kristian__ has joined #openstack-nova07:55
*** yushb has joined #openstack-nova07:56
*** thorst has joined #openstack-nova07:59
*** thorst has quit IRC08:03
janguttersean-k-mooney: would you have an example handy on that alternate way to set the bridge name in a VIF? (something about trunk ports, I think...)08:07
*** markvoelker has quit IRC08:08
*** ralonsoh has quit IRC08:09
*** ralonsoh_ has joined #openstack-nova08:09
*** gjayavelu has quit IRC08:10
*** lucas-afk is now known as lucasagomes08:15
*** slaweq_ has joined #openstack-nova08:21
*** slaweq_ has quit IRC08:25
*** alexchadin has quit IRC08:29
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925708:30
*** belmoreira has joined #openstack-nova08:31
*** slaweq_ has joined #openstack-nova08:34
*** slaweq_ has quit IRC08:34
*** ralonsoh_ is now known as ralonsoh08:37
*** sbezverk has quit IRC08:39
openstackgerritLiChunlin proposed openstack/nova master: update the description of hypervisor statistics response  https://review.openstack.org/46235508:40
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177708:49
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache  https://review.openstack.org/47277308:50
*** dtantsur|afk is now known as dtantsur08:52
*** FL1SK has joined #openstack-nova08:53
*** karimb has joined #openstack-nova08:56
*** alexchadin has joined #openstack-nova08:58
*** efoley has joined #openstack-nova09:05
openstackgerritMerged openstack/nova master: Add BDM to InstancePayload  https://review.openstack.org/44877909:05
*** efoley has quit IRC09:05
*** efoley has joined #openstack-nova09:05
*** edmondsw has joined #openstack-nova09:16
*** kristia__ has joined #openstack-nova09:18
*** edmondsw has quit IRC09:20
*** kristian__ has quit IRC09:21
*** sambetts_ is now known as sambetts09:23
*** owalsh has joined #openstack-nova09:28
*** mvk has quit IRC09:28
*** chenying_ has quit IRC09:29
openstackgerritChangBo Guo(gcb) proposed openstack/nova master: rbd: Remove unnecessary 'encode' calls  https://review.openstack.org/41235609:30
*** zsli__ has quit IRC09:31
bauzasstephenfin: hola09:32
stephenfinbauzas: Morning09:32
bauzasstephenfin: could you please remember me if we need to add a _LW() for any warning log ?09:32
bauzasI think so09:32
bauzasonly debug logs are not needed09:32
*** tuanluong has joined #openstack-nova09:32
openstackgerrityuanyue proposed openstack/nova master: Remove ports on create_port timeout during instance build  https://review.openstack.org/48328009:33
stephenfinbauzas: No, IIRC we don't need any _L* translations. We only translate exceptions now09:33
bauzasokay09:33
stephenfin...which use the '_' symbol09:33
bauzasyeah that I know09:33
bauzasI thought we said to only translate logs having no debug level09:34
bauzasbut I didn't remember we said to stop translating any level log09:34
stephenfinHmm, lemme go check09:35
*** slaweq_ has joined #openstack-nova09:35
*** yingjun has quit IRC09:35
stephenfingcb: Can you say, do we still need to translate LOG.warning logs?09:35
stephenfinbauzas: https://docs.openstack.org/nova/latest/i18n.html09:36
stephenfinLog messages should not be translated. Any use of _LI(), _LW(), _LE(), _LC() are vestigial and will be removed over time.09:36
stephenfingcb: It's good - we figured it out :)09:37
bauzasstephenfin: cool thanks09:37
gcbstephenfin:  yes, look at https://docs.openstack.org/oslo.i18n/latest/user/usage.html09:37
openstackgerritAlex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications  https://review.openstack.org/46552609:37
gcbStarting with the Pike series, OpenStack no longer supports log translation. It is not necessary to add translation instructions to new code, and the instructions can be removed from old code. Refer to the email thread understanding log domain change on the openstack-dev mailing list for more details.09:37
gcbstephenfin^^09:38
stephenfingcb: Excellent. Thank you :)09:38
*** cdent has joined #openstack-nova09:39
*** slaweq_ has quit IRC09:40
cdentstephenfin: I'm not wise in the ways of policy, does this change (or the linked bug) make sense: https://review.openstack.org/#/c/481443/09:40
stephenfincdent: I was purposefully leaving that until if/when johnthetubaguy was back. Apparently we were removing all default rules, so I'd imagine that will have some bearing on that09:42
cdentstephenfin: ah okay, so the burden of setting the default is on the deployment?09:43
*** liusheng has quit IRC09:43
stephenfincdent: Wait, no - that was _discoverable_ policies (because they no longer make sense)09:44
*** karimb has quit IRC09:44
*** liusheng has joined #openstack-nova09:44
*** liusheng has quit IRC09:44
stephenfinLooks like claudiub added a load of those default rules. Maybe he can review https://review.openstack.org/#/c/481443/ and see if it makes sense?09:44
*** liusheng has joined #openstack-nova09:45
*** karimb has joined #openstack-nova09:46
*** mvk has joined #openstack-nova09:55
*** tylerderosagrund has joined #openstack-nova09:56
openstackgerritRikimaru Honjo proposed openstack/nova master: compute: Do not create bdms before successfully reserving a volume  https://review.openstack.org/29079309:56
*** tylerderosagrund has quit IRC10:00
*** thorst has joined #openstack-nova10:00
*** thorst has quit IRC10:04
*** alexchadin has quit IRC10:06
*** yufei has quit IRC10:06
openstackgerritStephen Finucane proposed openstack/nova master: Always set MTU when plugging devices  https://review.openstack.org/40878110:07
*** markvoelker has joined #openstack-nova10:09
*** kiennt has quit IRC10:09
*** amotoki is now known as amotoki_away10:10
*** pallavi_ has joined #openstack-nova10:10
pallavi_Hi , I have done ubuntu manual installation  for ocata release. I have launched instance. But accessing to the instance is not happening.?? Can anyone please help me out to figure it out10:12
*** Guest93990 has quit IRC10:16
*** esberglu has joined #openstack-nova10:16
*** esberglu has quit IRC10:16
claudiubstephenfin: replied. that patch LGTM.10:17
stephenfinclaudiub: Thanks :) If you're happy, so am I10:18
*** owalsh has quit IRC10:19
*** owalsh has joined #openstack-nova10:21
*** liujiong has quit IRC10:22
*** yushb has quit IRC10:22
*** owalsh has quit IRC10:22
claudiubstephenfin: if you're happy and you know it, clap your hands! *clap* *clap*. :D10:23
cdentthanks claudiub10:23
*** nicolasbock has joined #openstack-nova10:25
gibibauzas: hi! There is a small test and doc only patch that needs a second +2. Could you check it please? https://review.openstack.org/#/c/480469/10:28
bauzasgibi: roger this.10:28
gibibauzas: thanks!10:28
*** yamamoto has quit IRC10:31
*** amotoki_away is now known as amotoki10:32
*** mvk has quit IRC10:34
sean-k-mooneyjangutter: hi the bridge name is currently set here https://github.com/openstack/neutron/blob/8326a65a5e3c70c66a5297c618d0202c9395cc3f/neutron/plugins/ml2/drivers/openvswitch/mech_driver/mech_openvswitch.py#L94-L10510:35
*** slaweq_ has joined #openstack-nova10:36
*** udesale has quit IRC10:36
*** karimb_ has joined #openstack-nova10:36
sean-k-mooneyjangutter: so in you mech driver if you just overrided that fuction you can pass the bridge name via the vif binding details.10:37
*** owalsh has joined #openstack-nova10:38
*** tbachman has quit IRC10:38
*** amotoki is now known as amotoki_away10:38
sean-k-mooneyjangutter: this then get populated into the os-vif network object here https://github.com/openstack/nova/blob/d955eb1db753f191acb02eb9758080329da5b866/nova/network/os_vif_util.py#L225-L22610:38
*** karimb has quit IRC10:38
*** tbachman has joined #openstack-nova10:39
sean-k-mooneyjangutter: and its used in os-vif here: https://github.com/openstack/os-vif/blob/4a8e07f6c502e5e3b99db9ba543a501fbb138596/vif_plug_ovs/ovs.py#L97 and anywayer else we need the bridge name10:40
*** owalsh has quit IRC10:41
*** slaweq_ has quit IRC10:41
*** yamahata has quit IRC10:41
*** owalsh has joined #openstack-nova10:43
*** markvoelker has quit IRC10:43
*** zhurong has quit IRC10:43
openstackgerritStephen Finucane proposed openstack/nova master: Remove plug_ovs_hybrid, unplug_ovs_hybrid  https://review.openstack.org/48303010:43
openstackgerritStephen Finucane proposed openstack/nova master: tests: Remove useless test  https://review.openstack.org/48303110:43
bauzasstephenfin: gibi: something got messed up with our man pages10:44
stephenfin?10:44
bauzaseg. <http://docs.openstack.org/developer/nova/man/nova-manage.html#nova-cells-v2 is pointing nowhere10:44
stephenfinYeah, that's a pbr bug10:44
stephenfin*Oh, that's like that pbr bug10:44
stephenfin*likely10:44
stephenfindammit10:44
stephenfinbauzas: https://bugs.launchpad.net/pbr/+bug/170287210:45
openstackLaunchpad bug 1702872 in PBR "pbr + Sphinx 1.6+ ignores the '[sphinx_build] builders' setuptools option" [Undecided,In progress] - Assigned to Stephen Finucane (stephenfinucane)10:45
*** lpetrut has joined #openstack-nova10:45
stephenfinI'll tack a fix onto the end of the https://review.openstack.org/#/q/topic:doc-migration+project:openstack/nova series, once pep8 fix patch goes in (it needs a rebase again)10:46
bauzasstephenfin: I just +2d https://review.openstack.org/#/c/478472/7 because it workarounds the issue10:46
bauzasstephenfin: I'm not sure it should be a cli/ section, but meh10:46
bauzasat least it's understandable that nova-manage and nova-status are CLI tools10:47
stephenfinYeah, sdague said the same thing. However, we've no where else to put those man pages10:47
stephenfinI wanted that section called 'man' instead, but didn't make my case strongly enough :(10:48
* gibi is glad stephenfin knew the answer10:48
*** amotoki_away is now known as amotoki10:49
*** mvk has joined #openstack-nova10:50
*** gcb has quit IRC10:52
*** acormier has joined #openstack-nova10:53
*** acormier has quit IRC10:55
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366010:55
*** acormier has joined #openstack-nova10:55
*** amotoki is now known as amotoki_away10:57
*** bkopilov has quit IRC10:57
*** jamesdenton has joined #openstack-nova10:59
*** thorst has joined #openstack-nova11:01
*** thorst has quit IRC11:05
*** esberglu has joined #openstack-nova11:10
*** jamesdenton has quit IRC11:12
*** yamamoto has joined #openstack-nova11:13
*** mvk has quit IRC11:13
*** jamesdenton has joined #openstack-nova11:13
openstackgerritChris Dent proposed openstack/nova master: Add functional test for two-cell scheduler behaviors  https://review.openstack.org/45200611:13
*** tbachman has quit IRC11:14
*** esberglu has quit IRC11:14
*** tbachman has joined #openstack-nova11:15
*** thorst has joined #openstack-nova11:17
*** yamamoto has quit IRC11:18
*** amotoki_away is now known as amotoki11:21
*** yamamoto has joined #openstack-nova11:21
*** smatzek has joined #openstack-nova11:24
*** liangy has joined #openstack-nova11:26
*** mvk has joined #openstack-nova11:26
*** yamamoto has quit IRC11:26
*** jamesden_ has joined #openstack-nova11:34
*** ekuris_ has quit IRC11:35
*** trinaths1 has left #openstack-nova11:36
*** yamamoto has joined #openstack-nova11:36
*** moshele has quit IRC11:37
*** slaweq_ has joined #openstack-nova11:37
*** jamesdenton has quit IRC11:37
*** damien_r has joined #openstack-nova11:37
*** lpetrut_ has joined #openstack-nova11:38
*** tbachman_ has joined #openstack-nova11:39
*** tbachman has quit IRC11:39
*** tbachman_ is now known as tbachman11:39
*** yamamoto has quit IRC11:40
*** markvoelker has joined #openstack-nova11:41
*** lpetrut has quit IRC11:41
*** slaweq_ has quit IRC11:42
*** gszasz has joined #openstack-nova11:44
*** acormier has quit IRC11:46
*** acormier has joined #openstack-nova11:47
*** gbarros has joined #openstack-nova11:49
*** ekuris_ has joined #openstack-nova11:49
*** yamamoto has joined #openstack-nova11:50
*** acormier has quit IRC11:51
*** huanxie has quit IRC11:54
gibicdent: thank you for your detailed answer on my placement question on the ML. It helps a lot11:54
*** yamamoto has quit IRC11:54
*** tylerderosagrund has joined #openstack-nova11:56
openstackgerritClaudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM  https://review.openstack.org/47894311:57
janguttersean-k-mooney: hmmm, I still get: ERROR os_vif NotImplementedError: Cannot load 'bridge' in the base class11:57
*** ratailor has quit IRC11:57
*** tbachman has quit IRC11:57
*** Oku_OS has quit IRC11:58
sean-k-mooneyjangutter: how are you accessing the bridge? via the network object?11:58
*** owalsh has quit IRC11:59
janguttersean-k-mooney: vif.network.bridge -> I think that might not be populated for vhostuser11:59
*** litao__ has quit IRC11:59
sean-k-mooneyjangutter: if you do a neutron port show or openstack port show do you see the bridge correctly set in the vif_binding_details11:59
sean-k-mooneyjangutter: it should be populated or it means that vlan aware vms dose not work for vhost-user and this is a nova bug we need to fix12:00
*** tbachman has joined #openstack-nova12:00
*** dave-mccowan has joined #openstack-nova12:01
*** udesale has joined #openstack-nova12:01
sean-k-mooneyjangutter: yep you are right.... https://github.com/openstack/nova/blob/d955eb1db753f191acb02eb9758080329da5b866/nova/network/neutronv2/api.py#L2157-L217912:01
sean-k-mooneyjangutter: do you want to submit the patch or will i12:02
janguttersean-k-mooney: binding_vif_details   -> ovs_hybrid_plug='False', port_filter='True'12:02
sean-k-mooneyactully no i misread that https://github.com/openstack/nova/blob/d955eb1db753f191acb02eb9758080329da5b866/nova/network/neutronv2/api.py#L2172-L2173 it is saved for vhost-user12:03
janguttersean-k-mooney: depends on what will get it merged faster :-)12:03
sean-k-mooneyjangutter: so ya the bug is in the neutron ml2 driver yuou are missing the bridge key in the vif_binding_details12:04
janguttersean-k-mooney: Aah, and the workaround is to add: elif vif_type == network_model.VIF_TYPE_AGILIO_OVS to that list.12:05
sean-k-mooneyjangutter: you really should not have a new vif_type12:05
*** pallavi_ has quit IRC12:06
sean-k-mooneythere will not be a new vif object in os-vif and you should not need one for nova or neutron12:06
*** thorst has quit IRC12:06
sean-k-mooneythe vif type shoud just be VIF_TYPE_VHOSTUSER12:06
*** owalsh has joined #openstack-nova12:07
*** owalsh has quit IRC12:07
janguttersean-k-mooney: I am using VIF_TYPE_VHOSTUSER for the actual plugging.12:07
sean-k-mooneyjangutter: you should also be using it for port binding12:07
*** owalsh has joined #openstack-nova12:07
sean-k-mooneywe are trying to stop using vif types as a flag for how to plug interfaces and retrun them to being a statemant about what type if interface you want12:08
*** fragatina has joined #openstack-nova12:08
sean-k-mooneyor in otherwords the vif_type is an abstion for the hyperviser interface type and is not related to how that interface is pluged to the network backend at all12:09
janguttersean-k-mooney: yeah. so one thing that kinda breaks the abstraction is that the "forwarder" is actually a kind of translation layer on top of the hypervisor.12:10
sean-k-mooneyjangutter: that is captured by the vnic_type12:10
janguttersean-k-mooney: and introducing another layer just leads to madness.12:10
janguttersean-k-mooney: yep. let me see if I can figure out if I'm doing the wrong thing in my ML2 driver.12:11
*** thorst has joined #openstack-nova12:11
janguttersean-k-mooney: (facepalm) network_model.VIF_DETAILS_VHOSTUSER_OVS_PLUG12:12
*** markvoelker has quit IRC12:13
sean-k-mooneyjangutter: ya you might need to set that12:13
*** sudhirag0987 has joined #openstack-nova12:13
sean-k-mooneythat is a legacy thing that should go away when we can specify the plugin to use in the vif details12:14
*** tbachman_ has joined #openstack-nova12:14
*** tbachman has quit IRC12:14
*** tbachman_ is now known as tbachman12:14
*** foutatoro has joined #openstack-nova12:15
*** tuanluong has quit IRC12:16
foutatorohi all, I would like to know how to deploy hypervisors VMware vSphere and Hyper-V   on compute nodes ?12:17
*** yamamoto has joined #openstack-nova12:17
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709112:18
openstackgerritBéla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification  https://review.openstack.org/46062512:18
openstackgerritBalazs Gibizer proposed openstack/nova master: use already loaded BDM in instance.<action>  https://review.openstack.org/48332412:19
*** baoli has joined #openstack-nova12:23
*** bkopilov has joined #openstack-nova12:24
*** pchavva has joined #openstack-nova12:26
*** baoli has quit IRC12:28
*** baoli has joined #openstack-nova12:29
cdentgibi: glad it was of some use, looking forward to seeing more of what you're doing12:30
*** cdent has quit IRC12:31
stephenfinsean-k-mooney, jaypipes: Fancy taking a look at this? Easy +1/2, imo https://review.openstack.org/#/c/483030/12:32
*** markvoelker has joined #openstack-nova12:32
toabctlis there a way to list hosts which are not attached to a cell (v2) ?12:34
*** vvargaszte has quit IRC12:34
*** sridharg has quit IRC12:34
*** deep-book-gk_ has joined #openstack-nova12:35
*** tbachman has quit IRC12:36
*** sridharg has joined #openstack-nova12:37
*** deep-book-gk_ has left #openstack-nova12:37
*** sridharg has quit IRC12:38
*** slaweq_ has joined #openstack-nova12:38
sean-k-mooneystephenfin: sure, looking at it now12:38
*** gcb has joined #openstack-nova12:39
*** sridharg has joined #openstack-nova12:39
*** prateek has quit IRC12:41
*** liverpooler has joined #openstack-nova12:41
stephenfinsean-k-mooney: Cheers. Out of curiosity, is the expectation that all the old, non-os-vif plugging/unplugging code would eventually be replaced by code that harnesses vendor-developed os-vif plugins?12:42
*** slaweq_ has quit IRC12:43
stephenfine.g. All of these https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L599-L764 would be replaced by these https://github.com/openstack/nova/blob/master/nova/network/os_vif_util.py#L346-L40312:43
openstackgerritMerged openstack/nova master: doc: Enable pep8 on doc generation code  https://review.openstack.org/48046912:44
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48333712:44
sean-k-mooneystephenfin: yes that was the original intention was to add an os-vif pluging for all the old types. modify neutron ml2 driver to pass plugin name in vif details and then eventurlly remove os_vif_util when we finally have neutron retrunin serialised os-vif objects from its api12:47
*** jmlowe has joined #openstack-nova12:47
*** alexchadin has joined #openstack-nova12:48
stephenfinsean-k-mooney: OK, so regarding the comment here https://review.openstack.org/#/c/483030/2/nova/virt/libvirt/vif.py ...12:50
stephenfinwould we be submitting a bug against os-vif, or would that be a separate plugin maintained in its own repo by the floodlight team?12:52
sean-k-mooneystephenfin: yep i was suggesting opening a bug against https://github.com/openstack/networking-bigswitch12:52
stephenfinahhh12:52
*** jwcroppe has quit IRC12:53
sean-k-mooneystephenfin: IVS is a bigswitch networks competitor to ovs . it is the only thing that uses the IVS vif type https://github.com/openstack/networking-bigswitch/blob/master/networking_bigswitch/plugins/ml2/drivers/mech_bigswitch/driver.py#L422-L43112:54
stephenfinsean-k-mooney: Yeah, so they'd "own" the repo. It would be merged into 'openstack/os-vif'?12:54
*** kristia__ has quit IRC12:55
sean-k-mooneystephenfin: no its an non reference backend so im proposing that we wright an os-vif plugin for them and proposes it to there repo then deprecate the nova code and remove it once it is merged12:56
*** kristian__ has joined #openstack-nova12:56
*** catintheroof has joined #openstack-nova12:56
*** kristian__ has quit IRC12:56
sean-k-mooneystephenfin: alternitively we can create our only os-vif-ivs repo and that we own for this backend12:56
*** kristian__ has joined #openstack-nova12:56
*** links has quit IRC12:57
sean-k-mooneyit depend on whetere you want to tie the packaging of the plugin to the packaging of the ml2 driver which arguably you might want to keep seprate12:57
stephenfinWe'll let them make that decision, I guess13:00
stephenfinsean-k-mooney: ...though IMO they're separate products so they should be. We'll see13:00
stephenfinI'll file that bug now13:01
sean-k-mooneyam well if we have a seperate repo we dont need a bug on networking-bigswtich though we could use that bug to disucss this with them i guess?13:02
*** sbezverk has joined #openstack-nova13:02
*** mdnadeem has quit IRC13:03
stephenfinsean-k-mooney: Yeah, that's what I think. They will probably have their own ideas too13:05
stephenfinIf nothing else, they'll need to give us a repo to populate13:05
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48333713:06
*** MVenesio has joined #openstack-nova13:06
*** kylek3h has joined #openstack-nova13:07
*** yassine has joined #openstack-nova13:07
*** alexchadin has quit IRC13:09
*** mriedem has joined #openstack-nova13:09
*** gmann_ has joined #openstack-nova13:12
stephenfinsean-k-mooney: https://bugs.launchpad.net/networking-bigswitch/+bug/170412913:16
openstackLaunchpad bug 1704129 in networking-bigswitch "Add an IVS os-vif plugin" [Undecided,New]13:16
*** ekuris_ has quit IRC13:18
openstackgerritBalazs Gibizer proposed openstack/nova master: use already loaded BDM in instance.<action>  https://review.openstack.org/48332413:19
sean-k-mooneystephenfin: cool i have subscribed to the bug so i should get pinged if they  respond13:22
sean-k-mooneystephenfin: the only thing i hate about the contrail plugin is that they do not use the setup.cfg to declare there entrypoitns and instead to it all in setup.py https://github.com/Juniper/contrail-nova-vif-driver/blob/master/setup.py13:24
stephenfinsean-k-mooney: That's because they're not using pbr13:25
sean-k-mooneystephenfin: but its a good example of how to do in correctly otherwise13:25
stephenfinpbr had setuptools plugins that allow reading of things like that from 'setup.cfg'. By default though, setuptools doesn't support that13:25
stephenfin*has13:25
*** acormier has joined #openstack-nova13:25
stephenfinsean-k-mooney: We could always draft a cookiecutter template if it would help people?13:25
sean-k-mooneystephenfin: oh really . there is one13:26
sean-k-mooneyoh you mean for os-vif plugins?13:26
*** jwcroppe has joined #openstack-nova13:26
sean-k-mooneystephenfin: the cookicutter template for new openstack project i think has not been updated for pbr yet13:26
stephenfinYup, a cookiecutter template for plugins13:26
sean-k-mooneystephenfin: unless you did it13:26
stephenfinI think I did13:26
*** acormier has quit IRC13:26
*** eharney has joined #openstack-nova13:27
sean-k-mooneyoh i have been meaning to aks you how to fix one of our repos to use pbr correctly13:27
*** acormier has joined #openstack-nova13:27
stephenfinsean-k-mooney: link?13:28
*** esberglu has joined #openstack-nova13:28
sean-k-mooneystephenfin: https://github.com/openstack/devstack-plugin-libvirt-qemu13:28
gmann_mriedem: i replied on https://review.openstack.org/#/c/482994/113:28
*** gabor_antal_ is now known as gabor_antal13:28
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538513:28
sean-k-mooneystephenfin: hum need to fix the readme too...13:29
gmann_mriedem: 'events' can be empty fro coding we have on controller side :) but very rare to rare timing things13:29
gmann_*from13:29
sean-k-mooneystephenfin: we can gret rid of all the mail map stuff correct and there is some other boiler plate stuff that pbr will do for us too i think13:29
gmann_https://github.com/openstack/nova/blob/d955eb1db753f191acb02eb9758080329da5b866/nova/api/openstack/compute/instance_actions.py#L10613:29
mriedemgmann_: ok, but an empty events list is different from an event with no 'event.start_time' key13:31
*** sudhirag0987 has quit IRC13:31
stephenfinsean-k-mooney: What's wrong with that project exactly? Looks a-ok to me?13:33
gmann_mriedem: is it. 'events' has direct 'start_time' so 'events' will be with no sub key13:33
*** thorst has quit IRC13:33
*** Oku_OS-away has joined #openstack-nova13:34
*** Oku_OS-away is now known as Oku_OS13:34
gmann_mriedem: i commented the event record on compute service side and then we will have like this - http://paste.openstack.org/show/615266/13:34
sean-k-mooneystephenfin: it may be fine now. i know orginally after teh cookie cotter template it had the old change log and a few other things but rodolfo may have allready cleaned up everythin13:35
mriedemgmann_: consider it like the os-extended-volumes:volumes_attached field in the server details response13:35
mriedemthat can be an empty list, which doesn't mean os-extended-volumes:volumes_attached.id is marked optional in the api ref13:35
mriedemit means os-extended-volumes:volumes_attached will be in the response, but can be an empty list13:36
mriedemAnd the description reflects that, " The attached volumes, if any."13:36
mriedemnow if you have volumes attached, the os-extended-volumes:volumes_attached.id and os-extended-volumes:volumes_attached.delete_on_termination keys will be in there,13:36
mriedemand they are required13:36
mriedemi'm saying it's the same thing for instance action events and the start_time field per event13:37
mriedem*Except* the 'events' list is optional13:37
mriedemwhich was what i changed in my patch13:37
mriedemhttps://review.openstack.org/#/c/480792/13:37
*** slaweq_ has joined #openstack-nova13:39
gibimriedem: hi! I proposed the first patch to use already in scope BDM for the notification payload as you suggested yesterday. Could take a brief look to see if it is a good directions? https://review.openstack.org/#/c/483324/13:41
gmann_mriedem: i see your point. and from api-ref yes your point makes more sense to understand the api response. but from tempest response schema those should not be marked required13:42
mriedemgibi: yeah maybe later13:42
mriedemgmann_: but why? if there are events, and there is an event within the list of events, tempest should care that start_time is there13:43
*** smatzek has quit IRC13:44
mriedemthe keys for the event will be in the response, the values may be null if the event is not done, like finish_time and traceback13:44
mriedemand result13:44
*** slaweq_ has quit IRC13:44
mriedemrequired == you expect a key in the body dict, right?13:44
mriedemrequired != that key must map to a non-None value13:44
*** awaugama has joined #openstack-nova13:44
gmann_mriedem: i am saying it can eb emtpy list means no start_time inside 'events' key13:45
*** takashin has joined #openstack-nova13:45
gmann_mriedem: if api controller always insert those subkey even as null then i agree but that is not case13:46
gmann_because we insert 'events'  key in response based on policy and insterts its subkey based on event we have in db or not13:46
gmann_so if later is not there we end up with 'events' with no sub  key13:47
mriedemgmann_: is jsonschema going to validate against an entry that isn't in the response?13:48
mriedemwe should have 2 schemas,13:48
mriedem1 for the instanceAction response, which has the 'events' key which is optional and is an array13:49
stephenfinsean-k-mooney, ralonsoh: https://review.openstack.org/48339813:49
mriedemand 1 for the instanceAction.events, which is an object type, which has fields like start_time, finish_time, traceback, etc - those are required13:49
stephenfinYe silly gooses :)13:49
stephenfingmann_: Can you take a second look at my replies here when you have a chance? https://review.openstack.org/#/c/478484/13:50
ralonsohstephenfin: thanks! I'll review the patch now13:51
gmann_mriedem: current schema will check 'events' as optional as it is policy based. as on some cloud it can be present and on other it cannot be13:51
*** acormier has quit IRC13:52
gmann_mriedem: and my point is to have 'events' array schema also same way which validate start_time etc if those are present in response otherwise skip13:53
gmann_mriedem: but if we say those as mandatory in api-ref i am ok to validate those strictly by not looking at the code.13:53
gmann_which i should not when i work from tempest perspective :)13:53
*** acormier has joined #openstack-nova13:53
*** acormier has quit IRC13:54
gmann_stephenfin: sure, ll check tomorrow morning. may be i still argue on those doc in reference but let me rethink on those13:54
*** acormier has joined #openstack-nova13:54
*** amotoki is now known as amotoki_away13:54
stephenfingmann_: Sounds good to me13:55
*** roeyc has quit IRC13:56
*** sree has joined #openstack-nova13:59
mriedemnova meeting in 1 minute13:59
* bauzas fastens the belt14:00
*** gmann_ has quit IRC14:00
*** gmann_ has joined #openstack-nova14:00
sean-k-mooneystephenfin: hehe thanks for taking a look at   openstack/devstack-plugin-libvirt-qemu most of the crft can be removed but we shoudl keep the docs/release note stuff14:02
*** belmoreira has quit IRC14:02
*** cdent has joined #openstack-nova14:03
stephenfinsean-k-mooney: Yeah, you can rework it to add that14:03
stephenfinfine by me :)14:03
sean-k-mooneystephenfin: yep ill be happy too. i didnt actully expect a patch. i was just hopeing for some pointers to make sure we were uptodate with the new ways to handel pbr and the other project config stuff14:05
*** yushb has joined #openstack-nova14:05
stephenfinAh, it's all good. Easy fix14:05
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector  https://review.openstack.org/46835314:05
*** hongbin has joined #openstack-nova14:06
*** smatzek has joined #openstack-nova14:06
*** acormier_ has joined #openstack-nova14:07
*** acormier_ has quit IRC14:07
*** acormier_ has joined #openstack-nova14:07
*** yushb has quit IRC14:09
*** acormier has quit IRC14:10
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847314:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section  https://review.openstack.org/47847214:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847514:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775114:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847414:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext  https://review.openstack.org/48296114:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs  https://review.openstack.org/48007414:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext  https://review.openstack.org/47935814:11
openstackgerritStephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues  https://review.openstack.org/48051614:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848414:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848514:11
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead files  https://review.openstack.org/47847014:11
janguttersean-k-mooney: so how would I write an external plugin _without_ creating a new VIF type?14:19
*** fragatina has quit IRC14:21
*** aarefiev is now known as aarefiev_afk14:21
*** acormier has joined #openstack-nova14:22
*** psachin has quit IRC14:22
*** acormier has quit IRC14:23
*** itlinux_ has joined #openstack-nova14:23
*** acormier has joined #openstack-nova14:23
*** acormier_ has quit IRC14:25
*** burt has joined #openstack-nova14:26
*** thorst has joined #openstack-nova14:29
bauzasjaypipes: I saw your rant yesterday about the scheduler, could you please clarify your concerns ?14:31
bauzasjaypipes: the fact that we're amending the ReqSpec by adding the found host ?14:31
jaypipesbauzas: hmm? no, nothing to do with amending reqspec14:31
bauzasit's just because of the affinity filter, but not sure I understand your problem honestly14:32
bauzasjaypipes: you were telling dan you had a problem with that yesterday night :)14:32
jaypipesbauzas: oh, yeah, sorry, that was a different rant :)14:32
bauzasanway, if I can help you...14:33
bauzasat least by sharing thoughts14:33
jaypipesbauzas: my general annoyance with the scheduler is its tight coupling of state with decision-making.14:33
jaypipesbauzas: And I'd much prefer to deal with things like num_instances > 0 outside of the filter scheduler.14:34
*** gmann_ has quit IRC14:34
bauzasjaypipes: the real problem is that since we really only supported FilterScheduler (ChanceScheduler being only for tests or crazy operators), we really didn't had a clear interface14:34
bauzasso we built a pile of dependencies between the driver, the host manager and the filters14:35
jaypipesbauzas: that and the unit tests are a pile of spaghetti that test way too much surface area and therefore if you change any little thing, a million tests crash and burn.14:35
bauzasthe code itself isn't very large, that's just the fact that we're poorly defining our internal interfaces14:35
jaypipesthat, and there's no reason for the WeighedHost class to begin with.14:35
bauzasI just feel we need to be pragmatic for Pike and do the nice stuff later14:36
jaypipesbut whatever, I'm back off the cliff edge and making slow progress.14:36
jaypipesbauzas: not sure what you mean by "nice stuff"./14:36
bauzashonestly, our internal interfaces are... internal, so I don't care with us changing those a couple of times14:36
*** acormier_ has joined #openstack-nova14:37
bauzasjaypipes: I'm talking of getting results first, and then clean up our model later14:37
*** gouthamr has joined #openstack-nova14:37
*** marst has quit IRC14:37
jaypipesbauzas: yeah... that's what has caused all of the issues in the first place :) shoot first, ask questions later.14:37
*** acormier_ has quit IRC14:37
bauzasI know14:38
*** acormier_ has joined #openstack-nova14:38
bauzasdo you remember I started changing those interfaces in 2014 ?14:38
bauzasthat's just a funny story14:38
*** slaweq_ has joined #openstack-nova14:40
*** acormier has quit IRC14:40
*** itlinux_ has quit IRC14:42
*** slaweq_ has quit IRC14:45
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895514:46
*** lucasxu has joined #openstack-nova14:46
jaypipesbauzas: I know you know. And I know you know that I know you know.14:47
*** marst has joined #openstack-nova14:47
bauzasjaypipes: wake me before you know-know14:48
bauzasaaaaaah14:48
bauzaswake me *up before you know-know14:48
* bauzas totally crashed his joke14:48
*** coreywright has quit IRC14:48
cdentgood effort though14:48
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove unnecessary function  https://review.openstack.org/48343314:49
jaypipesbauzas: hehe14:51
*** tylerderosagrund has quit IRC14:52
*** tylerderosagrund has joined #openstack-nova14:52
mriedemso we're making wham! jokes now?14:52
dansmithoh jeez, you'll get mriedem all worked up14:53
mriedemwell look who it is14:53
mriedemsomeone has a 'wham!' keyword listener14:53
*** felipemonteiro has joined #openstack-nova14:53
dansmithheh14:53
bauzas:)14:54
efriedmordred mriedem Now a good time to talk about ksa/sta/os-service-types releases in support of use-service-catalog-for-endpoints ?14:55
*** felipemonteiro_ has joined #openstack-nova14:55
*** andreas_s has quit IRC14:55
mriedemi need some time14:56
edleafejaypipes: I've bookmarked that quote14:56
jaypipesedleafe: ha! it was paraphrasing White Goodman from "Dodgeball: A True Underdog Story" (http://www.imdb.com/title/tt0364725/quotes)14:58
jaypipes"I know you. You know you. And I know you know that I know you."14:58
edleafeoh, not that quote. http://p.anticdent.org/38I14:58
jaypipesedleafe: ooohhhhh, that one.14:59
*** felipemonteiro has quit IRC14:59
edleafejaypipes: :)14:59
*** efried is now known as efried_bbiab15:00
*** Swami has joined #openstack-nova15:01
*** coreywright has joined #openstack-nova15:01
mriedemstvnoyes: i've gone through the compute manager changes on https://review.openstack.org/#/c/463987/ - there is a definite bug in the way that _detach_volume is called15:02
mriedemstvnoyes: beyond that, i'm concerned that since we don't always call rollback_live_migration_at_destination, we might not cleanup properly15:03
*** jmlowe has quit IRC15:03
mordredefried_bbiab, mriedem: ping me when you're ready15:04
*** lucasxu has quit IRC15:09
*** armax has joined #openstack-nova15:11
*** yingjun has joined #openstack-nova15:20
*** Oku_OS is now known as Oku_OS-away15:20
*** acormier has joined #openstack-nova15:22
*** acormier has quit IRC15:22
*** acormier has joined #openstack-nova15:23
*** gcb has quit IRC15:23
edleafejaypipes: let me check my understanding of compute-land with you15:25
edleafejaypipes: There is a compute service. It has a single resource tracker. That RT can have 1 or more compute nodes15:25
edleafejaypipes: Normally it's one (VMs), but for Ironic it can be (and usually is) >115:25
edleafejaypipes: is that correct?15:25
jaypipesyup15:25
edleafejaypipes: ok thx15:25
*** acormier_ has quit IRC15:25
dansmithcorrect in the technical sense15:25
jaypipesedleafe: always a single resource tracker. that resource tracker tracks >1 compute node objects in case of ironic.15:26
dansmithnot correct in the moral should-this-be-done sense15:26
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060815:26
edleafedansmith: my soul is already unclean15:26
*** isantosp has joined #openstack-nova15:26
bauzasedleafe: because jaypipes fixed that :)15:26
bauzasedleafe: we previously had 1 RT per compute "node"15:27
bauzas<3 jaypipes15:27
*** rcernin has quit IRC15:27
edleafebauzas: get a room, you two15:28
jaypipeslol15:28
*** damien_r has quit IRC15:28
isantospHi, I have an issue on nova newton, im trying to set ipv6 taking it from the port creation in neutron newton, apparently neutron is doing his job perfectly, the problem is when nova try to call port_crete_minimal, I have the following: ```15:29
bauzasedleafe: you mean #openstack-loves-jaypipes IRC room ? :p15:29
isantospsubnet_id_v4 = host_subnets['host']['available_random_subnet_v4']15:29
isantosp                port_req_body['port']['fixed_ips'] = [15:29
isantosp                    {'subnet_id': str(subnet_id_v4)}]15:29
isantospyes15:29
edleafebauzas: there are several such rooms, I imagine15:29
isantospIn the creation of the instance, i see this error: {"message": "'subnets'", "code": 500}15:30
dansmithmelwitt: I think you should roll on with the update to that patch. I'm to the tests, and my will to live is waning15:31
*** udesale has quit IRC15:31
isantospand this one in the logs15:31
isantospFile "/usr/lib/python2.7/site-packages/nova/network/neutronv2/api.py", line 392, in _create_port_minimal15:31
isantosphost_subnets = port_client.show_host(instance.host)15:31
isantospKeyError: 'subnets'15:31
*** foutatoro has quit IRC15:31
isantospany ideas? the line 392 that is pointed there, it has nothing to solve15:32
*** acormier_ has joined #openstack-nova15:35
openstackgerritAlex Szarka proposed openstack/nova master: Transform the transformed notifications functional tests  https://review.openstack.org/48344815:36
*** jwcroppe has quit IRC15:36
*** pcaruana has quit IRC15:38
*** efried_bbiab is now known as efried15:38
*** MVenesio has quit IRC15:38
*** MVenesio has joined #openstack-nova15:39
*** acormier has quit IRC15:39
sean-k-mooneyjangutter: well what specifically is the gap. you can use https://github.com/Juniper/contrail-nova-vif-driver/blob/master/vif_plug_vrouter/vrouter.py as an example of an external plugin15:40
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896415:40
*** slaweq_ has joined #openstack-nova15:41
sean-k-mooneyjangutter: for the sriov type you should call your plugins plug method with a VIFHostDevice and for the virtio forworder case you should use VIFVHostUser with a costom port profile if you need to pass vendor specific info.15:43
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004215:43
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004215:44
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492615:45
*** thorst has quit IRC15:45
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492615:45
*** thorst has joined #openstack-nova15:46
*** slaweq_ has quit IRC15:46
openstackgerritAlex Szarka proposed openstack/nova master: Transform the transformed notifications functional tests  https://review.openstack.org/48344815:48
sean-k-mooneyjangutter: was your question more how do i tell nova to allocate the vf and call the plugin?15:48
sean-k-mooneyjangutter: without a review too look at its a little hard to advise what do next15:49
*** lucasxu has joined #openstack-nova15:49
*** thorst has quit IRC15:50
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366015:52
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.resize.error notifications  https://review.openstack.org/41788215:52
mriedemjaypipes: dansmith: can one of you put the priority reviews docs patch through https://review.openstack.org/#/c/470420/15:53
dansmithsure15:53
janguttersean-k-mooney: your wish is my command, just as we speak the clearance came through.15:53
*** karimb_ has quit IRC15:54
jangutterSo my next newbie question to the room is: should I throw away the old reviews and create new ones, or replace them? what's the status quo?15:54
mriedemdansmith: https://review.openstack.org/#/c/445142/ also needs a rebase15:55
*** hshiina has joined #openstack-nova15:56
dansmithmriedem: ac15:56
dansmithk15:56
sean-k-mooneyjangutter: cool. am it really up to you. you can keep the change ids if the new code is a nautal evolution of the previous patches or abandon the old patchs and leave a link to the new patchset if its substially different15:56
jaypipesjangutter: just Abandon the old patches.15:57
jaypipesjangutter: patches are cheap :)15:57
jangutter*sniff* they're going to data heaven now.15:58
*** Apoorva has joined #openstack-nova15:58
*** acormier_ has quit IRC15:58
sean-k-mooneyjangutter: openstack never delete the review so they will live forever in bit rot land15:59
*** acormier has joined #openstack-nova15:59
jangutter*sniff* they're going to data purgatory now.15:59
openstackgerritMerged openstack/nova-specs master: Document Pike release review priorities  https://review.openstack.org/47042016:01
*** mriedem has left #openstack-nova16:01
*** mriedem has joined #openstack-nova16:01
sean-k-mooneyjangutter: jaypipes apparently this is the oldest review still visable https://review.openstack.org/#/c/131 jay you were a core on keystone once?16:01
*** dtantsur is now known as dtantsur|afk16:02
*** jwcroppe has joined #openstack-nova16:02
bretonlol. Even Dolph was not a core.16:03
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects  https://review.openstack.org/47489216:03
*** acormier has quit IRC16:03
jaypipessean-k-mooney: lol16:03
*** kristia__ has joined #openstack-nova16:04
*** lucasxu has quit IRC16:04
*** kristian__ has quit IRC16:07
sean-k-mooneyjangutter: am when you push the new patches add me to them and ill take a look at what you have end to end16:07
*** gyee has joined #openstack-nova16:09
*** jwcroppe_ has joined #openstack-nova16:09
*** sridharg has quit IRC16:11
jaypipesjangutter: yes, purgatory is more appropriate for the patches that I personally abandon (there have been many, many of them)16:12
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948316:13
*** jwcroppe has quit IRC16:13
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - List/Show all server migration types  https://review.openstack.org/43083916:15
openstackgerritJan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement  https://review.openstack.org/48345916:16
*** jwcroppe_ has quit IRC16:16
jangutterWarning: these patches have been ripped from the Netronomicon.16:17
openstackgerritAlex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update  https://review.openstack.org/46629616:17
*** saneax has quit IRC16:18
*** tesseract has quit IRC16:19
*** rajathagasthya has joined #openstack-nova16:19
*** jamielennox has quit IRC16:19
*** slaweq has quit IRC16:20
jangutterjaypipes, sean-k-mooney: thanks very much for the infinite patience. The ML2/OS-VIF external plugin's on github, and I'll upload the neutron bits in an hour or so.16:20
*** lucasxu has joined #openstack-nova16:21
*** lucasagomes is now known as lucas-afk16:22
*** eharney has quit IRC16:23
jaypipesjangutter: no problem at all.16:23
*** eharney has joined #openstack-nova16:26
*** jwcroppe has joined #openstack-nova16:27
*** gbarros has quit IRC16:27
*** kfarr has joined #openstack-nova16:30
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - Enable cold migration with target host  https://review.openstack.org/40670716:33
*** takashin has left #openstack-nova16:34
*** tbachman has joined #openstack-nova16:34
*** jwcroppe_ has joined #openstack-nova16:34
mriedembauzas: ok email to the ML sent about bps close to done16:35
openstackgerritzhang zhenzhong proposed openstack/nova master: fix resource-provider regist failed in db while placement endpoint created after nova-compute start  https://review.openstack.org/48346016:36
*** jwcroppe has quit IRC16:37
openstackgerritzhang zhenzhong proposed openstack/nova master: fix resource-provider regist failed in db while placement endpoint created after nova-compute start  https://review.openstack.org/48346016:38
*** hshiina has quit IRC16:38
*** gouthamr has quit IRC16:41
*** zzzeek_ has quit IRC16:41
*** dtp has joined #openstack-nova16:41
*** gouthamr has joined #openstack-nova16:42
*** lpetrut_ has quit IRC16:42
*** slaweq has joined #openstack-nova16:42
*** zzzeek has joined #openstack-nova16:43
*** itlinux has joined #openstack-nova16:45
mriedemjaypipes: https://review.openstack.org/#/c/482383/ is the current focus right now right?16:45
mriedemit's not https://review.openstack.org/#/c/482381/16:45
mriedemthe dep tree is confusing me16:46
*** thorst has joined #openstack-nova16:46
jaypipesmriedem: I needed to go in a different direction on those. I will abandon both.16:47
mriedemok16:47
mriedemwell claim_resources() is +W so yay16:47
jaypipesyeah, that's fine.16:47
jaypipesself contained.16:47
jaypipesshould have new series pushed today.16:47
mriedemack16:47
*** slaweq has quit IRC16:47
*** sree has quit IRC16:50
*** sree has joined #openstack-nova16:51
*** felipemonteiro_ has quit IRC16:51
*** sree has quit IRC16:55
*** Apoorva_ has joined #openstack-nova16:56
*** dtp has quit IRC16:57
* cdent sends jaypipes some beer, pizza, coffee, a pillow, two bottle of jack, some crunchie bars and spare pug16:59
cdentuse as you see fit16:59
*** ralonsoh has quit IRC16:59
*** Apoorva has quit IRC17:00
jaypipescdent: you can keep the spare pug. just double the jack.17:00
*** fragatina has joined #openstack-nova17:00
cdentnoted17:00
cdentI’ve kinda sorta always wanted a pug of my very own17:00
openstackgerritzhang zhenzhong proposed openstack/nova master: fix rp regist failed while placement endpoint created after nova-compute start  https://review.openstack.org/48346017:01
jaypipescdent: they are great, in moderation.17:03
jaypipescdent: three is my limit.17:03
*** damien_r has joined #openstack-nova17:03
openstackgerritzhang zhenzhong proposed openstack/nova master: fix rp registration failed while placement endpoint created after compute start  https://review.openstack.org/48346017:06
openstackgerritMatt Riedemann proposed openstack/nova master: Use oslo.polcy DocumentedRuleDefault  https://review.openstack.org/47263417:06
openstackgerritMatt Riedemann proposed openstack/nova master: Update policy descriptions for base.  https://review.openstack.org/47262217:06
openstackgerritMatt Riedemann proposed openstack/nova master: Consistent policies  https://review.openstack.org/47977417:06
openstackgerritMatt Riedemann proposed openstack/nova master: Remove 'create_rule_default'  https://review.openstack.org/47977317:06
mriedemeasy +W here https://review.openstack.org/#/c/479773/17:07
jaypipes143gg17:08
jaypipesguh17:08
*** yamahata has joined #openstack-nova17:08
*** fragatina has quit IRC17:11
*** sambetts is now known as sambetts|afk17:14
*** tylerderosagrund has quit IRC17:15
*** tylerderosagrund has joined #openstack-nova17:16
*** jwcroppe has joined #openstack-nova17:20
openstackgerritMatt Riedemann proposed openstack/nova master: Update policy descriptions for base.  https://review.openstack.org/47262217:20
openstackgerritMatt Riedemann proposed openstack/nova master: Consistent policies  https://review.openstack.org/47977417:20
*** damien_r has quit IRC17:21
*** jwcroppe_ has quit IRC17:22
*** liusheng has quit IRC17:25
cdentmriedem: if you’re in a policy kind of way, does this make sense: https://review.openstack.org/#/c/481443/17:25
*** liusheng has joined #openstack-nova17:26
*** harlowja has quit IRC17:27
*** kristia__ has quit IRC17:28
*** kristian__ has joined #openstack-nova17:28
*** yingjun has quit IRC17:29
*** sshwarts has quit IRC17:30
*** kristian__ has quit IRC17:32
*** slaweq has joined #openstack-nova17:35
*** slaweq has quit IRC17:35
*** slaweq has joined #openstack-nova17:36
melwittinternet is on the fritz again. comcast ...17:37
melwittdansmith: roger that. will update17:37
mriedemcdent: oh boy17:38
mriedemso, it usually takes me about 3 days to figure out where the rules are defined17:39
melwittdansmith, mriedem I have been wondering about the behavior change for resize confirm/revert. currently, it's possible to fail confirm/revert with OverQuota. during earlier discussions on the quota patch review (the monolith) we were thinking we should change the behavior and not block confirm/revert and instead limit max pending resizes17:39
melwittshould we keep with that change or should we maintain the current behavior?17:39
cdentthis is part of why I’m asking because me<->policies, not friends17:39
mriedemmelwitt: i'm generally not in favor of behavior changes slipping in with major refactors17:40
mriedembecause it adds all the extra complication17:40
mriedembut i don't know why the original decision was made17:40
mriedemyou said that was from johnthetubaguy17:40
mriedem?17:40
melwittI thought it was johnthetubaguy who suggested it but I might be remembering that wrong. it was months ago17:41
melwittyeah, I feel like I'm not too into the idea of changing the behavior either. I was thinking about it and I don't think you can get stuck in a limbo state with a pending resize17:41
mriedemcdent: so i think the answer is it comes from somewhere in the bowels of keystone17:42
melwittif you get blocked on confirm, you can revert. if you get blocked on revert, you confirm. unless I'm missing something17:42
mriedemso you'd get blocked on confirm if you are resizing up, yes?17:42
*** tbachman has quit IRC17:42
melwittyes17:43
mriedemhow do you get blocked on revert? resizing down?17:43
mriedemso the revert goes up17:43
melwittyes17:43
melwittresize down generally isn't a thing but I think it's allowed for BFV17:43
mriedemsince we don't have reservations for resize now,17:43
*** slaweq_ has joined #openstack-nova17:43
mriedemwell,17:43
*** feefifofum has joined #openstack-nova17:43
mriedemnvm17:43
mriedemi was thinking somehow reservations in the old way helped this here17:43
mriedembut that would only help when you're resizing up17:44
*** gjayavelu has joined #openstack-nova17:44
mriedemwell either way i guess,17:44
mriedembut yeah if this was a way you could fail over quota before, i don't know why we wouldn't just continue to count quota on confirm/revert17:44
mriedemand fail if the current state of the world has changed and you're screwd17:44
mriedem*screwed even17:44
openstackgerritMerged openstack/nova master: api-ref: Add missing parameters in limits.inc  https://review.openstack.org/47564017:45
feefifofumhi, i'm working on upgrading from liberty to ocata and noticed a difference in how hypervisor memory usage is reported17:45
feefifofumin liberty, memory_mb - memory_mb_used = free_ram_mb17:46
feefifofumhowever in ocata, there is a different quantity now17:46
melwittyeah17:46
feefifofummemory_mb - mb_used - x = free_ram_mb17:47
feefifofumi don't know what x is though17:47
mriedemfeefifofum: you know that skipping from L to O isn't supported upstream right? and you'd need to check the release notes for M, N and O first17:47
*** slaweq_ has quit IRC17:48
mriedemlbragstad: where is the default policy rule defined in keystone again?17:48
mriedemi can never find it17:48
mriedemit's not this https://github.com/openstack/keystone/blob/master/etc/policy.v3cloudsample.json#L1117:48
*** fragatina has joined #openstack-nova17:48
feefifofummriedem: yes i'm aware of the release notes17:50
sean-k-mooneyjangutter: i think this is legacy code that you dont need https://github.com/Netronome/agilio-ovs-openstack-plugin/blob/90c38378b772c8f82d5bc1752df60ec5055dca23/vif_plug_agilio_ovs/objects/vif.py#L24-L30 also multiple inheritence for multiple oslo_versioned objects will likely not work out well for you17:50
mriedemfeefifofum: are you looking in code?17:50
mriedemit could be an overhead value17:50
sean-k-mooneyjangutter: on the plus side https://github.com/Netronome/agilio-ovs-openstack-plugin/search?utf8=%E2%9C%93&q=VIFAgilioOpenVSwitch&type= its not used anyware in the code17:50
mriedemcalculated from the virt driver17:50
*** jwcroppe_ has joined #openstack-nova17:51
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L19217:51
*** liangy has quit IRC17:51
feefifofumhmmm, this looks interesting. i'll take a closer look17:51
feefifofumthanks mriedem17:51
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/claims.py#L10517:51
*** jwcroppe has quit IRC17:52
*** krtaylor has quit IRC17:55
*** feefifofum has quit IRC17:55
lbragstadmriedem: we removed it17:57
lbragstadmriedem: all of our policy bits are here - https://github.com/openstack/keystone/tree/b3fc06c9ef6a0a3d317f414962bab3d309c2e6c0/keystone/common/policies17:58
lbragstadthe basic rules are in base.py - https://github.com/openstack/keystone/blob/b3fc06c9ef6a0a3d317f414962bab3d309c2e6c0/keystone/common/policies/base.py#L15-L3117:58
*** Swami has quit IRC17:59
lbragstadmriedem: edmondsw has proposed the removal of the default rule - https://review.openstack.org/#/c/482164/17:59
*** rajathag_ has joined #openstack-nova18:00
*** rajathagasthya has quit IRC18:01
*** mvk has quit IRC18:01
*** tbachman has joined #openstack-nova18:01
*** felipemonteiro has joined #openstack-nova18:03
*** pchavva has quit IRC18:03
*** eharney has quit IRC18:03
*** felipemonteiro_ has joined #openstack-nova18:04
*** FL1SK has quit IRC18:04
*** slaweq has quit IRC18:06
*** roeyc has joined #openstack-nova18:06
*** Apoorva_ has quit IRC18:06
*** Apoorva has joined #openstack-nova18:07
mriedemhmm18:07
mriedemhttps://github.com/openstack/keystone/blob/b3fc06c9ef6a0a3d317f414962bab3d309c2e6c0/keystone/common/policies/base.py#L61 ok18:07
mriedemwell that's what i was looking for anyway18:07
mriedemcdent: ^18:08
*** felipemonteiro has quit IRC18:08
lbragstadmriedem: yeah - we're going to be removing that since every operation will be required to have a default18:08
cdentwhat does it mean?18:08
mriedemlbragstad: so does that mean if nova didn't defined a default rule, we wouldn't have a default at all then?18:09
mriedemif keystone didn't define it?18:09
mriedemor something else18:09
mriedemhttps://review.openstack.org/#/c/481443/4/nova/policies/base.py18:09
lbragstadthat default is specific to keystone APIs18:09
mriedemok that's what i was wondering18:09
lbragstadso if you made a request to keystone that we didn't have a policy entry for - we would normally apply that default18:09
mriedemand these are the nova-api specific defaults https://review.openstack.org/#/c/481443/4/nova/policies/base.py18:09
lbragstadbut since each policy is formally moved into code and documented - a default of defaults doesn't really make sense18:09
*** efoley has quit IRC18:10
lbragstadaha - sure18:10
lbragstadyeah - removing the keystone-specific defaults shouldn't impact that for nova18:10
*** roeyc has quit IRC18:10
mriedemnova's default is different anyway18:10
lbragstadright - and that makes sense18:11
lbragstadi wouldn't expect each service's default rule to be the same18:11
cdent(I think the issue in the review is more that policy file generation was leaving the default out? https://review.openstack.org/#/c/481443/ and if you try to use that file elsewhere, things go awry?)18:12
mriedemcdent: i left a question in there18:12
cdentthanks18:12
mriedemcompute:get_all is an old legacy v2 API rule i think18:12
mriedemseems like a horizon bug actually18:13
lbragstadspeaking of policy - i was thinking about organizing a session in Denver focused on next steps (much like what we did in atlanta)18:13
lbragstadis anyone here interested in that?18:13
cdentcould very welll be18:13
mriedemi'm not interested in thinking about denver at all right now18:14
lbragstad:) that's fair18:15
robcresswellAh, we still have references to the old style policy rules18:15
robcresswellI can fix that.18:15
*** kristian__ has joined #openstack-nova18:15
mriedemrobcresswell: someone already did https://github.com/openstack/horizon/blob/master/openstack_dashboard/conf/nova_policy.json18:16
mriedemcompute:get_all isn't in there18:16
*** slaweq has joined #openstack-nova18:16
mriedemunless it's hard-coded18:16
robcresswellmriedem: I co-authored on that. We've left a check on the rule in our admin > instances panel18:16
robcresswellhttps://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/admin/instances/panel.py#L2918:17
robcresswellIs there a relevant bug open in Nova I can link to?18:17
mriedemhttps://bugs.launchpad.net/horizon/+bug/170306618:17
openstackLaunchpad bug 1703066 in OpenStack Compute (nova) " Add "default" rule when generate policy.yaml.sample" [Undecided,Incomplete] - Assigned to KaiLi (damonl1)18:17
mriedemjust added horizon to it18:17
robcresswellThanks18:17
*** slaweq has quit IRC18:17
mriedemnp18:17
mriedemcdent: -1 on the nova change then18:17
cdentthanks to all three of you18:18
cdentyeah, totes18:18
*** kristian__ has quit IRC18:18
*** lyan has joined #openstack-nova18:20
robcresswellWhats the equivalent of "get_all"? "os_compute_api:servers:index"?18:21
mriedemyeah18:21
mriedemwell,18:21
mriedemdepends, do you need details?18:21
mriedembecause then it would be os_compute_api:servers:detail18:21
mriedem"os_compute_api:servers:detail"18:21
mriedemblast18:21
robcresswellThat particular check is just saying "can I list all instances" I believe.18:22
robcresswellon the horizon side, I mean18:22
mriedemi'll check some old nova code18:22
*** rajathag_ has quit IRC18:23
robcresswellIt looks like that last patch replaced get_all with "os_compute_api:servers:index:get_all_tenants" in a couple of places18:23
mriedemthat would be wrong18:23
mriedemthe old version of that was compute:get_all_tenants18:23
*** acormier has joined #openstack-nova18:23
robcresswellgotcha18:23
robcresswellthanks18:23
mriedemthe all tenants stuff is admin only by default18:23
mriedemsince tenant A shouldn't be able to list instances from tenant B18:24
mriedemcompute:get_all was used for both index and detail in the v2 API18:24
mriedemso you can use the index or detail one, either is fine18:24
mriedemwell, i mean it really depends on what you're going to be doing - if you're going to be getting servers with details to show details, then you want the details one18:25
robcresswellIs that the same as viewing the details of a specific server?18:25
mriedemnope18:25
mriedem:)18:25
robcresswellWhats the policy for that?18:26
* robcresswell looks18:26
mriedemi think that's "os_compute_api:servers:show"18:26
robcresswellAh! Yes18:26
robcresswellVery good18:26
robcresswellmriedem: https://review.openstack.org/#/c/483501/18:28
openstackgerritzhang zhenzhong proposed openstack/nova master: fix rp registration failed while placement endpoint created after compute start  https://review.openstack.org/48346018:29
dansmithmelwitt: agree that less behavioral change is better18:29
melwittcool. I think we all agree, so I'll nix the pending resizes stuff18:30
robcresswellmriedem: Right, I'll get that merged ASAP and backport it. Thanks for the help.18:30
melwittand put back the current behavior18:30
mriedem+118:30
*** krtaylor has joined #openstack-nova18:31
mriedemmelwitt: cool, that means you can also drop the 2nd change in the series too right?18:31
dansmithI think so18:31
mriedemf yes18:31
mriedemthat's like 15% better18:31
dansmithhehe18:31
melwitt /fistpump18:32
dansmithif we round to 20% less (code to merge) we could all take tomorrow off18:32
*** Swami has joined #openstack-nova18:32
melwitthaha18:32
*** mvk has joined #openstack-nova18:35
*** rajathagasthya has joined #openstack-nova18:35
*** tbachman_ has joined #openstack-nova18:35
mriedemi have to work on my pbc tomorrow18:36
mriedemso no dice18:36
*** shaohe_feng has quit IRC18:36
*** jamielennox has joined #openstack-nova18:37
*** shaohe_feng has joined #openstack-nova18:37
*** tbachman has quit IRC18:38
*** tbachman_ is now known as tbachman18:38
*** pcaruana has joined #openstack-nova18:38
dansmithmriedem: oh horrors of horror18:39
dansmither, horror of horrors?18:39
dansmithwhatever. horror-ible.18:39
mriedemha18:39
mriedemdon't make me get out ralph wiggum again18:39
dansmithhehe18:39
melwittwhat's pbc18:40
mriedempersonal business commitments18:40
mriedemgoals18:41
mriedemlike, "i'm going to synergize for cloud enterprise adoption 2.0"18:41
*** kristian__ has joined #openstack-nova18:41
melwitthah18:42
*** lyan has quit IRC18:43
edleafemriedem: don't forget the cover page for the TPS report18:44
edleafemriedem: you did get the memo, right?18:44
*** thorst has quit IRC18:44
jaypipesmriedem: don't forget to unprison your think rhino.18:44
mriedemi did put a cover page on my expense claim and throw that in the mail today18:44
openstackgerritzhang zhenzhong proposed openstack/nova master: fix rp registration failed while placement endpoint created after compute start  https://review.openstack.org/48346018:45
* cdent visualizes his think rhino, free at last18:46
melwittlol think rhino? unprison?18:46
cdentjaypipes: you may have some opinions on that ^^18:46
jaypipesmelwitt: https://twitter.com/e_monty/status/85254010631762739218:50
melwittlol18:51
edleafejaypipes: ah, that totally clears everything up18:51
jaypipeshehe18:51
openstackgerritzhang zhenzhong proposed openstack/nova master: remove un-necessary update() in _init_compute_node of rt  https://review.openstack.org/48350618:53
*** harlowja has joined #openstack-nova18:54
* melwitt opens the tome of review comments to address18:56
*** cdent has quit IRC19:00
dansmithmriedem: this event rebase is taking a long time because of your silly volume extend nonsense, but I'm about done19:04
openstackgerritDan Smith proposed openstack/nova master: Fix and optimize external_events for multiple cells  https://review.openstack.org/44514219:04
*** nicolasbock has quit IRC19:04
mriedemboohoo19:07
melwitthehehe19:07
dansmithmriedem: you guys are only on cloud 2.0?19:08
dansmithI think we're on 4.0 now19:09
*** gbarros has joined #openstack-nova19:10
*** kristian__ has quit IRC19:14
*** kristian__ has joined #openstack-nova19:15
*** kfarr has quit IRC19:17
*** slaweq has joined #openstack-nova19:18
*** sbezverk has quit IRC19:21
*** sbezverk has joined #openstack-nova19:22
*** slaweq has quit IRC19:23
*** acormier_ has joined #openstack-nova19:24
*** acormier_ has joined #openstack-nova19:24
melwittcomcast is here to replace a line outside so I might drop off while they're working19:25
*** acormier has quit IRC19:26
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Microversion 2.50 - fix quota class sets resource usage  https://review.openstack.org/48306019:27
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Add support for the 2.51 microversion  https://review.openstack.org/48314919:27
*** kristian__ has quit IRC19:32
mriedemdansmith: melwitt: so we talked about some testing we can do against the quotas change,19:35
mriedemnovaclient functional runs in serial with a single tenant19:35
mriedemnormal tempest jobs are a unique tenant per test so if we're leaking quotas we won't find it in the normal runs19:35
mriedemso i was thinking about doing something in novaclient as a DNM change and point it at the quotas patch19:36
mriedemlike run something 20 times19:36
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48333719:36
mriedemthoughts on what we'd want to cover there? i think normal server create/delete is already covered by the existing novaclient functional tests19:36
mriedemso was thinking about adding a novaclient functional test for resize that we could then run like 20 times in a loop19:36
mriedem*we talked about this in the nova meeting i mean19:37
mriedemthoughts on what to test there? like resize up/confirm, resize down/confirm * 20, and resize up/revert, resize down/revert * 2019:38
*** slaweq has joined #openstack-nova19:38
*** kristian__ has joined #openstack-nova19:39
*** tbachman has quit IRC19:39
*** tbachman has joined #openstack-nova19:39
melwittsoft delete/restore19:41
mriedemok19:41
melwittmulti create19:41
mriedemhmm, actually, i suppose this is why tempest doesn't test soft delete,19:42
mriedemis that the soft delete behavior is based on config...19:42
mriedemand by default it doesn't soft delete19:42
melwittyeah19:42
mriedemso would have to test that one manually19:42
*** kristian__ has quit IRC19:43
*** kristian__ has joined #openstack-nova19:45
*** slaweq has quit IRC19:49
*** slaweq has joined #openstack-nova19:50
*** lucasxu has quit IRC19:50
*** pcaruana has quit IRC19:52
*** Apoorva_ has joined #openstack-nova19:54
*** Apoorva has quit IRC19:57
*** pchavva has joined #openstack-nova19:57
openstackgerritJackie Truong proposed openstack/nova master: [WIP] Implement certificate_utils  https://review.openstack.org/47994919:58
*** tbachman_ has joined #openstack-nova20:02
*** liverpooler has quit IRC20:03
*** FL1SK has joined #openstack-nova20:03
*** armax has quit IRC20:03
openstackgerritJackie Truong proposed openstack/nova master: [WIP] Add trusted certificates to InstanceExtras  https://review.openstack.org/45771120:05
*** tbachman has quit IRC20:05
*** tbachman_ is now known as tbachman20:05
*** mtanino__ has joined #openstack-nova20:08
dansmithmriedem: do you recall anyone saying they actually use soft delete?20:08
mriedemdansmith: no20:09
dansmithgiven it's not user-visible (control anyway) we might want to seriously consider yanking it out, IMHO20:10
cburgessdansmith: +120:10
mriedemi'm asking in ops20:10
mriedemso will get back to you in 3 days20:10
dansmithmaybe do some targeted data gathering on the idea20:10
cburgessdansmith Its mostly useless in its current form.20:10
dansmithhah20:10
dansmithcburgess: agreed, and it's constantly broken, untested, etc20:10
mriedemsdague and i have kicked around ideas on data mining api usage20:10
mriedemb/c we've asked this about shelve too20:11
mriedemthis would also be a good user survey question20:11
cburgessmriedem Our product team keeps asking for that. So maybe its something of value to add in optionally for people to turn on/off20:11
dansmithI know people that use shelve20:11
cburgessdansmith I'm sorry20:11
mriedemcburgess: keeps asking for what?20:11
mriedemcburgess: api usage data mining?20:11
cburgessmriedem Our product team wants to be able to data mine the API calls to figure out which clients use which features etc.20:11
mriedemah20:11
mriedemyeah, i don't think we'd need to turn it on/off,20:12
mriedemit'd be nova-manage20:12
mriedemmine the instance actions table20:12
dansmithcburgess: that's a script of processing the log right?20:12
mriedempretty simple actually20:12
mriedemthis data is all in instance actions already20:12
cburgessWhich one can do by scraping the logs but.. they want want something a bit more sophsiticated20:12
cburgessdansmith Yeah we have been trying to get the data team to do something with it... mixed results.20:12
mriedemselect * from instance_actions where action == 'shelve'20:12
cburgessmriedem I don't think instance_action is enough for all the stuff product has in mind. You have to look at args at times to figure out whats being done in the call. For instannce boot from volume etc.20:13
dansmithfor client versions it does nothing20:13
melwittnotifications would be a good way to mine20:13
cburgessType of security groups rules (nested vs not) etc.20:13
cburgessmelwitt yeah true...20:13
mriedemi'm just looking at basic things right now20:13
cburgessmriedem Fair enough...20:14
mriedem"of all instance actions in this cloud, what % is shelve?"20:14
mriedembasically first iteration is a report of the count of instance actions in a cloud20:14
mriedemwhich is pretty simple20:14
dansmithhmm, not sure what use that is unless you're comparing to another20:14
mriedem?20:15
mriedemif we know there are 100 instance actions in the db, and 5 are shelve, then we know 5% is shelve20:15
dansmithon an active cloud, 10% shelve would be huge. on a static "we just moved our workloads to cloud", 10% wouldn't mean as much20:15
mriedemyeah, again, baby steps20:15
mriedemyou have to be able to mine the data in a single install, and then start aggregating that probably in the user survey, or some other tool, idk20:15
mriedembased on the size of the cloud, which is probably based on # of vms and/or # of computes20:16
dansmiththat's what I'm saying, it only really means something in comparison20:16
mriedemif i'm asking a binary yes/no question,20:16
mriedemit means something20:16
mriedemala user survey, does your cloud use shelve20:16
mriedemor,20:16
mriedemdo users of your cloud use shelve20:16
mriedemyes/no20:16
melwittI think the user survey is out right now20:17
mriedemyeah this would be 2018 at this point20:17
mriedemfor the user survey20:17
melwittyeah20:17
mriedembut,20:17
dansmithmriedem: but you don't want someone with one shelve ever to answer yes20:17
dansmithanyway, not arguing with data of course, I'm just not sure what I'd make of "10% shelve" as an answer20:17
mriedemif we provided a thing in nova-manage, we could also ask people to run that in the ops list and report back their findings20:17
melwittdo you use shelve A Lot(tm)?20:17
mriedemwhich is skewed i know, but so is the user survey20:17
dansmithI mostly care about "would you freak if we removed feature X?"20:18
mriedemyes you'd have to account for outliers like 1 usage of 100020:18
mriedemgdi you bastards20:18
mriedemi'm going home20:18
melwittlol20:18
mriedemgetting something like this fed into the interop wg could also be a possibility20:19
mriedemas they are feeding their interop results in there regularly20:19
mriedembut,20:19
dansmith1/1000 is probably a typo, 10/10000 is probably one user that loves shelve20:19
mriedeminterop is also like products like distros, not necessarily active clouds20:19
*** harlowja has quit IRC20:19
mriedemso that doesn't work as well20:19
mriedemif we can get the data, we can give it to sdague who can give it to watson20:20
mriedemwho will then turn it into trivia20:20
dansmithhah20:20
dansmithWhat is "kill soft delete" alex/20:20
mriedemfor 50020:20
mriedemooo that's the mothertruckin daily double!20:20
*** smatzek has quit IRC20:21
*** lpetrut_ has joined #openstack-nova20:21
* melwitt cues sound effect20:21
*** harlowja has joined #openstack-nova20:22
dansmithhttps://i.ytimg.com/vi/3J2PlJmkvog/maxresdefault.jpg20:22
melwittlol20:22
mriedemso,20:23
mriedemif i'm writing this resize test that gets the quota usage before the resize,20:23
mriedemand then gets the reserved quota usage after the resize,20:23
mriedembefore counting quotas they are going to be the same,20:23
mriedembut after counting quotas they are going to be different b/c we don't reserve, right?20:23
mriedemlooking at this https://github.com/openstack/nova/blob/ac20980702030424ce4bb3ddf2610d31716a8bd7/nova/api/openstack/compute/used_limits.py#L6520:24
melwittwell, with counting quotas we won't be touching the usages table, so neither usage nor reservation will be changed. so they should be the same (unchanged)20:24
mriedemthat tells the limits API whether or not to include 'reserved' quota in the response for usage20:24
melwittoh, you mean get usage as in, from the used limits API?20:24
melwittthat will show you the new counted usage20:25
mriedemyes20:25
melwittokay20:25
mriedemso once the server is in verify_resize state, it's going to count against quota?20:25
melwittin the change it always sets reserved to show 020:25
mriedemfor the new size, if i'm resizing up for example?20:25
melwittit's not pulling it from the usages table, it's just hard-coded to be 020:25
dansmithis reserved for resizing instances?20:26
mriedemreserved is just the placeholder for quota usage until the operation is complete20:26
melwittit's for everything. yeah20:26
mriedemusually committed or rolled back later20:26
melwittot20:26
dansmithoh I guess reserved is reservations that haven't been committted20:26
dansmithgotcha20:26
melwittit's for all ReservableResource20:26
mriedemlike, that's why the reservations are passed from api to compute20:26
mriedemyeah, so i'm trying to figure out how i should write the functional test,20:26
*** slaweq has quit IRC20:26
mriedembecause that's what i was goign to do, check usage before the resize, and compare to usage after the resize is in verify_resize state20:27
mriedemand today they will be different if you include the reserved value in the results20:27
mriedemhttps://developer.openstack.org/api-ref/compute/?expanded=show-a-quota-detail,show-rate-and-absolute-limits-detail#id22520:27
mriedem120:27
mriedem"Specify whether the result of resource total includes reserved resources or not."20:27
melwittwell, I see that resize reservation is committed in compute after the resize request. so that happens before confirm/revert20:29
melwittso when it's in verify_resize it should have reserved == usage currently, right?20:29
melwittsorry20:30
melwittreserved will be 020:30
mriedembut in_use would be incremented20:30
melwittright20:30
mriedemok20:30
mriedemso what will happen with counting quotas?20:30
melwittusage should be incremented and reserved will be 0 (reserved is hard-coded to 0 always)20:30
mriedembecause instance.memory_mb is taken from new_instance_type['memory_mb'] somewhere?20:31
mriedemb/c the count is now taken from instance.memory_mb right?20:31
melwittlemme see ...20:31
melwittyeah, as long as the new flavor has been saved to the instance20:31
melwittI'm looking where that happens. pretty sure it's regardless of confirm/revert20:32
melwittyeah, in finish_resize() (before confirm/revert) it sets the new flavor. so at that time the new usage should be counted20:32
melwittif an instance is sitting in verify_resize it's consuming more resources (if a resize up)20:33
mriedemah Specify whether the result of resource total includes reserved resources or not.20:33
mriedemoops20:33
mriedem_set_instance_info20:33
mriedemin the compute manager20:33
melwittyeah20:34
mriedemwe should probably remove this comment then :) https://github.com/openstack/nova/blob/ac20980702030424ce4bb3ddf2610d31716a8bd7/nova/compute/manager.py#L400620:34
mriedemsince the quota count is goign to be completely based on these now20:34
*** tbachman_ has joined #openstack-nova20:34
melwittyeah, true20:35
mriedemi've noted it in your change20:35
mriedemok on with the test20:35
melwittI'm trying to think how a confirm could ever fail quota check. I think it couldn't because all confirm does is remove the stuff from the source host and other cleanup. it wouldn't do anything to change the usage20:35
melwittjust thinking out loud20:36
mriedemthat sounds right20:36
mriedemso i'll get this test up and it will test the existing behavior, and then put a change on top to depend on your patch to test the new behavior20:36
melwittso just makes me wonder why it checks quota on a confirm at all right now20:36
melwittk, cool.20:37
mriedemshrug20:37
*** tbachman has quit IRC20:37
*** tbachman_ is now known as tbachman20:37
melwittin a code comment says it does it only for a decrease in usage. ah, because on resize it only makes a reservation for an upsize. never for a downsize20:38
*** gszasz has quit IRC20:39
melwittso this is adjusting usage for a downsize only on confirm. guh, how confusing20:39
melwittso for our new counting stuff, we shouldn't need to check quota on confirm. only on revert20:39
*** krtaylor has quit IRC20:41
mriedemand only on revert when upsizing?20:44
mriedemor we just won't make a distinction?20:44
melwittyeah. it could save wasted query to only do it on a revert of a downsize20:44
melwittso I think that's a good idea20:45
*** armax has joined #openstack-nova20:45
*** MVenesio has quit IRC20:51
*** lyan has joined #openstack-nova20:53
*** tylerder_ has joined #openstack-nova21:01
*** awaugama has quit IRC21:02
*** tylerderosagrund has quit IRC21:04
*** tylerder_ has quit IRC21:06
*** esberglu has quit IRC21:12
*** smatzek has joined #openstack-nova21:15
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation  https://review.openstack.org/48355921:16
*** smatzek has quit IRC21:20
*** kylek3h has quit IRC21:20
*** jmlowe has joined #openstack-nova21:21
*** zzzeek has quit IRC21:22
*** zzzeek has joined #openstack-nova21:24
*** esberglu has joined #openstack-nova21:26
*** krtaylor has joined #openstack-nova21:26
*** pchavva has quit IRC21:27
*** Apoorva_ has quit IRC21:29
*** Apoorva has joined #openstack-nova21:30
mriedemhttps://bugs.launchpad.net/cinder/+bug/169450921:31
openstackLaunchpad bug 1694509 in Cinder "eventlet errors and cannot switch to a different thread" [Undecided,Confirmed]21:31
mriedemoops21:31
openstackgerritJay Pipes proposed openstack/nova master: scheduler: isolate _get_sorted_hosts()  https://review.openstack.org/48356421:33
openstackgerritJay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests()  https://review.openstack.org/48356521:33
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356621:33
jaypipescdent, mriedem, dansmith, edleafe, bauzas: ok, I know it's late in the day, but I got it pushed ^^21:33
*** thorst has joined #openstack-nova21:42
*** jaypipes has quit IRC21:42
*** gouthamr has quit IRC21:47
*** thorst has quit IRC21:48
*** gouthamr has joined #openstack-nova21:49
*** tylerderosagrund has joined #openstack-nova21:50
*** sree has joined #openstack-nova21:51
*** jdandrea has quit IRC21:51
*** tylerderosagrund has quit IRC21:54
*** tylerderosagrund has joined #openstack-nova21:54
*** claudiub has quit IRC21:55
*** sree has quit IRC21:56
*** tylerderosagrund has quit IRC21:56
*** tylerderosagrund has joined #openstack-nova21:57
*** saneax has joined #openstack-nova22:01
*** tylerderosagrund has quit IRC22:04
*** tylerderosagrund has joined #openstack-nova22:06
*** lpetrut_ has quit IRC22:06
*** burt has quit IRC22:09
*** gouthamr has quit IRC22:10
*** slaweq has joined #openstack-nova22:10
*** tylerderosagrund has quit IRC22:11
*** tylerderosagrund has joined #openstack-nova22:19
*** tylerder_ has joined #openstack-nova22:22
*** tylerderosagrund has quit IRC22:22
*** tylerder_ has quit IRC22:23
*** tylerderosagrund has joined #openstack-nova22:23
*** tylerderosagrund has quit IRC22:24
*** tylerderosagrund has joined #openstack-nova22:24
*** slaweq has quit IRC22:25
*** tylerderosagrund has quit IRC22:25
*** tylerderosagrund has joined #openstack-nova22:25
*** felipemonteiro_ has quit IRC22:27
*** slaweq has joined #openstack-nova22:42
*** baoli has quit IRC22:45
*** slaweq has quit IRC22:46
*** kristian__ has quit IRC22:47
*** raunak has joined #openstack-nova22:47
*** lyan has quit IRC22:48
*** baoli has joined #openstack-nova22:52
*** jmlowe has quit IRC22:57
*** itlinux has quit IRC22:58
openstackgerritMerged openstack/nova master: Fix wrong log parm.  https://review.openstack.org/48283223:01
*** catintheroof has quit IRC23:03
*** feefifofum has joined #openstack-nova23:04
*** mtanino__ has quit IRC23:05
*** slaweq has joined #openstack-nova23:08
*** jwcroppe_ has quit IRC23:09
*** jwcroppe has joined #openstack-nova23:09
*** kristian__ has joined #openstack-nova23:12
*** slaweq has quit IRC23:12
*** tylerderosagrund has quit IRC23:12
*** jwcroppe has quit IRC23:13
*** rajathagasthya has quit IRC23:14
*** kristian__ has quit IRC23:17
*** Swami has quit IRC23:30
*** jamesden_ has quit IRC23:33
*** hongbin has quit IRC23:34
*** jamesdenton has joined #openstack-nova23:34
openstackgerritMerged openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vm_utils.py  https://review.openstack.org/47874923:36
*** slaweq has joined #openstack-nova23:37
*** slaweq has quit IRC23:41
*** jwcroppe has joined #openstack-nova23:44
*** thorst has joined #openstack-nova23:48
*** vishwanathj has quit IRC23:53
*** vishwanathj has joined #openstack-nova23:54
*** thorst has quit IRC23:55
*** itlinux has joined #openstack-nova23:56

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