| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-25 | |||
| 19:43:50 | mriedem | that's where the nodes on the host are all claimed? | |
| 19:44:01 | dansmith | no, there's an easier way | |
| 19:44:12 | dansmith | jan will take a crack at it while marcia brushes her hair | |
| 19:44:33 | mriedem | alright, wfm | |
| 19:44:39 | mriedem | works. for. marcia. | |
| 19:44:58 | mriedem | oh btw i'm in full summer feather hair mode atm | |
| 19:45:09 | mriedem | but it will be shorn before denver | |
| 19:49:07 | sean-k-mooney | so ya that just finished running all the nova fuctional tests. nova.tests.functional.libvirt.test_numa_servers.NUMAServersWithNetworksTest.test_rebuild_server_network_changes is the only one that failed in dans version so its just the attch suff thats failing. | |
| 20:03:17 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Add additional functional tests for NUMA networks https://review.openstack.org/585385 | |
| 20:16:58 | artom | mriedem, wait, it failed and logged nothing at all o_O | |
| 20:19:49 | dansmith | mriedem: so you'll convert your +1 to +2 on the functional move patch? | |
| 20:21:50 | efried | Does he have to deny you three times, or something? | |
| 20:22:07 | efried | not that kind of conversion, maybe. | |
| 20:58:39 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add support for multiple fixed-ips in metadata https://review.openstack.org/580742 | |
| 21:00:12 | mriedem | dansmith: yeah | |
| 21:00:27 | mriedem | artom: right | |
| 21:02:09 | melwitt | does anyone know anything about metadata API versions and what's valid? for the addition of 'ip_addresses' to network_data.json, it means we have a second version update this cycle and we apparently already used a date in the future 2018-08-27 https://review.openstack.org/#/c/580742/2/nova/api/metadata/base.py@77 | |
| 21:03:32 | mriedem | apparently we already did that once with NEWTON_TWO even though NEWTON_ONE was using the wrong date | |
| 21:03:44 | melwitt | what's proposed is 2018-08-27-2 which is a format we haven't done before | |
| 21:03:52 | mriedem | # NOTE(mikal): think of these strings as version numbers. They traditionally | |
| 21:03:52 | mriedem | # correlate with OpenStack release dates, with all the changes for a given | |
| 21:03:53 | mriedem | # release bundled into a single version. Note that versions in the future are | |
| 21:03:53 | mriedem | # hidden from the listing, but can still be requested explicitly, which is | |
| 21:03:53 | mriedem | # required for testing purposes. We know this isn't great, but its inherited | |
| 21:03:53 | mriedem | # from EC2, which this needs to be compatible with. | |
| 21:04:03 | mriedem | i would -1 on the fomrat | |
| 21:04:04 | mriedem | *format | |
| 21:04:24 | mriedem | based on "They traditionally | |
| 21:04:24 | mriedem | # correlate with OpenStack release dates, with all the changes for a given | |
| 21:04:24 | mriedem | # release bundled into a single version." we could lump the changes into the existing ROCKY version | |
| 21:04:42 | mriedem | which isn't CI/CD friendly, so if we cared, i'd just make the ROCKY_TWO a day later than ROCKY_ONE | |
| 21:04:48 | mriedem | i'm a big meh on either approach | |
| 21:05:10 | melwitt | okay, I was wondering about that. else we could use 2018-08-30 the actual rocky release date? I haven't been able to find examples of consumers using version strings to request versions | |
| 21:05:30 | mriedem | it says versions in the future are hidden from the listing, | |
| 21:05:38 | mriedem | so if that's true, rocky doesn't even show up today, | |
| 21:05:40 | mriedem | but would need to confirm | |
| 21:06:11 | melwitt | and it looks like this new version won't break anyone because it adds a field, doesn't change any fields. so cloud-init defaulting to latest (I think it does, based on the code) will still work with the new version with 'ip_addresses' in it | |
| 21:06:19 | melwitt | ohh, I see | |
| 21:06:34 | mriedem | i have a hell of a time ever knowing where the route code in this thing works | |
| 21:06:44 | melwitt | so rolling together should work on that basis. I didn't understand what "hidden" meant until you said that | |
| 21:07:00 | mriedem | well, i'd want to know where that hiding happens | |
| 21:07:04 | mriedem | i don't have a devstack handy to test this | |
| 21:07:09 | dansmith | right, should be rocky release date, and we shouldn't have multiple versions for rocky | |
| 21:07:26 | dansmith | I think that the date-based one is hidden, | |
| 21:07:32 | dansmith | but current takes you to it, | |
| 21:07:41 | mriedem | https://review.openstack.org/#/c/580742/2/nova/api/metadata/base.py@597 | |
| 21:07:49 | dansmith | and the idea is that until the release happens, it's not really codified as the date-based version, | |
| 21:07:51 | dansmith | so we can add stuff to it | |
| 21:07:56 | mriedem | yup, | |
| 21:07:58 | mriedem | found that code | |
| 21:10:17 | melwitt | a-ha, cool. thanks for all that info | |
| 21:12:49 | melwitt | fyi mgagne ^ (backscroll for more context on the latest review comment) | |
| 21:13:46 | mgagne | melwitt: so no new metadata api version and call it a day? | |
| 21:14:28 | melwitt | mgagne: yeah, the code mriedem highlighted on the review will hide version dates that are in the future, so that they stay 'unreleased' until the release date. so you can roll your changes into the existing ROCKY version | |
| 21:14:35 | mgagne | +1 | |
| 21:16:38 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add support for multiple fixed-ips in metadata https://review.openstack.org/580742 | |
| 21:16:43 | artom | mriedem, yeah, am looking into it | |
| 21:16:46 | artom | Really weird | |
| 21:18:24 | mriedem | dansmith: so does that mean we're good with the move patch for my +2 since we have the functional WIP patch for the move ops? | |
| 21:18:27 | artom | mriedem, no chance we can merge and later revert really excessive logging? | |
| 21:18:35 | mriedem | like, are you ok approving the stack and cleaning up the functional patch test later or? | |
| 21:18:48 | artom | Back in a bit, picking up daughter from daycare | |
| 21:18:52 | mriedem | i also have to do ^ | |
| 21:19:20 | dansmith | mriedem_afk: oh I should have removed the WIP on that as I think it's okay | |
| 21:19:35 | dansmith | mriedem_afk: but yeah, I think we need to merge that stack if you're good with it | |
| 21:19:42 | dansmith | that's what I meant when I asked about your +2 | |
| 21:21:18 | mriedem_afk | want me to +W the bottom change too? | |
| 21:22:00 | mriedem_afk | did it anyway | |
| 21:22:04 | mriedem_afk | gotta run | |
| 21:47:08 | openstackgerrit | Dan Smith proposed openstack/nova master: Add additional functional tests for NUMA networks https://review.openstack.org/585385 | |
| 22:12:29 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add support for multiple fixed-ips in metadata https://review.openstack.org/580742 | |
| 22:13:40 | mgagne | mriedem_afk: sorry, I was a bit too hasty in updating the change. Let me know about the expected services field behavior. I will update accordingly. | |
| 22:18:10 | melwitt | mgagne: well, tbh, I'm not sure what it should be, just pointing out that my prior understanding of the change was that it just adds the ip_addresses field and nothing else changes | |
| 22:19:24 | melwitt | it might be fine to include more services in 'services' too. I'll add a comment and ask mriedem_afk and dansmith about it tomorrow to see what they think | |
| 22:25:42 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | |
| 23:13:50 | mgagne | melwitt: thanks for the review, much appreciated. sorry if I didn't make the full extend of the change clear in the initial spec :-/ | |
| 23:16:19 | melwitt | mgagne: that's okay, I'm guessing it's obvious to those more familiar than I. thanks for the prompt replies | |
| 23:16:50 | mgagne | I shouldn't assume people are familiar with that part ;) | |
| 23:18:13 | melwitt | I learned new things, that's a plus :) | |
| 23:18:55 | openstackgerrit | Eric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | |
| 23:18:56 | openstackgerrit | Eric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033 | |
| 23:18:57 | openstackgerrit | Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider sane https://review.openstack.org/584598 | |
| 23:18:58 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599 | |
| 23:18:59 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648 | |
| 23:19:00 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034 | |
| 23:19:01 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049 | |
| 23:19:02 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| #openstack-nova - 2018-07-26 | |||
| 00:19:46 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Making consistent used of GiB and MiB in API ref https://review.openstack.org/559985 | |
| 00:20:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | |
| 01:01:20 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Extract base functional test case from test_direct https://review.openstack.org/585777 | |
| 01:01:21 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Use base test in placement functional tests https://review.openstack.org/585778 | |
| 01:11:34 | openstackgerrit | Merged openstack/nova master: Remove support for /os-floating-ips-bulk REST API https://review.openstack.org/582912 | |
| 01:14:04 | openstackgerrit | Merged openstack/nova master: Remove support for /os-floating-ip-dns REST API https://review.openstack.org/582943 | |
| 01:14:11 | openstackgerrit | Merged openstack/nova master: Remove "DEPRECATED" tag from Obsolete APIs https://review.openstack.org/583092 | |
| 01:14:17 | openstackgerrit | Merged openstack/nova stable/pike: cleanup mapping/reqspec after archive instance https://review.openstack.org/580775 | |
| 01:14:23 | openstackgerrit | Merged openstack/nova master: doc: remove rocky-specific nova-scheduler min placement version https://review.openstack.org/585520 | |
| 02:06:16 | naichuans | Txt files which store openstack IRC history reported as trojan and deleted by McAFee, anyone meet such kind of issues before? | |
| 02:16:14 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: DNM? Extra logs for volume detach device tags cleanup https://review.openstack.org/584032 | |
| 02:18:27 | openstackgerrit | Xiaohan Zhang proposed openstack/nova master: compute node local_gb_used include swap disks The ComputeNode.local_gb_used value is set in the ResourceTracker._update_usage() method: https://review.openstack.org/585928 | |
| 06:22:34 | openstackgerrit | huanhongda proposed openstack/nova master: hypervisor-stats shows wrong disk usages with shared storage https://review.openstack.org/149878 | |