| opendevreview | Comfort Oyewole proposed openstack/manila master: Add metadata to share replica API docs https://review.opendev.org/c/openstack/manila/+/982140 | 04:37 |
|---|---|---|
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982352 | 04:41 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 05:18 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 06:03 |
| opendevreview | Jonathan Koerber proposed openstack/manila master: api: Add Schema to share: create POST /v2/share https://review.opendev.org/c/openstack/manila/+/974649 | 06:24 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 06:24 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 07:07 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: Fix wrong type for group_snapshot_id parameter https://review.opendev.org/c/openstack/manila/+/982055 | 07:38 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: [CI] set grenade to non-voting and drop grenade-skip-level https://review.opendev.org/c/openstack/manila/+/982295 | 07:42 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: Migrate grenade resources.sh from manila CLI to openstack CLI https://review.opendev.org/c/openstack/manila/+/981631 | 07:42 |
| opendevreview | Merged openstack/manila master: api-ref: Fix wrong type for group_snapshot_id parameter https://review.opendev.org/c/openstack/manila/+/982055 | 08:05 |
| lesmwendwa | carloss goutham: can i have this assigned to me? https://bugs.launchpad.net/manila/+bug/2146271 | 11:01 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 11:20 |
| Asmamothana[m] | carloss: Opps thought I put the link. If no one took it yet, https://bugs.launchpad.net/manila/+bug/2146271 | 12:00 |
| Asmamothana[m] | What's the difference between triaged and in-progress? Does in-progress mean a fix has already been pushed and still in review? | 12:07 |
| lesliem | Triaged means comments contain a full analysis on how to fix the issue, in-progress means the bug has an assignee and work on the fix is in progress | 12:11 |
| opendevreview | Silvia Wachira proposed openstack/python-manilaclient master: Fix DictColumn JSON serialization error https://review.opendev.org/c/openstack/python-manilaclient/+/981744 | 12:22 |
| hong-p[m] | carloss: hi i need help with a grenade failure on zuul. It fails at resources.sh create, but there are no errors in the manila-api logs. the change is just a one-liner for RequestContext initialization. could you help me?" https://review.opendev.org/c/openstack/manila/+/981930 | 12:22 |
| carloss | hong-p[m]: thanks for the ping... we're currently looking into that issue | 12:24 |
| carloss | https://review.opendev.org/c/openstack/manila/+/982295 - this should get gate passing | 12:24 |
| carloss | https://review.opendev.org/c/openstack/manila/+/981631/9 - this should fix resource.sh | 12:25 |
| carloss | https://review.opendev.org/q/I229a46393ed956ff9fb4c823e0054a75b510d2de - this should fix a regression grenade is catching | 12:25 |
| carloss | https://review.opendev.org/c/openstack/manila/+/980846/7 - this should re-enable the job after things get back to passing | 12:25 |
| carloss | hong-p[m]: I'm trying to get more core reviewers to look into such changes | 12:26 |
| hong-p[m] | Oh, sorry for the duplicate message. My element app acted up and resent it. Please ignore the extra ping! | 12:42 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 13:12 |
| carloss | hong-p[m]: no worries :) | 13:39 |
| opendevreview | Asma Mothana proposed openstack/manila master: api-ref: Add network_allocation_update_support to share network responses https://review.opendev.org/c/openstack/manila/+/982324 | 13:53 |
| nathan_akin[m] | opendevreview: Thanks for the merge, ill work on more fixes | 13:56 |
| opendevreview | Asma Mothana proposed openstack/manila master: api-ref: Add network_allocation_update_support to share network responses https://review.opendev.org/c/openstack/manila/+/982324 | 14:17 |
| opendevreview | Cynthia Uche proposed openstack/manila master: Fix incorrect response codes in api-ref for shares and snapshots https://review.opendev.org/c/openstack/manila/+/982431 | 14:30 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982458 | 15:35 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 15:45 |
| Asmamothana[m] | Hi. I've submitted two documentation patches https://review.opendev.org/c/openstack/manila/+/982241 and https://review.opendev.org/c/openstack/manila/+/982324. I noticed #982324 has a NetApp CI failure on patchset 3 which I think is infrastructure related since both are documentation only changes. I re triggered it to check just in case. Could anyone take a look and let me know if I'm on the right track? Thanks :) @carloss | 16:26 |
| Asmamothana[m] | @gouthamr | 16:26 |
| Asmamothana[m] | * Hi. I've submitted two documentation patches https://review.opendev.org/c/openstack/manila/+/982241 and https://review.opendev.org/c/openstack/manila/+/982324. I noticed #982324 has a NetApp CI failure on patchset 3 which I think is infrastructure related since both are documentation only changes. I re triggered it to check just in case. Could anyone take a look and let me know if I'm on the right track? Thanks :) | 16:26 |
| Asmamothana[m] | carloss: gouthamr Hi! I've submitted two documentation patches https://review.opendev.org/c/openstack/manila/+/982241 and https://review.opendev.org/c/openstack/manila/+/982324. One has a NetApp CI failure on patchset 3 which I think is infrastructure related since its only documentation only changes. I retriggered it to check just in case. Could anyone take a look and let me know if I'm on the right track? Thanks :) | 16:27 |
| Priestess[m] | <Asmamothana[m]> "carloss: gouthamr Hi! I've..." <- I got the same NetApp CI build failed notification too. | 16:34 |
| Priestess[m] | And I only made changes to documentation. | 16:34 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 16:53 |
| carloss | Asmamothana[m] Priestess[m] hey, so NetApp CI is what we call third party CI, which pretty much runs manila tests against NetApp hardware. In this case, as your changes are mostly targeting documentation, these failures won't necessarily be related to your changes, so you can ignore them for now | 17:23 |
| carloss | if it was a change in the code then it would be definitely worth investigating the third party driver CI failures | 17:24 |
| nathan_akin[m] | carloss: i noticed the failures in my Prs too, i figured out it was an existing issue, can i take a look at whats wrong? | 17:27 |
| carloss | nathan_akin[m]: what's the failure you're referring to? | 17:27 |
| carloss | I mean which voting job is failing? | 17:27 |
| nathan_akin[m] | this one, NetApp-CI | 17:29 |
| nathan_akin[m] | Build failed. | 17:29 |
| nathan_akin[m] | but i noticed in one of my merged Prs it passed | 17:29 |
| nathan_akin[m] | for example , this also. NetApp-CI | 17:30 |
| nathan_akin[m] | Build failed. | 17:30 |
| nathan_akin[m] | manila-tempest-plugin-ontap-dhss https://netappopenstacklogserver.s3.us-west-1.amazonaws.com/index.html#logs/52/982052/3/upstream-check-manila/manila-tempest-plugin-ontap-dhss/7f58534/ : RETRY_LIMIT in 7m 07s | 17:30 |
| carloss | nathan_akin[m]: I think the same thing applies to you | 17:30 |
| carloss | but the retry limit doesn't mean the build failed in this case | 17:30 |
| carloss | the job itself is not running due to some issues | 17:30 |
| nathan_akin[m] | oh i understand | 17:30 |
| nathan_akin[m] | thats actually true | 17:30 |
| carloss | you can ignore it for now | 17:30 |
| nathan_akin[m] | carloss: is it a bug that can be checked? | 17:31 |
| nathan_akin[m] | or its not really necessary | 17:31 |
| carloss | nope, that's on the NetApp engineering team, as we don't have access to their hardware and infra to understand what exactly is going on | 17:31 |
| nathan_akin[m] | carloss: thats good to know, thanks for clarifying | 17:32 |
| nathan_akin[m] | i was confused the first time i saw it | 17:32 |
| opendevreview | Silvia Wachira proposed openstack/manila master: api-ref: fix share_id parameter reference in share-replicas https://review.opendev.org/c/openstack/manila/+/982481 | 17:46 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Shared File Systems telemetry https://review.opendev.org/c/openstack/manila/+/982149 | 18:25 |
| Asmamothana[m] | carloss: Is there a limit to how many contributions we can make during the application period? And are there specific types of contributions that are weighted more heavily than others when evaluating applicants? | 18:26 |
| opendevreview | Joses De Carlos proposed openstack/manila master: add share network get schema validation https://review.opendev.org/c/openstack/manila/+/979348 | 18:37 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Add unit tests for manila.cmd.status module https://review.opendev.org/c/openstack/manila/+/982050 | 18:49 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Replace assertTrue/assertFalse with assertEqual/assertNotEqual https://review.opendev.org/c/openstack/manila/+/982052 | 18:49 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: add missing fields to share server responses https://review.opendev.org/c/openstack/manila/+/982058 | 18:49 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: fix share_server_id used as path param but defined in body https://review.opendev.org/c/openstack/manila/+/982328 | 18:49 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: add missing availability_zone to replica responses https://review.opendev.org/c/openstack/manila/+/982329 | 18:49 |
| Asmamothana[m] | Also, this is still available, right? https://bugs.launchpad.net/manila/+bug/1981626 If so, can I have it assigned to me? | 19:19 |
| carloss | hey Asmamothana[m] - the bug in question is being addressed by https://review.opendev.org/c/openstack/manila/+/858356 - which is in merge conflict | 19:22 |
| carloss | if you'd like, you can also rebase the fix and follow-up on comments if any | 19:22 |
| Asmamothana[m] | carloss: Yes, will give it a shot | 19:39 |
| opendevreview | Comfort Oyewole proposed openstack/manila master: Add metadata to share replica API docs https://review.opendev.org/c/openstack/manila/+/982140 | 20:18 |
| opendevreview | Merged openstack/manila master: [CI] set grenade to non-voting and drop grenade-skip-level https://review.opendev.org/c/openstack/manila/+/982295 | 21:18 |
| opendevreview | Cynthia Uche proposed openstack/manila master: Fix incorrect response codes in api-ref for shares and snapshots https://review.opendev.org/c/openstack/manila/+/982431 | 22:00 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: document metadata param in manage snapshot https://review.opendev.org/c/openstack/manila/+/982508 | 22:10 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Remove deprecated contrib/ci scripts https://review.opendev.org/c/openstack/manila/+/982509 | 22:29 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!