Friday, 2017-09-01

*** Apoorva has quit IRC00:01
*** Apoorva_ has quit IRC00:02
*** edmondsw has quit IRC00:03
*** edmondsw has joined #openstack-nova00:05
*** aries has quit IRC00:06
*** edmondsw has quit IRC00:08
*** edmondsw has joined #openstack-nova00:09
*** rajathagasthya has joined #openstack-nova00:11
*** edmondsw has quit IRC00:13
*** tetsuro has joined #openstack-nova00:17
*** gouthamr has joined #openstack-nova00:18
openstackgerritMerged openstack/nova master: Clean variable names and docs around neutron allocate_for_instance  https://review.openstack.org/48926700:21
*** ijw has joined #openstack-nova00:26
*** diga has joined #openstack-nova00:27
*** ijw has quit IRC00:31
*** ijw has joined #openstack-nova00:32
*** slaweq has joined #openstack-nova00:34
*** slaweq has quit IRC00:39
*** thorst_afk has joined #openstack-nova00:39
*** thorst_afk has quit IRC00:44
*** kiennt has joined #openstack-nova00:45
*** Swami has quit IRC00:46
*** thorst_afk has joined #openstack-nova00:47
*** thorst_afk has quit IRC00:47
*** licanwei has joined #openstack-nova00:48
*** thorst_afk has joined #openstack-nova00:49
*** thorst_afk has quit IRC00:49
*** mriedem has quit IRC00:51
*** baoli has joined #openstack-nova00:53
*** baoli has quit IRC00:54
*** baoli has joined #openstack-nova00:55
*** rajathagasthya has quit IRC00:57
*** chyka has joined #openstack-nova00:59
*** Shunli has joined #openstack-nova01:01
*** phuongnh has joined #openstack-nova01:01
*** mriedem has joined #openstack-nova01:02
*** chyka has quit IRC01:04
*** litao__ has joined #openstack-nova01:05
*** ijw has quit IRC01:08
*** yangyapeng has joined #openstack-nova01:10
*** zhurong has joined #openstack-nova01:12
*** hongbin has joined #openstack-nova01:13
*** tetsuro has quit IRC01:27
*** tonygunk has quit IRC01:33
*** awaugama has quit IRC01:35
*** slaweq has joined #openstack-nova01:35
*** kbaegis has joined #openstack-nova01:38
*** slaweq has quit IRC01:40
*** trungnv has quit IRC01:40
*** priteau has joined #openstack-nova01:50
*** yingjun has joined #openstack-nova01:51
*** thorst_afk has joined #openstack-nova01:54
*** priteau has quit IRC01:55
*** yingjun has quit IRC01:58
*** thorst_afk has quit IRC01:59
*** catintheroof has quit IRC02:00
*** catintheroof has joined #openstack-nova02:01
*** yamahata has joined #openstack-nova02:01
*** thorst_afk has joined #openstack-nova02:05
*** thorst_afk has quit IRC02:05
*** itlinux has joined #openstack-nova02:12
*** esberglu has joined #openstack-nova02:20
*** esberglu has quit IRC02:20
*** tbachman has quit IRC02:21
*** TuanLA has joined #openstack-nova02:21
*** zhurong has quit IRC02:28
*** thegreenhundred has joined #openstack-nova02:31
*** thorst_afk has joined #openstack-nova02:36
*** coreywright has quit IRC02:36
*** tbachman has joined #openstack-nova02:36
*** diga has quit IRC02:37
*** thorst_afk has quit IRC02:41
openstackgerritMatt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure  https://review.openstack.org/49987402:49
*** slaweq has joined #openstack-nova02:49
*** zhurong has joined #openstack-nova02:53
*** slaweq has quit IRC02:55
*** coreywright has joined #openstack-nova02:55
*** cdent has joined #openstack-nova02:56
openstackgerritMatt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure  https://review.openstack.org/49987403:02
openstackgerritMatt Riedemann proposed openstack/nova master: Add a test to make sure failed evacuate cleans up dest allocation  https://review.openstack.org/49987703:02
*** gbarros has quit IRC03:06
*** nicolasbock has quit IRC03:07
*** cdent has quit IRC03:14
*** esberglu has joined #openstack-nova03:14
*** esberglu has quit IRC03:18
*** Nel1x has quit IRC03:20
openstackgerritMatt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails  https://review.openstack.org/49987803:21
*** mriedem has quit IRC03:23
*** baoli has quit IRC03:26
*** baoli has joined #openstack-nova03:27
*** baoli has quit IRC03:31
*** links has joined #openstack-nova03:32
*** tonygunk has joined #openstack-nova03:34
*** rajathagasthya has joined #openstack-nova03:37
*** slaweq has joined #openstack-nova03:37
*** udesale has joined #openstack-nova03:37
*** thorst_afk has joined #openstack-nova03:38
*** tonygunk has quit IRC03:39
*** psachin has joined #openstack-nova03:41
*** slaweq has quit IRC03:42
*** thorst_afk has quit IRC03:42
*** baoli has joined #openstack-nova03:51
*** priteau has joined #openstack-nova03:51
*** crushil has quit IRC03:52
*** gouthamr has quit IRC03:53
*** priteau has quit IRC03:56
*** vladikr has quit IRC04:02
*** vladikr has joined #openstack-nova04:03
*** hongbin has quit IRC04:07
*** vladikr has quit IRC04:17
*** claudiub has joined #openstack-nova04:17
*** vladikr has joined #openstack-nova04:17
*** chyka has joined #openstack-nova04:27
*** yamamot__ has quit IRC04:28
*** abhi89 has joined #openstack-nova04:28
*** trinaths has joined #openstack-nova04:29
*** chyka has quit IRC04:31
*** rajathagasthya has quit IRC04:32
*** kbaegis has quit IRC04:33
*** baoli has quit IRC04:34
*** yamamoto_ has joined #openstack-nova04:34
*** slaweq has joined #openstack-nova04:38
*** thorst_afk has joined #openstack-nova04:38
*** gyee has quit IRC04:39
*** yangyapeng has quit IRC04:41
*** catintheroof has quit IRC04:41
*** slaweq has quit IRC04:43
*** diga has joined #openstack-nova04:43
*** thorst_afk has quit IRC04:43
*** vladikr has quit IRC04:48
*** moshele has joined #openstack-nova04:51
*** vvargaszte has joined #openstack-nova04:54
*** vvargaszte has quit IRC04:54
*** yangyapeng has joined #openstack-nova04:57
*** hoonetorg has quit IRC05:05
*** moshele has quit IRC05:14
*** ssmith has joined #openstack-nova05:16
*** yamamoto_ has quit IRC05:17
*** yamamoto_ has joined #openstack-nova05:19
*** hoonetorg has joined #openstack-nova05:22
*** ratailor has joined #openstack-nova05:29
*** thorst_afk has joined #openstack-nova05:39
*** Shunli has quit IRC05:43
*** thorst_afk has quit IRC05:44
*** josecastroleon has joined #openstack-nova05:44
*** links has quit IRC05:50
*** priteau has joined #openstack-nova05:52
*** marst has joined #openstack-nova05:52
openstackgerritMerged openstack/nova master: De-duplicate two delete_allocation_for_* methods  https://review.openstack.org/49693605:55
*** priteau has quit IRC05:57
*** marst has quit IRC05:57
*** yangyapeng has quit IRC05:57
*** slaweq has joined #openstack-nova05:58
*** esberglu has joined #openstack-nova05:59
*** links has joined #openstack-nova06:01
*** edmondsw has joined #openstack-nova06:02
*** esberglu has quit IRC06:03
*** abalutoiu has joined #openstack-nova06:03
*** yangyapeng has joined #openstack-nova06:04
*** edmondsw has quit IRC06:06
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595406:07
*** Oku_OS-away is now known as Oku_OS06:11
*** slaweq has quit IRC06:12
*** pcaruana has joined #openstack-nova06:17
openstackgerritLei Zhang proposed openstack/nova master: placement: extract traits from flavor extra spec  https://review.openstack.org/49202606:24
*** hferenc has quit IRC06:26
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595406:28
*** mlakat has quit IRC06:29
*** phuongnh has quit IRC06:32
*** rcernin has joined #openstack-nova06:33
*** phuongnh has joined #openstack-nova06:33
*** sridharg has joined #openstack-nova06:33
*** lajoskatona has joined #openstack-nova06:35
*** thorst_afk has joined #openstack-nova06:40
*** thorst_afk has quit IRC06:45
*** Tom has joined #openstack-nova06:46
*** lpetrut has joined #openstack-nova06:52
*** esberglu has joined #openstack-nova06:52
openstackgerritLiChunlin proposed openstack/nova master: update the description of hypervisor statistics response  https://review.openstack.org/46235506:52
*** esberglu has quit IRC06:53
*** esberglu has joined #openstack-nova06:53
*** esberglu has quit IRC06:53
*** esberglu has joined #openstack-nova06:54
*** esberglu has quit IRC06:58
*** abhi89 has quit IRC07:00
*** slaweq has joined #openstack-nova07:01
*** slaweq has quit IRC07:06
*** tesseract has joined #openstack-nova07:12
*** slaweq has joined #openstack-nova07:13
*** cfriesen has quit IRC07:24
*** lajoskatona has quit IRC07:28
*** cfriesen has joined #openstack-nova07:28
*** lpetrut has quit IRC07:33
openstackgerritDavid Rabel proposed openstack/nova master: Adds support for graceful shutdown for VMware instances  https://review.openstack.org/49416907:35
gibidansmith: you are right that https://review.openstack.org/#/c/498950 was failed notification tests due to the service version bump07:36
*** tonygunk has joined #openstack-nova07:36
*** mhenkel has quit IRC07:36
*** jpena|off is now known as jpena07:39
*** lifeless_ is now known as lifeless07:39
*** tonygunk has quit IRC07:40
*** thorst_afk has joined #openstack-nova07:41
*** itlinux has quit IRC07:42
*** mhenkel has joined #openstack-nova07:43
openstackgerritDavid Rabel proposed openstack/nova master: VMware: add support for graceful shutdown of instances  https://review.openstack.org/49416907:44
*** thorst_afk has quit IRC07:45
*** ralonsoh has joined #openstack-nova07:49
*** kiennt has quit IRC07:50
*** TuanLA has quit IRC07:50
*** lpetrut has joined #openstack-nova07:50
*** TuanLA has joined #openstack-nova07:51
*** kiennt has joined #openstack-nova07:51
*** priteau has joined #openstack-nova07:53
*** abalutoiu has quit IRC07:54
*** priteau has quit IRC07:58
*** jaosorior has quit IRC07:59
*** ygl has joined #openstack-nova07:59
*** jaosorior has joined #openstack-nova08:00
*** abalutoiu has joined #openstack-nova08:00
*** shan has joined #openstack-nova08:00
*** slaweq has quit IRC08:00
*** shan is now known as shan|afk08:00
*** slaweq has joined #openstack-nova08:01
*** yangyapeng has quit IRC08:01
*** mlakat has joined #openstack-nova08:01
*** avolkov has joined #openstack-nova08:02
*** chyka has joined #openstack-nova08:03
*** zxy has joined #openstack-nova08:04
*** lucas-hungry is now known as lucasagomes08:04
*** lpetrut has quit IRC08:04
*** yangyapeng has joined #openstack-nova08:05
*** chyka has quit IRC08:08
*** slaweq has quit IRC08:09
*** slaweq has joined #openstack-nova08:09
*** ssmith has quit IRC08:10
*** yangyapeng has quit IRC08:31
*** yangyapeng has joined #openstack-nova08:32
*** ygl_ has joined #openstack-nova08:35
*** ygl has quit IRC08:35
*** efoley has joined #openstack-nova08:36
*** yangyapeng has quit IRC08:36
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment  https://review.openstack.org/48945108:40
*** thorst_afk has joined #openstack-nova08:42
* gibi was entertained by the spoon talk 08:45
*** thorst_afk has quit IRC08:46
*** adisky__ has quit IRC08:49
*** syqian__ has quit IRC08:50
*** syqian__ has joined #openstack-nova08:50
*** yangyapeng has joined #openstack-nova08:50
*** yangyapeng has quit IRC08:51
*** syqian__ has quit IRC08:53
*** psachin has quit IRC09:00
*** yangyapeng has joined #openstack-nova09:00
*** priteau has joined #openstack-nova09:02
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin  https://review.openstack.org/49308209:08
*** yuchenghui has joined #openstack-nova09:10
*** sambetts|afk is now known as sambetts09:11
*** efoley has quit IRC09:15
*** efoley_ has joined #openstack-nova09:15
*** shan|afk is now known as shan09:15
*** hshiina has joined #openstack-nova09:19
*** cfriesen has quit IRC09:20
*** lpetrut has joined #openstack-nova09:23
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: [WIP][POC] Host plugin information from os-vif  https://review.openstack.org/49994109:24
ygl_hi all09:24
ygl_my instance creaiton is failing09:24
*** salv-orlando has quit IRC09:24
ygl_its is saying "Connection to glance host failed:"09:24
ygl_but i can list the images09:25
*** udesale__ has joined #openstack-nova09:27
*** udesale has quit IRC09:27
*** udesale has joined #openstack-nova09:29
*** salv-orlando has joined #openstack-nova09:29
*** udesale__ has quit IRC09:31
*** nicolasbock has joined #openstack-nova09:33
*** ygl_ has quit IRC09:35
*** cfriesen has joined #openstack-nova09:38
*** udesale has quit IRC09:40
*** udesale has joined #openstack-nova09:41
*** cfriesen_ has joined #openstack-nova09:42
*** thorst_afk has joined #openstack-nova09:42
*** cfriesen has quit IRC09:43
*** udesale has quit IRC09:44
*** cfriesen_ has quit IRC09:47
*** thorst_afk has quit IRC09:47
*** cfriesen_ has joined #openstack-nova09:49
*** avolkov has quit IRC09:50
*** yangyapeng has quit IRC09:52
*** yangyapeng has joined #openstack-nova09:55
*** avolkov has joined #openstack-nova09:57
*** karthiks is now known as karthiks_afk09:58
*** salv-orlando has quit IRC10:01
*** salv-orlando has joined #openstack-nova10:04
*** Tom has quit IRC10:11
*** kbaegis has joined #openstack-nova10:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Default the service version in the notification tests  https://review.openstack.org/49995710:16
*** yangyapeng has quit IRC10:21
*** rmart04 has joined #openstack-nova10:28
*** TuanLA has quit IRC10:30
*** Tom has joined #openstack-nova10:33
*** efoley_ has quit IRC10:34
*** gszasz has joined #openstack-nova10:34
*** Tom has quit IRC10:38
openstackgerritBalazs Gibizer proposed openstack/nova master: doc: fix show-hide sample in notification devref  https://review.openstack.org/49996310:40
*** zxy has quit IRC10:43
*** thorst_afk has joined #openstack-nova10:43
*** zxy has joined #openstack-nova10:43
*** psachin has joined #openstack-nova10:45
*** cfriesen__ has joined #openstack-nova10:46
*** cfriesen_ has quit IRC10:46
*** thorst_afk has quit IRC10:48
*** hieulq has quit IRC10:52
*** cfriesen__ has quit IRC10:56
*** yassine has quit IRC10:57
*** zhurong has quit IRC10:59
*** yassine has joined #openstack-nova11:02
*** lucasagomes is now known as lucas-afk11:06
*** cfriesen__ has joined #openstack-nova11:08
*** thorst_afk has joined #openstack-nova11:09
*** thorst_afk has quit IRC11:11
*** esberglu has joined #openstack-nova11:26
*** esberglu has quit IRC11:26
*** zxy has quit IRC11:29
*** salv-orlando has quit IRC11:29
*** kbaegis has quit IRC11:29
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova master: docs: Explain the flow of the "serial console" feature  https://review.openstack.org/47618811:33
*** cfriesen__ has quit IRC11:34
*** jmlowe has quit IRC11:38
*** jmlowe has joined #openstack-nova11:41
*** cfriesen__ has joined #openstack-nova11:42
*** kiennt has quit IRC11:43
*** sdague has joined #openstack-nova11:43
*** abalutoiu has quit IRC11:52
*** ratailor has quit IRC11:53
*** trinaths has quit IRC12:01
*** thorst_afk has joined #openstack-nova12:04
*** litao__ has quit IRC12:07
*** vladikr has joined #openstack-nova12:12
*** wlfightup has joined #openstack-nova12:14
wlfightuphttps://blueprints.launchpad.net/nova/+spec/instance-mem-and-cpu-hot-plugging12:15
*** catintheroof has joined #openstack-nova12:17
*** esberglu has joined #openstack-nova12:19
*** smatzek has joined #openstack-nova12:22
*** esberglu has quit IRC12:23
*** mdnadeem has joined #openstack-nova12:24
*** jichen has joined #openstack-nova12:26
*** udesale has joined #openstack-nova12:29
*** phuongnh has quit IRC12:33
*** catintheroof has quit IRC12:34
*** catintheroof has joined #openstack-nova12:34
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin  https://review.openstack.org/49308212:36
*** sbezverk has quit IRC12:38
*** krtaylor has joined #openstack-nova12:39
openstackgerritStephen Finucane proposed openstack/nova master: Add '_has_dns_extension' function  https://review.openstack.org/48067512:41
openstackgerritStephen Finucane proposed openstack/nova master: Always refresh cache when checking extensions  https://review.openstack.org/48067412:41
openstackgerritStephen Finucane proposed openstack/nova master: metadata: Don't use nova-net DNS values with neutron  https://review.openstack.org/48061612:41
openstackgerritStephen Finucane proposed openstack/nova master: Store dns_domain value from neutron  https://review.openstack.org/48067612:41
openstackgerritStephen Finucane proposed openstack/nova master: Add '_has_qos_queue_extension' function  https://review.openstack.org/50001012:41
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/50001112:42
*** licanwei has quit IRC12:44
*** sbezverk has joined #openstack-nova12:45
*** jpena is now known as jpena|lunch12:46
*** psachin has quit IRC12:46
*** abalutoiu has joined #openstack-nova12:46
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/50002812:47
*** wlfightup has quit IRC12:50
*** mdnadeem has quit IRC12:51
*** yuchenghui has quit IRC12:57
*** edleafe is now known as figleaf12:57
*** jichen has quit IRC12:58
*** thegreenhundred has quit IRC13:02
*** esberglu has joined #openstack-nova13:07
*** esberglu has quit IRC13:07
*** markvoelker has joined #openstack-nova13:11
*** mriedem has joined #openstack-nova13:11
*** crushil has joined #openstack-nova13:13
*** baoli has joined #openstack-nova13:14
*** catintheroof has quit IRC13:16
*** catintheroof has joined #openstack-nova13:16
*** scottda_ has joined #openstack-nova13:18
*** efried is now known as fried_rice13:20
*** lyan has joined #openstack-nova13:21
*** bauzas is now known as bauwser13:22
openstackgerritEric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link  https://review.openstack.org/49982613:23
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Make 'get_domain' private  https://review.openstack.org/41737813:23
*** cdent has joined #openstack-nova13:23
*** hshiina has quit IRC13:26
cdentfried_rice: I’m fairly in favor of making that links change, but I dunno if it will pass the rules13:29
fried_ricecdent Yuh; I'm happy to take my guidance from whoever - but who is that?13:30
cdentI would guess a final arbiter is someone with +2, which is not me, nor figleaf13:30
*** burt has joined #openstack-nova13:30
cdentI’ll go ahead and +1 it to express my opinion13:30
fried_riceAh, gotcha.  Someone like dansmith or leakypipes or mriedem13:31
fried_riceor sdague13:31
figleafcdent: it's not fixing an error in the sense we intended. It's correcting an oversight13:32
*** dave-mccowan has joined #openstack-nova13:32
mriedemis it changing the response?13:32
figleafyes13:32
mriedemif it's changing the response then it's a new microversion13:32
cdentsdague is probably the main arbiter on this kind of thing13:32
cdentmriedem: the squiggle is that we changed the content of that section before without a microversion13:33
cdentthus the lawyering going on now13:33
figleaftwo clouds requesting the same microversion would get different data13:33
mriedempast performance is not an indicator of future earnings13:33
mriedemwhat was the previous non-versioned change?13:34
cdentYeah, the rules are it must be a new microversion. The reason I’m being squishy about it is just to reinforce that just because I wrote the latest version of the rules doesn’t mean I’m in love with microversions and want to marry them.13:34
fried_ricemriedem https://review.openstack.org/#/c/468923/1/nova/api/openstack/placement/handlers/resource_provider.py13:34
cdentmriedem: it’s linked in fried_rice’s comments13:34
cdentjinx13:34
mriedemlet me note that it's too early in my day to have to deal with the age old, "does this require a microversion?" question13:35
figleafit's the usual problem: a seemingly backwards-compatible change13:35
mriedemso alex_xu and sdague approved https://review.openstack.org/#/c/468923/13:36
mriedemwith no comment on the microversion part of this13:36
fried_riceright13:36
*** esberglu has joined #openstack-nova13:36
*** tonygunk has joined #openstack-nova13:37
*** esberglu has quit IRC13:37
cdentfried_rice: did you find this because you are actually using the links, or just because you were having a poke around. If someone’s using genuine hypertextuality, i’m going to need to sit down.13:38
cdent(somewhat related, I’ve started using genuine hypertextuality in my gabbi tests of placement, just for the sake of it)13:39
fried_ricecdent Poking around.  Larnin' up the placement API and resource providers and whatnot.13:39
*** esberglu has joined #openstack-nova13:39
fried_riceWorking towards being able to do generic device management via placement.13:39
cdentLarnin’!!!??? Ain’t no body got time for that.13:39
fried_riceThe alternative is putting together a huge pile of incomprehensible sh*t that will waste everybody's time in Denver.13:40
fried_riceThis way it gets to be a huge pile of slightly-comprehensible sh*t.13:40
kashyapHuh, the documentation for 'live_migration_scheme' says: "Override the default libvirt live migration scheme", but it doesn't mention _what_ are the possible values?13:41
kashyapFrom here: https://docs.openstack.org/ocata/config-reference/compute/config-options.html13:41
* kashyap goes to look at source13:41
cdentI hope to introduce you to some vmware folk who are curious about that stuff too, and are also planning to do some funky not-just-for-the-sake-of-nova resource mgt via placement13:41
cdents/you/fried_rice/13:41
mriedemkashyap: sec, i have a ML thread link13:42
kashyapmriedem: Ah, I came across it (as I sometimes filter openstack-dev list by your e-mail ID :P)13:42
*** udesale has quit IRC13:42
fried_ricecdent Cool, I've been wanting to talk to VMWare about this.13:42
kashyapLet me go look13:42
fried_riceCause I think they have some of the same issues as claudiub and I do.13:42
fried_ricethat we want to make sure the new solution solves.13:43
cdentfried_rice: yup13:43
*** jpena|lunch is now known as jpena13:43
mriedemkashyap: http://lists.openstack.org/pipermail/openstack-dev/2017-June/117987.html13:43
* kashyap clicks13:44
*** hferenc has joined #openstack-nova13:44
*** links has quit IRC13:44
kashyapmriedem: BTW, thanks for kickstarting the cleanup of the spaghetti that is LM doc - 9c2518c2 ("Cleanup the plethora of libvirt live migration options")13:44
mriedemyeah it's impossible13:45
kashyapHeh, "landmine of terrible"13:46
openstackgerritElod Illes proposed openstack/nova master: Functional test for regression bug #1713783  https://review.openstack.org/50005713:47
openstackbug 1713783 in OpenStack Compute (nova) pike "After failed evacuation the recovered source compute tries to delete the instance" [High,Triaged] https://launchpad.net/bugs/171378313:47
*** shan has quit IRC13:47
*** hferenc has quit IRC13:49
*** ratailor has joined #openstack-nova13:49
kashyapmriedem: Darn, I missed your e-mail as it got filtered to the 'operators' mail dir.13:50
*** hongbin has joined #openstack-nova13:54
*** pchavva has joined #openstack-nova13:54
*** niraj_singh has quit IRC13:54
mriedemelod: thanks for https://review.openstack.org/#/c/500057/ - just a few small things inline13:55
*** gouthamr has joined #openstack-nova13:56
elodmriedem: thanks too, checking13:57
* kashyap wishes people just _quote_ the relevant code in e-mails when feasible, instead of GitHub line numbers, they all change13:58
*** salv-orlando has joined #openstack-nova13:58
kashyapAlthough you could figure out for context, but the URL will be useless in future13:59
kashyaps/for/from/13:59
*** baoli has quit IRC13:59
gibikashyap: you can have a github link that points to place via a specfic commit hash. That is futureproof14:01
*** catintheroof has quit IRC14:04
*** catintheroof has joined #openstack-nova14:05
*** awaugama has joined #openstack-nova14:06
*** aries has joined #openstack-nova14:06
mriedeme.g. https://github.com/openstack/nova/blob/7815108d4892525b0047c787cbd2fe2f26c204c2/nova/virt/libvirt/driver.py#L65214:07
mriedemlike in my email14:07
stephenfinAny chance of a +W on this? Been sitting around for a while https://review.openstack.org/#/c/407055/14:07
kashyapgibi: Ah, noted14:08
mriedemstephenfin: the change under it has a -1 on it14:08
mriedemso what's the rush14:08
mriedemunless you re-order14:08
kashyapJust reading this one (oh dear, yet more live migration options) - https://review.openstack.org/#/c/456571/14:08
stephenfinAha, correct. nvm me14:08
stephenfinMy list of open patches has grown rather large/unmanageable :/14:09
kashyapI know it intends to solve the said valid bug where one can't specify one of the URI variants14:09
mriedemkashyap: the point of that is to remove the URI option and make it specific14:09
mriedemso you can configure the parts of the URI14:09
mriedemright14:09
mriedemlive_migration_query_parameters :)14:09
kashyapmriedem: :-)  I see the point14:09
mriedemoh i guess that's extra_params14:10
* kashyap first reads the complete set of comments14:10
mriedemi was just joking about that one14:10
kashyapYeah, I've read your comment where you lament about the proliferation of the options14:10
*** gbarros has joined #openstack-nova14:10
mriedemyou know, it makes me think, if we merged that and removed live_migration_uri, in 2 years time someone will say, "why do i have to configure separately the 4 parts of this URI? why can't i just specify the single live migration URI as a config option?"14:11
mriedemthat someone will likely be me14:11
*** tidwellr has joined #openstack-nova14:12
mriedemoh i guess it's because only nova knows the host14:12
mriedemso we have to sub in the host/ip in the URI14:12
kashyapOn a call, /me is a bit slow to read the scroll here14:12
kashyapmriedem: Exactly!  About the future complainer14:13
kashyapBut yeah, it's a bit hair-splitty14:13
*** catintheroof has quit IRC14:15
*** catintheroof has joined #openstack-nova14:15
*** cshen has quit IRC14:15
openstackgerritElod Illes proposed openstack/nova master: Functional test for regression bug #1713783  https://review.openstack.org/50005714:17
openstackbug 1713783 in OpenStack Compute (nova) pike "After failed evacuation the recovered source compute tries to delete the instance" [High,Triaged] https://launchpad.net/bugs/171378314:17
*** edmondsw has joined #openstack-nova14:17
*** diga has quit IRC14:17
mriedemgibi: we might want to think about how to highlight http://docs-draft.openstack.org/63/499963/1/check/gate-nova-docs-ubuntu-xenial/2c3bf74//doc/build/html/reference/notifications.html#existing-versioned-notifications at a higher level in the docs,14:17
mriedemgibi: because right now you have to go main index > technical reference deep dives > notifications > samples14:18
mriedemif i'm a consumer of the versioned notifications and want samples, like we have for the compute api reference, it would be nice to get to those from a higher level since i dont think i'd expect to find it in 'technical reference deep dives'14:18
gibimriedem: sure, I have to scan the new structure to see where a link can fit14:19
openstackgerritMatt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails  https://review.openstack.org/49987814:19
openstackgerritMatt Riedemann proposed openstack/nova master: Add a test to make sure failed evacuate cleans up dest allocation  https://review.openstack.org/49987714:19
openstackgerritMatt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure  https://review.openstack.org/49987414:19
*** eharney has joined #openstack-nova14:19
mriedemgibi: maybe a new section in https://docs.openstack.org/nova/latest/#for-end-users ?14:20
mriedembecause the tech reference deep dive is under 'For Contributors'14:20
mriedembut if i'm a notification consumer i'm not necessarily a contributor14:20
mriedemi'll hack something up quick to fight over14:20
*** cleong has joined #openstack-nova14:20
gibimriedem: I think similar to what is in https://docs.openstack.org/nova/latest/#writing-to-the-api14:21
openstackgerritEric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link  https://review.openstack.org/49982614:21
mriedemyeah14:21
gibimriedem: or maybe even inside it as a sub section14:21
gibimriedem: as notification is also an API14:21
mriedemok i can stack that on top of this https://review.openstack.org/#/c/498977/14:22
gibimriedem: cool. but if you have more important things then I can do that doc change14:23
mriedemthis is the most important thing in my life right this second14:23
gibimriedem: thank go for it :)14:23
*** gyee has joined #openstack-nova14:24
gibiI mean thanks, then go for it14:24
*** catintheroof has quit IRC14:25
*** catintheroof has joined #openstack-nova14:25
openstackgerritEric Fried proposed openstack/nova master: Glance download: only fsync files  https://review.openstack.org/45109414:27
openstackgerritStephen Finucane proposed openstack/nova master: conf: Remove '[conductor] topic' opt  https://review.openstack.org/49917914:27
mriedemelod: can you rebase https://review.openstack.org/#/c/499237/ on top of https://review.openstack.org/#/c/500057/ now?14:28
mriedembauwser: i've got all of the patches up for the various evacuate failure scenarios where we need to cleanup allocations https://review.openstack.org/#/q/topic:bug/171378614:29
mriedemi plan on getting those backported to pike14:29
*** marst has joined #openstack-nova14:29
bauwsermriedem: in a meeting but I'll look asap14:29
mriedemthey are really in two separate series for failures in conductor and failures in compute, i could stack them though (and probably should to avoid conflicts in the functional test)14:30
mriedemso, maybe i'll do that quick14:30
openstackgerritChris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007314:30
bauwsermmm, your call14:30
bauwsermriedem: I'm not fully onboard yet about all the problems you folks found during my offline, so I'll trust you14:31
openstackgerritStephen Finucane proposed openstack/nova master: conf: Remove deprecated 'null_kernel' opt  https://review.openstack.org/49961114:31
openstackgerritStephen Finucane proposed openstack/nova master: WIP! conf: Remove deprecated 'multi_instance_display_name_template' opt  https://review.openstack.org/49961214:31
mriedemthese are pretty straightforward14:31
mriedemsomething fails, cleanup the dest allocation14:31
bauwseryeah I can see14:31
cdentmriedem, dansmith ^ that post allocations wip is very wippy, but wanted to get some early commentary14:31
sean-k-mooneyo/ i dont see jay pipes online. is he traveling today. i think he was in europe this week?14:32
*** armax_ has joined #openstack-nova14:32
openstackgerritStephen Finucane proposed openstack/nova master: conf: Rename two VNC options  https://review.openstack.org/49838714:32
mriedemsean-k-mooney: he's on vacation today14:32
mriedemso out until tuesday14:32
mriedemsince monday is a US holiday14:32
sean-k-mooneyah ok noting urgent.14:33
openstackgerritStephen Finucane proposed openstack/nova master: conf: Deprecate 'keymap' options  https://review.openstack.org/48399414:34
*** armax has quit IRC14:35
*** armax_ is now known as armax14:35
*** rabel has joined #openstack-nova14:35
*** catintheroof has quit IRC14:35
*** catintheroof has joined #openstack-nova14:36
*** catintheroof has quit IRC14:37
*** catintheroof has joined #openstack-nova14:37
rabeli think https://review.openstack.org/#/c/494169/ is ready for a final review (again). it even passed VMware NSX CI this time. ;)14:38
*** baoli has joined #openstack-nova14:39
*** vladikr_ has joined #openstack-nova14:39
*** vladikr has quit IRC14:40
*** jaosorior has quit IRC14:40
mriedemrabel: question inline14:40
openstackgerritMatt Riedemann proposed openstack/nova master: doc: link to placement api-ref and history docs from main index  https://review.openstack.org/49897714:44
openstackgerritMatt Riedemann proposed openstack/nova master: doc: link to versioned notification samples from main index  https://review.openstack.org/50008114:44
mriedemgibi: ^14:44
mriedemsee how that looks14:44
openstackgerritElod Illes proposed openstack/nova master: Set error state after failed evacuation  https://review.openstack.org/49923714:45
*** gbarros has quit IRC14:45
gibimriedem: looking14:46
*** hrw has quit IRC14:47
*** hrw has joined #openstack-nova14:48
elodmriedem: bug-fix rebased14:48
mriedemyup, just looked it over14:49
mriedemwaiting on tests but lgtm14:49
elodok, thx14:51
*** catintheroof has quit IRC14:52
gibimriedem: in the index the first sentence of the section explicity talks about REST API which now conflicts with the added notification bullet point14:52
*** catintheroof has joined #openstack-nova14:52
gibimriedem: "All end user (and some administrative) features of nova are exposed via a REST"14:52
gibimriedem: what if we don't add the notifications directly to the bullet list but add an extra sentence talking about the notifications after the bullet list?14:54
*** slaweq has quit IRC14:54
mriedemok i'll take a look14:55
*** ratailor has quit IRC14:55
*** crushil has quit IRC14:56
*** catintheroof has quit IRC15:00
*** catintheroof has joined #openstack-nova15:00
mriedemlpetrut: do you have some time to discuss https://bugs.launchpad.net/nova/+bug/1714247 ?15:02
openstackLaunchpad bug 1714247 in OpenStack Compute (nova) "Cleaning up deleted instances leaks resources" [Undecided,New]15:02
*** jpena is now known as jpena|away15:03
*** lpetrut has quit IRC15:05
*** crushil has joined #openstack-nova15:05
*** gbarros has joined #openstack-nova15:05
alex_xusdague: a patch related 'microversion or not' https://review.openstack.org/#/c/490722, it will be good if you can take a look at15:06
*** lpetrut has joined #openstack-nova15:10
*** gbarros has quit IRC15:10
gibimriedem: I'm looking at: https://review.openstack.org/#/c/499399/2/nova/conductor/manager.py@760 . Does ironic custom resource provided by the compute resource provider or a separate resource provider per compute?15:10
*** vikrant has joined #openstack-nova15:12
*** EmilienM has quit IRC15:12
*** EmilienM has joined #openstack-nova15:12
figleafalex_xu: if you're still around, do you want to give your opinion on whether this needs a microversion or not? https://review.openstack.org/#/c/499826/15:12
stephenfinsean-k-mooney, ralonsoh: Resolved your comments in https://review.openstack.org/#/c/483398/ finally15:12
ralonsohstephenfin: I'll put this patch in my todo list for today or monday morning15:13
ralonsohthanks!15:13
stephenfinno problemo :)15:14
*** josecastroleon has quit IRC15:17
*** links has joined #openstack-nova15:18
cdentnot clear if I’m going to have time for a rp update today. If I do it will be later in the day15:18
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: doc: fix show-hide sample in notification devref  https://review.openstack.org/50010015:21
*** yangyapeng has joined #openstack-nova15:22
*** thegreenhundred has joined #openstack-nova15:22
*** gmann has quit IRC15:23
*** gmann has joined #openstack-nova15:23
*** yangyapeng has quit IRC15:26
*** rcernin has quit IRC15:29
*** pcaruana has quit IRC15:30
*** Oku_OS is now known as Oku_OS-away15:32
openstackgerritMerged openstack/nova master: doc: fix show-hide sample in notification devref  https://review.openstack.org/49996315:35
*** kenperkins has joined #openstack-nova15:35
*** chyka has joined #openstack-nova15:36
*** abalutoiu_ has joined #openstack-nova15:39
*** abalutoiu__ has joined #openstack-nova15:40
openstackgerritMatt Riedemann proposed openstack/nova master: add online_data_migrations to nova docs  https://review.openstack.org/49344215:41
stephenfinmriedem, sean-k-mooney: Where did we come down on https://bugs.launchpad.net/nova/+bug/1698010? Do we actually want to use a FQDN for hostname?15:41
openstackLaunchpad bug 1698010 in OpenStack Compute (nova) "neutron-based instances should not use the nova-network 'dhcp_domain' option" [High,In progress] - Assigned to Stephen Finucane (stephenfinucane)15:41
*** rabel has quit IRC15:41
*** abalutoiu has quit IRC15:42
openstackgerritStephen Finucane proposed openstack/nova master: add online_data_migrations to nova docs  https://review.openstack.org/49344215:43
*** armax_ has joined #openstack-nova15:44
*** abalutoiu_ has quit IRC15:44
*** armax has quit IRC15:45
*** armax_ is now known as armax15:45
*** abalutoiu__ has quit IRC15:45
mriedemdansmith: can you clarify a question i have in here? https://review.openstack.org/#/c/493442/615:47
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Set HostPluginInfo plugin_name as an Enum  https://review.openstack.org/50011115:47
dansmithmriedem: yeah, so really it should be only the controllers, but we had one incident where it wouldn't complete until all the computes have been upgraded15:49
dansmithmriedem: the pci one15:49
dansmithI was talking to someone recently about this, not sure if it was this person15:49
*** trinaths has joined #openstack-nova15:49
dansmithI dunno what to say here, because in some cases it'd be one or the other, but almost always just controllers15:49
*** trinaths1 has joined #openstack-nova15:50
*** salv-orlando has quit IRC15:50
mriedemin the case of the pci issue,15:51
mriedemdid running online_data_migrations break anything? or just not migrate things as expected?15:51
dansmithnope it would just warn you that it couldn't finish because it needed the computes to be upgraded first15:51
mriedemah, ok15:51
mriedemso, then i think we leave the ideal case in the man page, and if we have a known weird wrinkle, then we release note that15:52
dansmithsounds good15:52
*** salv-orlando has joined #openstack-nova15:53
*** trinaths has quit IRC15:53
*** yamahata has quit IRC15:54
*** ssurana has joined #openstack-nova15:54
*** takedakn has joined #openstack-nova15:55
*** rmart04 has quit IRC15:57
*** yamahata has joined #openstack-nova15:57
*** Apoorva has joined #openstack-nova15:57
*** ssurana has quit IRC15:58
mriedemevacuating local git15:59
mriedemforce=True16:00
openstackgerritMatt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails  https://review.openstack.org/49987816:00
openstackgerritMatt Riedemann proposed openstack/nova master: Add recreate test for forced host evacuate not setting dest allocations  https://review.openstack.org/49967816:00
openstackgerritMatt Riedemann proposed openstack/nova master: Add a test to make sure failed evacuate cleans up dest allocation  https://review.openstack.org/49987716:00
openstackgerritMatt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure  https://review.openstack.org/49987416:00
openstackgerritMatt Riedemann proposed openstack/nova master: Create allocations against forced dest host during evacuate  https://review.openstack.org/49939916:00
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor out claim_resources_on_destination into a utility  https://review.openstack.org/49971816:00
openstackgerritMatt Riedemann proposed openstack/nova master: Modernize set_vm_state_and_notify  https://review.openstack.org/49979916:00
stephenfinsdague: If you get a chance, would appreciate a sanity check on https://bugs.launchpad.net/nova/+bug/170445816:00
openstackLaunchpad bug 1704458 in OpenStack Compute (nova) "The use_ipv6 flag not only influences nova networking" [High,Invalid] - Assigned to Stephen Finucane (stephenfinucane)16:00
*** rajathagasthya has joined #openstack-nova16:01
*** ralonsoh has quit IRC16:02
*** annegentle has joined #openstack-nova16:02
*** cdent has quit IRC16:03
*** david-lyle has quit IRC16:03
*** itlinux has joined #openstack-nova16:05
*** baoli_ has joined #openstack-nova16:06
mriedemstephenfin: regarding that dhcp_domain thing for dns, i think it makes sense that if you're using neutron, you pull it from the neutron network resource as it appears your series is doing16:07
mriedemstephenfin: i just have to get through all of your unrelated changes to get to the meat of the fix first... :16:07
mriedem:)16:07
*** baoli has quit IRC16:08
*** annegentle has quit IRC16:08
*** annegentle has joined #openstack-nova16:09
stephenfinmriedem: Right, but the question is whether we should actually be using FQDN at all, rather than from neutron instead. sean-k-mooney seemed to suggest it was a bad idea, but my network/sysadmin foo is not good enough to say why not :)16:09
stephenfinAlso, I like refactoring. Sorry :D I can pull those out16:09
* stephenfin runs home. Back in 1016:09
mriedemrun swift young stephen16:10
mriedemdon't look back....16:10
*** gbarros has joined #openstack-nova16:10
*** annegentle has quit IRC16:10
*** annegentle has joined #openstack-nova16:14
*** takedakn has quit IRC16:17
openstackgerritMatt Riedemann proposed openstack/nova master: doc: fix online_data_migrations option in upgrades doc  https://review.openstack.org/50012416:18
*** dave-mccowan has quit IRC16:20
*** esberglu has quit IRC16:24
*** aarefiev is now known as aarefiev_afk16:29
*** links has quit IRC16:31
openstackgerritMatt Riedemann proposed openstack/nova master: Set error state after failed evacuation  https://review.openstack.org/49923716:34
*** tonygunk has quit IRC16:34
mriedemwe're going to want to backport this fix ^ back through newton16:35
mriedemso it'd be nice to start that, it's pretty trivial16:35
*** catintheroof has quit IRC16:36
mriedemprevents the source compute from deleting local instance if the evacuation fails in conductor to find a dest host16:36
*** catintheroof has joined #openstack-nova16:36
*** jpena|away is now known as jpena|off16:38
dansmithmriedem: so just thinking about this quickly,16:39
dansmithmriedem: are you thinking that if we fail to schedule a new place and the source comes back up that there's value in having it still there?16:39
dansmithbecause the way this was designed, once an evacuation starts we're *going* to delete it from the source compute to avoid a race between rebuilding it elsewhere and the compute coming back up and deciding to keep it or not16:40
mriedemmelwitt: commented on https://review.openstack.org/#/c/407346/ - cinder would also need changes if we do something like this in nova16:40
*** rajathagasthya has quit IRC16:41
* stephenfin had a lovely run16:41
*** rajathagasthya has joined #openstack-nova16:41
mriedemdansmith: hmm16:42
mriedemthat's a good point16:42
dansmithknowing the folks that use this a lot from a script,16:42
melwittmriedem: cool thanks. looks like it's not as straightforward was I initially thought16:42
dansmiththe last thing they want is vague behavior from this16:43
mriedemi wish that would have been a comment in the _delete_evacuated_instances code because i spent a good chunk of time yesterday trying to figure out why we include 'accepted' in the filter16:43
dansmithmriedem: was it in my robustify spec? I think the logic was sussed out there16:43
dansmithanyway,16:43
mriedemi didn't read the spec16:43
dansmithI would think that what we want is to make sure you can call rebuild (or at least reset-state) on the instance and have it rebuilt somewhere else16:44
dansmithbecause I think we've nulled out the instance.host here at this point right? meaning it won't be a simple restart on the source host16:44
* dansmith looks16:44
openstackgerritMerged openstack/nova master: Functional test for regression bug #1713783  https://review.openstack.org/50005716:45
openstackbug 1713783 in OpenStack Compute (nova) "After failed evacuation the recovered source compute tries to delete the instance" [High,In progress] https://launchpad.net/bugs/1713783 - Assigned to Matt Riedemann (mriedem)16:45
mriedemdansmith: i don't think the instance.host is nulled out16:45
mriedemwhich is part of the problem16:45
mriedemreading the docstring for _destroy_evacuated_instances, it says,16:45
mriedem"Check that the instances reported by the driver are still associated with this host.  If they are not, destroy them"16:45
openstackgerritMerged openstack/nova master: add online_data_migrations to nova docs  https://review.openstack.org/49344216:45
mriedembut it doesn't actually compare the instances that it finds against self.host16:45
dansmithmriedem: right, it used to, which is why it would delete everything if your hostname changed16:46
mriedemit just says, oh you had an evacuation from this host, we assume it was cool, delete local16:46
*** trinaths1 has left #openstack-nova16:47
mriedemit also says, "with the exception of instances which are in the MIGRATING, RESIZE_MIGRATING, RESIZE_MIGRATED, RESIZE_FINISH task state or RESIZED vm state"16:47
dansmithmriedem: so it might not be nulled out, but if we're in the middle of it starting on the new destination  host, we don't know, so if we don't delete it, we'll end up with it in two places16:47
mriedembut i don't see that filtering happening anywhere16:47
*** esberglu has joined #openstack-nova16:47
dansmithmriedem: yeah that's not from me, that's from dd6fb1246ff2789bd78b772b45e1fcac21eda67a16:48
dansmithwhich looks like it had a filter in it which is now gone16:48
dansmiththat makes no sense to me though,16:48
dansmithsince that code already only looks at evacuations16:48
mriedemhttps://review.openstack.org/#/c/101803/ predates the robustify that used the migration record for filtering16:49
dansmithah yep16:50
mriedemso we just need to cleanup the comments in here a bit,16:50
dansmithso maybe my bad for not removing that then16:50
mriedemso if we don't take this change,16:50
*** edmondsw has quit IRC16:50
mriedemand the compute is up and running, need to think about what the RT is going to do16:50
*** catintheroof has quit IRC16:50
mriedemso we'd remove the allocation on the source node when it starts up16:51
dansmithwhich is correct16:51
mriedemright b/c the guest was deleted,16:51
*** esberglu has quit IRC16:51
mriedemthe instance isn't actually gone, and it points at the source host still16:51
*** edmondsw has joined #openstack-nova16:51
dansmithand it looks to me like we would be able to run rebuild on it16:51
dansmithyou can run rebuild from error state and it seems like it'll just pick up the evacuation and try again,16:52
dansmithbut not if we error it out like this patch does16:52
mriedemthe update_available_resource task is going to query for instances on this host/node, and pull this in since it's not deleted16:52
mriedempass that to _update_usage_from_instances16:53
mriedemwhich calls _update_usage_from_instance with has_ocata_computes=True16:53
mriedem*False16:53
mriedemassuming you're upgraded16:53
mriedemand it won't recreate the allocation in placement for the source node, which again is correct since we don't have the guest anymore16:54
mriedemif you have ocata computes, it would, but...16:54
mriedemmeh?16:54
dansmithwe'll create the new allocation during the rebuild16:54
dansmithand when build finishes it'll replace the doubled (due to migration) allocation with the single one for the new host16:54
mriedemthe instance wouldn't necessarily be in ERROR state,16:54
mriedemand conductor doesn't set it to ERROR state if it fails to find a host16:55
dansmithit would from that novalidhost16:55
mriedemnope16:55
mriedemit just resets the task_state to None16:55
dansmithit does _set_vm_state_and_notify()16:55
mriedem                        self._set_vm_state_and_notify(context, instance.uuid,16:55
mriedem                                'rebuild_server',16:55
mriedem                                {'vm_state': instance.vm_state,16:55
mriedem                                 'task_state': None}, ex, request_spec)16:55
mriedemi've been all up in this code for a week16:55
dansmithoh, does that not set it to error?16:55
mriedemnope16:55
dansmithI see16:55
dansmithwell a reset state will put it in error16:55
mriedemif you tell it to :)16:55
mriedemreset-state takes the state you want it in16:56
mriedemi htink16:56
dansmithdoesn't reset state default to error?16:56
dansmithno, it's just error or --active as an option I think16:56
mriedemoh i don't know what the CLI does16:56
mriedembut the API doesn't default16:56
mriedem"The state of the server to be set, active or error are valid."16:56
dansmithwell, sure16:56
mriedemyeah default on the CLI is 'error'16:57
dansmithI guess we could make it go to error state in here as a change if you think that's better16:57
*** lpetrut has quit IRC16:57
dansmithI think that won't affect rebuilds because we don't schedule if we do a rebuild, but do for evac16:57
*** penick has joined #openstack-nova16:57
mriedemright we bypass the scheduler for rebuild16:58
mriedemso yeah we could set the instance to ERROR state in conductor...16:59
mriedemif it's an evac16:59
mriedem*failed evac16:59
mriedemwell, now i'm not sure what to do17:01
*** Apoorva has quit IRC17:01
*** Apoorva has joined #openstack-nova17:02
*** penick has quit IRC17:02
dansmithhave a beer?17:03
*** pchavva has quit IRC17:04
mriedemi left a summary in https://review.openstack.org/#/c/499237/17:05
mriedemfeel free to r'ar or whatever17:05
mriedemi'm going to make lunch17:05
*** annegentle has quit IRC17:06
*** gszasz has quit IRC17:07
*** mlakat has quit IRC17:10
*** Tom_ has joined #openstack-nova17:13
*** ragasthya has joined #openstack-nova17:13
*** salv-orlando has quit IRC17:16
*** Tom_ has quit IRC17:17
*** sambetts is now known as sambetts|afk17:19
*** esberglu has joined #openstack-nova17:20
*** thegreenhundred has quit IRC17:24
*** gbarros has quit IRC17:30
*** harlowja has quit IRC17:30
*** gbarros has joined #openstack-nova17:31
*** yangyapeng has joined #openstack-nova17:37
*** yangyapeng has quit IRC17:42
*** shan has joined #openstack-nova17:43
*** lpetrut has joined #openstack-nova17:44
*** cdent has joined #openstack-nova17:44
*** elod has quit IRC17:45
*** carthaca_ has quit IRC17:46
*** sapcc-bot has quit IRC17:46
*** sapcc-bot has joined #openstack-nova17:46
*** carthaca_ has joined #openstack-nova17:46
*** awaugama has quit IRC17:53
*** bnemec is now known as beekneemech17:56
openstackgerritMatt Riedemann proposed openstack/nova master: Update docs for _destroy_evacuated_instance  https://review.openstack.org/50014417:58
openstackgerritSean Dague proposed openstack/nova master: DNM: test cells v1/nova-net without screen  https://review.openstack.org/50015118:03
sdaguemriedem: doesn't seem like cellsv1 job is even in experimental queue for devstack?18:03
mriedemit should be18:04
mriedemused to be anyway18:04
mriedembut yeah i don't see it in there now18:05
mriedemonly in experimental for tempest and nova from what i see18:06
mriedemsdague: are you going to add that or should i?18:09
*** egonzalez has joined #openstack-nova18:09
openstackgerritMatt Riedemann proposed openstack/nova master: doc: link to placement api-ref and history docs from main index  https://review.openstack.org/49897718:10
openstackgerritMatt Riedemann proposed openstack/nova master: doc: link to versioned notification samples from main index  https://review.openstack.org/50008118:10
*** harlowja has joined #openstack-nova18:12
*** Swami has joined #openstack-nova18:17
*** salv-orlando has joined #openstack-nova18:17
*** gszasz has joined #openstack-nova18:20
*** salv-orlando has quit IRC18:22
openstackgerritMatt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails  https://review.openstack.org/49987818:26
openstackgerritMatt Riedemann proposed openstack/nova master: Modernize set_vm_state_and_notify  https://review.openstack.org/49979918:26
fried_riceProcedural question: can I repoint the spec link for a blueprint?  Need to submit a q spec for https://blueprints.launchpad.net/nova/+spec/use-service-catalog-for-endpoints18:28
openstackgerritMatt Riedemann proposed openstack/nova master: Update docs for _destroy_evacuated_instances  https://review.openstack.org/50014418:28
mriedemfried_rice: i have edit rights18:29
mriedemso i can18:29
fried_ricemriedem Ah, cool.  So I'll shoot ya the link once I have it.  Thanks.18:29
*** itlinux has quit IRC18:30
openstackgerritMerged openstack/nova stable/pike: Fix nova assisted volume snapshots  https://review.openstack.org/49897918:30
mriedemthanks for making me think of this https://www.youtube.com/watch?v=GP1KHL0j0GU18:31
*** yamahata has quit IRC18:32
*** david-lyle has joined #openstack-nova18:33
sdaguemriedem: either way, I was just surprised. I feel like we had this issue before18:40
mriedemsdague: it comes up every time a devstack change breaks the cells v1 job and blocks nova18:40
mriedemthen we fix whatever and forget about it18:41
*** annegentle has joined #openstack-nova18:43
*** vikrant has quit IRC18:45
fried_ricemriedem So the other thing is, this is no longer going to be "use service catalog for endpoints" - it's going to be "use keystoneauth1 discovery for endpoints".  Can the blueprint-identifying name be changed; or do I need to file a new blueprint; or do we not care that the identifier is out of sync?18:47
sdagueok, I really thought we merged it the last time :)18:47
*** ragasthya has quit IRC18:47
*** rajathagasthya has quit IRC18:47
mriedemfried_rice: changing the bp name in lp would break the link in the old spec18:49
sdaguemriedem: https://review.openstack.org/50016918:49
mriedemwe could file a new blueprint and supersede the old one18:49
fried_ricemriedem Okay, I'll do that.  Thanks.18:49
fried_ricelong as there's a backtrail.18:49
fried_ricesupersede works18:49
mriedemsdague: +118:50
mriedemdansmith: superdan: sdague: btw, i'm huawei now but guess what i get to do18:51
mriedemBCG18:51
mriedemjealous!?18:51
dansmithwooooo18:51
*** cleong has quit IRC18:57
*** cfriesen__ has quit IRC19:00
*** cfriesen__ has joined #openstack-nova19:00
*** rajathagasthya has joined #openstack-nova19:02
*** ragasthya has joined #openstack-nova19:02
mriedembut no videos...19:03
mriedem:(19:03
*** ijw has joined #openstack-nova19:03
mriedem"what do you think? should wanda record the future sale to her friend's sister's government contractor for the 3rd quarter even though the sale isn't complete but they say they will sign?"19:04
mriedem....YES!19:04
*** esberglu has quit IRC19:04
*** esberglu has joined #openstack-nova19:05
sdaguemriedem: ok, well at least one cells incompat found and fixed, I'll see if that gets it to completion19:09
*** rajathagasthya has quit IRC19:09
*** ragasthya has quit IRC19:09
*** gbarros has quit IRC19:09
*** cfriesen__ has quit IRC19:10
*** tonygunk has joined #openstack-nova19:13
*** gbarros has joined #openstack-nova19:14
*** cfriesen__ has joined #openstack-nova19:15
*** nicolasbock has quit IRC19:18
*** gbarros has quit IRC19:18
*** salv-orlando has joined #openstack-nova19:18
*** cfriesen__ has quit IRC19:20
*** cfriesen__ has joined #openstack-nova19:22
*** penick has joined #openstack-nova19:23
*** salv-orlando has quit IRC19:24
*** cfriesen__ has quit IRC19:28
openstackgerritMatt Riedemann proposed openstack/nova master: Pass migration from API to conductor for evacuate  https://review.openstack.org/50017619:28
*** shaner has quit IRC19:29
*** cfriesen__ has joined #openstack-nova19:32
*** cfriesen__ has quit IRC19:37
*** cfriesen__ has joined #openstack-nova19:38
*** sridharg has quit IRC19:41
*** annegentle has quit IRC19:43
*** tonygunk has quit IRC19:44
*** shan7 has joined #openstack-nova19:45
*** gbarros has joined #openstack-nova19:45
*** tonygunk has joined #openstack-nova19:45
*** shan has quit IRC19:49
*** cfriesen__ has quit IRC19:49
*** shan7 has quit IRC19:49
*** shan has joined #openstack-nova19:50
*** cfriesen__ has joined #openstack-nova19:50
*** gouthamr has quit IRC19:51
*** annegentle has joined #openstack-nova19:53
*** yangyapeng has joined #openstack-nova19:54
*** annegentle has quit IRC19:54
*** shan7 has joined #openstack-nova19:55
*** shan has quit IRC19:55
*** shan has joined #openstack-nova19:57
*** yangyapeng has quit IRC19:58
*** tbachman has quit IRC19:59
*** shan7 has quit IRC20:00
*** cfriesen__ has quit IRC20:02
*** cfriesen__ has joined #openstack-nova20:03
*** tbachman has joined #openstack-nova20:03
*** jmlowe_ has joined #openstack-nova20:03
*** baoli_ has quit IRC20:04
*** jmlowe has quit IRC20:06
*** yangyapeng has joined #openstack-nova20:07
*** rajathagasthya has joined #openstack-nova20:08
*** ragasthya has joined #openstack-nova20:08
*** bhagyashri_s has joined #openstack-nova20:09
*** ps_jadhav has joined #openstack-nova20:10
*** shaner has joined #openstack-nova20:11
*** pooja_jadhav has quit IRC20:11
*** yangyapeng has quit IRC20:11
*** bhagyashris has quit IRC20:12
*** penick has quit IRC20:13
*** cfriesen__ has quit IRC20:13
*** abalutoiu__ has joined #openstack-nova20:13
*** cfriesen__ has joined #openstack-nova20:15
*** smatzek has quit IRC20:16
*** salv-orlando has joined #openstack-nova20:21
*** cfriesen__ has quit IRC20:24
*** salv-orlando has quit IRC20:26
*** gouthamr has joined #openstack-nova20:28
*** rcernin has joined #openstack-nova20:31
*** rajathagasthya has quit IRC20:32
*** marst has quit IRC20:37
cdentmriedem: I recall from earlier in the week there was stuff happening about placement-osc-plugin. Where did that end up? Still in progress? Stall? Abort? Done?20:45
openstackgerritEric Fried proposed openstack/nova-specs master: Spec: Use keystoneauth1 Adapter for endpoints  https://review.openstack.org/50019020:45
fried_ricemriedem mordred sdague ^ kablam20:46
*** thorst_afk has quit IRC20:46
*** esberglu has quit IRC20:49
*** esberglu has joined #openstack-nova20:50
*** ijw has quit IRC20:50
*** vladikr_ has quit IRC20:51
*** egonzalez has quit IRC20:52
*** slaweq has joined #openstack-nova20:52
*** ijw has joined #openstack-nova20:52
*** esberglu has quit IRC20:54
*** vladikr_ has joined #openstack-nova20:54
*** rcernin has quit IRC20:55
*** ijw has quit IRC20:56
*** ijw has joined #openstack-nova20:56
*** catintheroof has joined #openstack-nova20:57
*** shan has quit IRC20:57
*** tinwood has quit IRC20:57
*** catinthe_ has joined #openstack-nova20:58
*** tinwood has joined #openstack-nova20:58
*** tonygunk has quit IRC20:59
*** slaweq has quit IRC21:00
*** catintheroof has quit IRC21:01
*** Apoorva has quit IRC21:03
*** ijw has quit IRC21:05
*** Apoorva has joined #openstack-nova21:10
*** yamahata has joined #openstack-nova21:12
*** Tom__ has joined #openstack-nova21:14
*** gouthamr has quit IRC21:16
*** cfriesen__ has joined #openstack-nova21:18
*** Tom__ has quit IRC21:18
*** salv-orlando has joined #openstack-nova21:21
*** salv-orlando has quit IRC21:26
*** thorst_afk has joined #openstack-nova21:32
*** ijw has joined #openstack-nova21:36
*** thorst_afk has quit IRC21:36
*** priteau has quit IRC21:39
*** abalutoiu__ has quit IRC21:41
*** gszasz has quit IRC21:42
*** lyan has quit IRC21:44
*** prometheanfire has joined #openstack-nova21:45
prometheanfirelooks like online_data_migrations are broken in pike for postgresql21:46
*** gyee has quit IRC21:46
prometheanfiremriedem: http://paste.openstack.org/show/h1ABPkHTtlL8i29IIF3i/21:47
prometheanfirethat's after a db sync (which was successful)21:47
*** thorst_afk has joined #openstack-nova21:48
prometheanfirehttp://paste.openstack.org/show/Oen7ZwvnFNlHfjSXyx9S/ has a second/third run too21:49
*** lpetrut has quit IRC21:50
*** thorst_afk has quit IRC21:50
cdentprometheanfire: is this actually true: “column build_requests.tags does not exist”?21:54
prometheanfirewhich table?21:55
prometheanfireguess build_requests?21:55
* cdent nods21:55
prometheanfirethat table is absent21:56
cdentthat’s nova/db/sqlalchemy/api_migrations/migrate_repo/versions/042_build_requests_add_tags.py21:56
cdentdiid you sync both all your databases?21:57
prometheanfirethat may be it21:57
prometheanfire...21:57
cdentyou need an api_db sync too21:57
prometheanfireya, I forget that every time21:57
cdent:( it ought to be simpler21:58
prometheanfireyep, happier now, NOTABUG21:58
* cdent schwews21:58
prometheanfire:D21:58
prometheanfireso far so good this time around with postgres21:59
cdentI was actually kind of hoping that it was a bug so we could have the “is postgres supported _really_?” argument all over again21:59
cdentbecause gosh isn’t that fun?21:59
prometheanfireoh ya...21:59
*** slaweq has joined #openstack-nova21:59
*** slaweq has quit IRC22:00
prometheanfireif I could migrate to galera or something I would at this point, simply because online migrations and all22:00
*** burt has quit IRC22:00
prometheanfirethis install is grizzly I think, maybe havana22:01
*** tesseract has quit IRC22:09
*** baoli has joined #openstack-nova22:22
*** salv-orlando has joined #openstack-nova22:22
*** baoli has quit IRC22:26
*** salv-orlando has quit IRC22:27
*** thorst_afk has joined #openstack-nova22:29
*** figleaf is now known as edleafe22:31
*** thorst_afk has quit IRC22:32
*** ijw has quit IRC22:33
*** ijw has joined #openstack-nova22:36
*** tonygunk has joined #openstack-nova22:47
cdentIf you like micro-optimizations, here’s one that may be useful: https://review.openstack.org/#/c/486829/22:52
*** catinthe_ has quit IRC22:54
*** Swami has quit IRC22:57
*** catintheroof has joined #openstack-nova23:03
*** sdague has quit IRC23:11
*** cdent has quit IRC23:12
*** gouthamr has joined #openstack-nova23:13
*** prometheanfire has left #openstack-nova23:13
*** thegreenhundred has joined #openstack-nova23:16
*** salv-orlando has joined #openstack-nova23:23
*** thegreenhundred has quit IRC23:24
*** hongbin has quit IRC23:26
*** gbarros has quit IRC23:26
*** ragasthya has quit IRC23:27
*** salv-orlando has quit IRC23:27
*** gyee has joined #openstack-nova23:29
*** markvoelker has quit IRC23:36
*** chyka has quit IRC23:44
*** catintheroof has quit IRC23:51
*** cdent has joined #openstack-nova23:53
*** cdent has quit IRC23:56
*** gyee has quit IRC23:59

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