Earlier  
Posted Nick Remark
#openstack-nova - 2019-10-04
16:07:18 mriedem after that we verify the instance created on the old side
16:07:19 mriedem 2019-10-03 22:20:00.703 | + /opt/stack/new/grenade/projects/60_nova/resources.sh:verify:175 : nova-manage cell_v2 verify_instance --uuid cb3d18d0-3266-43b8-9288-50151fe3b68b
16:08:55 fungi is there any formal specification for network_data.json? so far i've only found an example at https://docs.openstack.org/nova/latest/user/metadata.html#openstack-format-metadata
16:09:44 mriedem fungi: not formal no, i think there is an abandoned spec somewhere that people reference...
16:10:08 fungi ahh, okay. is it treated as a stable api at least?
16:10:35 fungi came up in working on improving ipv6 support in glean on rh-derived distros
16:11:36 mriedem this is what i could find from the original spec https://specs.openstack.org/openstack/nova-specs/specs/liberty/implemented/metadata-service-network-info.html#rest-api-impact
16:12:24 mriedem and https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/multiple-fixed-ips-network-information.html#rest-api-impact but the multiple fixed ips thing never landed...
16:12:36 mriedem fungi: it's not microversioned,
16:12:40 mriedem it's supposed to be stable,
16:12:53 mriedem but i wouldn't be surprised if things slip through the nested dicts that get in htere
16:13:15 fungi no worries, that's probably good enough, thanks!!!
16:13:31 mriedem i remember this coming up in that latter spec because it says, "The old ip_address and netmask fields found in network_data.json need to be preserved for backward compatible reasons. This means those fields will continue to contain information of the first IP address."
16:13:48 mriedem so it's like old school backward compat pre-microversion where we add keys but don't remove old ones
16:13:53 fungi neat, i'd call that fairly stable
16:14:01 mriedem the metadata api also rarely changes
16:14:07 fungi i figured
16:14:10 dansmith yeah that's what we did for the multiple dns or multple subnets or whatever change we made
16:14:18 fungi thanks again, this is helpful
16:20:48 sean-k-mooney do the peole that maintain cloud-init have a specfication for what whoudl be returned
16:21:14 sean-k-mooney i know initlaly we based our metadata api on the ec2 version but we have diveraged along time ago
16:25:41 sean-k-mooney stephenfin: i added some info on what case will not work with sev currently https://review.opendev.org/#/c/686414/1/releasenotes/notes/bug-1845986-95cbede0a296b088.yaml
16:31:48 sean-k-mooney aspiers: are you working on a new version of the sev fix by the way?
16:50:30 stephenfin efried: Comment left on https://review.opendev.org/#/c/677443/
16:50:32 stephenfin *reply
16:54:05 efried stephenfin: but I mean, did something in this patch expose the need?
16:54:25 stephenfin efried: Yeah, the option directives are indexed
16:54:36 efried or just because this is the first patch hitting docs builds since whatever
16:54:57 efried ah, thx
16:56:26 efried +A
17:01:44 stephenfin ]ta
17:24:58 openstackgerrit Merged openstack/nova master: Add Selection.availability_zone field https://review.opendev.org/685807
17:25:07 openstackgerrit Merged openstack/nova master: FUP to I30916d8d10d70ce25523fa4961007cedbdfe8ad7 https://review.opendev.org/676228
17:25:07 openstackgerrit Balazs Gibizer proposed openstack/nova-specs master: Support move operations with qos ports - Ussuri https://review.opendev.org/686779
17:25:22 gibi efried, mriedem_lunch: ^^
17:26:13 gibi and now I run away to have weekend...
17:50:55 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup https://review.opendev.org/684343
17:50:56 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests https://review.opendev.org/684344
17:50:56 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests https://review.opendev.org/684345
17:50:57 openstackgerrit Stephen Finucane proposed openstack/nova master: fixtures: Beef up NeutronFixture https://review.opendev.org/686795
17:50:57 openstackgerrit Stephen Finucane proposed openstack/nova master: fixtures: Handle iterable params for 'NeutronFixture.list_*' https://review.opendev.org/686796
17:50:58 openstackgerrit Stephen Finucane proposed openstack/nova master: fixtures: Store 'device_id' when creating port in NeutronFixture https://review.opendev.org/686797
17:50:58 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Change name of network provided by NeutronFixture https://review.opendev.org/686798
17:50:59 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Stop mocking the instance network cache https://review.opendev.org/686799
17:50:59 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Make it obvious where we're getting our names from https://review.opendev.org/686800
17:51:00 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Make even more nova-net stuff optional https://review.opendev.org/686801
17:51:00 openstackgerrit Stephen Finucane proposed openstack/nova master: fixtures: Add support for security groups https://review.opendev.org/686802
17:58:23 mriedem ociuhandu: looks like hyperv cloudbase ci is having problems
17:58:29 mriedem http://cloudbase-ci.com/nova/656656/30/devstack-build.log.gz
17:58:40 mriedem fatal: [10.222.0.14]: FAILED! => { "msg": "The conditional check 'devstack_clone_result.rc != 0' failed. The error was: error while evaluating conditional (devstack_clone_result.rc != 0): 'dict object' has no attribute 'rc'
17:59:36 ociuhandu mriedem: thank you, I'll ping the team asap and ask them to check and fix it
17:59:43 mriedem thanks
18:03:54 openstackgerrit Eric Fried proposed openstack/nova-specs master: WIP: Spec: Ussuri: Emulated Virtual TPM https://review.opendev.org/686804
18:11:26 ociuhandu mriedem: I've checked with the team, the errors started from late last evening and by noon today the fix was commited: https://github.com/cloudbase/openstack-cis/commit/6b2f428d22dcd9f99d396ac3c596c7d9618e9796
18:13:02 mriedem ah cool, i must have just been looking at an old result
18:13:03 mriedem thanks again
18:13:55 ociuhandu here's a recent run: https://review.opendev.org/#/c/684334/
18:21:33 ociuhandu mriedem: thanks, it's definitely better to check twice than miss some problems :)
18:22:04 mriedem yp
18:22:06 mriedem *yup
18:22:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'os-security-group-default-rules' REST API https://review.opendev.org/686807
18:22:21 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove unused '*_default_rules' security group DB APIs https://review.opendev.org/686808
18:22:22 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove (most) '/os-networks' REST APIs https://review.opendev.org/686809
18:22:22 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove '/os-tenant-networks' REST API https://review.opendev.org/686810
18:22:23 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'USE_NEUTRON' from functional tests https://review.opendev.org/686811
18:22:23 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'networks' quota https://review.opendev.org/686812
18:22:24 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova-manage network' commands https://review.opendev.org/686813
18:22:24 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove associate, disassociate network APIs https://review.opendev.org/686814
18:22:25 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-dhcpbridge' binary https://review.opendev.org/686815
18:22:25 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-network' binary https://review.opendev.org/686816
18:22:26 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Blast most references to nova-network https://review.opendev.org/686817
18:22:26 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818
18:48:13 mriedem while on the topic if migrating off old volume attachments, https://review.opendev.org/#/c/541420/ has a +2 and gets us to stop creating legacy attachments during boot from volume which will make the eventually data migration less bad
18:48:17 mriedem *of
18:48:27 mriedem the patch is mostly comments and test code
19:11:12 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove Rocky compute compat checks for live migration with port bindings https://review.opendev.org/680005
19:26:21 mriedem efried: aspiers: i take it there isn't much urgency in getting this fixed? https://review.opendev.org/#/c/684825/ - there have been no updates on the patch since tuesday.
19:26:50 mriedem for an RC kind of thing that's a bit disappointing
19:26:54 efried agree
19:27:33 efried that it's disappointing. No idea if there's urgency. Kinda seemed like there should be.
19:28:28 efried last time I heard from aspiers was on the "document this is broke" patch, yesterday morning.
19:32:23 mriedem yeah i saw that
19:38:11 openstackgerrit Merged openstack/nova master: FUP to I4d181b44494f3b0b04537d5798537831c8fdf400 https://review.opendev.org/676231
19:38:19 openstackgerrit Merged openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests https://review.opendev.org/684334
19:55:18 mriedem gibi: you said you were weekending
19:57:35 openstackgerrit Merged openstack/nova master: tests: Correctly mock out security groups in NeutronFixture https://review.opendev.org/685927
20:16:31 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Functional tests for NUMA live migration https://review.opendev.org/672595
20:16:32 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Helper to start computes with different HostInfos https://review.opendev.org/686832
20:17:29 artom I think I've done the hardest - get (re-) started and overcome that initial hurdle
20:17:50 artom And with that - kids pick up time
20:39:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Cache image GETs for multi-create/multi-BDM requests https://review.opendev.org/686835
20:44:47 mriedem hmm, nova.tests.functional.libvirt.test_numa_servers.ReshapeForPCPUsTest.test_vcpu_to_pcpu_reshape might be a problem for the subunit parser bug
20:44:52 mriedem https://zuul.opendev.org/t/openstack/build/4f649747fd9c44f29a105e5100a6232e/log/job-output.txt#2574
20:44:57 mriedem that test dumps a lot of stuff in the output stream
20:45:29 mriedem only 1 fail in logstash though so maybe rare enough to not care http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22INFO%20%5Bnova.virt.libvirt.driver%5D%20Requesting%20provider%20tree%20reshape%20in%20order%20to%20move%20VCPU%20to%20PCPU%20allocations%20to%20the%20compute%20node%20provider%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d
20:51:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Filter out alembic logs below WARNING in tests https://review.opendev.org/686836
21:15:06 dking_desktop My attempts to boot an ironic instance seem to be stuck in BUILD/scheduling/NOSTATE. I'm seeing this message a lot in nova-compute.log: "Unknown column 'instance_extra_1.resources' in 'field list'". Could somebody confirm whether that column should be in the db?
21:17:33 efried dking_desktop: I'm not the expert here, but it sounds like you may need to run some of the nova-manage db migratey things
21:19:11 dking_desktop efried: Do you know where I might find out more about nova-manage? I saw that before, but it's not a command I currently have installed, and so I thought it might have been something older.
21:21:14 efried mriedem and dansmith would probably know your answer off the top, but I can find the docs, sec...

Earlier   Later