| opendevreview | Folasade Falodun proposed openstack/manila master: Add schema validation for share access update https://review.opendev.org/c/openstack/manila/+/1011029 | 01:12 |
|---|---|---|
| opendevreview | Fiyin Peter proposed openstack/manila master: Add JSON Schema for share metadata show response https://review.opendev.org/c/openstack/manila/+/1011033 | 03:44 |
| FiyinPeter[m] | @carlos I'd like to claim #98. I have yet to find an open patch on it and it's so far unassigned | 04:04 |
| FiyinPeter[m] | s/@carlos/carloss:/ | 04:04 |
| opendevreview | Fiyin Peter proposed openstack/manila master: Add JSON Schema for share access show response https://review.opendev.org/c/openstack/manila/+/1011033 | 04:10 |
| opendevreview | Fiyin Peter proposed openstack/manila master: Add JSON Schema for share metadata show response https://review.opendev.org/c/openstack/manila/+/1011033 | 04:16 |
| opendevreview | Olamide Ojo proposed openstack/manila master: Add JSONSchema validation for shares.update_metadata_item() [v2.0+] https://review.opendev.org/c/openstack/manila/+/1011052 | 05:54 |
| opendevreview | Namrata Thakur proposed openstack/manila master: Add JSON Schema validation for share create_metadata https://review.opendev.org/c/openstack/manila/+/1011055 | 06:55 |
| OlamideOjo[m] | hello carloss gouthamr , id love to claim Add JSONSchema validation: shares.deny_access_legacy() [v2.0-2.6] | 07:24 |
| RaginiSharma[m] | Hello gouthamr and carloss , I was working with "quota_sets.detail().... (full message at <https://matrix.org/oftc/media/v1/media/download/AawXgSxfd8kAITPoU-cm0lh_3S0PYQkaS134hDmPQ0hPsYam1aVAcwDV9oWjMWdrNNOkMmVp6udqouGVbmnjrkVCehkN_vVgAG1hdHJpeC5vcmcvSVBzRkpTTWhNaHlHRUpNWlVlQ2VGVWpz>) | 07:32 |
| tessyjames | carloss and gouthamr: Hi Carlos and Goutham. Good day from this side of the world. the NetApp CI ontap-no-dhss job for my change 1010906 keeps failing due to what appears to be a CI setup issue. The logs show Setup device allocating IPs failed with ERROR: Not found available devices from device_group_type manila-cDOT-no-dhss! and an HTTP 404 from /setup_device. The job failed before the Tempest tests ran. Teardown also reported that | 08:54 |
| tessyjames | /home/zuul/teste.yaml was missing. The DHSS job passed in one run, but subsequent runs have also shown immediate NODE_FAILURE errors. Please what's the next step forward to resolve this CI issue? Thanks. | 08:54 |
| opendevreview | Joshua Bamson proposed openstack/manila master: Adding JSON Schema validation for quota set defaults https://review.opendev.org/c/openstack/manila/+/1011060 | 09:54 |
| JoshuaBamson[m] | Hello carloss i submitted my patch for the task 41 that was asigned to me, i also noticed after submittion an existing working in progress review patch by Jonathan Koerber #924467 and that patch modifies the same files, so i wanted to know if you would keep my focused patch as a separate contribution coordinating with Jonathan or will you adjust the scope to avoid duplication? | 10:17 |
| JoshuaBamson[m] | carloss and gouthamr while my patch i submitted is being reviewed and task #84 is still currently listed as unassigned. Would it be possible to assign this task to me so I can start working on it? | 10:26 |
| JoshuaBamson[m] | Thank you | 10:26 |
| delightubah[m] | [carloss](https://matrix.to/#/@_oftc_carloss:matrix.org) [gouthamr](https://matrix.to/#/@_oftc_gouthamr:matrix.org) following up on my with_count question, I filed it as bug 2171294 (https://bugs.launchpad.net/manila/+bug/2171294) and reproduced it with unit tests to verify. I assigned it to myself as I'd be happy to fix it. Would you prefer adding the 2.42 check to detail, or keeping the current behavior and updating the API | 12:39 |
| delightubah[m] | reference? I’ve also added it to the bug triage etherpad for the Oct 29 meeting. | 12:39 |
| *** ubuntu is now known as Guest2940 | 13:17 | |
| opendevreview | Namrata Thakur proposed openstack/manila master: Add JSON schema validation for share network delete https://review.opendev.org/c/openstack/manila/+/1010523 | 13:32 |
| opendevreview | Namrata Thakur proposed openstack/manila master: Add JSON Schema validation for share create_metadata https://review.opendev.org/c/openstack/manila/+/1011055 | 13:35 |
| opendevreview | Ubah Delight Godson Okechukwu proposed openstack/manila master: api-ref: Document exact description filter for shares https://review.opendev.org/c/openstack/manila/+/1011066 | 14:07 |
| opendevreview | Ubah Delight Godson Okechukwu proposed openstack/manila master: api-ref: Add exact description filter for shares https://review.opendev.org/c/openstack/manila/+/1011066 | 14:10 |
| delightubah[m] | opendevreview: I proposed an api-ref fix documenting the exact description filter for shares (new in 2.36), following up on the question I raised in my review of https://review.opendev.org/c/openstack/manila/+/956053: https://review.opendev.org/c/openstack/manila/+/1011066 | 14:13 |
| Namrata | opendevreview: carloss: gouthamr regarding this i have added req schema with additional properties = false like all other schemas but i think the old code used to allow additional prop by ignoring them so this might reject old req that got 200, i hope that's fine but if its an issue lmk....a;so i updated test_create_metadata to pass body as keyword argument because of this line - return func(*args, **kwargs) of api/validat./init.py, | 15:02 |
| Namrata | the wrapper is throwing key err if i am passing it positionally, so i hope that's fine too | 15:02 |
| Namrata | Olamide Ojo: hey i saw ur work on update metadata item and delete metadta for shares, are u also working on update all metadata? If not i will take it and if anybody else is working on it please lmk | 15:14 |
| Namrata | <adeolaadesina[m]> "I am working on share access..." <- Ah okay thanks for replying | 15:47 |
| Namrata | <adeolaadesina[m]> "I am working on share access..." <- [@adeolaadesina:matrix.org](https://matrix.to/#/@adeolaadesina:matrix.org) hey uh so u are working on patch 36? The schema for share access rule metadata? I was asking about shares.update_all_metadata, if it's free then I will take it or is it a part of ur change? | 15:53 |
| Namrata | [@adeolaadesina:matrix.org](https://matrix.to/#/@adeolaadesina:matrix.org) I am taking that as a, yes to go ahead. If I meant something else ping me :) | 15:58 |
| OlamideOjo[m] | <Namrata> "Olamide Ojo: hey i saw ur work..." <- no im not working on update all metadata | 16:15 |
| opendevreview | Fiyin Peter proposed openstack/manila master: Add JSON Schema for legacy share access listing https://review.opendev.org/c/openstack/manila/+/1011075 | 17:27 |
| opendevreview | Olamide Ojo proposed openstack/manila master: Add JSONSchema validation: shares.deny_access_legacy() [v2.0-2.6] https://review.opendev.org/c/openstack/manila/+/1011076 | 17:31 |
| OlamideOjo[m] | hello carloss gouthamr , id love to add jsonschemas for the deny_access endpoint | 17:34 |
| opendevreview | Adeola Adesina proposed openstack/manila master: Schemas for share access rule metadata https://review.opendev.org/c/openstack/manila/+/1011077 | 17:56 |
| opendevreview | Fiyin Peter proposed openstack/manila master: Add JSONSchema validation: shares.access_list_legacy() [v2.0-2.6] https://review.opendev.org/c/openstack/manila/+/1011075 | 18:02 |
| FiyinPeter[m] | opendevreview: Hi carloss. This is submission for my earlier claimed #98 | 18:15 |
| FiyinPeter[m] | * This is my submission for | 18:16 |
| FiyinPeter[m] | Is anyone working on #89 Add JSONSchema validation: share_types._add_project_access() [all] [CRITICAL]? If no, I'd like to claim this | 18:34 |
| opendevreview | Meer Hashaam Khan proposed openstack/manila master: api: Add schemas for share_replicas https://review.opendev.org/c/openstack/manila/+/1010072 | 23:48 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!