| opendevreview | Youngjun Yoo proposed openstack/manila master: refactoring: modify function parameters and annotations to match https://review.opendev.org/c/openstack/manila/+/912507 | 00:54 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: service_instance: Ensure create_service_image is a boolean https://review.opendev.org/c/openstack/puppet-manila/+/912688 | 01:48 |
| opendevreview | Merged openstack/manila master: refactoring: modify function parameters and annotations to match https://review.opendev.org/c/openstack/manila/+/912507 | 04:47 |
| opendevreview | jayaanand borra proposed openstack/manila master: Human readable export location NetApp driver changes https://review.opendev.org/c/openstack/manila/+/908907 | 05:18 |
| opendevreview | jayaanand borra proposed openstack/manila master: Human readable export location NetApp driver changes https://review.opendev.org/c/openstack/manila/+/908907 | 05:33 |
| opendevreview | jayaanand borra proposed openstack/manila master: Human readable export location NetApp driver changes https://review.opendev.org/c/openstack/manila/+/908907 | 05:34 |
| opendevreview | Ke Niu proposed openstack/manila master: Add user defined extra capabilities https://review.opendev.org/c/openstack/manila/+/911085 | 08:24 |
| opendevreview | Ke Niu proposed openstack/manila master: Add user defined extra capabilities https://review.opendev.org/c/openstack/manila/+/911085 | 08:26 |
| opendevreview | christian okeke proposed openstack/manila master: Fix the link to get the next share replica https://review.opendev.org/c/openstack/manila/+/911709 | 10:48 |
| opendevreview | christian okeke proposed openstack/manila master: Add delete CLI helper to ShareCommands https://review.opendev.org/c/openstack/manila/+/911694 | 11:47 |
| opendevreview | christian okeke proposed openstack/manila master: Add delete CLI helper to ShareCommands https://review.opendev.org/c/openstack/manila/+/911694 | 11:56 |
| opendevreview | kiran pawar proposed openstack/manila master: Add more tests for share/snapshot deferred deletion https://review.opendev.org/c/openstack/manila/+/912765 | 12:47 |
| opendevreview | Carlos Eduardo proposed openstack/manila master: [WIP] testing pylint https://review.opendev.org/c/openstack/manila/+/912777 | 13:59 |
| *** ChanServ sets mode: +o carloss | 14:04 | |
| *** ChanServ changes topic to "OpenStack Shared File System Service | Documentation: https://docs.openstack.org/manila/latest/ | Meetings: https://wiki.openstack.org/wiki/Manila/Meetings | Channel History: https://meetings.opendev.org/irclogs/%23openstack-manila/ | Dalmation PTG (8th to 12th Apr 2024) Planning Etherpad: https://etherpad.opendev.org/p/dalmatian-ptg-manila-planning" | 14:06 | |
| opendevreview | Merged openstack/manila master: Forbid resetting state of active replicas https://review.opendev.org/c/openstack/manila/+/899320 | 14:08 |
| opendevreview | Ashley Rodriguez proposed openstack/manila stable/2023.2: Conditional Import for FIPS Compliance https://review.opendev.org/c/openstack/manila/+/912742 | 14:58 |
| opendevreview | Merged openstack/manila-tempest-plugin master: Add access rules restriction tests https://review.opendev.org/c/openstack/manila-tempest-plugin/+/889331 | 16:16 |
| opendevreview | Merged openstack/manila master: [CI] Drop glusterfs jobs https://review.opendev.org/c/openstack/manila/+/908680 | 16:42 |
| tspyderboy[m] | Hi gouthamr Can you please assign me: https://bugs.launchpad.net/python-manilaclient/+bug/1898312 | 16:46 |
| gouthamr | tspyderboy[m]: o/ please go ahead and submit a fix; your comment suggesting that you are looking into this is sufficient for the moment | 16:47 |
| gouthamr | when you submit a fix, one of us will assign the bug to you.. | 16:47 |
| tspyderboy[m] | I am unable to push changes: | 16:48 |
| tspyderboy[m] | `fatal: unable to access 'https://review.opendev.org/openstack/python-manilaclient/': The requested URL returned error: 403` | 16:48 |
| gouthamr | tspyderboy[m]: how are you submitting your change? | 16:48 |
| tspyderboy[m] | git review command | 16:49 |
| tspyderboy[m] | Followed the documentation with proper commit message:... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/TOXXgtoDOftbhRdjBOJKLHjz>) | 16:50 |
| gouthamr | tspyderboy[m]: a couple of things to check; have you added your machine's SSH public key to your profile on review.opendev.org? Are your git remotes correct? you can check the o/p of "git remote -v" | 16:50 |
| tspyderboy[m] | git remote -v... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/NiZZZtcgBNpHnhHBaUFmtmsQ>) | 16:52 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila stable/2023.2: Forbid resetting state of active replicas https://review.opendev.org/c/openstack/manila/+/912743 | 16:53 |
| gouthamr | tspyderboy[m]: hmmm; i've not used https with git review; is your https password correct? | 16:59 |
| gouthamr | tspyderboy[m]: i presume you followed this https://docs.opendev.org/opendev/infra-manual/latest/developers.html#accessing-gerrit-over-https | 17:00 |
| tspyderboy[m] | Yes, I tested it on sandbox with HTTPS password, there it worked fine, thanks will explore the other way of SSH. Just a question, there is no hook based validation on bug-id or branch as error is 403? | 17:07 |
| gouthamr | tspyderboy[m]: what kind of validation are you expecting; maybe "git review -vvv" will give you a bit more detail about why the 403 occurred.. i suggest using git-review with SSH in general | 17:08 |
| opendevreview | Merged openstack/manila master: Human readable export location NetApp driver changes https://review.opendev.org/c/openstack/manila/+/908907 | 18:04 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.1: Fix stable CI https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/909055 | 19:13 |
| tspyderboy[m] | <gouthamr> "tspyderboy: what kind of..." <- If I enter wrong password, error is:... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/tRozwOtQfHZuMoRGltXCdTpQ>) | 19:55 |
| opendevreview | Merged openstack/manila master: Add more tests for share/snapshot deferred deletion https://review.opendev.org/c/openstack/manila/+/912765 | 19:56 |
| opendevreview | Merged openstack/manila stable/2023.2: Forbid resetting state of active replicas https://review.opendev.org/c/openstack/manila/+/912743 | 20:31 |
| opendevreview | jordan kinlocke proposed openstack/manila master: [Doc] replace "manila" with "openstack" cli https://review.opendev.org/c/openstack/manila/+/912815 | 21:13 |
| opendevreview | jordan kinlocke proposed openstack/manila master: [Doc] replace "manila" with "openstack" cli https://review.opendev.org/c/openstack/manila/+/912815 | 21:35 |
| gouthamr | tspyderboy[m]: hi, noticing your question now... if you're trying to fix up the gerrit remote for ssh, just delete the existing gerrit remote, and retry: "git remote rm gerrit" ... then use "git review".. it should automatically try using ssh and form the right URL (it mght ask you for your gerrit username, you'd need to set that with "git config --global gitreview.username <username>") | 21:44 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila stable/2023.1: Forbid resetting state of active replicas https://review.opendev.org/c/openstack/manila/+/912744 | 22:12 |
| opendevreview | Merged openstack/manila master: Deprecate the Ceph NFSProtocolHelper https://review.opendev.org/c/openstack/manila/+/912648 | 22:27 |
| opendevreview | Merged openstack/manila master: [doc] Update CephFS admin guide https://review.opendev.org/c/openstack/manila/+/911632 | 22:27 |
| opendevreview | Merged openstack/manila master: Change admin metadata config option name https://review.opendev.org/c/openstack/manila/+/912300 | 22:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!