| opendevreview | Raghavendra Tilay proposed openstack/cinder master: [WIP] HPE 3par: Add comment for cloned volumes https://review.opendev.org/c/openstack/cinder/+/916063 | 09:21 |
|---|---|---|
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Support get manageable resources. https://review.opendev.org/c/openstack/cinder/+/913211 | 09:35 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Support manage volume and snapshot https://review.opendev.org/c/openstack/cinder/+/913217 | 09:37 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Support volume replication https://review.opendev.org/c/openstack/cinder/+/914229 | 09:46 |
| crohmann | Could someone kindly look into my comment https://review.opendev.org/c/openstack/cinder/+/484729/comments/b6da8f16_5c9481e3 on cinder-backup doing incremental backups of snapshots? I'd like to know who is at fault here so I can fix the right thing. | 10:56 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Fix reimage with snapshot backed image https://review.opendev.org/c/openstack/cinder/+/916408 | 11:21 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Fix reimage with snapshot backed image https://review.opendev.org/c/openstack/cinder/+/916408 | 11:21 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 12:52 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 12:57 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: Add wsgi module https://review.opendev.org/c/openstack/cinder/+/902876 | 13:59 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/cinder/+/902877 | 13:59 |
| zaitcev | rosmaita: do you know if we have review festival today? | 14:04 |
| rosmaita | i believe so | 14:04 |
| rosmaita | though i guess we are running late | 14:04 |
| crohmann | zaitcev: rosmaita: May I kindly add my two remaining patches from the PTG to your list? | 14:05 |
| rosmaita | crohmann: you may, and you have my permission to bug me on monday when i haven't reviewed them | 14:05 |
| crohmann | Where is the list to add them? | 14:05 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: docs: Migrate docs from cinderclient to OSC (part 1) https://review.opendev.org/c/openstack/cinder/+/916430 | 14:06 |
| rosmaita | crohmann: you mean for the review festival? | 14:06 |
| crohmann | yes | 14:07 |
| zaitcev | Dashboard says " | 14:07 |
| zaitcev | XS patches proposed to master that are not mine, have passed Zuul, no merge conflict, not reviewed by me, no negative reviews (9)". It's automated. | 14:07 |
| rosmaita | https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:07 |
| jbernard | rosmaita: meetpad is not cooperating for me | 14:07 |
| crohmann | ah "XS-only" | 14:07 |
| rosmaita | jbernard: i think Rajat switched to using google meet | 14:07 |
| crohmann | In any case they would be https://review.opendev.org/c/openstack/cinder/+/484729 and https://review.opendev.org/c/openstack/cinder/+/810457 | 14:07 |
| rosmaita | so you can do that too | 14:07 |
| jbernard | rosmaita: the cal event i have shows meetpad | 14:08 |
| whoami-rajat | I've started the meeting, everyone can join now https://meet.google.com/jqg-eigw-rku | 14:08 |
| rosmaita | jbernard: ok, we really need to go over the list of dev cycle tasks (you, me, and rajat) | 14:09 |
| zaitcev | jbernard: er, the Etherpad says https://meet.google.com/jqg-eigw-rku | 14:09 |
| zaitcev | OK, Etherpad is updated with new meeting, ushering in the Jon's regime | 14:12 |
| opendevreview | Merged openstack/rbd-iscsi-client master: Update CI for Dalmatian https://review.opendev.org/c/openstack/rbd-iscsi-client/+/915462 | 14:49 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Move rate limiting middleware to its own module https://review.opendev.org/c/openstack/cinder/+/916434 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove v2 limits API controller https://review.opendev.org/c/openstack/cinder/+/916435 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Merge v2, v3 volume metadata controller https://review.opendev.org/c/openstack/cinder/+/916436 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: tests: Don't use mocks as requests https://review.opendev.org/c/openstack/cinder/+/916437 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: tests: Merge v2, v3 volume API tests https://review.opendev.org/c/openstack/cinder/+/916438 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Merge v2, v3 volume controller https://review.opendev.org/c/openstack/cinder/+/916439 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: tests: Merge v2, v3 snapshot API tests https://review.opendev.org/c/openstack/cinder/+/916440 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove v2 snapshots API controller https://review.opendev.org/c/openstack/cinder/+/916461 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: tests: Merge v2, v3 fakes https://review.opendev.org/c/openstack/cinder/+/916462 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove v2 prefixes https://review.opendev.org/c/openstack/cinder/+/916463 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove unnecessary router split https://review.opendev.org/c/openstack/cinder/+/916464 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Add missing functools.wraps https://review.opendev.org/c/openstack/cinder/+/916465 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove errant actions https://review.opendev.org/c/openstack/cinder/+/916466 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Keep track of extension controllers https://review.opendev.org/c/openstack/cinder/+/916467 | 14:51 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove AdminController controller https://review.opendev.org/c/openstack/cinder/+/916468 | 14:51 |
| stephenfin | In case it's not obvious, that's the start of the openapi series. Basically I'm making the router and controllers easier to introspect. We'd like to have a 1:1 mapping of HTTP method + path (e.g. GET /volumes) to controller method | 14:53 |
| stephenfin | It also cleans up the remnants of the v2/v3 API split, which I can only assume is an unalloyed "Good Thing(TM)" | 14:54 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 16:07 |
| zaitcev | stephenfin: interesting! | 16:07 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 16:09 |
| opendevreview | Merged openstack/os-brick master: Update CI for Dalmatian https://review.opendev.org/c/openstack/os-brick/+/915425 | 16:26 |
| opendevreview | Merged openstack/cinder master: Update CI for Dalmatian https://review.opendev.org/c/openstack/cinder/+/915422 | 16:36 |
| opendevreview | Merged openstack/cinder master: [docs] Add recheck advice https://review.opendev.org/c/openstack/cinder/+/913789 | 16:36 |
| opendevreview | Merged openstack/cinder master: docs: Migrate docs from cinderclient to OSC (part 1) https://review.opendev.org/c/openstack/cinder/+/916430 | 16:36 |
| opendevreview | Merged openstack/os-brick master: Cleanup pylintrc adjustments for six https://review.opendev.org/c/openstack/os-brick/+/914237 | 17:17 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove unnecessary router split https://review.opendev.org/c/openstack/cinder/+/916464 | 17:27 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Add missing functools.wraps https://review.opendev.org/c/openstack/cinder/+/916465 | 17:27 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove errant actions https://review.opendev.org/c/openstack/cinder/+/916466 | 17:27 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Keep track of extension controllers https://review.opendev.org/c/openstack/cinder/+/916467 | 17:27 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api: Remove AdminController controller https://review.opendev.org/c/openstack/cinder/+/916468 | 17:27 |
| opendevreview | Merged openstack/cinder master: Fujitsu driver: Improve volume deletion https://review.opendev.org/c/openstack/cinder/+/898199 | 17:54 |
| opendevreview | Pete Zaitcev proposed openstack/cinder master: Speed up starting cinder-backup https://review.opendev.org/c/openstack/cinder/+/657543 | 22:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!