openstackgerrit | karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620 | 01:13 |
---|---|---|
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | 01:34 |
*** edmondsw has quit IRC | 01:51 | |
openstackgerrit | JiaJunsu proposed openstack/nova master: Remove args(os=False) in monkey_patch https://review.openstack.org/568999 | 02:02 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: update the document related to vdi streaming https://review.openstack.org/568444 | 02:19 |
*** edmondsw has joined #openstack-placement | 02:56 | |
*** edmondsw has quit IRC | 03:01 | |
*** tetsuro has quit IRC | 03:05 | |
*** tetsuro has joined #openstack-placement | 03:05 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (2) https://review.openstack.org/568757 | 03:29 |
openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703 | 03:41 |
openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | 03:44 |
openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703 | 03:44 |
openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: ProviderTree.has_inventory_changed for new fields https://review.openstack.org/569228 | 04:13 |
openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Normalize inventory from update_provider_tree https://review.openstack.org/569132 | 04:13 |
openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 04:13 |
openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 04:13 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (3) https://review.openstack.org/569320 | 04:21 |
*** gjayavelu has quit IRC | 04:29 | |
*** gjayavelu has joined #openstack-placement | 04:30 | |
*** edmondsw has joined #openstack-placement | 04:45 | |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 04:45 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 04:45 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 04:45 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 04:45 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 04:45 |
*** edmondsw has quit IRC | 04:50 | |
openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | 06:01 |
*** edmondsw has joined #openstack-placement | 06:33 | |
*** edmondsw has quit IRC | 06:38 | |
openstackgerrit | Robin Naundorf proposed openstack/nova master: Fix inconsistency in docs https://review.openstack.org/569348 | 06:54 |
*** tssurya has joined #openstack-placement | 07:06 | |
*** e0ne has joined #openstack-placement | 07:09 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | 07:32 |
openstackgerrit | Robin Naundorf proposed openstack/nova master: Fix inconsistency in docs https://review.openstack.org/569348 | 07:37 |
*** tetsuro has quit IRC | 07:41 | |
openstackgerrit | sahid proposed openstack/nova master: pci: don't consider case when match tags specs https://review.openstack.org/565808 | 08:32 |
openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 08:32 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | 08:32 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: add vf_trusted field for network metadata https://review.openstack.org/566343 | 08:32 |
openstackgerrit | sahid proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | 08:32 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.compute.cpu_shared_set https://review.openstack.org/510897 | 08:52 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877 | 09:01 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877 | 09:08 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877 | 09:11 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 09:26 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 09:26 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | 09:26 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Configure multiple vGPU types https://review.openstack.org/564214 | 09:33 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: mdevs returning parent and vendor PCI info https://review.openstack.org/562304 | 09:33 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Get inventories per vGPU type https://review.openstack.org/564215 | 09:33 |
*** edmondsw has joined #openstack-placement | 10:10 | |
*** edmondsw has quit IRC | 10:14 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | 10:16 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | 10:18 |
*** avolkov has joined #openstack-placement | 11:11 | |
*** edmondsw has joined #openstack-placement | 11:26 | |
*** edleafe is now known as figleaf | 12:59 | |
*** jaypipes has joined #openstack-placement | 13:04 | |
*** mriedem has joined #openstack-placement | 13:07 | |
jaypipes | efried: you had mentioned something on Wednesday about https://github.com/openstack/nova/blob/4b4e005c639204f2c680eda61131a6930379d70e/nova/compute/resource_tracker.py#L884-L886. I was out all day yesterday dealing with some health problems. what was it you were getting at? | 13:13 |
*** diga has joined #openstack-placement | 13:49 | |
openstackgerrit | Eric Berglund proposed openstack/nova stable/pike: Stringify instance UUID https://review.openstack.org/569421 | 14:14 |
*** rubasov has quit IRC | 14:18 | |
jaypipes | alex_xu: nice work on https://review.openstack.org/#/c/569011. Left some suggestions and wording fixes inline. | 14:27 |
gibi | jaypipes: could you look at https://review.openstack.org/#/c/515811/ again? | 14:31 |
jaypipes | gibi: yessir | 14:35 |
gibi | jaypipes: thank you sir | 14:35 |
jaypipes | gibi: welcomesir | 14:36 |
gibi | :) | 14:36 |
jaypipes | dansmith, mriedem, bauzas: your thoughts requested on https://review.openstack.org/#/c/515811/ please (see release note comment) | 14:59 |
jaypipes | gibi: I'm essentially +2 on the patch -- the code at least. I'm slightly concerned about the impact on upgrade for operators that modify a flavor and that flavor goes from working properly to resulting in an error about missing group_policy for some amount of time. | 14:59 |
bauzas | jaypipes: ack, but I'll be on travels, so I'll review it during the summit | 15:00 |
jaypipes | gibi: wondering if we should indeed have a default group_policy... | 15:00 |
jaypipes | bauzas: ack, no prob | 15:00 |
jaypipes | bauzas: safe travels | 15:00 |
bauzas | jaypipes: by the "group policy", that means the default behaviour when asking for multiple groups ? | 15:01 |
gibi | jaypipes: It will be fun to re-iterate the question of default policy. But better talking about it now than in after the release | 15:01 |
gibi | bauzas: the policy of overlapping RPs for different numbered request groups | 15:02 |
bauzas | if we need to update flavors, it's a huuuuuge upgrade impact | 15:03 |
mriedem | jaypipes: replied | 15:03 |
jaypipes | mriedem: ty matt | 15:03 |
gibi | jaypipes: to have more than one group in the flavor we need the patch we are talking about | 15:03 |
bauzas | mriedem: your comment works if we explicitely say which default behaviour there is | 15:04 |
gibi | jaypipes: basically whant mriedem said in the review | 15:04 |
bauzas | mriedem: which is why I think we could default | 15:04 |
jaypipes | gibi: yes, I completely see that. I'm just concerned about the time period when a deployer edits a flavor to add in the granular stuff, doesn't see the requirement about group_policy extra spec, and there's a time period where all builds of that flavor die :) | 15:04 |
bauzas | what jaypipes said | 15:05 |
bauzas | it could be errorprone | 15:05 |
gibi | jaypipes: but that can be handled with proper documentation | 15:05 |
jaypipes | gibi: and yes, I understand it's only a temp situation :) | 15:05 |
bauzas | if we say 'if you use granular request groups', this is the default behaviour | 15:05 |
bauzas | then we don't need to automatically provide that extra spec | 15:05 |
jaypipes | bauzas: right, but that comes with its own set of bugaboos where the flavor behaviour could temporarily be different from what the admin expected... it's a weird situation alright. | 15:06 |
bauzas | probably it's a nit, I dunno | 15:06 |
bauzas | but I feel we make it verbose for something not really important | 15:06 |
bauzas | group_policy=none is semantically equal to not providing the extra spec to me | 15:06 |
bauzas | which makes that policy ("none") to be the default | 15:07 |
bauzas | we could rathole on that, so I'm not particularly attached to that, I just feel it would help operators | 15:07 |
jaypipes | bauzas: to be clear, you feel adding a default would help operators, yes? | 15:08 |
bauzas | jaypipes: repliede | 15:10 |
bauzas | replied* even | 15:10 |
bauzas | but like I said, I'm not opposed to make the extra spec mandatory, I just feel it's unnecessary from a semantical pov | 15:10 |
bauzas | either way, I need to review that change before I can +2 | 15:11 |
bauzas | and that's not something I can do soon today | 15:11 |
jaypipes | bauzas: I see both sides... we're kinda dammed if we do, damned if we don't. | 15:11 |
bauzas | so, feel free to cut the cord | 15:11 |
jaypipes | heh, ok :) | 15:11 |
bauzas | cut the rope$ | 15:11 |
bauzas | jaypipes: FWIW, I will hail Placement API in my slides :p | 15:12 |
bauzas | I even made it a pinned tweet | 15:13 |
efried | jaypipes: Turns out we *do* need to normalize inventory. I put up https://review.openstack.org/#/c/569132/ to fix. | 15:13 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/queens: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/569442 | 15:13 |
bauzas | efried: seen my comment btw. ? | 15:13 |
efried | bauzas: looking at it now, thanks for the review. | 15:13 |
bauzas | efried: I was about to say we could make https://review.openstack.org/#/c/560444/7 the bottom change (as it would help my own series) but if so, we'd regress | 15:14 |
bauzas | efried: so I'm all up for making those changes to be merged ASAP | 15:14 |
efried | The bottom two changes were made to fix that one. | 15:14 |
bauzas | I know | 15:14 |
efried | cool. I'll get to work updating the comments ASAP. | 15:14 |
bauzas | hence my "we'd regress" | 15:14 |
bauzas | efried: understood my opinion on that ? | 15:15 |
bauzas | efried: it's more an opiniated comment, rather than technical | 15:15 |
efried | bauzas: Yeah, totally. I didn't mean to imply that we would ever pass the compute node to the virt driver. | 15:15 |
efried | but I'll fix it to make that clearer. | 15:15 |
bauzas | I just feel that we should never pass the ComputeNode object to the virt drivers, or terrible things would happen | 15:15 |
jaypipes | efried, bauzas: I've +W'd the change | 15:15 |
efried | jaypipes: which one? | 15:16 |
bauzas | jaypipes: the numbered group thing ? cool. | 15:16 |
bauzas | I'm way behind reviewing, I need to catch up | 15:16 |
bauzas | I also need to understand the state of art with allocation candidates and nested resource providers | 15:16 |
bauzas | efried: jaypipes: I saw tetsuro having yet another spec ? that's sad. | 15:17 |
efried | bauzas: tetsuro owns the series to make nrp-in-alloc-cands happen. I haven't seen the new spec yet. | 15:17 |
bauzas | that creates more complexity for having nested resource providers be a thing for Rocky | 15:17 |
mriedem | i haven't been following, i'm in -nova with dansmith | 15:17 |
efried | I was at my kid's talent show this morning, looks like I missed a lot. | 15:17 |
mriedem | this is simple, if you use granular request groups in your flavor, you MUST specify a policy | 15:17 |
*** jaypipes is now known as leakypipes | 15:17 | |
*** efried is now known as fried_rice | 15:17 | |
fried_rice | that too | 15:18 |
mriedem | anyone that f'ed that up and fails didn't read the docs | 15:18 |
mriedem | we specifically said in the spec we wouldn't do a default policy because we shouldn't pick, | 15:18 |
mriedem | we want the user of the API to be explicit | 15:18 |
mriedem | s/pick/assume/ | 15:18 |
*** dansmith is now known as superdan | 15:18 | |
bauzas | mriedem: either way, the patch is +Wd | 15:18 |
bauzas | we said it's a doc problem | 15:18 |
fried_rice | leakypipes: This one needs your eye, want to make sure you agree: https://review.openstack.org/#/c/569228/ | 15:19 |
leakypipes | fried_rice: k. | 15:19 |
*** e0ne has quit IRC | 15:19 | |
mriedem | bauzas: well, we also talked about this a few weeks ago and agreed on the course | 15:19 |
fried_rice | leakypipes: Let me know if you need the back story, or if it's clear from the context of the series. | 15:20 |
leakypipes | fried_rice: will do, thx. | 15:20 |
bauzas | mriedem: in the spec, yeah, I just feel we ratholed on that one | 15:20 |
bauzas | anyway, it's done | 15:21 |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300 | 15:21 |
fried_rice | bauzas: So if I just change to "...the virt driver doesn't (and never will) have access..." will that satisfy? | 15:21 |
bauzas | that's all good with me | 15:21 |
bauzas | fried_rice: one other option could be to not normalize, but rather explicitely ask the virt driver to pass the whole inventory | 15:21 |
bauzas | fried_rice: but then, update the inventory with the values we get from the compute node | 15:22 |
fried_rice | bauzas: Which it can, if it wishes. If those fields are set by the virt driver, they're *not* overridden by the rt. | 15:22 |
bauzas | fried_rice: I see, so normalization is good | 15:22 |
fried_rice | bauzas: But if they aren't set by the virt driver, we want to set them based on the vals in the compute node. | 15:22 |
bauzas | fried_rice: yup, I understood that | 15:23 |
bauzas | anyway, I need to pack my stuff | 15:23 |
*** jroll is now known as jrollinhatin | 15:23 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Normalize inventory from update_provider_tree https://review.openstack.org/569132 | 15:24 |
openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 15:24 |
openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 15:24 |
fried_rice | bauzas: fixed the comment & commit message ^ | 15:25 |
bauzas | looking qucikly | 15:25 |
fried_rice | bauzas: Hopefully making it clear that what I meant was "we need to do this unless some day we stop pushing those values through the compute_node entirely" | 15:25 |
bauzas | fried_rice: you mean, stopping to lookup the ComputeNode object ? well, maybe. | 15:26 |
bauzas | it's just an implementation details | 15:26 |
bauzas | allocation ratios are set per compute, hence defined by the compute service, not the hypervisor | 15:26 |
bauzas | how those values are passed to the inventory is just an implementation detail to me | 15:26 |
bauzas | fried_rice: the fact we used the object facade was only for upgrade concerns | 15:27 |
bauzas | fried_rice: if we keep the things as they are, I think we could just drop that | 15:27 |
bauzas | and simplify all that crap | 15:28 |
fried_rice | bauzas: I find it odd that the virt driver is responsible for all the other fields in the inventory, but allocation ratio and reserved amounts can be set outside of that. | 15:28 |
fried_rice | I'm mollified by the fact that the virt driver still *can* set those values if it wishes. | 15:29 |
fried_rice | cause we're certainly going to do that in powervm | 15:29 |
bauzas | for the reserved bit, yeah | 15:29 |
fried_rice | allocation ratio too. | 15:29 |
fried_rice | in power, this isn't an overcommit; it's a real thing, because we have micropartitioning. | 15:30 |
bauzas | fried_rice: then you would mess up the conf option | 15:30 |
fried_rice | bauzas: Actually we would use the conf options to decide on the micropartitioning size. | 15:30 |
fried_rice | bauzas: So technically I suppose it's the same thing. | 15:31 |
bauzas | fried_rice: but you need to use the object facade to get the right thing | 15:31 |
fried_rice | bauzas: Point is, we don't intend to let the rt set those values. | 15:31 |
bauzas | fried_rice: because atm. some ops can have 0.0 in the conf files, and things are fine | 15:31 |
fried_rice | which is how the code is set up anyway - we're not violating any rules. | 15:31 |
fried_rice | We can haggle over that when we cut powervm over to using update_provider_tree. Which we'll do as soon as nrp-in-alloc-cands lands, because we want to make use of trees for devices. | 15:32 |
openstackgerrit | Allen proposed openstack/nova master: Update the deprecate os_region_name option https://review.openstack.org/567829 | 15:35 |
bauzas | fried_rice: one comment on unittest with https://review.openstack.org/#/c/569132/5 | 15:37 |
fried_rice | bauzas: Responded. | 15:41 |
*** tssurya has quit IRC | 15:44 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/pike: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/569449 | 15:46 |
mriedem | leakypipes: not sure if you saw, but there was some review on the consumer generation spec update yesterday - https://review.openstack.org/#/c/565565/ - i was holding off on reviewing the code series until the spec was updated otherwise i'm coming into the code changes kind of blind | 15:49 |
*** melwitt is now known as jgwentworth | 15:49 | |
leakypipes | mriedem: yes, I planned to get to those shortly. | 15:49 |
leakypipes | jgwentworth: both quota-related specs reviewed now. | 15:49 |
leakypipes | jgwentworth: short summary of my opinion: we should move forward on both. | 15:51 |
jgwentworth | leakypipes: I saw, thanks! I was about to ping you to clarify what I meant when I brought up the "owner" or "type" issue on my spec. I explained it on alex_xu's spec if you could take a look and let me know what you think | 15:51 |
leakypipes | jgwentworth: yup, I'm on it. | 15:51 |
jgwentworth | that is, I just now posted some comments on his spec | 15:51 |
jgwentworth | awesome | 15:51 |
leakypipes | :) yep, I saw em | 15:51 |
jgwentworth | :) | 15:52 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip ServerShowV254Test.test_rebuild_server in cells v1 job https://review.openstack.org/569454 | 16:06 |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/569127 | 16:07 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | 16:20 |
*** e0ne has joined #openstack-placement | 16:26 | |
openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Testing a thing https://review.openstack.org/569461 | 16:28 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove the remaining of the removed option https://review.openstack.org/569462 | 16:28 |
openstackgerrit | Merged openstack/nova master: Update the deprecate os_region_name option https://review.openstack.org/567829 | 16:29 |
openstackgerrit | Merged openstack/nova master: Fix inconsistency in docs https://review.openstack.org/569348 | 16:30 |
*** e0ne has quit IRC | 16:32 | |
*** e0ne has joined #openstack-placement | 16:33 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300 | 16:34 |
jgwentworth | leakypipes: thanks for your reply, replied again :) trying to explain what I meant there about the potential for ambiguity | 16:43 |
jgwentworth | I realize the example of VCPU and MEMORY_MB might be silly (can we assume no other service will use those resource classes?) | 16:44 |
jgwentworth | but maybe it opens up more in alex_xu's use case where we would add quota for custom resource classes | 16:45 |
superdan | jgwentworth: leakypipes: I think I've said this before, but two novas using the same placement would both use VCPU, not to mention something else using it to provide compute resources (like magnum) | 16:47 |
leakypipes | superdan: ah, yes, that's true indeed. | 16:48 |
leakypipes | jgwentworth: I see now, yes. | 16:49 |
jgwentworth | superdan: you have and my brain lost the info | 16:49 |
* jgwentworth puts information back in brain | 16:49 | |
* superdan hands jgwentworth some tape | 16:49 | |
* jgwentworth uses tape to hold information for a time | 16:49 | |
* leakypipes hands jgwentworth a funnel, which is the correct tool for the job. | 16:51 | |
superdan | a funnel for insertion, tape for retention | 16:51 |
leakypipes | and to gross everyone out... here's actual brain losing info: https://www.cnn.com/2018/05/08/health/cerebrospinal-fluid-leak-nebraska/index.html | 16:51 |
jgwentworth | dear lord | 16:52 |
leakypipes | hehe | 16:52 |
jgwentworth | but yeah, the two novas use case messes up the namespace idea. and the "type" idea wouldn't work in that case either. which is probably how we got onto discussing how to represent a unique "owner" at dublin | 17:01 |
jgwentworth | wah wah | 17:01 |
leakypipes | jgwentworth: added a potential solution for the 2 Novas, 1 Placement situation. | 17:11 |
leakypipes | jgwentworth: same solution could be used for per-cell quotas. | 17:12 |
jgwentworth | looking | 17:13 |
leakypipes | jgwentworth: we can call it Multi-Dimension Mass Allocation. or MDMA. | 17:14 |
jgwentworth | lol | 17:14 |
jgwentworth | okay, so thinking of ... how nova aggs get mirrored into placement aggs, what you would do is draw a new aggregate for quotas that covers the entire deployment (if you want quota that spans the entire deployment) | 17:15 |
leakypipes | jgwentworth: or cell, yeah. | 17:16 |
leakypipes | jgwentworth: just basically telling placement that X set of providers is related. | 17:16 |
jgwentworth | neat. yeah, seems like that works. and per cell quota is something I think people would like too | 17:16 |
*** diga has quit IRC | 17:19 | |
*** avolkov has quit IRC | 17:20 | |
*** gjayavelu has quit IRC | 17:46 | |
*** gjayavelu has joined #openstack-placement | 17:47 | |
*** e0ne has quit IRC | 18:10 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make instance.refresh() avoid recursion better https://review.openstack.org/569461 | 18:13 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make instance able to lazy-load almost everything https://review.openstack.org/569267 | 18:13 |
openstackgerrit | Dan Smith proposed openstack/nova master: Expose instance_get_all_uuids_by_host() from DB API and use it https://review.openstack.org/569268 | 18:13 |
openstackgerrit | Merged openstack/nova master: Fix typo in NUMATopologyFilter docs https://review.openstack.org/543505 | 18:18 |
*** mriedem has quit IRC | 18:54 | |
*** mriedem has joined #openstack-placement | 18:54 | |
openstackgerrit | Matthew Edmonds proposed openstack/nova master: Update PowerVM hypervisor docs https://review.openstack.org/569497 | 19:02 |
openstackgerrit | Arvind Nadendla proposed openstack/nova master: WIP:Handle rebuild of instances with image traits https://review.openstack.org/569498 | 19:04 |
fried_rice | leakypipes: Still around? | 19:29 |
fried_rice | I haven't gotten through review email for the last couple hours yet, but wanted to clarify that the _normalize patch has nothing to do with the bug that bhagyashris tagged it with. | 19:29 |
*** gjayavelu has quit IRC | 19:34 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Normalize inventory from update_provider_tree https://review.openstack.org/569132 | 19:36 |
openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 19:36 |
openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 19:36 |
fried_rice | leakypipes: I removed the bogus Partial-Bug tag from those two patches. | 19:36 |
fried_rice | leakypipes: And tried to explain in comments. Let me know if you want the unabridged version. | 19:37 |
openstackgerrit | Matthew Edmonds proposed openstack/nova master: Update PowerVM hypervisor docs https://review.openstack.org/569497 | 19:52 |
*** e0ne has joined #openstack-placement | 19:55 | |
openstackgerrit | Arvind Nadendla proposed openstack/nova master: WIP:Handle rebuild of instances with image traits https://review.openstack.org/569498 | 19:57 |
fried_rice | leakypipes: Are you going to get a chance to take a swing at tetsuro's series today? | 20:29 |
*** e0ne has quit IRC | 20:30 | |
openstackgerrit | Arvind Nadendla proposed openstack/nova master: WIP:Handle rebuild of instances with image traits https://review.openstack.org/569498 | 21:05 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update PowerVM hypervisor docs https://review.openstack.org/569497 | 21:12 |
*** mriedem1 has joined #openstack-placement | 21:24 | |
*** mriedem has quit IRC | 21:24 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make instance.refresh() avoid recursion better https://review.openstack.org/569461 | 21:36 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make instance able to lazy-load almost everything https://review.openstack.org/569267 | 21:36 |
openstackgerrit | Dan Smith proposed openstack/nova master: Expose instance_get_all_uuids_by_host() from DB API and use it https://review.openstack.org/569268 | 21:36 |
openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300 | 21:38 |
*** figleaf is now known as edleafe | 21:40 | |
openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.has_inventory_changed for new fields https://review.openstack.org/569228 | 21:41 |
openstackgerrit | Eric Fried proposed openstack/nova master: Normalize inventory from update_provider_tree https://review.openstack.org/569132 | 21:41 |
openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 21:41 |
openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 21:41 |
openstackgerrit | Eric Fried proposed openstack/nova master: Granular requests to get_allocation_candidates https://review.openstack.org/515811 | 21:43 |
openstackgerrit | Eric Fried proposed openstack/nova master: Update nova-status and docs for required placement 1.24 https://review.openstack.org/568923 | 21:46 |
openstackgerrit | Eric Fried proposed openstack/nova master: Enforce placement minimum in nova.cmd.status https://review.openstack.org/568965 | 21:47 |
*** mriedem1 has quit IRC | 21:54 | |
openstackgerrit | Matthew Edmonds proposed openstack/nova master: Update PowerVM hypervisor docs https://review.openstack.org/569497 | 21:55 |
openstackgerrit | Eric Fried proposed openstack/nova master: Update PowerVM hypervisor docs https://review.openstack.org/569497 | 22:00 |
*** gjayavelu has joined #openstack-placement | 22:15 | |
*** edmondsw has quit IRC | 22:54 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Placement: allow to set reserved value equal to total for inventory https://review.openstack.org/564838 | 23:04 |
openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | 23:04 |
openstackgerrit | Eric Fried proposed openstack/nova master: Avoid unnecessary joins in HostManager._get_instances_by_host https://review.openstack.org/569218 | 23:05 |
openstackgerrit | Eric Fried proposed openstack/nova master: Update placement upgrade docs for nova-api dependency on placement https://review.openstack.org/568925 | 23:06 |
openstackgerrit | karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620 | 23:16 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!