| opendevreview | Goutham Pacha Ravi proposed openstack/manila stable/yoga: Fix DriverFilter/GoodnessWeigher string evaluations https://review.opendev.org/c/openstack/manila/+/857474 | 05:29 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/python-manilaclient master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/python-manilaclient/+/856256 | 09:13 |
| opendevreview | kiran pawar proposed openstack/manila master: [NetApp] Consider last transfer size and error for replica state https://review.opendev.org/c/openstack/manila/+/856756 | 09:51 |
| opendevreview | kiran pawar proposed openstack/manila master: Add defaultadsite to security service https://review.opendev.org/c/openstack/manila/+/855220 | 10:34 |
| opendevreview | Pierre Riteau proposed openstack/python-manilaclient master: Respect OS_CACERT when using Manila OSC plugin https://review.opendev.org/c/openstack/python-manilaclient/+/857632 | 10:42 |
| *** dviroel|brb is now known as dviroel | 11:37 | |
| opendevreview | Lucas Morais de oliveira proposed openstack/manila master: Remove status field from security services API https://review.opendev.org/c/openstack/manila/+/845586 | 12:16 |
| opendevreview | Merged openstack/manila-ui stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/manila-ui/+/857452 | 14:03 |
| gouthamr | o/ carloss - dunno if we want to file a blocker for this: https://review.opendev.org/c/openstack/python-manilaclient/+/857632 | 14:17 |
| carloss | gouthamr: o/ | 14:42 |
| * carloss looks | 14:42 | |
| carloss | gouthamr: hmm important fix to have | 14:47 |
| carloss | we haven't claimed full support yet - we mentioned the feature parity and some limitations. it is pretty annoying to have this though because it will prevent people to use the osc with a private CA | 14:57 |
| carloss | gouthamr ^ | 14:57 |
| carloss | wdyt? | 14:57 |
| gouthamr | yes | 15:02 |
| gouthamr | its annoying - we'd have to wait until the coordinated release date to request the next release | 15:02 |
| gouthamr | which may not be too bad; we should probably let priteau know on launchpad | 15:03 |
| carloss | sure gouthamr - we can commit to doing the next release asap and let priteau know that we'll make a new release available with the fix | 15:04 |
| carloss | o/ gouthamr felipe_rodrigues ashrodri vkmc luizsantos[m] lucasmoliveira059 HelenaDantas[m] MatheusAndrade[m] vhari nahimsouza[m] rfluisa_ caiquemello[m] felipe_rodrigues - we are ~5 minutes past the start of our "mid-point" check for the bugsquash :D | 15:06 |
| * gouthamr carloss wishes "@here" or "@channel" worked on irc | 15:07 | |
| felipe_rodrigues | o/ | 15:07 |
| carloss | there's a chance I forgot to add someone's IRC nick to the ping - I tried my best to add everyone :) | 15:07 |
| nahimsouza[m] | hi carloss | 15:07 |
| vhari | o/ | 15:07 |
| carloss | haha gouthamr - that would possibly piss off some people :p | 15:07 |
| caiquemello[m] | hello | 15:07 |
| carloss | it would make irc more slack~ish | 15:08 |
| carloss | hello all :D | 15:08 |
| carloss | I hope you are already having some fun with this bugsquash | 15:09 |
| carloss | I have seen some changes being updated | 15:09 |
| carloss | but first I'd like to ask to the NetAppers: were you able to reassign the bugs within your team? | 15:09 |
| opendevreview | Lucas Morais de oliveira proposed openstack/manila master: Remove status field from security services API https://review.opendev.org/c/openstack/manila/+/845586 | 15:10 |
| carloss | and do you need some help with the bugs in the list? | 15:11 |
| carloss | felipe_rodrigues nahimsouza[m] caiquemello[m] | 15:12 |
| felipe_rodrigues | carloss we reassigned some of them.. | 15:12 |
| caiquemello[m] | carloss: yeap, we separated some bugs to work in this bugsquash | 15:12 |
| caiquemello[m] | felipe_rodrigues: do we updated the assignment in lauchpad? | 15:13 |
| carloss | ack, thank you! | 15:13 |
| felipe_rodrigues | not yet.. we need to update there | 15:14 |
| caiquemello[m] | Ok, I can do that | 15:14 |
| felipe_rodrigues | I updated now on the ethercalc | 15:14 |
| felipe_rodrigues | actulaly, caiquemello[m] you can only update yours | 15:15 |
| caiquemello[m] | ok | 15:15 |
| carloss | okay, thank you | 15:15 |
| felipe_rodrigues | we need privilege mode to assign bug to another person | 15:16 |
| carloss | oh, okay - if you'd like to, I could do that | 15:16 |
| carloss | are you updating the assignees here: https://lite.framacalc.org/lep5gvp0jl-9wgg or here https://ethercalc.net/urx9rn9z6lnb ? | 15:16 |
| felipe_rodrigues | https://bugs.launchpad.net/manila/+bug/1920937 --> goes to caiquemello[m] | 15:16 |
| felipe_rodrigues | https://bugs.launchpad.net/manila/+bug/1966198 --> goes to nahimsouza[m] | 15:16 |
| felipe_rodrigues | here: https://ethercalc.net/urx9rn9z6lnb | 15:18 |
| carloss | ack | 15:19 |
| *** dviroel is now known as dviroel|lunch | 15:21 | |
| carloss | gouthamr: thank you for working on CI for https://bugs.launchpad.net/manila/+bug/1970661 - reassigning it to you makes more sense to me | 15:21 |
| carloss | wdyt? | 15:21 |
| gouthamr | ack sure | 15:24 |
| carloss | I see some failures in the scenario tests for the proposed change https://review.opendev.org/839438 - but they seem like scenario tests | 15:24 |
| carloss | with timeouts | 15:24 |
| gouthamr | yes ^ | 15:24 |
| carloss | so I'd say it is not related to the change you are proposing at all | 15:24 |
| carloss | I'm okay merging it | 15:24 |
| carloss | we'd need some more reviewers | 15:24 |
| gouthamr | yes, its been failing awhile - the failures are random and due to vms not spawning in time, and we are looking at long term strategies - will discuss some more at the PTG | 15:25 |
| carloss | thanks for the context | 15:25 |
| carloss | vhari vkmc - could you please take a look at this change too? | 15:25 |
| vkmc | gouthamr++ | 15:27 |
| vkmc | sure thing | 15:27 |
| carloss | thanks | 15:28 |
| carloss | after yesterday's session we had another addition to the list of this bugsquash: https://review.opendev.org/c/openstack/manila/+/856756 | 15:29 |
| carloss | me and gouthamr are okay with this change and the NetApp CI is happy as well | 15:29 |
| carloss | to the NetAppers: could you please take a look at this change? | 15:29 |
| nahimsouza[m] | sure | 15:31 |
| felipe_rodrigues | for sure, I'm reviewing now | 15:31 |
| carloss | thank you | 15:31 |
| felipe_rodrigues | yw | 15:31 |
| carloss | okay, these are some of the changes that had some movement since yesterday | 15:34 |
| carloss | is there some change/bug you'd like to add to the list? | 15:34 |
| carloss | or something to highlight and have the attention of everyone | 15:34 |
| felipe_rodrigues | I'm working on this issue: https://bugs.launchpad.net/manila/+bug/1918845 | 15:36 |
| felipe_rodrigues | I didn't get the expected behavior described on the report | 15:37 |
| felipe_rodrigues | "Some backends expect that the parent share server do not get removed from the reusable share servers list, considering that they must reuse the share server to create the share from the snapshots" | 15:37 |
| felipe_rodrigues | If we don't remove the shre server from the reusable share server and the limit is just 1, the new share would be placed in the server, not respecting the share server limit of 1 share | 15:38 |
| felipe_rodrigues | is that the expected behavior ? | 15:39 |
| carloss | yes, that was what I expected initially with that feature too | 15:40 |
| carloss | but there is a chance that some backends will not be able to create a share in a different share server | 15:41 |
| carloss | so if the share is created on share server A and the snapshot is sent to share server B due to the limit, the creation may fail | 15:41 |
| felipe_rodrigues | so, the idea is ignore the limit when creating the share from snapshot ? | 15:42 |
| carloss | yes - and make the administrator aware of that | 15:43 |
| carloss | we could also try to fail the operation if it makes sense | 15:43 |
| felipe_rodrigues | got it | 15:43 |
| carloss | but ideally we wouldn't let the share created from the snapshot be placed in a different share server | 15:44 |
| felipe_rodrigues | yeah, that's my doubt.. if we should fail differently.. or we don't want to respect the limit | 15:44 |
| felipe_rodrigues | I don't understand much about the use case for the limit feature | 15:44 |
| felipe_rodrigues | Should the limit be forced always or it can be overestimated in some situations (like creating from snapshot shot or even migrating a share) ? | 15:46 |
| carloss | it's mostly about balancing the amount of shares among share servers | 15:47 |
| felipe_rodrigues | uhhhh... | 15:47 |
| carloss | https://github.com/openstack/manila-specs/blob/master/specs/wallaby/spec-lite-add-max-shares-on-share-server-limit.rst - there are some more details here | 15:47 |
| carloss | the share servers can grow too much in a large cloud | 15:48 |
| carloss | and that makes the share servers tricky to manage | 15:48 |
| felipe_rodrigues | got it | 15:48 |
| carloss | the the behavior of letting the limit for the share server be exceeded in case of creating shares from snapshots sound okay to me | 15:51 |
| carloss | in case we register that in the documentation | 15:51 |
| felipe_rodrigues | so, let's ignore the limit during creating from snapshhot | 15:51 |
| carloss | and make the administrator aware | 15:51 |
| felipe_rodrigues | agree | 15:51 |
| felipe_rodrigues | since there is no balancing during create from snapshot... the share server must be reused | 15:52 |
| felipe_rodrigues | nice | 15:52 |
| felipe_rodrigues | thanks carloss | 15:52 |
| carloss | anytime felipe_rodrigues | 15:53 |
| carloss | okay - is there another change you'd like to highlight? or something else to discuss while we have more people around? | 15:53 |
| gouthamr | NetAppers, please for the love of holy, make your CI trigger simpler :) | 15:57 |
| gouthamr | would be lovely if it were just "run netapp-ci" | 15:57 |
| felipe_rodrigues | lol | 15:57 |
| nahimsouza[m] | :D | 15:58 |
| carloss | haha, and pleeeease get back the filters for error, debug, warning levels on the screen files | 15:58 |
| carloss | :D | 15:58 |
| felipe_rodrigues | I'd love too | 15:58 |
| felipe_rodrigues | we moved to Zuul v3 and kept as "run-NetApp CI" pattern.. don't know why | 15:59 |
| gouthamr | backwards compatibility for humans :D | 15:59 |
| felipe_rodrigues | yeap | 15:59 |
| carloss | ok, so I don't have much more to add to our "meeting" :D | 16:01 |
| carloss | if you have questions or would like to discuss something, please ping on this exact channel :) | 16:01 |
| carloss | see you tomorrow in the manila meeting slot, using our bluejeans bridge! | 16:02 |
| carloss | otherwise, see... * | 16:02 |
| carloss | thank you everyone! :D | 16:04 |
| *** dviroel|lunch is now known as dviroel | 16:43 | |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: [RBAC] Retain legacy admin behaviour https://review.opendev.org/c/openstack/manila/+/856394 | 16:55 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Add cephfs_nfs_cluster_id option https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/857742 | 17:12 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: [CI] pin test plugin in manila-grenade-skip-level https://review.opendev.org/c/openstack/manila/+/857759 | 19:19 |
| opendevreview | Merged openstack/manila master: [NetApp] Consider last transfer size and error for replica state https://review.opendev.org/c/openstack/manila/+/856756 | 20:22 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: [RBAC] Retain legacy admin behaviour https://review.opendev.org/c/openstack/manila/+/856394 | 20:22 |
| *** dviroel is now known as dviroel|afk | 20:26 | |
| opendevreview | Merged openstack/manila-tempest-plugin master: [CI] Stop suppressing cleanup errors in CephFS jobs https://review.opendev.org/c/openstack/manila-tempest-plugin/+/839438 | 21:21 |
| *** dviroel|afk is now known as dviroel | 22:12 | |
| *** dviroel is now known as dviroel|afk | 22:52 | |
| opendevreview | Merged openstack/python-manilaclient stable/wallaby: Switch FIPS job to CentOS 9 https://review.opendev.org/c/openstack/python-manilaclient/+/855976 | 23:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!