| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-30 | |||
| 11:48:08 | sean-k-mooney | tobias-urdin: this could be a blueprint/spec | |
| 11:48:14 | sean-k-mooney | but its a new feature not a bug | |
| 11:48:41 | tobias-urdin | sean-k-mooney: ok, thanks :) | |
| 11:48:57 | sean-k-mooney | tobias-urdin: its one i would not mind seeing added by the way. i think it would be useful | |
| 11:49:27 | tobias-urdin | sean-k-mooney: i agree | |
| 11:55:32 | noonedeadpunk | sean-k-mooney: looking at code, I have one question - should I need another aggregate which would include the rest of the hosts? | |
| 11:56:00 | sean-k-mooney | noonedeadpunk: off the top of my head i dont think so | |
| 11:56:14 | noonedeadpunk | ok, just to double check, as I don't have one | |
| 11:56:29 | noonedeadpunk | I mean I have the only aggregate with required trait | |
| 11:56:52 | sean-k-mooney | all the rest will be in the implcit nova aggreate | |
| 11:59:20 | noonedeadpunk | was just thinking how this works. https://github.com/openstack/nova/blob/stable/train/nova/scheduler/request_filter.py#L74-L76 Like we get list of aggregates without the trait? | |
| 12:02:15 | sean-k-mooney | yes so we first get all the required traits from the instance | |
| 12:03:06 | sean-k-mooney | then we all the aggreates that have require traits that are not in the set we require for the instnace | |
| 12:03:20 | sean-k-mooney | we then add those as forbiden aggreates to the placment query | |
| 12:03:42 | noonedeadpunk | ah, ok, yeah | |
| 12:03:45 | sean-k-mooney | so since you have no required traits in the flaovr or image for that instance | |
| 12:03:55 | sean-k-mooney | it will get back all aggreates with a require trait | |
| 12:04:06 | sean-k-mooney | in this case the one aggreate you created | |
| 12:04:16 | sean-k-mooney | and then add those to the forbidden list | |
| 12:04:21 | noonedeadpunk | ok, thanks so much for explanation | |
| 12:04:42 | sean-k-mooney | that becmoes a "mem_of=!<aggreate uuid>" paramter to placemnt | |
| 12:05:08 | sean-k-mooney | ! being not | |
| 12:05:09 | openstack | sean-k-mooney: Error: "being" is not a valid command. | |
| 12:05:25 | noonedeadpunk | lol | |
| 12:05:26 | sean-k-mooney | hehe | |
| 12:05:49 | sean-k-mooney | sorry that hsoul be member_of | |
| 12:06:07 | sean-k-mooney | anyway you get the point we prefix the uuid with ! whenever we want to exclude it | |
| 12:07:36 | noonedeadpunk | yes, sure, thanks again) | |
| 13:33:16 | dansmith | lyarwood: I may have missed something about the live migration ceph job.. this is a grenade job, but this is the one I care most about: https://review.opendev.org/#/c/742056/6/.zuul.yaml@90 | |
| 13:33:31 | dansmith | which looks like it has already been removed and we're not running, right? | |
| 13:36:48 | lyarwood | dansmith: it's still there just hidden in the original nova-live-migration job | |
| 13:36:57 | lyarwood | dansmith: https://github.com/openstack/nova/blob/2beb18444fccac2ffe73292423ff801a9f327f49/.zuul.yaml#L59-L70 | |
| 13:37:01 | lyarwood | dansmith: https://github.com/openstack/nova/blob/2beb18444fccac2ffe73292423ff801a9f327f49/playbooks/legacy/nova-live-migration/run.yaml#L51-L54 | |
| 13:37:26 | lyarwood | dansmith: https://github.com/openstack/nova/blob/2beb18444fccac2ffe73292423ff801a9f327f49/gate/live_migration/hooks/run_tests.sh#L55-L65 | |
| 13:37:50 | dansmith | lyarwood: confusing, but okay.. so *that* is the thing I think is important not to lose, but grenade I guess I'm less worried about | |
| 13:38:54 | lyarwood | kk ack, I've -W'd my switch over change until I've covered that in a ceph specific job | |
| 13:38:55 | dansmith | I'm still worried about losing pressure to close the loop and these gaps sticking around for a while, but if the basic multinode live migration switcheroo bit is still there, then I don't want to block smaller process hacks like the grenade bit if that makes it easier | |
| 13:39:39 | lyarwood | np this shouldn't take long to work out and backport to stable/victoria so we can close the zuulv3 stuff out there tbh | |
| 13:39:53 | lyarwood | I've worked out how to copy the config and keys around already | |
| 13:40:08 | lyarwood | in here https://review.opendev.org/#/c/755270/ | |
| 13:40:22 | lyarwood | but I think the role needs to land in devstack itself | |
| 13:40:58 | dansmith | ah, wait wait, | |
| 13:41:12 | dansmith | gmann's patch *does* remove that chunk: https://review.opendev.org/#/c/752557/8/.zuul.yaml@a68 | |
| 13:41:22 | dansmith | you were saying that job isn't already gone, | |
| 13:41:34 | dansmith | but his patch *does* reduce that coverage... | |
| 13:41:45 | lyarwood | that's my change | |
| 13:41:48 | lyarwood | and is -W'd | |
| 13:42:15 | lyarwood | until we get the coverage sorted | |
| 13:42:17 | dansmith | okay, I guess I'm not sure which that is, | |
| 13:42:18 | lyarwood | I'll add a comment | |
| 13:43:53 | dansmith | ah, this guy: https://review.opendev.org/#/c/752557/ | |
| 13:44:25 | lyarwood | that's the same change as above | |
| 13:44:40 | dansmith | lol | |
| 13:45:25 | lyarwood | https://review.opendev.org/#/c/742056/ is the grenade change | |
| 13:45:26 | dansmith | I know, I just didn't realize that was yours, thought it was gmann's because he had the -W until a minute ago | |
| 13:45:38 | lyarwood | ah np | |
| 13:45:46 | dansmith | yeah, I got it now | |
| 13:46:12 | dansmith | btw, I'm super happy to see you guys also getting spanked by zuul on syntax stuff | |
| 13:46:33 | dansmith | fixing up glance's jobs, zuul was getting sick of me throwing up patches that didn't even parse :) | |
| 13:46:55 | lyarwood | haha yeah, all the damn time | |
| 13:47:05 | lyarwood | wish there was a way of running some of this stuff locally | |
| 13:47:08 | dansmith | there needs to be a separately-installable zuulint tool or something we can throw into a tox env | |
| 13:47:11 | dansmith | yeah | |
| 14:02:09 | sean-k-mooney | lyarwood: there is a docker compose file for running zull quickly | |
| 14:02:18 | sean-k-mooney | but ya gettign the gate jobs to work is non trivial | |
| 14:03:16 | sean-k-mooney | the devstack/tempest based jobs pull in a lot of repos so you need to donload a lot of repos to get them to work unless you take a few steps that mean your nolonger really mirrioring what happens in the gate | |
| 14:04:00 | sean-k-mooney | dansmith: but ya a zuul-lint tool would be nice | |
| 14:47:36 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557 | |
| 14:47:36 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Merge nova-evacuate into nova-multinode-live-migration https://review.opendev.org/755309 | |
| 17:46:32 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: [stable-only] nova-live-migration: Disable libvirtd during negative tests https://review.opendev.org/754688 | |
| 17:49:07 | openstackgerrit | melanie witt proposed openstack/nova master: Add note about setting CONF.max_disk_devices_to_attach = 0 https://review.opendev.org/755360 | |
| 17:56:27 | openstackgerrit | melanie witt proposed openstack/nova master: Add note about setting CONF.max_disk_devices_to_attach = 0 https://review.opendev.org/755360 | |
| 20:11:08 | ganso | Hi Nova folks! I know most people are very busy with bugs for the RC1, but just in case you have 1 minute to spare, here is a backport to stable/ussuri that is waiting for +2s, if you could please take a look at. Thanks in advance! https://review.opendev.org/#/c/752245/ | |
| 20:31:32 | sean-k-mooney | lyarwood: elod ^ you might be able to help ganso when ye are around | |
| 21:58:14 | ganso | sean-k-mooney: thanks! | |
| #openstack-nova - 2020-10-01 | |||
| 08:05:21 | hemanth_n | Hi folks, is this time the unofficial office hours slot , if so want to discuss on the following review https://review.opendev.org/#/c/749175/ | |
| 08:11:16 | lyarwood | hemanth_n: I don't think we have any office hours at the moment but looking at the change gibi / sean-k-mooney1 / stephenfin might be able to help once they are online | |
| 08:11:42 | lyarwood | In other news, stable/victoria is blocked until we switch the nova-live-migration job out :/ | |
| 08:11:45 | lyarwood | ++ /opt/stack/new/devstack-gate/devstack-vm-gate.sh:setup_localrc:L316: /usr/bin/python3 roles/test-matrix/library/test_matrix.py -n -b stable/victoria -f roles/test-matrix/files/features.yaml -r primary | |
| 08:11:47 | lyarwood | 2020-09-30 21:04:43,670 ERROR: branch not allowed by features matrix: victoria | |
| 08:12:32 | hemanth_n | thanks lyardwood | |
| 08:13:02 | hemanth_n | lyarwood* thank you | |
| 08:13:27 | lyarwood | np | |
| 08:13:52 | gibi | hemanth_n: that patch is on my review list for long | |
| 08:14:57 | gibi | hemanth_n: I see that sean-k-mooney1 is +1 on it already so I will try to read it today | |
| 08:15:08 | gibi | hemanth_n: is there any special aspect I should be aware of? | |
| 08:15:53 | hemanth_n | gibi: cool thanks .. nope except its a specific scenario which is mentioned in the commit message and bug description | |
| 08:17:07 | gibi | hemanth_n: I saw that you managed to add a functional test for it. thanks that helps a lot | |
| 08:17:32 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: [stable-only] nova-live-migration: Disable libvirtd during negative tests https://review.opendev.org/754688 | |
| 08:25:09 | frickler | lyarwood: the right solution would be to move that job away from using ds-gate, I'm not sure if someone has started doing that yet. even if we keep ds-gate working for victoria, that should be considered unstable and very much deprecated | |
| 08:25:56 | lyarwood | frickler: yeah I'm working on that in master, just need sort out a multinode ceph job and we should be good to backport to stable/victoria | |
| 08:26:09 | lyarwood | frickler: and agreed, this is just for a few days hopefully | |
| 08:27:26 | lyarwood | gibi: are you aware of https://zuul.opendev.org/t/openstack/builds?job_name=nova-next - looks like gate/post_test_hook.sh is failing when it calls `openstack port unset --binding-profile allocation port-normal-qos` | |
| 08:27:41 | lyarwood | on master at least | |
| 08:28:02 | gibi | lyarwood: thanks for the ping | |
| 08:28:10 | gibi | I haven't seen it yet | |
| 08:29:50 | lyarwood | gibi: np, can't see anything obvious in nova master that could be causing it, I assume something in Neutron has changed? | |
| 08:32:22 | gibi | lyarwood: I will ping lajoskatona on Neutron side, he did some recent changes around qos | |
| 08:33:28 | lyarwood | awesome thanks | |
| 08:35:12 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: [stable-only] nova-live-migration: Disable libvirtd during negative tests https://review.opendev.org/754688 | |