| caiqilong | akawai: hi, i am a contributer as you. if the zuul gives -1 to your review, you could check the result and make modification if the failed result is due to your driver coder | 01:03 |
|---|---|---|
| caiqilong | code | 01:04 |
| caiqilong | akawai: after you proposed you patch, the ZUUL CI need time to run tests, after that, it could give you +1 or -1 result. | 01:05 |
| caiqilong | akawai: you cold get a real time viewing about your ZUUL tasks at: https://zuul.opendev.org/t/openstack/status | 01:06 |
| caiqilong | akawai: your patch could not be merged without +1 by Zuul test | 01:07 |
| akawai | caiqilong: Thank you for your reply. The URL is very useful. | 01:16 |
| akawai | caiqilong: But my asking is : I would like to run Zuul test for my patch https://review.opendev.org/c/openstack/cinder/+/885986, but the test has never been run automatically and manually(e.g. recheck command) on the patch. I guess the cause is that my patch does not have the reviewer "Zuul". Do you know what to do? | 01:24 |
| caiqilong | akawai: it seems you patch trigger a task queued in ZUUl, Generally, the manually trigger way is just leave a comment "recheck" | 01:31 |
| caiqilong | akawai: if it doesn't work, maybe you should ask core reviews for further solutions. | 01:32 |
| akawai | caiqilong: Thanks. | 01:33 |
| caiqilong | akawai: you do not need to add reviewer ZUUL manually | 01:33 |
| caiqilong | akawai: it will be auto added once it report a result | 01:34 |
| caiqilong | the key problem is your patch can't trigger into ZUUL at https://zuul.opendev.org/t/openstack/status | 01:35 |
| akawai | caiqilong: I see. | 01:37 |
| opendevreview | caiqilong proposed openstack/cinder master: Add Cinder driver for TOYOU NetStor TYDS https://review.opendev.org/c/openstack/cinder/+/886942 | 02:43 |
| Supriya_Kotwal | Hi Sofia Enriquez- I have update commit message for https://review.opendev.org/c/openstack/cinder/+/886612/. It will be very great and helpful for us if you all review it again. Sean McGinnis reviewed yesterday. | 05:22 |
| opendevreview | caiqilong proposed openstack/cinder master: Add Cinder driver for TOYOU NetStor TYDS https://review.opendev.org/c/openstack/cinder/+/886942 | 07:26 |
| opendevreview | Biser Milanov proposed openstack/os-brick master: storpool.py: Use StorPool's API for Attach/Detach https://review.opendev.org/c/openstack/os-brick/+/887575 | 08:31 |
| opendevreview | Biser Milanov proposed openstack/os-brick master: StorPool: Raise on spopenstack, SP_OURID issues https://review.opendev.org/c/openstack/os-brick/+/887576 | 08:31 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/887578 | 08:35 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/887578 | 08:50 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/887586 | 08:53 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/887587 | 08:55 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/887588 | 09:12 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/887589 | 09:13 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/862490 | 09:26 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Fix temp volume project id for Tatlin storage https://review.opendev.org/c/openstack/cinder/+/858293 | 09:29 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin Unified driver - remove unused code https://review.opendev.org/c/openstack/cinder/+/862489 | 10:16 |
| sfv880_ | Hello reviewers, could you please review backports https://review.opendev.org/c/openstack/cinder/+/886067 and https://review.opendev.org/c/openstack/cinder/+/886141 ? Thank you very much! | 11:58 |
| chris218 | Hi, sorry to bother you but wanted to ask for your recommendations regarding openstack setup with slow networking. | 12:06 |
| chris218 | I'm setting up small openstack installation with 3 low powered nodes which only have single 1gig ethernet and from what I've read so far people discourage using ceph with only 1G networking | 12:07 |
| chris218 | (it's mostly homelab/for learning purposes) | 12:07 |
| *** chuanm3 is now known as chuanm | 13:40 | |
| admin1 | chris218, you can use nfs or iscsi/lvm directly .. so that yuo save the bandwidth on the 1gbit for the 3x replication | 17:15 |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 17:18 |
| chris218 | So like without using ceph at all? | 17:19 |
| tosky | yes! (cinder-tempest-plugin) | 17:25 |
| admin1 | as far as openstack (cinder) is concerned, ceph is just one of the backend which is abstracted/hidden to the user by cinder | 17:27 |
| chris218 | Awesome thanks for help | 17:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!