| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-13 | |||
| 03:58:49 | openstackgerrit | Merged openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070 | |
| 04:05:20 | openstackgerrit | Merged openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | |
| 04:05:26 | openstackgerrit | Merged openstack/nova master: Handle unicode characters in migration params https://review.openstack.org/591080 | |
| 04:05:33 | openstackgerrit | Merged openstack/nova master: improve migration script https://review.openstack.org/591030 | |
| 06:20:06 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/591275 | |
| 06:23:33 | prometheanfire | :D | |
| 06:23:41 | prometheanfire | one step at a time | |
| 07:34:35 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345 | |
| 07:39:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629 | |
| 07:44:12 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | |
| 07:57:37 | openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for PCI devices https://review.openstack.org/591037 | |
| 08:34:16 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP scheduler: check request_spec.instance_group before updating https://review.openstack.org/590733 | |
| 09:11:13 | mriedem | gibi: oh hi | |
| 09:11:18 | mriedem | we are timezone buddies! | |
| 09:12:07 | gibi | mriedem: Ohh welcome in this side of the globe! | |
| 09:12:46 | gibi | mriedem: are you in China? | |
| 09:18:06 | openstackgerrit | garyk proposed openstack/nova master: Docs: remove invalid link for remote debugging https://review.openstack.org/591316 | |
| 09:23:07 | mriedem | mdbooth: http://lists.openstack.org/pipermail/openstack-dev/2018-August/133198.html - the functional test job runs mysql (and postgresql) "opportunistic" tests | |
| 09:23:11 | mriedem | with mysql and pg backends | |
| 09:24:15 | mdbooth | mriedem: So both are configured? IIRC the opportunistic tests run only if the specific DB is actually available. | |
| 09:24:34 | mdbooth | mriedem: You're up early, btw. In Europe somewhere? | |
| 09:25:54 | mriedem_ | mdbooth: china | |
| 09:26:10 | mdbooth | Ah, so you've been up for a while :) | |
| 09:26:26 | mriedem_ | https://github.com/openstack/nova/blob/master/tools/test-setup.sh sets up the mysql and pg dbs for the opportunistic tests | |
| 09:26:47 | mdbooth | mriedem_: Thank you! | |
| 09:33:24 | mriedem | mdbooth: btw is rajesh aware of the spec for that metadata case sensitivity issue? | |
| 09:33:42 | mdbooth | mriedem: I was just putting a +1 on it and I noticed your comments. Reading... | |
| 09:34:02 | mdbooth | ratailor: ^^^ ? | |
| 09:35:33 | ratailor | mdbooth, I don't know about spec. | |
| 09:35:42 | gibi | mriedem: do you know if we can remove this in Stein? https://github.com/openstack/nova/blob/16f89fd093217d22530570e8277b561ea79f46ff/nova/virt/ironic/driver.py#L147-L149 | |
| 09:36:00 | mdbooth | At first glance it doesn't sound like a very good idea to me, tbh. | |
| 09:36:58 | ratailor | mdbooth, let me go through the comments on patch. | |
| 09:36:58 | mdbooth | So.. we've allowed users to define their own metadata for however long, and now we want to change it just because of a mysql misconfiguration? | |
| 09:37:01 | mriedem | gibi: maybe... | |
| 09:37:24 | mdbooth | That seems user hostile | |
| 09:37:33 | mriedem | gibi: we have a nova-status upgrade check for the ironic instance / flavor migration to resource classes | |
| 09:37:42 | ratailor | mdbooth, mriedem in that case, should I abandon the patch ? | |
| 09:37:48 | mriedem | gibi: i think we have something in the ptg etherpad about this, removing support for granular resources with ironic | |
| 09:38:39 | gibi | mriedem: Is it L25 in the etherpad? | |
| 09:39:17 | gibi | mriedem: it seems so. So I'm plugging my pain (https://review.openstack.org/#/c/583667/19/nova/tests/functional/test_servers.py@5135) into that item. thanks | |
| 09:40:04 | mdbooth | ratailor: It's up to you, but personally I'd see if the bp and spec still have weight behind them, especially given that they're user hostile and your patch shows that fixing the underlying issue isn't hard. | |
| 09:40:45 | mriedem | ratailor: mdbooth: right it's obviously still an issue, | |
| 09:40:53 | mriedem | and the spec was merged at some point, so there was a plan with code, | |
| 09:41:03 | mriedem | but then auggy was gone and sdague was gone and that whole HP team | |
| 09:41:11 | mriedem | er IBM team or whatever | |
| 09:41:46 | mriedem | gibi: yes that's the one | |
| 09:41:54 | mriedem | definitely want to remove that in stein if we can | |
| 09:41:55 | gibi | mriedem: thanks. I've added a note | |
| 09:43:03 | sean-k-mooney | mriedem: are you in china right now or is this very early for you. im guessing its quite late but just wondering what time zone your in since your not normally on at this time | |
| 09:43:37 | mriedem | sean-k-mooney: i'm in china this week | |
| 09:44:02 | mdbooth | ratailor: Looks like the proposed code wasn't merged and was abandoned. | |
| 09:45:02 | mdbooth | mriedem: Thanks for pointing that out. I'll follow up on the ML with a question around if this is still something we want to do. | |
| 09:45:33 | mdbooth | ratailor: Unless you want to do that? | |
| 09:45:55 | ratailor | mdbooth, no problem. | |
| 09:54:17 | mriedem | Kevin_Zheng: i think resize to same host might have fixed the doubled allocation issue after all - we move the "source" allocations to the migration record and the target (new flavor) allocations to the instance | |
| 09:55:21 | mriedem | Kevin_Zheng: although we'll still have allocations for both flavors against the same resource provider (compute node), so they are essentially doubled in placement... | |
| 09:57:09 | Kevin_Zheng | you mean already fixed? | |
| 09:57:46 | mriedem | well, depends on what is considered the issue; the instance, as a consumer, will only have allocations for the new_flavor during the resize, | |
| 09:58:02 | mriedem | but the resource provider (compute node) will have allocations for both the old_flavor and new_flavor, but to different consumers | |
| 09:58:10 | mriedem | so that is still an issue | |
| 09:58:38 | mriedem | if i resize from 2 VCPU to 4 VCPU, during the resize to same host, the host RP will have 6 VCPU being used, rather than 4 | |
| 09:58:46 | Kevin_Zheng | Ok got it | |
| 09:59:09 | Kevin_Zheng | But if it only has two more? | |
| 09:59:11 | sean-k-mooney | mriedem: that is only while the resize is happening correct | |
| 09:59:23 | Kevin_Zheng | Like only has 4 | |
| 09:59:39 | Kevin_Zheng | And we do 2->4 | |
| 09:59:40 | mriedem | sean-k-mooney: correct | |
| 09:59:55 | mriedem | Kevin_Zheng: if the host only has 4 VCPU inventory, the resize to same host would fail | |
| 10:00:16 | Kevin_Zheng | Yeah, but that’s our case | |
| 10:00:22 | mriedem | yeah... | |
| 10:00:25 | mriedem | alright, going to dinner | |
| 10:00:39 | Kevin_Zheng | Coll | |
| 10:00:41 | Kevin_Zheng | Cool | |
| 10:08:11 | mdbooth | ratailor: I wonder if we can get zzzeek to comment on http://lists.openstack.org/pipermail/openstack-dev/2018-August/133202.html | |
| 10:11:20 | ratailor | mdbooth, Thanks for putting it on ML nicely, I think we should get comment from him. | |
| 10:12:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Add missing aggregates example https://review.openstack.org/591332 | |
| 10:25:46 | openstackgerrit | Rajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys https://review.openstack.org/504885 | |
| 10:38:03 | nalhat | Hi, Is any one aware about the filters used by nova: 'AggregateInstanceExtraSpecsFilter' while instance reservation(Blazar) | |
| 10:40:43 | nalhat | I was referring this doc: https://docs.openstack.org/blazar/latest/cli/instance-reservation.html for instance reservation by Blazar, at the time of creating an instance in nova with reservation id this filter comes in picture | |
| 11:52:07 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4) https://review.openstack.org/568462 | |
| 11:54:43 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: VMware: fix TypeError while get console log https://review.openstack.org/591365 | |
| 12:00:12 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | |
| 12:05:15 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842 | |
| 12:10:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330 | |
| 12:13:18 | openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367 | |
| 12:14:46 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | |
| 12:18:32 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | |
| 12:22:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104 | |
| 13:10:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Add missing aggregates example https://review.openstack.org/591332 | |
| 13:47:46 | openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367 | |
| 13:50:22 | efried | Scheduler meeting in 10 minutes in #openstack-meeting-alt | |
| 14:30:38 | dansmith | sahid: we wanted to make sure you saw this: https://review.openstack.org/#/c/591275/ | |
| 14:30:54 | dansmith | sahid: and that the rationale was clear, because we discussed mostly on irc | |
| 14:31:28 | dansmith | basically, I think that doing that in the libvirt driver will always be racy because of where the plug happens, | |
| 14:32:03 | dansmith | but since mriedem added it generically in the compute manager (where it can be done properly) it has the same effect you were going for | |
| 14:32:22 | dansmith | and, the two conflict of course because the wait in compute manager will "eat" the notification so that the libvirt wait will never see it | |
| 14:33:13 | dansmith | sahid: I also wonder if the race we identified is why you were seeing it not always work sometimes | |
| 14:33:30 | dansmith | the person that reported it on pike was always seeing it fail to win the race and thus never increase the speed limit | |
| 14:33:40 | gibi | tetsuro: would you mind if I rebase you report client 1.29 patches? | |
| 14:34:10 | gibi | tetsuro: I would like to see if 1.28 and 1.29 are still compatible codewise before I move forward with the split | |