| opendevreview | Oleg proposed openstack/cinder master: PowerMax Driver - Manage volume into correct storage group https://review.opendev.org/c/openstack/cinder/+/797333 | 08:20 |
|---|---|---|
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add QoS support https://review.opendev.org/c/openstack/cinder/+/847730 | 08:38 |
| opendevreview | OpenStack Release Bot proposed openstack/cinderlib stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/cinderlib/+/847728 | 08:52 |
| opendevreview | OpenStack Release Bot proposed openstack/cinderlib master: Update master for stable/yoga https://review.opendev.org/c/openstack/cinderlib/+/847729 | 08:52 |
| *** vkmc_ is now known as vkmc | 09:10 | |
| whoami-rajat | geguileo, fyi, cinderlib yoga released! but we still need to unblock wallaby gate so can you take a look at this backport? https://review.opendev.org/c/openstack/cinderlib/+/847622 | 09:11 |
| geguileo | whoami-rajat: ok | 09:12 |
| geguileo | whoami-rajat: btw, we are going to have problem in RDO :'-( https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43730 | 09:13 |
| whoami-rajat | hmm, doesn't look good, so does changing the shebang from python->python3 fixes it? | 09:15 |
| whoami-rajat | we don't support python2 anymore so not sure why we used python instead of python3 | 09:16 |
| geguileo | whoami-rajat: Looking at /usr/lib/rpm/redhat/brp-mangle-shebangs it seems like it will replace the env call | 09:16 |
| geguileo | python does not necessarily mean python2 | 09:16 |
| geguileo | it's just ambiguous | 09:17 |
| whoami-rajat | python means choose from python2 or python3, whichever is available right? | 09:17 |
| whoami-rajat | available/default | 09:17 |
| geguileo | yup | 09:18 |
| geguileo | but even if you change it to python3 the brp-mangle-shebands will mess it up (though it may not fail) | 09:18 |
| geguileo | shebang=$(echo "$shebang" | sed -r -e 's@^(.+/)env (.+)$@\1\2@') | 09:18 |
| geguileo | according to the comments the script will change /whatsoever/env foo → /whatsoever/foo | 09:18 |
| geguileo | so it won't use the venv python, which is the whole point of that script | 09:19 |
| whoami-rajat | oh, then that doesn't look right ... can we have custom shebang for this cinderlib case? one that doesn't replace the env | 09:20 |
| whoami-rajat | s/replace/remove | 09:20 |
| geguileo | I don't know | 09:21 |
| geguileo | I don't know how to disable the call to that script on the RPM building process | 09:21 |
| whoami-rajat | ok, I don't have much idea about rdo as well | 09:22 |
| geguileo | tosky: you can do the next backport for https://review.opendev.org/c/openstack/cinderlib/+/847622 | 09:30 |
| tosky | geguileo: uh, I thought I did it already | 10:25 |
| tosky | geguileo: I think you have overridden one of the change for testing purpose, let me restore it | 10:26 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: WIP: Add Cinder NFS driver for Dell EMC PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 10:27 |
| opendevreview | Merged openstack/cinder master: Log when waiting to acquire coordinator lock https://review.opendev.org/c/openstack/cinder/+/847216 | 10:38 |
| opendevreview | Merged openstack/cinderlib stable/xena: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847622 | 10:39 |
| opendevreview | Luigi Toscano proposed openstack/cinderlib stable/wallaby: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847623 | 10:39 |
| tosky | geguileo: ^^ yep, it was overwritten, I've restored the initial commit message | 10:40 |
| geguileo | tosky: ouch, sorry | 11:07 |
| HappyStacker1 | hello team! I have a few patches which needs to have workflow triggered | 11:57 |
| HappyStacker1 | https://review.opendev.org/c/openstack/cinder/+/844563 | 11:58 |
| HappyStacker1 | https://review.opendev.org/c/openstack/cinder/+/821749 | 11:59 |
| HappyStacker1 | https://review.opendev.org/c/openstack/cinder/+/816705 | 11:59 |
| HappyStacker1 | can someone trigger them? | 11:59 |
| *** dviroel|out is now known as dviroel | 12:01 | |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: WIP: Add Cinder NFS driver for Dell EMC PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 13:57 |
| opendevreview | Merged openstack/cinderlib stable/wallaby: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847623 | 14:10 |
| opendevreview | Eric Harney proposed openstack/cinder master: Bump moto version to support py3.9 https://review.opendev.org/c/openstack/cinder/+/847798 | 14:39 |
| opendevreview | Eric Harney proposed openstack/cinder master: Move bandit requirements to tox.ini https://review.opendev.org/c/openstack/cinder/+/847801 | 15:03 |
| *** dviroel is now known as dviroel|lunch | 15:27 | |
| *** dviroel|lunch is now known as dviroel | 16:37 | |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 16:54 |
| opendevreview | Walt proposed openstack/cinder master: Add reporting of capacity factors in API https://review.opendev.org/c/openstack/cinder/+/844601 | 17:36 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp NFS: Clone image using copy file operation https://review.opendev.org/c/openstack/cinder/+/847732 | 20:24 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp NFS ONTAP: Deprecate Copy Offload Tool https://review.opendev.org/c/openstack/cinder/+/847733 | 20:26 |
| *** dviroel is now known as dviroel|out | 21:40 | |
| opendevreview | Oleg proposed openstack/cinder stable/yoga: PowerMax Driver - Add a release note for 821739 https://review.opendev.org/c/openstack/cinder/+/847864 | 22:53 |
| opendevreview | Merged openstack/cinder stable/wallaby: NetApp ONTAP: Fix check QoS min support for SVM account https://review.opendev.org/c/openstack/cinder/+/845799 | 22:55 |
| opendevreview | Oleg proposed openstack/cinder stable/yoga: PowerMax Driver - Add a release note for 784603 https://review.opendev.org/c/openstack/cinder/+/847864 | 23:00 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Driver - RDF checks https://review.opendev.org/c/openstack/cinder/+/768105 | 23:15 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Driver - RDF checks https://review.opendev.org/c/openstack/cinder/+/768105 | 23:16 |
| *** hemna0 is now known as hemna | 23:38 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!