| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/846842 | 02:17 |
|---|---|---|
| opendevreview | Friedrich Hiekel proposed openstack/cinder master: add netapp copyoffload provider location https://review.opendev.org/c/openstack/cinder/+/834822 | 06:20 |
| opendevreview | Friedrich Hiekel proposed openstack/cinder master: add netapp copyoffload provider location https://review.opendev.org/c/openstack/cinder/+/834822 | 07:19 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Send the correct location URI to the Glance v2 API https://review.opendev.org/c/openstack/cinder/+/845178 | 07:50 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add the clone_across_pools driver capability https://review.opendev.org/c/openstack/cinder/+/847131 | 07:50 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: drop _attach_volume() and _detach_volume() https://review.opendev.org/c/openstack/cinder/+/787296 | 07:50 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: really detach after copy to/from image. https://review.opendev.org/c/openstack/cinder/+/845990 | 07:50 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool driver: remove the obsolete backup_volume() https://review.opendev.org/c/openstack/cinder/+/803823 | 07:50 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements https://review.opendev.org/c/openstack/cinder/+/843277 | 07:50 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: declare the clone_across_pools capability https://review.opendev.org/c/openstack/cinder/+/847132 | 07:50 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 07:50 |
| Roamer` | so here ^^ (847536) is something I mentioned at a meeting a month or two ago: the StorPool driver can now be configured to export all the volumes via iSCSI (sometimes it is difficult to bring the StorPool kernel modules to a hypervisor, sometimes it is difficult to run iscsid in a container, so it's configurable). As (I think it was) Rajat rightly pointed out, there are indeed places in Cinder | 07:52 |
| Roamer` | and Nova that do different things for iSCSI, so we also declare the storage protocol as constants.ISCSI now. | 07:52 |
| Roamer` | I have two questions though. First, is it expected that there could be multiattach failures when exporting volumes using iSCSI? Our CI system's experimental iSCSI job seems to have hit some, and I am still investigating them, it may turn out to be our implementation, but I thought I'd ask just in case :) | 07:53 |
| Roamer` | and also, the StorPool iSCSI implementation will always export separate volumes as separate iSCSI targets; there is no overlap, there is no "this is a volume store that we export to make a dozen volumes available". Does this mean that we can declare shared_targets: False as a capability? | 07:54 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 9 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/729766 | 09:16 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: DNM: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847170 | 09:16 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM WIP Try to import cinder tests from barbican-tempest-plugin https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/743492 | 09:40 |
| opendevreview | Arun KV proposed openstack/cinder master: Reintroduce DataCore driver https://review.opendev.org/c/openstack/cinder/+/836996 | 09:50 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Read HBA information from sysfs https://review.opendev.org/c/openstack/os-brick/+/847567 | 11:09 |
| *** dviroel|out is now known as dviroel | 11:19 | |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: DNM: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847170 | 11:26 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847170 | 13:27 |
| opendevreview | Gorka Eguileor proposed openstack/cinder stable/wallaby: DNM: Test cinderlib fix for privsep https://review.opendev.org/c/openstack/cinder/+/847189 | 13:32 |
| opendevreview | Walt proposed openstack/cinder master: Add reporting of capacity factors in API https://review.opendev.org/c/openstack/cinder/+/844601 | 13:45 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Ceph set minimum client version https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/782624 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: cinder/api/common.py https://review.opendev.org/c/openstack/cinder/+/841125 | 14:26 |
| *** dviroel is now known as dviroel|lunch | 15:19 | |
| opendevreview | Gorka Eguileor proposed openstack/cinder stable/wallaby: DNM: Test cinderlib fix for privsep https://review.opendev.org/c/openstack/cinder/+/847189 | 15:36 |
| *** dviroel|lunch is now known as dviroel | 16:18 | |
| opendevreview | David Sariel proposed openstack/os-brick master: CacheManager: avoid old style declaration https://review.opendev.org/c/openstack/os-brick/+/847593 | 16:25 |
| Zer0Byte | hey | 17:28 |
| Zer0Byte | im having a issue while reisizing disk on a NFS backend | 17:28 |
| Zer0Byte | ERROR cinder.scheduler.manager [req-98be4d5c-5dc4-4811-9162-f43e944542d7 992ccdcb7d674c5181480bd51fb82adb bdca1685c1d840d9975948c7adf03179 - 01dfac4014394e32bec4f9dd5687a2a0 01dfac4014394e32bec4f9dd5687a2a0] Failed to schedule_extend_volume: Cannot place volume 53e645e1-4457-4da0-8fb9-495f991459f9 on cinder@netapp-vol-openstack#10.10.20.250:/vol_cloud_openstack.: cinder.exception.NoValidBackend: Cannot place volume | 17:28 |
| Zer0Byte | 53e645e1-4457-4da0-8fb9-495f991459f9 on cinder@netapp-vol-openstack#10.10.20.250:/vol_cloud_openstack. | 17:28 |
| Zer0Byte | disk have enough space | 17:29 |
| whoami-rajat | geguileo, (don't want to bother you again and again but) in the latest run of the DNM wallaby patch, we're still facing failures ... https://ee70e0179e6cb74fd85f-56e56a993b15e32d01b886b8d73f896b.ssl.cf1.rackcdn.com/847189/3/check/cinder-tempest-plugin-lvm-lio-barbican/f045d6e/testr_results.html | 17:31 |
| geguileo | whoami-rajat: No bother, but the problem is that depends-on doesn't seem to be working | 17:32 |
| whoami-rajat | oh, if that's the case then any other way of checking if the patch works as expected? | 17:33 |
| geguileo | whoami-rajat: It looks like checkouts the right patch first, but 2 lines later it checkouts wallaby: https://zuul.opendev.org/t/openstack/build/e1d56e58db9f4644a7cc431b45d93c6e/log/job-output.txt#629 | 17:36 |
| geguileo | whoami-rajat: I would just merge the cinderlib patch, as I think it should work | 17:36 |
| geguileo | whoami-rajat: though we may also need this one: https://review.opendev.org/c/openstack/cinderlib/+/845607/ | 17:38 |
| geguileo | not 100% sure on whether we need that one or not... | 17:39 |
| whoami-rajat | geguileo, i agree, and even if i doesn't fix the gate, we won't end up in a worse state then we currently are in | 17:39 |
| whoami-rajat | and cinderlib gate passes with the patch so ... | 17:40 |
| geguileo | yeah, and even the tempest LVM job | 17:40 |
| geguileo | which is not overriding the checkout of the branches | 17:40 |
| geguileo | so it's running tempest with master, and then running cinderlib functional tests inside tox's venv | 17:41 |
| whoami-rajat | cool, at a quick glance, Brian's patch looks right, but i will take your word if we want that in the release or not | 17:42 |
| whoami-rajat | ok, that's some assurance | 17:42 |
| geguileo | whoami-rajat: fortunately Brian's patch only affects tox, so it won't affect the release even if merged | 17:44 |
| whoami-rajat | geguileo, do we want to keep the debug and logs enabled in jobs as done in the patch? | 17:44 |
| geguileo | whoami-rajat: I think it will be better for a while | 17:45 |
| geguileo | that way if it fails in Cinder jobs we have some info on what's going on | 17:45 |
| geguileo | I think it doesn't make sense to run CI jobs without debug logs | 17:46 |
| whoami-rajat | ok, makes sense, i asked since it was disabled before | 17:47 |
| geguileo | yeah, and then I realized how stupid I was for not having it enabled | 17:47 |
| geguileo | lol | 17:47 |
| whoami-rajat | :D | 17:48 |
| whoami-rajat | geguileo, so the pros of merging Brian's patch in yoga seems to be updating the release head pointing to his patch and maybe it helps in packaging of cinderlib? | 17:49 |
| whoami-rajat | so what do you say should we merge it or not? | 17:50 |
| geguileo | I'm ok with merging it | 17:50 |
| geguileo | but I would first merge the other patch, then test the DNM cinder patch to confirm things are fixed | 17:50 |
| geguileo | and then we can decide | 17:50 |
| whoami-rajat | cool | 17:51 |
| whoami-rajat | i had a chat with elodilles in the releases channel and we've time till monday, I will look over this during the weekend but would be good if you can leave your vote on the patch so it becomes easier to merge (for me) | 17:52 |
| geguileo | whoami-rajat: reviewed | 18:19 |
| whoami-rajat | geguileo, thank you! | 18:35 |
| opendevreview | Merged openstack/cinderlib master: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847170 | 19:38 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/wallaby: DNM: Test cinderlib fix for privsep https://review.opendev.org/c/openstack/cinder/+/847189 | 19:40 |
| Zer0Byte | noone with this | 19:54 |
| Zer0Byte | ? | 19:54 |
| Zer0Byte | ERROR cinder.scheduler.manager [req-98be4d5c-5dc4-4811-9162-f43e944542d7 992ccdcb7d674c5181480bd51fb82adb bdca1685c1d840d9975948c7adf03179 - 01dfac4014394e32bec4f9dd5687a2a0 01dfac4014394e32bec4f9dd5687a2a0] Failed to schedule_extend_volume: Cannot place volume 53e645e1-4457-4da0-8fb9-495f991459f9 on cinder@netapp-vol-openstack#10.10.20.250:/vol_cloud_openstack.: cinder.exception.NoValidBackend: Cannot place volume | 19:55 |
| *** dviroel is now known as dviroel|out | 20:50 | |
| opendevreview | Jesper Schmitz Mouridsen proposed openstack/cinder master: Add benji-backup.me driver to ceph and other backends https://review.opendev.org/c/openstack/cinder/+/847620 | 21:42 |
| opendevreview | Luigi Toscano proposed openstack/cinderlib stable/xena: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847622 | 21:48 |
| opendevreview | Luigi Toscano proposed openstack/cinderlib stable/wallaby: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847623 | 21:48 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib stable/wallaby: DNM: Fix privsep issues inside virtual env https://review.opendev.org/c/openstack/cinderlib/+/847623 | 22:50 |
| opendevreview | Gorka Eguileor proposed openstack/cinder stable/wallaby: DNM: Test cinderlib fix for privsep https://review.opendev.org/c/openstack/cinder/+/847189 | 22:51 |
| opendevreview | David White proposed openstack/cinder master: Update docs for powervault driver compatability https://review.opendev.org/c/openstack/cinder/+/847624 | 23:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!