| *** mhen_ is now known as mhen | 02:04 | |
| opendevreview | Yosi Ben Shimon proposed openstack/tempest master: Add new test case for volume retype https://review.opendev.org/c/openstack/tempest/+/956225 | 06:21 |
|---|---|---|
| opendevreview | Yosi Ben Shimon proposed openstack/tempest master: Add new test case for volume retype https://review.opendev.org/c/openstack/tempest/+/956225 | 06:23 |
| opendevreview | Maxim Sava proposed openstack/tempest master: Add test for parallel image upload https://review.opendev.org/c/openstack/tempest/+/945320 | 08:51 |
| stephenfin | frickler: gmaan: When you're about, I would appreciate your thoughts on https://review.opendev.org/c/openstack/devstack/+/959213/ (as an alternative to https://review.opendev.org/c/openstack/devstack/+/936963) | 09:54 |
| frickler | stephenfin: I like the idea, but it looks like some plugin use case is still failing, see https://zuul.opendev.org/t/openstack/build/4c69d65028c64d3696ec9380561f3f23 | 10:38 |
| stephenfin | hmm, I'd missed that | 10:39 |
| stephenfin | now to figure out how that job is working | 10:40 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/devstack-plugin-prometheus master: Adds openstack-exporter deployment script https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/959063 | 10:40 |
| opendevreview | Maxim Sava proposed openstack/tempest master: Add test for parallel image upload https://review.opendev.org/c/openstack/tempest/+/945320 | 10:50 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/devstack-plugin-prometheus master: Adds openstack-exporter deployment script https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/959063 | 11:09 |
| sean-k-mooney | long term i would love ot use uv to manage the creation of the global virtual env and the provisioning of the python interperter, at least as configurable option | 11:23 |
| sean-k-mooney | that woudl effectivly allow use to use any python verison on any distro | 11:23 |
| sean-k-mooney | i have not looked at stephen pathc yet but i assuem tox is being repalced by natvily using python3 -m venv? | 11:24 |
| sean-k-mooney | ok more or less its using VIRTUALENV_CMD to create the venv | 11:25 |
| sean-k-mooney | then manually doing the install using constratigs ectra | 11:25 |
| sean-k-mooney | i belive we have a wrapper that can do pip isntall with uc but i guess that does nto supprot specifying which venv to install into so it woudl insall in teh global one rather then the tempest one | 11:27 |
| sean-k-mooney | which woudl make sense why we are not actully using that today | 11:27 |
| sean-k-mooney | so ya i generally like the approch and i can see how we could evolve it in the future | 11:28 |
| sean-k-mooney | but also yes we do need to make sure we can correctly install the tempest plugins into this venv | 11:29 |
| sean-k-mooney | its odd why that is not workign given https://review.opendev.org/c/openstack/devstack/+/959213/2/lib/tempest#852 | 11:29 |
| sean-k-mooney | i guess the problem is that we use tox env to actully specify the regex in the jobs in many cases too | 11:34 |
| sean-k-mooney | i.e. we dont just use tox to create teh venv we actully encode the regex in the tox config as well in many cases | 11:35 |
| sean-k-mooney | that how we do the two stage run for example | 11:36 |
| sean-k-mooney | i.e. concurrent api tests followed by serial senario test then we combind both stestr runs to have one subunti db | 11:37 |
| sean-k-mooney | stephenfin: https://github.com/openstack/tempest/blob/master/roles/run-tempest/tasks/main.yaml#L122-L127 that how we actully run tempst in the jobs | 11:38 |
| sean-k-mooney | so for example the tempest integrated comptue jobs is using this tox env https://github.com/openstack/tempest/blob/master/tox.ini#L189-L201 | 11:39 |
| sean-k-mooney | some of the target like all and full allow passign regexs in via the commandline which we expose as job paramter in the zull job as vars | 11:40 |
| sean-k-mooney | but without rewiting that you wont be able to externalise the creation of the vnev unless you can tell tox to just use the one you pre created | 11:41 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/devstack-plugin-prometheus master: Adds openstack-exporter deployment script https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/959063 | 12:05 |
| fungi | frickler: correct, it didn't dawn on me at the time that we might be running propose jobs on platforms with a git version old enought to lack the --trailer option (also i incorrectly assumed that option had been implemented for longer | 12:21 |
| opendevreview | Saikumar Pulluri proposed openstack/devstack master: Add service type to keystone authtoken middleware https://review.opendev.org/c/openstack/devstack/+/959436 | 12:51 |
| opendevreview | Saikumar Pulluri proposed openstack/devstack master: Add service type to keystone authtoken middleware https://review.opendev.org/c/openstack/devstack/+/959436 | 13:56 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/devstack-plugin-prometheus master: Adds openstack-exporter deployment script https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/959063 | 14:11 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/devstack-plugin-prometheus master: Adds openstack-exporter deployment script https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/959063 | 14:58 |
| opendevreview | Eric Harney proposed openstack/devstack-plugin-nfs master: DNM: Test removing tempest_test_regex https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/959544 | 15:14 |
| opendevreview | Saikumar Pulluri proposed openstack/devstack master: Add service type to keystone authtoken middleware https://review.opendev.org/c/openstack/devstack/+/959436 | 16:47 |
| whoami-rajat | gmaan, hey, are you around? | 20:13 |
| whoami-rajat | so our changes are working fine, there is just one problem, the grenade jobs fail on the GET location API implementation | 20:14 |
| whoami-rajat | my theory is that grenade creates a N-1 devstack deployment (that doesn't contain the glance section in cinder.conf), upgrades it, and now requests the get_location call again | 20:16 |
| whoami-rajat | ideally it should work, the GET location code should be backward compatible, which i can make with handling the not authroized exception | 20:17 |
| whoami-rajat | but i was also wondering if we want to merge and backport the devstack patch, that is another fix for the problem | 20:17 |
| whoami-rajat | anyways, i will go ahead with the cinder change but wanted to know your thoughts on the same | 20:17 |
| whoami-rajat | but I'm also skeptical that the new section we introduce might be a breaking change for upgrades | 20:19 |
| whoami-rajat | in any case we should have a releasenote here https://review.opendev.org/c/openstack/cinder/+/958716 stating the requirement of a new [glance] section | 20:20 |
| whoami-rajat | gmaan, i took the liberty to add the upgrade section myself to make the review process faster https://review.opendev.org/c/openstack/cinder/+/958716/6/releasenotes/notes/add-glance-service-section-3e73daee0e995442.yaml | 20:58 |
| opendevreview | Rajat Dhasmana proposed openstack/devstack master: Configure glance user in cinder conf https://review.opendev.org/c/openstack/devstack/+/958718 | 21:03 |
| whoami-rajat | also updated your devstack patch failing on pep8 ^ | 21:04 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!