| *** baojg has quit IRC | 01:09 | |
| openstackgerrit | pengyuesheng proposed openstack/manila-ui master: Use openstack-python3-train-jobs for python3 test runtime https://review.opendev.org/666776 | 01:24 |
|---|---|---|
| *** baojg has joined #openstack-manila | 01:36 | |
| openstackgerrit | pengyuesheng proposed openstack/manila master: Modify the url of upper_constraints_file https://review.opendev.org/666213 | 01:44 |
| openstackgerrit | Brin Zhang proposed openstack/manila master: Remove the redunant table from windows' editor https://review.opendev.org/668771 | 01:51 |
| openstackgerrit | Brin Zhang proposed openstack/manila master: Remove the redunant table from windows' editor https://review.opendev.org/668771 | 01:55 |
| *** brinzhang has joined #openstack-manila | 01:56 | |
| brinzhang | gouthamr: please deal with this patch, it result in 'git pull' an error on windows, beacuse of that file have an 3 '\t' at the end of the file. | 01:58 |
| gouthamr | brinzhang: looking | 01:58 |
| brinzhang | https://review.opendev.org/#/c/668771/ | 01:58 |
| brinzhang | gouthamr: thanks. | 01:59 |
| gouthamr | brinzhang: you're welcome. thanks for finding the problem and fixing it! | 01:59 |
| brinzhang | gouthamr: np | 02:01 |
| openstackgerrit | Merged openstack/manila master: Remove the redunant table from windows' editor https://review.opendev.org/668771 | 02:10 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/stein: Remove the redunant table from windows' editor https://review.opendev.org/668774 | 02:10 |
| openstackgerrit | Merged openstack/manila stable/stein: Remove the redunant table from windows' editor https://review.opendev.org/668774 | 02:26 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/rocky: Remove the redunant table from windows' editor https://review.opendev.org/668775 | 02:26 |
| gouthamr | brinzhang: does fixing the master allow you to "git pull"? | 02:32 |
| brinzhang | gouthamr: I will have a try. | 02:32 |
| gouthamr | brinzhang: thank you, let me know... this fix is appropriate all the way till stable/queens, but i'll hold off ninja-merging if you're unblocked | 02:33 |
| gouthamr | other cores on this channel are probably offline right now | 02:33 |
| brinzhang | yeah, I am trying with others compute, but that network is not ok, wait please :P | 02:35 |
| *** haixin has joined #openstack-manila | 02:35 | |
| gouthamr | brinzhang: no rush.. :) | 02:36 |
| brinzhang | gouthamr: it's ok, thanks. | 02:36 |
| brinzhang | It does works well now on the windows env. | 02:37 |
| gouthamr | brinzhang: thanks for confirming! | 02:37 |
| gouthamr | brinzhang: and for the fix.. | 02:37 |
| brinzhang | gouthamr: NP. Maintaining a good open source environment is the responsibility of every open source enthusiast :P | 02:38 |
| gouthamr | brinzhang: true! | 02:39 |
| haixin | true | 02:39 |
| openstackgerrit | pengyuesheng proposed openstack/manila master: Modify the url of upper_constraints_file https://review.opendev.org/666213 | 03:25 |
| openstackgerrit | Merged openstack/manila stable/rocky: Remove the redunant table from windows' editor https://review.opendev.org/668775 | 04:23 |
| openstackgerrit | Liron Kuchlani proposed openstack/manila-tempest-plugin master: Look for api microversion header in "Vary" header data https://review.opendev.org/668622 | 04:27 |
| *** toabctl has quit IRC | 04:31 | |
| *** whoami-rajat has joined #openstack-manila | 04:34 | |
| *** brinzhang_ has joined #openstack-manila | 04:47 | |
| *** brinzhang has quit IRC | 04:50 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/queens: Remove the redunant table from windows' editor https://review.opendev.org/668784 | 04:59 |
| *** brinzhang_ has quit IRC | 05:05 | |
| *** brinzhang_ has joined #openstack-manila | 05:05 | |
| *** pcaruana has joined #openstack-manila | 05:06 | |
| *** e0ne has joined #openstack-manila | 05:20 | |
| *** e0ne has quit IRC | 06:12 | |
| *** haixin has quit IRC | 06:13 | |
| *** tosky has joined #openstack-manila | 07:20 | |
| *** toabctl has joined #openstack-manila | 07:30 | |
| *** trident has quit IRC | 08:08 | |
| *** trident has joined #openstack-manila | 08:09 | |
| *** e0ne has joined #openstack-manila | 09:37 | |
| *** tosky has quit IRC | 09:39 | |
| *** tosky has joined #openstack-manila | 09:40 | |
| *** brinzhang_ has quit IRC | 10:35 | |
| openstackgerrit | Helen Walsh proposed openstack/manila master: Manila PowerMax - rebrand from VMAX to PowerMax https://review.opendev.org/663089 | 10:49 |
| openstackgerrit | Helen Walsh proposed openstack/manila master: Manila PowerMax - rebrand from VMAX to PowerMax https://review.opendev.org/663089 | 11:22 |
| openstackgerrit | Merged openstack/manila-ui master: Use openstack-python3-train-jobs for python3 test runtime https://review.opendev.org/666776 | 11:25 |
| openstackgerrit | Helen Walsh proposed openstack/manila master: Manila PowerMax - rebrand from VMAX to PowerMax https://review.opendev.org/663089 | 11:33 |
| *** cosss_ has quit IRC | 12:06 | |
| *** carloss has joined #openstack-manila | 12:14 | |
| *** _erlon_ has joined #openstack-manila | 12:33 | |
| *** eharney has joined #openstack-manila | 12:37 | |
| *** baojg has quit IRC | 13:01 | |
| *** enriquetaso has joined #openstack-manila | 13:14 | |
| *** lseki has joined #openstack-manila | 13:17 | |
| *** brinzhang has joined #openstack-manila | 13:24 | |
| *** baojg has joined #openstack-manila | 13:25 | |
| *** lpetrut has joined #openstack-manila | 13:34 | |
| openstackgerrit | Tom Barron proposed openstack/manila stable/queens: DNM - test only https://review.opendev.org/668908 | 13:45 |
| *** brinzhang has quit IRC | 14:55 | |
| *** brinzhang has joined #openstack-manila | 15:00 | |
| *** hoonetorg has quit IRC | 15:20 | |
| *** brinzhang has quit IRC | 15:21 | |
| *** hoonetorg has joined #openstack-manila | 15:33 | |
| *** enriquetaso has quit IRC | 15:46 | |
| *** e0ne has quit IRC | 16:39 | |
| *** openstackgerrit has quit IRC | 17:04 | |
| *** lpetrut has quit IRC | 17:16 | |
| *** e0ne has joined #openstack-manila | 17:44 | |
| *** e0ne has quit IRC | 18:06 | |
| *** enriquetaso has joined #openstack-manila | 18:09 | |
| *** e0ne has joined #openstack-manila | 18:09 | |
| *** e0ne has quit IRC | 18:22 | |
| *** _erlon_ has quit IRC | 18:28 | |
| dviroel | Hi gouthamr, I'm working on a new patch for the spec *create share from snapshot in another pool or backend* | 18:31 |
| dviroel | Regarding this comment: https://review.opendev.org/#/c/609537/7/specs/train/create-share-from-snapshot-in-another-pool.rst@309 you suggest to modify the interface `create_share_from_snapshot` to return a model update. This should affect the existing drivers. Would be a better solution to create a new driver interface `create_share_from_snapshot_slow_copy` that returns a model update? In the latter, the share-manager | 18:31 |
| dviroel | will be responsible for calling the slow copy when the destination pool is different from the source's share. | 18:31 |
| *** raissa has joined #openstack-manila | 18:33 | |
| *** raissa has quit IRC | 18:42 | |
| *** whoami-rajat has quit IRC | 19:12 | |
| gouthamr | dviroel: hmmm, i'm okay either way.. but prefer the first method, we can make a change to the structure of the return - currently we accept a string or a list, we can check if the update is a dict, and parse the "export_locations" and "status" attrs out if a dictionary | 19:21 |
| gouthamr | dviroel: https://github.com/openstack/manila/blob/f4fc7aa08a8b2d4e7bba02af49bd84ea30d2fcac/manila/share/manager.py#L1730 | 19:21 |
| gouthamr | dviroel: https://github.com/openstack/manila/blob/4b6cfcf69023dc7a4b004c59513c50d92b979152/manila/db/sqlalchemy/api.py#L3137 | 19:22 |
| gouthamr | we already have compatibility code ^ | 19:22 |
| gouthamr | so my view is to evolve the existing driver interface rather than add a new one; we can check what other reviewers think | 19:23 |
| *** eharney has quit IRC | 19:27 | |
| *** whoami-rajat has joined #openstack-manila | 19:29 | |
| dviroel | gouthamr: sure, this solves the compatibility issue. I agree that is better to avoid the creation of a new interface when is not needed. thanks for the reply | 19:34 |
| gouthamr | dviroel: yep, you're welcome | 19:36 |
| gouthamr | tbarron: hey, the devstack backports to unblock us on stable/queens are going rather slowly ianw is paying attention, but he's also probably asleep now to act synchrnously | 19:37 |
| gouthamr | tbarron: how about i make the LVM job non-voting for a while, and make it voting when devstack is fixed | 19:37 |
| gouthamr | tbarron: we have a low risk of regression, because we have other non-voting jobs telling us everything is alright | 19:38 |
| gouthamr | https://review.opendev.org/#/c/668787/ is the backport to stable/rocky on devstack | 19:38 |
| tbarron | gouthamr: kk, ping me when it's ready to go | 19:38 |
| gouthamr | tbarron: ack, ty! | 19:39 |
| *** e0ne has joined #openstack-manila | 19:39 | |
| *** openstackgerrit has joined #openstack-manila | 19:41 | |
| openstackgerrit | Merged openstack/manila-tempest-plugin master: Look for api microversion header in "Vary" header data https://review.opendev.org/668622 | 19:41 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/queens: [CI] Add bindep.txt https://review.opendev.org/667721 | 19:45 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/queens: [CI] Add bindep.txt https://review.opendev.org/667721 | 19:46 |
| gouthamr | tbarron: ^ made lvm non-voting for now | 19:49 |
| gouthamr | tbarron: will propose a patch to fix https://bugs.launchpad.net/manila/+bug/1833696 | 19:50 |
| openstack | Launchpad bug 1833696 in Manila "Devstack installs version-less rdo-release package" [High,New] - Assigned to Goutham Pacha Ravi (gouthamr) | 19:50 |
| tbarron | gouthamr: ack | 19:51 |
| *** e0ne has quit IRC | 19:55 | |
| *** eharney has joined #openstack-manila | 20:04 | |
| *** e0ne has joined #openstack-manila | 20:20 | |
| *** trident has quit IRC | 20:43 | |
| *** trident has joined #openstack-manila | 20:45 | |
| *** s0ru has joined #openstack-manila | 20:56 | |
| dviroel | gouthamr: tbarron about the asynchronous share creation, how the manager can track this shares? There was a discussion in the last meeting, but I'm not sure if was a concern about it. We may need a new driver interface to check those status? | 20:58 |
| gouthamr | dviroel: we can put some breadcrumb down into the database, and retrieve these shares during the periodic health check thread | 21:00 |
| gouthamr | dviroel: you can introduce another state, that's only visible within manila; or use the share's task_state | 21:01 |
| *** pcaruana has quit IRC | 21:16 | |
| dviroel | ok, I will take a look on this. | 21:27 |
| *** e0ne has quit IRC | 21:28 | |
| *** e0ne has joined #openstack-manila | 21:29 | |
| *** e0ne has quit IRC | 21:46 | |
| gouthamr | tbarron bswartz xyang: https://review.opendev.org/#/c/667721/ - please review this when you have a min :) we have a couple backports waiting on that blocker (https://review.opendev.org/#/q/status:open+project:openstack/manila+branch:stable/queens) | 22:16 |
| *** carloss has quit IRC | 22:23 | |
| *** enriquetaso has quit IRC | 22:24 | |
| *** tosky has quit IRC | 22:38 | |
| *** whoami-rajat has quit IRC | 22:42 | |
| *** raissa has joined #openstack-manila | 23:13 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!