| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-09 | |||
| 12:16:45 | sean-k-mooney | its just a cache | |
| 12:16:49 | sean-k-mooney | IMO | |
| 12:17:21 | lyarwood | Partly, there's also some additional stuff we stash in there on the Nova side at the moment | |
| 12:17:36 | lyarwood | like the device path, multipath UUIDs etc | |
| 12:17:51 | lyarwood | I still think it's entirely valid in the API | |
| 12:18:12 | lyarwood | there's nothing a caller could do with it anyway outside of calling os-brick with it | |
| 12:18:40 | sean-k-mooney | well you could use it to connect to the backend | |
| 12:18:56 | lyarwood | right I mean with our APIs | |
| 12:19:09 | lyarwood | so the fact it's going away in the future shouldn't matter | |
| 12:19:17 | lyarwood | and there's always another way of getting it | |
| 12:20:07 | sean-k-mooney | i dont know to me this just feels like adding tech debt | |
| 12:20:25 | sean-k-mooney | the admin should not really need to know or care about this | |
| 12:20:48 | sean-k-mooney | i get that its for troble shooting | |
| 12:21:16 | sean-k-mooney | but wont the nova manage command provide a better way to do that | |
| 12:21:23 | sean-k-mooney | as that will also supprot refershing it | |
| 12:21:32 | sean-k-mooney | so fixing the problem if it exitis | |
| 12:24:08 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Unify 'nova.db.api', 'nova.db.sqlalchemy.api' https://review.opendev.org/c/openstack/nova/+/799524 | |
| 12:24:09 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Move remaining 'nova.db.sqlalchemy' modules https://review.opendev.org/c/openstack/nova/+/799525 | |
| 12:24:09 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Post reshuffle cleanup https://review.opendev.org/c/openstack/nova/+/799526 | |
| 12:24:10 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Avoid use of ALTER in initial migration https://review.opendev.org/c/openstack/nova/+/800076 | |
| 12:24:10 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Add initial alembic migration for main DB https://review.opendev.org/c/openstack/nova/+/799527 | |
| 12:24:11 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Add initial alembic migration for API DB https://review.opendev.org/c/openstack/nova/+/799528 | |
| 12:24:11 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Trivial style changes https://review.opendev.org/c/openstack/nova/+/799529 | |
| 12:24:12 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Normalize migrations tests https://review.opendev.org/c/openstack/nova/+/799684 | |
| 12:24:13 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Integrate alembic https://review.opendev.org/c/openstack/nova/+/799530 | |
| 12:24:13 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Enable auto-generation of migrations https://review.opendev.org/c/openstack/nova/+/800077 | |
| 12:24:15 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add documentation on database migrations https://review.opendev.org/c/openstack/nova/+/800078 | |
| 12:34:48 | viks__ | hi, i'm facing some issue in network interface naming... after i create an ubuntu instance, i assign a private interface, and then ubuntu instance gets an interface named `ens7`... but when i power off and then start the instance again, the interface name changes to `ens4`. I'm not sure how to avoid this name changing? Can someone plz guide? | |
| 12:39:30 | sean-k-mooney | viks__: that is because of two things one the pci adddreess of the instance likely change and 2 hotplug name are normally not stable in general | |
| 12:39:58 | sean-k-mooney | one way to adress this is with udev rules that mach on the mac to set the name | |
| 12:40:50 | sean-k-mooney | viks__: in generally if you shoudl leverage device role tagging https://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/virt-device-role-tagging.html | |
| 12:41:49 | opendevreview | Merged openstack/nova stable/ussuri: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787253 | |
| 12:44:30 | viks__ | sean-k-mooney: if we tag, the names will persist after instance comes up again? | |
| 12:45:41 | sean-k-mooney | no | |
| 12:45:53 | sean-k-mooney | the name is out of the contol of openstack | |
| 12:46:19 | sean-k-mooney | but if you tag it will provide a way for you to lookup the device and not depend on the name | |
| 12:46:48 | sean-k-mooney | in the metadata api you will get info like this | |
| 12:46:51 | sean-k-mooney | { | |
| 12:46:53 | sean-k-mooney | "type": "nic", | |
| 12:46:55 | sean-k-mooney | "bus": "pci", | |
| 12:46:57 | sean-k-mooney | "address": "0000:00:03.0", | |
| 12:46:59 | sean-k-mooney | "mac": "01:22:22:42:22:21", | |
| 12:47:01 | sean-k-mooney | "tags": ["nfvfunc2"] | |
| 12:47:03 | sean-k-mooney | }, | |
| 12:47:11 | sean-k-mooney | and you can have a script at boot name you interfaces as you wish | |
| 12:47:54 | sean-k-mooney | viks__: the nic names should not change again provide you dont add or remvoe any nics or volumes to the vm | |
| 12:49:58 | viks__ | sean-k-mooney: Thanks a lot.. will try out... | |
| 13:18:05 | sean-k-mooney | lyarwood: +1 on https://review.opendev.org/c/openstack/nova-specs/+/799624 i an alternitive approch inline and just want you to confirm it wont work | |
| 13:18:30 | opendevreview | Elod Illes proposed openstack/nova stable/train: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787316 | |
| 13:54:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 13:54:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 13:54:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 13:54:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port https://review.opendev.org/c/openstack/nova/+/800085 | |
| 14:00:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 14:01:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 14:03:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 14:03:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 14:03:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 14:06:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 15:02:48 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Switch the default video model from 'cirrus' to 'virtio' https://review.opendev.org/c/openstack/nova/+/798680 | |
| 16:17:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 16:17:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 16:17:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 16:17:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 16:19:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 17:09:54 | melwitt | lyarwood: the error in https://bugs.launchpad.net/nova/+bug/1931707 is the same but downstream it's happening during a server show | |
| 18:23:35 | opendevreview | Rodrigo Barbieri proposed openstack/nova stable/stein: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/800114 | |
| 18:53:45 | opendevreview | Rodrigo Barbieri proposed openstack/nova master: Improve prep_resize reschedule unit test https://review.opendev.org/c/openstack/nova/+/800297 | |
| 22:33:33 | opendevreview | Merged openstack/nova stable/train: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787316 | |
| 22:39:48 | opendevreview | melanie witt proposed openstack/nova stable/stein: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/791402 | |
| 22:40:34 | opendevreview | melanie witt proposed openstack/nova master: Make test_archive_task_logs deterministic https://review.opendev.org/c/openstack/nova/+/800313 | |
| #openstack-nova - 2021-07-10 | |||
| 05:44:17 | opendevreview | Merged openstack/nova stable/ussuri: Ignore PCI devices with 32bit domain https://review.opendev.org/c/openstack/nova/+/791771 | |
| 14:38:05 | opendevreview | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for Yoga https://review.opendev.org/c/openstack/nova-specs/+/800324 | |
| #openstack-nova - 2021-07-12 | |||
| 01:08:52 | opendevreview | Shuai Qian proposed openstack/nova master: support evacuating instance in soft_deleted https://review.opendev.org/c/openstack/nova/+/800357 | |
| 06:33:57 | opendevreview | ruishawn proposed openstack/nova master: Add armv6l to arch list https://review.opendev.org/c/openstack/nova/+/799964 | |
| 10:14:44 | sean-k-mooney | stephenfin: can you take a look again at this spec form lyarwood https://review.opendev.org/c/openstack/nova-specs/+/799624 | |
| 10:15:14 | sean-k-mooney | stephenfin: gibi and i are both +2 on it but wanted to give you a chance to take a look before applying +w | |
| 10:15:22 | stephenfin | ack, will do | |
| 10:19:05 | gibi | ack | |
| 10:21:50 | sean-k-mooney | while ye are both here im going to try and poc the claim in the sucuder utils today and update teh pci spec accourdingly. ill proably adress stephens other comment first and then do that update later today or tomorow | |
| 10:22:18 | sean-k-mooney | realistically however im not sure we will have time to agree the direction before thursday so this might just fall to next cycle | |
| 10:26:37 | gibi | sean-k-mooney: ack, I will be available for review | |
| 10:26:44 | gibi | but I agree that this might slip to next | |
| 10:29:21 | sean-k-mooney | trivial yoga spec directory path if people want take a look https://review.opendev.org/c/openstack/nova-specs/+/800324 | |
| 10:30:28 | gibi | sean-k-mooney: done | |
| 10:30:36 | sean-k-mooney | gibi++ | |
| 10:47:32 | opendevreview | Merged openstack/nova-specs master: Create specs directory for Yoga https://review.opendev.org/c/openstack/nova-specs/+/800324 | |
| 11:14:30 | opendevreview | Merged openstack/nova-specs master: Add nova-manage commands to show and refresh connection_info https://review.opendev.org/c/openstack/nova-specs/+/799624 | |
| 12:27:10 | opendevreview | Lee Yarwood proposed openstack/nova-specs master: Store and allow libvirt instance device buses and models to be updated https://review.opendev.org/c/openstack/nova-specs/+/799811 | |
| 12:28:03 | lyarwood | stephenfin sean-k-mooney gibi ; ^ I've submitted to the image-property command given it was 2 vs 1, can you take another swing at that this afternoon if you have time | |
| 12:38:44 | gibi | sure | |
| 12:41:57 | gibi | I'm still +2 :) | |
| 12:49:25 | lyarwood | cool thanks | |
| 13:13:02 | sean-k-mooney | lyarwood: im also +2 on that version ill wait for stephenfin to review | |
| 13:13:44 | sean-k-mooney | the imporant thing for me is if/when we expand the list we dont need to update any docs or add another command other then extending the list | |
| 13:58:17 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Enable auto-generation of migrations https://review.opendev.org/c/openstack/nova/+/800077 | |
| 14:10:20 | sean-k-mooney | ah i was going to ask the docs for that are here https://review.opendev.org/c/openstack/nova/+/800078/ in a followup | |
| 14:10:49 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add documentation on database migrations https://review.opendev.org/c/openstack/nova/+/800078 | |
| 14:11:38 | sean-k-mooney | oh and you are using the venv target for tox ok i was going to ask if add a specific venv to create migrations | |
| 14:11:52 | sean-k-mooney | venv works we use that for reno already | |