| opendevreview | Brian Rosmaita proposed openstack/cinder master: Implement Xena project personas for group snapshots https://review.opendev.org/c/openstack/cinder/+/759961 | 00:45 |
|---|---|---|
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Implement project personas for backups API https://review.opendev.org/c/openstack/cinder/+/804459 | 01:31 |
| venkatakrishnat | Hi Cores, Can someone please take a look at the patch https://review.opendev.org/c/openstack/cinder/+/784140 which got one CR +2. | 02:13 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Implement Xena project personas for group_actions https://review.opendev.org/c/openstack/cinder/+/759955 | 02:24 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 07:38 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 07:47 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 07:47 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group os-reset_status https://review.opendev.org/c/openstack/cinder/+/804735 | 07:47 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group-snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804757 | 07:47 |
| opendevreview | Arthur Outhenin-Chalandre proposed openstack/cinder master: doc: RBD: add documentation to configure RBD mirrroring https://review.opendev.org/c/openstack/cinder/+/803456 | 08:57 |
| opendevreview | Arthur Outhenin-Chalandre proposed openstack/cinder master: RBD: add support for snapshot mirroring based replication https://review.opendev.org/c/openstack/cinder/+/784945 | 09:09 |
| opendevreview | Arthur Outhenin-Chalandre proposed openstack/cinder master: backup: ceph: add support for snapshot mirroring based replication https://review.opendev.org/c/openstack/cinder/+/784956 | 09:09 |
| *** dviroel|out is now known as dviroel | 11:10 | |
| noonedeadpunk | hey everyone! | 11:36 |
| noonedeadpunk | after merging https://review.opendev.org/c/openstack/cinder/+/806010 I still face issue with migrations | 11:36 |
| noonedeadpunk | https://paste.opendev.org/show/809303/ | 11:38 |
| noonedeadpunk | so it was after "implemented" https://review.opendev.org/c/openstack/cinder/+/760178 and before https://review.opendev.org/c/openstack/cinder/+/806631 | 11:58 |
| noonedeadpunk | * implemented after https://review.opendev.org/c/openstack/cinder/+/806010 | 12:00 |
| noonedeadpunk | doh, between c995438ca2b0cfb078bfd29e585d49fa8f6a693a and 768b8996ba4cb24eb2e5cd5dc149cd114186debd | 12:06 |
| noonedeadpunk | even bccb3da0dde5593aab5f5637f753aca49fc89a6f was affected | 12:07 |
| * noonedeadpunk trying to find specific commit that breaks the thing | 12:23 | |
| rosmaita | noonedeadpunk: there's a bug filed already for that one, and whoami-rajat has a fix up | 12:26 |
| rosmaita | noonedeadpunk: https://launchpad.net/bugs/1942077 | 12:26 |
| rosmaita | noonedeadpunk: https://review.opendev.org/c/openstack/cinder/+/807083 | 12:27 |
| noonedeadpunk | well one of the issues has been fixed with https://bugs.launchpad.net/openstack-ansible/+bug/1941068 | 12:27 |
| noonedeadpunk | oh, ok, let me quickly patch it:) | 12:28 |
| noonedeadpunk | and check if that works | 12:29 |
| rosmaita | noonedeadpunk: that would be awesome, please leave your findings on the review | 12:29 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove usage of undefined CONF.quota_metadata_items https://review.opendev.org/c/openstack/cinder/+/798783 | 12:49 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Migrate all quota parameters to [quota] section. https://review.opendev.org/c/openstack/cinder/+/772181 | 12:49 |
| noonedeadpunk | rosmaita: patch works, and online migrations does not fail with it | 12:52 |
| noonedeadpunk | however, SAWarnings are still there https://paste.opendev.org/show/809305/ | 12:52 |
| noonedeadpunk | Can hardly understand if there's already a patch for that considering amount of recent patches that I see on gerrit... | 12:53 |
| *** lbragstad_ is now known as lbragstad | 13:15 | |
| opendevreview | Merged openstack/cinder master: db: Add initial alembic migration https://review.opendev.org/c/openstack/cinder/+/786931 | 13:22 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Docs - Xena release notes https://review.opendev.org/c/openstack/cinder/+/808959 | 13:49 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: RBD driver https://review.opendev.org/c/openstack/cinder/+/774288 | 13:52 |
| whoami-rajat__ | noonedeadpunk, rosmaita thanks for the discussion and testing it | 14:00 |
| *** whoami-rajat__ is now known as whoami-rajat | 14:00 | |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [SVF] Fix extend issue for mirroring volumes https://review.opendev.org/c/openstack/cinder/+/784140 | 14:08 |
| opendevreview | Merged openstack/cinder master: Xena project personas for volume type access API https://review.opendev.org/c/openstack/cinder/+/760184 | 14:12 |
| *** abhishekk is now known as abhishekk|away | 14:31 | |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for volume actions https://review.opendev.org/c/openstack/cinder/+/806965 | 14:41 |
| venkatakrishnat | whoami-rajat, Thanks for the review. I have updated unit tests and addressed all other comments for https://review.opendev.org/c/openstack/cinder/+/784140 . Please take a look. Thanks. | 14:44 |
| raghavendrat | hi reviewers, this is regarding https://review.opendev.org/c/openstack/cinder/+/806725 | 14:55 |
| raghavendrat | It is simple documentation update. It has one +2. Require another. | 14:55 |
| raghavendrat | Appreciate your feedback. | 14:56 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Fix: Online migration for volume_use_quota_online_data_migration https://review.opendev.org/c/openstack/cinder/+/807083 | 15:45 |
| *** abhishekk|away is now known as abhishekk | 15:51 | |
| opendevreview | Zohar Mamedov proposed openstack/os-brick master: NVMeOF connector support multipath-enabled kernels https://review.opendev.org/c/openstack/os-brick/+/806687 | 15:55 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: experimenting with the pdf generation https://review.opendev.org/c/openstack/cinder/+/808992 | 15:56 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: experimenting with the pdf generation https://review.opendev.org/c/openstack/cinder/+/808992 | 16:01 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Docs - Clarify Replication Group https://review.opendev.org/c/openstack/cinder/+/808995 | 16:15 |
| opendevreview | Merged openstack/cinder master: [SVF]:Storwize HyperSwap snapshot clone is failing https://review.opendev.org/c/openstack/cinder/+/776307 | 16:19 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: experimenting with the pdf generation https://review.opendev.org/c/openstack/cinder/+/808992 | 16:28 |
| venkatakrishnat_ | Hi core-reviewers, Can someone please review the patch https://review.opendev.org/c/openstack/cinder/+/784140 . Already, the patch got one CR +2 after addressing a few comments from Rajat. Thanks in adv. | 17:31 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: experimenting with the pdf generation https://review.opendev.org/c/openstack/cinder/+/808992 | 17:36 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Remove sample policy config from docs generation https://review.opendev.org/c/openstack/cinder/+/808992 | 18:04 |
| opendevreview | Merged openstack/cinder master: db: Add all migrations to initial alembic migration https://review.opendev.org/c/openstack/cinder/+/786932 | 18:21 |
| rosmaita | can i get a sanity check on https://review.opendev.org/c/openstack/cinder/+/808992 ? | 18:28 |
| rosmaita | e0ne eharney geguileo hemna jungleboyj smcginnis whoami-rajat enriquetaso ^^ | 18:28 |
| rosmaita | i'll merge it with just one +2, just need a quick look | 18:28 |
| rosmaita | unblocks the docs gate | 18:28 |
| jungleboyj | Looking. | 18:29 |
| rosmaita | ty | 18:29 |
| jungleboyj | I think it looks ok. You want me to merge rosmaita ? | 18:31 |
| rosmaita | sure | 18:31 |
| jungleboyj | Weird that that is breaking PDF build. | 18:32 |
| rosmaita | yeah, i can't figure out what's going on | 18:33 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/805887 | 18:37 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove unused config imports/objects https://review.opendev.org/c/openstack/cinder/+/805083 | 19:39 |
| opendevreview | Merged openstack/cinder master: [SVF] Fix extend issue for mirroring volumes https://review.opendev.org/c/openstack/cinder/+/784140 | 19:58 |
| opendevreview | Merged openstack/cinder master: Remove sample policy config from docs generation https://review.opendev.org/c/openstack/cinder/+/808992 | 20:41 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Better workaround for policy file in PDF docs https://review.opendev.org/c/openstack/cinder/+/809014 | 21:13 |
| *** dviroel is now known as dviroel|out | 21:36 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Implement project personas for volume actions https://review.opendev.org/c/openstack/cinder/+/806965 | 21:59 |
| opendevreview | Merged openstack/cinder master: Add request method to test requests https://review.opendev.org/c/openstack/cinder/+/808819 | 23:29 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!