| opendevreview | christian okeke proposed openstack/manila-ui master: add metadata create textbox to Create Snapshot Form https://review.opendev.org/c/openstack/manila-ui/+/923597 | 13:03 |
|---|---|---|
| kpdev | hi all, i need reviews on few PRs | 14:06 |
| kpdev | https://review.opendev.org/c/openstack/manila/+/925902 | 14:06 |
| kpdev | https://review.opendev.org/c/openstack/manila/+/924064 | 14:06 |
| kpdev | https://review.opendev.org/c/openstack/manila/+/925112 | 14:06 |
| kpdev | https://review.opendev.org/c/openstack/manila/+/921313 | 14:06 |
| carloss | kpdev: sure, will take a look | 14:57 |
| carloss | gireesh ashrodri gouthamr: could also use your eyes here :) | 14:58 |
| gireesh | Sure @carloss | 15:00 |
| carloss | ty | 15:25 |
| ccokeke[m] | <gouthamr> "ccokeke: hey there; so when..." <- Updated.Please take a look | 15:42 |
| ccokeke[m] | https://review.opendev.org/c/openstack/manila-ui/+/923597 | 15:42 |
| gouthamr | ccokeke[m]: will do thank you | 16:37 |
| fungi | tspyderboy[m]: it might be easier for me to reply to you here (gmail and my personal mailserver don't get along well these days, sorry). over what sort of medium did you want to conduct the interview (written, audio, video...)? i live just off the east coast of the usa so can mostly do anything between 12:00-22:00 utc this time of year, not sure what sort of overlap that has with your | 17:06 |
| fungi | waking yours. i have a lot of tuesday meetings, though other days are fairly open most weeks | 17:06 |
| fungi | your mail client implies you're in utc+0530, so i have a feeling your evenings/my mornings will have the greatest chance of overlap | 17:08 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: Skip test_share_extend in CephFS jobs https://review.opendev.org/c/openstack/manila-tempest-plugin/+/926004 | 17:21 |
| gouthamr | fungi: ty for doing this; /me is very excited for that conversation and the learnings tspyderboy[m] will have | 17:22 |
| fungi | me too! | 17:23 |
| fungi | i always learn something as well | 17:23 |
| gouthamr | ++ :) | 17:23 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: Delete package-based-installation test jobs https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/918951 | 17:31 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: Drop support for package based installation of Ceph https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/918952 | 17:31 |
| gouthamr | carloss: ashrodri: can one of you help me merge https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/915212 - the conflicts are confusing because there's a bunch of re-arranging happening within open patches.. my bad, but it's proabably useless to try to undo them | 17:36 |
| carloss | gouthamr: yep, taking a look | 17:37 |
| carloss | gouthamr: should we test it with a DNM manila-tempest-plugin change? | 17:40 |
| gouthamr | carloss: that job is only running on devstack-plugin-ceph and older stable branches of manila atm | 17:40 |
| carloss | ah, yes | 17:41 |
| carloss | true | 17:41 |
| gouthamr | carloss: we were rationing CI resources; but yes, something we must be aware when making changes that impact the use of standalone nfs-ganesha | 17:41 |
| carloss | yep... just asking because it would basically be a double check for us | 17:42 |
| carloss | but all good | 17:42 |
| carloss | left my vote in the change :) | 17:44 |
| gouthamr | any reason not to workflow? | 17:44 |
| ashrodri | nope, went ahead a started gate jobs. are you aiming to merge 918951 next? | 18:06 |
| gouthamr | ty ashrodri; yes please.. i'll fix up whatever gerrit's complaining about in https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/918952/ right after :) | 18:07 |
| carloss | > any reason not to workflow? | 18:11 |
| carloss | nope, was also trying to get ashrodri a change to look at it | 18:11 |
| carloss | gouthamr: sorry, missed the message | 18:11 |
| carloss | s/change/chance | 18:11 |
| carloss | what is wrong with me today? :D | 18:11 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Standalone nfs-ganesha with cephadm deployment https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/915212 | 20:00 |
| ashrodri | carloss, if i +1 the workflow for that patch and the skip tests patch merges during gate check, would it affect CI results? | 20:05 |
| carloss | ashrodri: you mean w+1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/918951 (Delete package-based-installation test jobs), right? | 20:06 |
| ashrodri | yes | 20:06 |
| carloss | if so, I don't think it would | 20:06 |
| ashrodri | alright, ill move to merge then thx | 20:07 |
| carloss | ashrodri: so yes, I don't think it would impact... both are in different repositories, the other change is already running in the gate. It should be smooth I think :) | 20:08 |
| carloss | ack, ty :D | 20:08 |
| opendevreview | Jonathan Koerber proposed openstack/manila master: Add OpenAPI Schema to user messages https://review.opendev.org/c/openstack/manila/+/922566 | 20:12 |
| opendevreview | Merged openstack/manila-tempest-plugin master: Skip test_share_extend in CephFS jobs https://review.opendev.org/c/openstack/manila-tempest-plugin/+/926004 | 20:16 |
| opendevreview | Carlos Eduardo proposed openstack/python-manilaclient master: WIP Add ensure shares command https://review.opendev.org/c/openstack/python-manilaclient/+/924675 | 20:50 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Delete package-based-installation test jobs https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/918951 | 22:08 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: Drop support for package based installation of Ceph https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/918952 | 23:33 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: DNM: Check dependency on devstack-plugin-ceph changes https://review.opendev.org/c/openstack/manila/+/918956 | 23:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!