| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-16 | |||
| 11:06:54 | sean-k-mooney | ack | |
| 11:07:19 | kashyap | Right; 'libosinfo' was what I was thinking; why isn't it feasible? Do outline your thoughts here; will come back and read | |
| 11:09:15 | sean-k-mooney | we have libosinfo "support" today but its not machine type aware and it does not know if you have the driver avaiable | |
| 11:09:29 | sean-k-mooney | they also can change the hardware modeles over time which has broken us in the past | |
| 11:09:44 | sean-k-mooney | and it currently missues the image propety fields | |
| 11:10:04 | sean-k-mooney | it forches you to put the version in the os name field ignoring the os_version field | |
| 11:10:26 | sean-k-mooney | so have to do os_name rhel8 not os_name rhel os_version 8 | |
| 11:10:41 | sean-k-mooney | so it breaks the standard usage of that attribute | |
| 11:11:16 | sean-k-mooney | the lib osinfo devs assume that the xml will be persisted so its ok for them to break compatiablity and change the resules in later versions | |
| 11:11:24 | sean-k-mooney | that is not how openstack works | |
| 11:12:20 | sean-k-mooney | libosinfo also has no awareness of the libvirt verison installed on the system as far as i am aware so it does not know if the models are supported by the libvirt/qemu installed | |
| 11:12:38 | lyarwood | stephenfin: LGTM | |
| 11:12:43 | sean-k-mooney | this is why i recommend that no one ever use this feature and why i have suggested removing it in the past | |
| 11:14:49 | lyarwood | stephenfin: wait, the `deps =` line in the tox env isn't a mistake is it? | |
| 11:16:35 | sean-k-mooney | lyarwood: i belive that ill install no deps | |
| 11:16:46 | sean-k-mooney | which in this case is fine | |
| 11:16:50 | sean-k-mooney | we just need bash | |
| 11:17:03 | lyarwood | yeah wasn't sure if tox would bork at it tis all | |
| 11:17:14 | sean-k-mooney | well the ci will tell us | |
| 11:18:20 | sean-k-mooney | lyarwood: stephenfin why are we doing this by the way | |
| 11:18:47 | sean-k-mooney | i agree we can do this but i dont expect this to really save much time in the gate | |
| 11:19:13 | lyarwood | sean-k-mooney: it should save at least one check run per backport | |
| 11:19:26 | lyarwood | sean-k-mooney: but really this is more about allowing stable cores to see acceptable backports earlier | |
| 11:19:44 | lyarwood | sean-k-mooney: so instead of them getting -1'd by the cherry pick script they just get held by the gate | |
| 11:19:49 | sean-k-mooney | ya so reading the commit message im not sure i buy that | |
| 11:20:00 | sean-k-mooney | but the 1 recheck i guess makes sense | |
| 11:20:23 | sean-k-mooney | althoguh we are loosing one thing that would be nice to keep | |
| 11:20:40 | sean-k-mooney | which is check that backports to stable acutlly are a backport or have stable only | |
| 11:20:54 | lyarwood | we still get that, it's just in the gate now | |
| 11:21:02 | sean-k-mooney | yep which is too late | |
| 11:21:11 | lyarwood | tbh we could make this non voting in the check queue | |
| 11:21:17 | sean-k-mooney | it should tell author that are unfamilar with our policy in check | |
| 11:21:27 | sean-k-mooney | this new job | |
| 11:21:30 | sean-k-mooney | that would work for me | |
| 11:21:31 | lyarwood | yeah | |
| 11:21:44 | lyarwood | stephenfin: ^ is that okay? | |
| 11:21:48 | sean-k-mooney | it lest you know if it will fail but no need to recheck it | |
| 11:59:15 | admin1 | where is the live_migration_downtime defined ? | |
| 11:59:19 | admin1 | is it on kvm or in nova ? | |
| 12:01:15 | sean-k-mooney | admin1: nova has options that we pass to libvirt which then modulates the qemu downtime | |
| 12:01:37 | sean-k-mooney | admin1: https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.live_migration_downtime | |
| 12:01:56 | sean-k-mooney | admin1: nova is not direcly involved in managing the vm during the livemigration | |
| 12:02:15 | sean-k-mooney | we invoke libvirt wiht a set of parmaters and then wait for ti to complete | |
| 12:04:38 | sean-k-mooney | melwitt: i think this is all you need to fix in the backports for bionic by the way https://review.opendev.org/c/openstack/nova/+/795433/1/roles/run-evacuate-hook/tasks/main.yaml#44 | |
| 12:05:12 | sean-k-mooney | well and lines 67-69 | |
| 12:05:53 | sean-k-mooney | the version of libvirt in bionic does not support socket activation so we just need to remove the socket lines | |
| 12:11:22 | admin1 | sean-k-mooney, so i just need to add those lines in nova.conf, restart the compute service ( systemctl ) and thats it ? | |
| 12:11:51 | admin1 | are those lines only needed in the actual compute nodes, or in the whole cluster ( nova scheduler, api etc ) ? | |
| 12:12:34 | sean-k-mooney | they are used on the source compute i think | |
| 12:12:45 | sean-k-mooney | so all the comptues that you want to tune | |
| 12:12:50 | admin1 | do they go under libvirt or under default ? | |
| 12:13:02 | sean-k-mooney | but i do not think they are used by any other services | |
| 12:13:11 | sean-k-mooney | they are in the libvirt section | |
| 12:13:12 | admin1 | libvirt. ( dot ) means its under libvirt i guess | |
| 12:13:20 | sean-k-mooney | yes | |
| 12:13:23 | sean-k-mooney | it does | |
| 12:13:50 | sean-k-mooney | so update the configs on the computes restart compute services and it will take effect | |
| 12:20:44 | admin1 | does these value make sense: live_migration_downtime: 5000 # ( default 500) && live_migration_downtime_steps: 20 # default (10) && live_migration_completion_timeout: 8000 # (default 800) | |
| 12:21:56 | admin1 | don't want to corrupt the instance , ram etc | |
| 12:28:22 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: libvirt: Set driver_iommu when attaching virtio devices to SEV instance https://review.opendev.org/c/openstack/nova/+/796642 | |
| 13:13:56 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: Reproduce bug 1897528 https://review.opendev.org/c/openstack/nova/+/792116 | |
| 13:13:57 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: Ignore PCI devices with 32bit domain https://review.opendev.org/c/openstack/nova/+/792117 | |
| 13:40:51 | ozzzo | The topic in this channel is a bit unfriendly, and gives bad advice. Nobody is answering questions in #openstack, and this is the correct place to ask Nova questions. Can we change the topic? | |
| 14:08:54 | opendevreview | Lee Yarwood proposed openstack/nova master: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/790519 | |
| 14:21:02 | sean-k-mooney | ozzzo: the topic is intentional | |
| 14:21:27 | ozzzo | i don't doubt that, but it is still bad advice | |
| 14:21:29 | sean-k-mooney | in the past we have had some user be quite unresonable and demand support form upstream for there sepcific thing | |
| 14:21:48 | lyarwood | stephenfin / gibi ; https://review.opendev.org/c/openstack/nova/+/796523 easy review if you have time | |
| 14:22:08 | sean-k-mooney | ozzzo: #openstack is inteded to be for general quetions | |
| 14:22:11 | ozzzo | it's true that there are idiots on irc, but asking questions in #openstack is a waste of time because nobody who knows any answers is watching there | |
| 14:22:17 | sean-k-mooney | but ill admign i dont tend to have it open | |
| 14:22:23 | ozzzo | I try to help out sometimes but I don't know much | |
| 14:22:57 | sean-k-mooney | i do not monitor it i used to have it open by default but i only responded if i was pingged | |
| 14:23:06 | sean-k-mooney | i shoudl set back up my default chanlles list after the mvoe | |
| 14:23:15 | sean-k-mooney | ozzzo: did you have a question by the way | |
| 14:23:52 | sean-k-mooney | ozzzo: we proably could just remove for support bit | |
| 14:23:58 | ozzzo | I don't, but the other day someone was fruitlessly asking questions in #openstack, and I sent him here, but the message scared him off | |
| 14:24:01 | sean-k-mooney | and just not this is for development mainly | |
| 14:24:24 | ozzzo | *nova questions | |
| 14:25:15 | opendevreview | Lee Yarwood proposed openstack/nova master: tests: Allow bindep and test-setup.sh to run successfully on RHEL https://review.opendev.org/c/openstack/nova/+/796428 | |
| 14:25:50 | sean-k-mooney | its hard to express what type of questions are ok here in a way that is aprochable to new commer but also provides a safty net for peopel to fall back on when peopel are lookign for upstream to provide support for free | |
| 14:26:21 | sean-k-mooney | you know that most of use are happy to help, at least to a point | |
| 14:31:55 | opendevreview | Lee Yarwood proposed openstack/nova master: DNM - Test bindep and test-setup.sh changes against centos8 https://review.opendev.org/c/openstack/nova/+/796684 | |
| 14:32:06 | lyarwood | stephenfin: ^ | |
| 14:43:09 | ganso | lyarwood, melwitt: thanks for the +2+W on the wallaby one! if you could please take a look at the victoria backport now when you have a minute: https://review.opendev.org/c/openstack/nova/+/795542 | |
| 14:43:49 | ganso | bauzas also as reviewed the original patch ^ | |
| 14:43:54 | ganso | thanks in advance | |
| 14:44:10 | bauzas | ganso: ack, opening a new tab | |
| 14:52:57 | opendevreview | Stephen Finucane proposed openstack/nova master: Move 'check-cherry-picks' test to gate, n-v check https://review.opendev.org/c/openstack/nova/+/796626 | |
| 14:53:36 | stephenfin | lyarwood: sean-k-mooney: ^ | |
| 14:54:29 | lyarwood | stephenfin: awesome thanks | |
| 14:56:17 | sean-k-mooney | looks good to me | |
| 14:56:41 | sean-k-mooney | we had the env varable beforefore for downstream? | |
| 14:56:56 | sean-k-mooney | or was it there for another reason | |
| 14:57:13 | sean-k-mooney | having it be a different tox enve removes the need for it but just wondering | |
| 14:57:40 | lyarwood | downstream | |
| 14:58:14 | sean-k-mooney | ack, thats what i tought but was not sure | |
| 14:58:19 | lyarwood | but I never got around to replacing the original hackaround with it | |
| 14:58:30 | lyarwood | the original being to truncate the file | |
| 14:58:38 | lyarwood | \o/ | |