| *** gthiemon1e is now known as gthiemonge | 06:07 | |
| sean-k-mooney | o/ Quick question. im removign the AZ filter from nova. im currently working on a devstack patch to stop it trying to use it (apparently i forgot to do that a few releases ago when it was nolonger enabled by default in nova) | 09:54 |
|---|---|---|
| sean-k-mooney | to make grenade work we have 3 options | 09:55 |
| sean-k-mooney | backport the default change | 09:55 |
| sean-k-mooney | update the grenade job to override the filters | 09:55 |
| sean-k-mooney | or modify grenade itself | 09:55 |
| sean-k-mooney | i would prefer to just backport the devstack change to zed but the other approches will also work any prefernce? | 09:56 |
| opendevreview | sean mooney proposed openstack/devstack master: The AZ filter is deprecated and planned for removal this cycle https://review.opendev.org/c/openstack/devstack/+/886972 | 10:00 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Remove unused file https://review.opendev.org/c/openstack/devstack/+/886975 | 10:24 |
| opendevreview | Artem Goncharov proposed openstack/devstack master: [WIP] Enable keystone token caching by OSC https://review.opendev.org/c/openstack/devstack/+/886339 | 11:50 |
| auniyal | hello, | 12:50 |
| auniyal | I am trying to setup devstack with os-vif | 12:50 |
| auniyal | this is the config which I am using - https://3a986fcbf1c7c0832f8d-9104b4cd302cb33eeeae5005d497f5db.ssl.cf2.rackcdn.com/886778/1/check/os-vif-ovs/9885325/controller/logs/local_conf.txt | 12:50 |
| auniyal | I updated the HOST_IP only. | 12:50 |
| auniyal | with this I have to clone the openstack services manually. | 12:50 |
| auniyal | still failing with ./stack.sh failed with this logs: https://paste.opendev.org/show/bXTwP218f4DfQVumCTDb/ | 12:50 |
| auniyal | so its failing on is_service_enabled q-agt neutron-agent, how can I disable q-agt, I could not find it from list-units | 12:52 |
| auniyal | gibi, sean-k-mooney ^^ | 13:38 |
| sean-k-mooney | you are using the ci cofnig as a base | 13:39 |
| sean-k-mooney | normally you would not do that | 13:39 |
| sean-k-mooney | you would jsut add LIBS_FROM_GIT=os-vif | 13:40 |
| sean-k-mooney | and 2 other parmaters to your normla lcoal.conf for the repo and branch | 13:40 |
| sean-k-mooney | OS_VIF_REPO and OS_VIF_BRANCH | 13:40 |
| sean-k-mooney | OS_VIF_REPO=https://review.opendev.org/openstack/os-vif | 13:41 |
| sean-k-mooney | OS_VIF_BRANCH=refs/changes/78/886778/3 | 13:42 |
| sean-k-mooney | you can get hte branch form the download patch optioni n the gerrit ui | 13:43 |
| sean-k-mooney | so the pull example is "git pull https://review.opendev.org/openstack/os-vif refs/changes/78/886778/3" for https://review.opendev.org/c/openstack/os-vif/+/886778 | 13:43 |
| sean-k-mooney | the qagent is disbale dby default in devstack | 13:44 |
| sean-k-mooney | on master in older banches its enabled by default | 13:44 |
| sean-k-mooney | if you want to use ml2 ovs then you need to actully cofnigure the QAGENT devstack macro | 13:46 |
| sean-k-mooney | Q_AGENT=openvswitch | 13:47 |
| sean-k-mooney | with that said you dont need to use ml2/ovs to test htis backprot ovn is fine so i would just test with devstack defaults and only set the minium config you need | 13:47 |
| sean-k-mooney | LIBS_FROM_GIT=os-vif OS_VIF_REPO=https://review.opendev.org/openstack/os-vif and OS_VIF_BRANCH=refs/changes/78/886778/3 | 13:48 |
| auniyal | I wanted to have a new devstack system where I can test os-vif changes | 13:50 |
| auniyal | so in existing one which I using I can just replace | 13:50 |
| auniyal | "LIBS_FROM_GIT=cinder,devstack,devstack-gate,glance,keystone,neutron,nova,os-vif,placement,requirements,swift,tempest" | 13:50 |
| auniyal | with | 13:50 |
| auniyal | LIBS_FROM_GIT=os-vif | 13:50 |
| auniyal | and will add below in default one [[local|localrc]] | 13:50 |
| auniyal | OS_VIF_REPO=https://review.opendev.org/openstack/os-vif | 13:50 |
| auniyal | =========== OR ==================== | 13:50 |
| auniyal | [[local|localrc]] | 13:50 |
| auniyal | ADMIN_PASSWORD=secret | 13:50 |
| auniyal | DATABASE_PASSWORD=$ADMIN_PASSWORD | 13:50 |
| auniyal | RABBIT_PASSWORD=$ADMIN_PASSWORD | 13:50 |
| auniyal | SERVICE_PASSWORD=$ADMIN_PASSWORD | 13:50 |
| auniyal | LIBS_FROM_GIT=os-vif | 13:50 |
| auniyal | OS_VIF_REPO=https://review.opendev.org/openstack/os-vif | 13:50 |
| auniyal | Q_AGENT=openvswitch | 13:50 |
| auniyal | once setup I can download change "886778" and then test it | 13:51 |
| auniyal | s/setup/setup is done | 13:51 |
| auniyal | sean-k-mooney, once devsstack setup is done, is there a way I can find out os-vif is in place (to confirm, I got what I wanted) | 13:56 |
| sean-k-mooney | if its in libs_from_git it will be cloned to /opt/stack/os-vif by default | 13:58 |
| sean-k-mooney | if its not it will be pip installed form pypi | 13:58 |
| auniyal | ack, so as I am going with LIBS_FROM_GIT=os-vif, there should be a repo os-vif in home dir and I should not see these package on pip freeze - https://github.com/openstack/os-vif/blob/master/requirements.txt | 14:02 |
| auniyal | thanks sean-k-mooney | 14:05 |
| opendevreview | melanie witt proposed openstack/tempest master: Disable immediate discard of unused blocks in make_fs https://review.opendev.org/c/openstack/tempest/+/886991 | 16:45 |
| gmann | sean-k-mooney: RE on AZ filter. yes we can go with backport the default change in devstack as it was deprecated. grenade on stable/2023.1 will keep testing it and from master onwards not | 17:52 |
| gmann | dansmith: whoami-rajat: do you know about this test_rebuild_volume_backed_server failure on ceph job, it seems consistent https://b932a1446345e101b3ef-4740624f0848c8c3257f704064a4516f.ssl.cf5.rackcdn.com/883557/2/check/nova-ceph-multistore/d7db64f/testr_results.html | 17:59 |
| dansmith | password isn't getting changed in the rebuild maybe/ | 18:00 |
| sean-k-mooney | that would only happen if you had the qemu guest agent and i dont think thats in the cirros image | 18:01 |
| sean-k-mooney | unless your trying to use the file injection way of doing that which has been deprecated for years | 18:01 |
| gmann | it rebuild on cirros 0.6.1, 0.6.2. I do not think it use file injection | 18:02 |
| sean-k-mooney | so it should be logging in via ssh key or the default cirros password | 18:04 |
| gmann | this is new test merged on Thursday and I doubt it ever passed on ceph job | 18:05 |
| gmann | yeah | 18:05 |
| gmann | with key | 18:05 |
| sean-k-mooney | i would not expect " User: cirros, Password: rebuildPassw0rd" | 18:05 |
| sean-k-mooney | to work in general. setting the instance password is genrelaly nto somethign you can assume will work form an interop persective | 18:06 |
| sean-k-mooney | it depend on the host config, the guest image and the vritdriver to work. so it sound liek this is either a bug in the test or a bug in the job config | 18:07 |
| sean-k-mooney | depending on where rebuildPassw0rd is comming form | 18:08 |
| dansmith | well, in that case, it is probably not actually rebuilding the instance or something | 18:08 |
| gmann | that is adminPass passed during rebuild request | 18:08 |
| dansmith | inject_partition = -2 | 18:08 |
| sean-k-mooney | yep and without file injection that is not soemthing you cna rely on being set | 18:09 |
| dansmith | inject_password is unset, so defaults to false | 18:09 |
| sean-k-mooney | it need the qemu guest agent or a user-data script if file inject is not on to set it | 18:09 |
| sean-k-mooney | yep | 18:09 |
| dansmith | okay there used to be a way to do password via metadata | 18:10 |
| sean-k-mooney | and cirror does nto have the guest agent and glen (wont set this automaticlly) | 18:10 |
| sean-k-mooney | there still is | 18:10 |
| dansmith | I remember because vish had to break the password hash into multiple key=value fields | 18:10 |
| sean-k-mooney | cloudbase-init supprots it on windows | 18:10 |
| sean-k-mooney | i dont think cloud-init or glean do it by defualt | 18:11 |
| gmann | yeah no inject password set. let me modify test to ssh via key. | 18:11 |
| gmann | i do not think we need to test the password injection and ssh via that in this test. if I am not wrong whoami-rajat ? | 18:11 |
| sean-k-mooney | well it should be in multiple tests since its testing multipe fucntionality if we were to test it right? | 18:12 |
| sean-k-mooney | but that not really the main point of the test | 18:12 |
| dansmith | gmann: we definitely need ssh for that test | 18:17 |
| gmann | dansmith: I mean ssh via key we can do instead of password right? | 18:18 |
| dansmith | gmann: and I thought that the password change was part of the test, but it's been like a year, so maybe I'm misremembering | 18:18 |
| gmann | ohk | 18:18 |
| dansmith | gmann: we should ask whoami-rajat or I can go look at the review comments.. I remember specifically calling that out, but I don't remember which way it was | 18:18 |
| gmann | that was my question if password change was testing part | 18:18 |
| gmann | ohk | 18:18 |
| gmann | and it fail on ceph job only and everywhere it pass | 18:19 |
| dansmith | right which is why it seems like it might be not properly rebuilding the image and thus not using the new password | 18:20 |
| dansmith | because otherwise the key would still work, I'd think | 18:20 |
| gmann | yeah, let me skip the test for now to unblock gate and we can wait for whoami-rajat in his morning | 18:21 |
| dansmith | is inject_password enabled on the jobs that pass? | 18:21 |
| dansmith | maybe that's the actual difference? | 18:22 |
| gmann | let me check | 18:22 |
| gmann | no, it is disabled there too. tempest-full-py3 job for example | 18:24 |
| dansmith | okay | 18:24 |
| gmann | https://zuul.opendev.org/t/openstack/build/eef23a7922404bc1931cd098ffa5d982/log/controller/logs/etc/nova/nova_conf.txt | 18:24 |
| opendevreview | Ghanshyam proposed openstack/devstack-plugin-ceph master: Skip test_rebuild_volume_backed_server test for ceph job https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887003 | 18:35 |
| opendevreview | Ghanshyam proposed openstack/devstack-plugin-ceph master: Skip test_rebuild_volume_backed_server test for ceph job https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887003 | 20:28 |
| opendevreview | Ghanshyam proposed openstack/devstack-plugin-ceph master: Skip test_rebuild_volume_backed_server test for ceph job https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887003 | 22:43 |
| gmann | dansmith: gouthamr ^^ can you check this test skip to unblock ceph job | 22:44 |
| gouthamr | ack gmann | 22:46 |
| dansmith | got it | 23:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!