Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-25
21:03:53 mriedem # from EC2, which this needs to be compatible with.
21:03:53 mriedem # required for testing purposes. We know this isn't great, but its inherited
21:03:53 mriedem # hidden from the listing, but can still be requested explicitly, which is
21:03:53 mriedem # release bundled into a single version. Note that versions in the future are
21:04:03 mriedem i would -1 on the fomrat
21:04:04 mriedem *format
21:04:24 mriedem # release bundled into a single version." we could lump the changes into the existing ROCKY version
21:04:24 mriedem # correlate with OpenStack release dates, with all the changes for a given
21:04:24 mriedem based on "They traditionally
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
07:12:12 karimull Hello , do the code freeze for the release applies to bugs related to documentation?
07:22:28 openstackgerrit Xiaohan Zhang proposed openstack/nova master: compute node local_gb_used include swap disks https://review.openstack.org/585928
07:31:00 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Increase sleep time in ServerGroupTestV21 to avoid potential failure https://review.openstack.org/585978
07:52:52 openstackgerrit Merged openstack/nova master: Online data migration for queued_for_delete flag https://review.openstack.org/584504
07:53:07 openstackgerrit Merged openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371
07:53:15 openstackgerrit Merged openstack/nova master: Remove stevedore extensions server_create method https://review.openstack.org/583882
07:59:08 kashyap ttx: Morning, do you have mod privileges on this channel? If so, can you kick the above spammers out, please?
08:02:46 lyarwood kashyap: They are being kicked pretty quickly already
08:03:08 kashyap lyarwood: Nod; some channels like Freenode's #kvm are still litterred with them :-(
08:03:31 kashyap lyarwood: But you're right, the Freenode Team is working on it
08:04:01 kashyap lyarwood: Morning, do you have a minute to look at something here?
08:04:09 kashyap lyarwood: It's seemingly embarssing PEP8 failure
08:04:34 lyarwood my forte
08:04:43 kashyap lyarwood: Hehe, so here we go:
08:04:46 kashyap On this line here: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L668
08:04:55 kashyap If I remove that: "# noqa"
08:05:08 kashyap I get this: ./nova/virt/libvirt/guest.py:665:60: F821 undefined name 'unicode'
08:05:17 kashyap How is the comment "solving" it?
08:06:01 kashyap lyarwood: It (that "# noqa") came in via this commit that was trying to fix the PEP8 on Py3 failures: a91af59f
08:06:30 lyarwood http://pep8.readthedocs.io/en/release-1.7.x/intro.html#error-codes - it's not a comment
08:06:31 kashyap lyarwood: Sorry for baiting you in the morning. If you're still making coffee or something, please go on. This can wait.

Earlier   Later