| *** eharney has quit IRC | 01:46 | |
| openstackgerrit | haixin proposed openstack/python-manilaclient master: Support query user message by timestamp https://review.opendev.org/708807 | 01:51 |
|---|---|---|
| *** eharney has joined #openstack-manila | 01:58 | |
| openstackgerrit | haixin proposed openstack/manila-specs master: the message-list command now support filter since and before. https://review.opendev.org/716467 | 02:08 |
| *** vhari has quit IRC | 02:11 | |
| *** maaritamm has joined #openstack-manila | 06:05 | |
| openstackgerrit | haixin proposed openstack/manila master: Support query user message by timestamp https://review.opendev.org/706438 | 06:06 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Update hacking for Python3 https://review.opendev.org/716499 | 06:46 |
| *** irclogbot_2 has quit IRC | 06:49 | |
| *** maaritamm has quit IRC | 06:50 | |
| *** maaritamm has joined #openstack-manila | 06:50 | |
| *** Reepicheep has quit IRC | 06:51 | |
| *** Reepicheep has joined #openstack-manila | 06:51 | |
| *** irclogbot_1 has joined #openstack-manila | 06:52 | |
| *** irclogbot_1 has quit IRC | 06:53 | |
| *** irclogbot_1 has joined #openstack-manila | 06:54 | |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix F632 https://review.opendev.org/716508 | 06:55 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix F841 https://review.opendev.org/716510 | 06:59 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix F601 https://review.opendev.org/716511 | 07:05 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix F601 https://review.opendev.org/716511 | 07:06 |
| openstackgerrit | haixin proposed openstack/manila master: Support query user message by timestamp https://review.opendev.org/706438 | 07:09 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix E241 https://review.opendev.org/716512 | 07:13 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Update hacking for Python3 https://review.opendev.org/716499 | 07:21 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix F632 https://review.opendev.org/716508 | 07:21 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix F841 https://review.opendev.org/716510 | 07:21 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix F601 https://review.opendev.org/716511 | 07:21 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix E241 https://review.opendev.org/716512 | 07:21 |
| *** maaritamm has quit IRC | 07:21 | |
| *** maaritamm has joined #openstack-manila | 07:23 | |
| *** tosky has joined #openstack-manila | 07:32 | |
| openstackgerrit | haixin proposed openstack/manila master: Support query user message by timestamp https://review.opendev.org/706438 | 08:12 |
| *** maaritamm has quit IRC | 08:21 | |
| *** maaritamm has joined #openstack-manila | 08:23 | |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix E226 https://review.opendev.org/716530 | 09:26 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix E117 https://review.opendev.org/716533 | 09:32 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix E305 https://review.opendev.org/716534 | 09:44 |
| *** maaritamm has quit IRC | 10:50 | |
| *** maaritamm has joined #openstack-manila | 11:11 | |
| openstackgerrit | Jose Castro Leon proposed openstack/manila master: Fixes pool capabilities reported on CephFS driver https://review.opendev.org/716552 | 11:18 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix E741 https://review.opendev.org/716553 | 11:29 |
| *** AJaeger has joined #openstack-manila | 11:29 | |
| AJaeger | manila cores, somehow you missed all the coding checks in hacking, please review the stack above ^ | 11:29 |
| AJaeger | I broke my initial change into smaller pieces to easily verify and review them | 11:30 |
| tbarron | AJaeger: will do, and thakns for the fixes! | 11:30 |
| AJaeger | tbarron, thanks! | 11:30 |
| AJaeger | one more will come - but first I'd like to get some feedback whether this is fine | 11:31 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila-image-elements master: Update hacking for Python3 https://review.opendev.org/716556 | 11:33 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila-ui master: Update hacking for Python3 https://review.opendev.org/716557 | 11:38 |
| *** maaritamm has quit IRC | 11:38 | |
| *** maaritamm has joined #openstack-manila | 11:47 | |
| *** enriquetaso has joined #openstack-manila | 11:48 | |
| openstackgerrit | Andreas Jaeger proposed openstack/manila-tempest-plugin master: Update hacking for Python3 https://review.opendev.org/716559 | 11:50 |
| AJaeger | so, that should cover all of manila repos. I'll continue on manila itself later. | 11:52 |
| tbarron | AJaeger: I really appreciate you splitting the manila fixes into separate gerrit reviews, makes it all much easier to check. | 11:53 |
| AJaeger | tbarron: the first one was so large and failed - splitting it up was the best way to bisect ;) | 11:54 |
| *** maaritamm has quit IRC | 12:08 | |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix E731 https://review.opendev.org/716563 | 12:11 |
| AJaeger | tbarron: I answered on https://review.opendev.org/716530 - this is a broken Zuul change reporting | 12:25 |
| tbarron | AJaeger:k, thanks | 12:29 |
| openstackgerrit | Andreas Jaeger proposed openstack/manila master: Hacking: Fix W605 https://review.opendev.org/716573 | 12:38 |
| AJaeger | final one ^ | 12:38 |
| AJaeger | tbarron: any idea what's up with manila-ui? https://review.opendev.org/716557 is failing tests and that looks unrelated | 12:39 |
| vkmc | AJaeger, I can look into manila ui o/ | 12:49 |
| tbarron | vkmc: thanks, on the face of it, seems unrelated -- maybe try a null patch? | 12:50 |
| vkmc | tbarron, yes, was about to do that | 12:50 |
| *** maaritamm has joined #openstack-manila | 12:53 | |
| AJaeger | thanks, vkmc ! | 12:54 |
| tbarron | vkmc: when I run 'rm -rf .tox; tox -epy37' on manila-ui with nod changes locally it's OK, same if I cherry-pick the change first | 13:02 |
| *** enriquetaso has left #openstack-manila | 13:05 | |
| openstackgerrit | Victoria Martinez de la Cruz proposed openstack/manila-ui master: [DNM] Testing Manila UI gates https://review.opendev.org/716585 | 13:05 |
| vkmc | tbarron, same, let's see what happens with the gate with the null change | 13:13 |
| *** maaritamm has quit IRC | 13:27 | |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Create share from snapshot in another pool or backend https://review.opendev.org/709697 | 13:33 |
| *** maaritamm has joined #openstack-manila | 13:36 | |
| *** sfernand has joined #openstack-manila | 14:00 | |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Create share from snapshot in another pool or backend https://review.opendev.org/709697 | 14:20 |
| AJaeger | vkmc, tbarron, the null change failed as well ;( | 14:22 |
| tbarron | AJaeger: ack, guess we can't blame you this time :D | 14:22 |
| AJaeger | tbarron: Uff ;) | 14:23 |
| vkmc | yes, it wasn't you AJaeger :) | 14:33 |
| vkmc | I'm on a meeting now, but will debug afterwards | 14:33 |
| AJaeger | vkmc: thanks - don't stress over it... | 14:33 |
| *** lpetrut has joined #openstack-manila | 14:59 | |
| *** lpetrut has quit IRC | 15:06 | |
| *** maaritamm has quit IRC | 15:36 | |
| *** maaritamm has joined #openstack-manila | 15:36 | |
| *** maaritamm has quit IRC | 15:38 | |
| *** maaritamm has joined #openstack-manila | 15:40 | |
| *** maaritamm has quit IRC | 15:42 | |
| gouthamr | tbarron: josecastroleon's not around, but i left a comment in https://review.opendev.org/#/c/716552/ - since we haven't shipped that bugfix yet, i'd be okay reverting the change and re-working it in light of https://launchpad.net/bugs/1870105 | 15:42 |
| openstack | Launchpad bug 1870105 in Manila "CephFS driver wrongly reports pool capabilities" [High,In progress] - Assigned to Jose Castro Leon (jose-castro-leon) | 15:42 |
| gouthamr | tbarron: what do you think? | 15:42 |
| tbarron | gouthamr: we can revert or he can just submit a fix for the fix | 15:45 |
| tbarron | gouthamr: just handle a list, right? | 15:46 |
| tbarron | where he was expecting a bool | 15:46 |
| tbarron | gouthamr: certainly if we don't think a fix will be forthcoming soon we could revert | 15:47 |
| gouthamr | tbarron: yes, the condition should expect a bool or a list of bools - if we changed the check he's conducting to invoke this: https://opendev.org/openstack/manila/src/commit/4249e94c6bb7f5efb1a2095c925398277046936d/manila/scheduler/utils.py#L121-L127 we'd be good | 15:50 |
| gouthamr | tbarron: cool, i'll ask on the patch - it'd be good to backport it as one patch or together | 15:50 |
| tbarron | gouthamr: i anticipate that he'll jump right on it | 15:50 |
| tbarron | if for some reason not, then we can do it | 15:51 |
| gouthamr | tbarron: ack | 15:51 |
| *** maaritamm has joined #openstack-manila | 16:06 | |
| *** maaritamm has quit IRC | 16:07 | |
| *** maaritamm has joined #openstack-manila | 16:12 | |
| *** maaritamm has quit IRC | 16:15 | |
| *** maaritamm has joined #openstack-manila | 16:15 | |
| *** maaritamm has quit IRC | 16:19 | |
| *** maaritamm has joined #openstack-manila | 16:19 | |
| *** maaritamm has quit IRC | 16:23 | |
| *** maaritamm has joined #openstack-manila | 16:23 | |
| *** maaritamm has quit IRC | 16:26 | |
| *** sfernand has quit IRC | 16:29 | |
| *** cvm_ has joined #openstack-manila | 17:06 | |
| *** josecastroleon has joined #openstack-manila | 17:21 | |
| josecastroleon | hi | 17:21 |
| *** josecastroleon has quit IRC | 17:25 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila master: Revert "Remove provisioned calculation on non thin provision backends" https://review.opendev.org/716683 | 17:33 |
| cvm_ | Hi all, I just installed Manila, but I get an error while creating a share. In /var/log/manila/share.log I see the following: ERROR oslo_messaging.rpc.server BadRequest: The requested availability zone is not available (HTTP 400) | 17:34 |
| cvm_ | I this a nova or a storage message? | 17:34 |
| gouthamr | Hi cvm_: depends - are you specifying an AZ during share creation? | 17:35 |
| gouthamr | cvm_: if the message is in /var/log/manila/share.log, it appears that you may have a config issue ... | 17:35 |
| cvm_ | Hi gouthamr, I don't specify an AZ during share creation. An AZ list returns only the nova share. I don't have a nova AZ on compute, renamed it to other AZ's. | 17:38 |
| gouthamr | cvm_: okay, that may be your problem.. you seem to be using the generic share backend driver - this uses cinder to provide the backing storage and nova to create a NAS server | 17:39 |
| gouthamr | cvm_: so, this backend's AZ should match the AZ you have on cinder and nova | 17:39 |
| cvm_ | Ah, on Cinder I have nova, but not on Nova... Indeed I'm using the generic share backend driver. Is it possible to define a different AZ? And must it be the same for both cinder and nova? | 17:41 |
| cvm_ | gouthamr: I expected manila to use the nova scheduler for creating the manila instance, but this is not the case? | 17:46 |
| gouthamr | cvm_: unfortunately no, it will seek the specific AZ | 17:46 |
| gouthamr | cvm_: do you have the config magic to allow cross az attaches in cinder | 17:47 |
| gouthamr | cvm_: the option seems to be in nova.conf's [cinder] section - i ask because that's something we don't support today | 17:49 |
| gouthamr | cvm_: manila will request a nova instance in the specific AZ manila knows about... (default "nova") | 17:50 |
| cvm_ | gouthamr: I'm still running Rocky, I guess I'm not able to use that | 17:50 |
| cvm_ | gouthamr: I just enable a node in the nova AZ, so that's available (for testing). Now I have the following: ERROR oslo_messaging.rpc.server ServiceInstanceException: Instance <ID> failed to reach active state in 300 seconds. Current status: ERROR. | 17:51 |
| cvm_ | gouthamr: It took by far less than 300 seconds... | 17:52 |
| gouthamr | cvm_: oh that is weird, the timer is set to loop/wait for a timeout, which is 300 seconds by default: https://opendev.org/openstack/manila/src/commit/198774292d8f53805c79fed05504a5d9f92eae26/manila/share/drivers/service_instance.py#L681-L704 | 17:55 |
| gouthamr | cvm_: do you see any other debug logging? | 17:56 |
| gouthamr | cvm_: if you're trying this on a test env, i'd suggest cleaning up any shares and share servers in "error" state and retrying to see if this happens again | 17:57 |
| cvm_ | gouthamr: yes, when I show the instance it mentions "No valid host was found.". I cannot find it in any log file (manila/nova) | 17:58 |
| cvm_ | gouthamr: I have the same when spawning a regular instance in Nova. I'll figure that out first. Thank you very much for your help! | 18:02 |
| gouthamr | cvm_: ack, sure thing | 18:02 |
| gouthamr | cvm_: just out of curiosity, are you trying to use manila's generic driver in your production cloud? | 18:04 |
| cvm_ | gouthamr: yes indeed, share_driver = manila.share.drivers.generic.GenericShareDriver | 18:07 |
| openstackgerrit | Vishal Manchanda proposed openstack/manila-ui master: Fix pyScss version in lower-constraints.txt https://review.opendev.org/716716 | 18:21 |
| *** maaritamm has joined #openstack-manila | 18:42 | |
| *** cvm_ has quit IRC | 19:22 | |
| *** maaritamm has quit IRC | 19:26 | |
| *** maaritamm has joined #openstack-manila | 19:27 | |
| openstackgerrit | Victoria Martinez de la Cruz proposed openstack/manila-ui master: [DNM] Testing Manila UI gates https://review.opendev.org/716585 | 20:30 |
| dviroel | hey gouthamr | 20:42 |
| gouthamr | hey dviroel | 20:42 |
| dviroel | gouthamr: I'm trying to skip the new create_from_snapshot filter and host_affinity weigher when creating share-replicas | 20:43 |
| dviroel | gouthamr: a test that is failing on zuul show me that | 20:44 |
| dviroel | since the active-replica is a share created from snapshot | 20:44 |
| dviroel | was looking for a request-spec attr do check for | 20:44 |
| dviroel | maybe "request_spec['active_replica_host']", any tips? | 20:46 |
| gouthamr | dviroel: hmm, that might work | 20:48 |
| *** jv has quit IRC | 20:51 | |
| gouthamr | dviroel: currently, aren't you checking for share_ref['snapshot_id']? | 20:52 |
| *** maaritamm has quit IRC | 20:52 | |
| dviroel | gouthamr: yes, only for snapshot_id | 20:53 |
| dviroel | gouthamr: creating a replica from a share created from snapshot is also giving me a snapshot_id | 20:54 |
| gouthamr | dviroel: you mean creating a replicated share.. | 20:54 |
| gouthamr | lemme think.. | 20:54 |
| gouthamr | dviroel: why is a replicated share different though? | 20:56 |
| dviroel | gouthamr: this test: manila_tempest_tests.tests.api.test_replication_snapshots.ReplicationSnapshotTest.test_create_replica_from_snapshot_share | 20:57 |
| gouthamr | dviroel: i mean, it still could be scheduled across backends? | 20:57 |
| gouthamr | dviroel: cool, and what fails? | 20:58 |
| dviroel | gouthamr: yes. its a replica creation, but its being filtered and weighted by create_from_snapshot filter and host_affinity weigher | 20:59 |
| dviroel | gouthamr: because it has a parent snapshot-id | 21:00 |
| dviroel | gouthamr: because the active replica was created from a snapshot | 21:00 |
| gouthamr | dviroel: so the replica of a cloned share fails to be scheduled? | 21:00 |
| dviroel | its failing because now the 'snapshot_host' is missing | 21:01 |
| dviroel | *because the 'snapshot_host' is missing | 21:01 |
| dviroel | instead of just checking this attr, I was thinking that this filter/weigher should be skipped for replicas. | 21:02 |
| dviroel | because its not their porpouse | 21:02 |
| dviroel | what do you think? | 21:02 |
| gouthamr | dviroel: step back for a sec with me, i'm a bit slow | 21:02 |
| gouthamr | dviroel: if there is no "snapshot_host", doesn't it mean that the source is not a snapshot | 21:03 |
| gouthamr | dviroel: https://review.opendev.org/#/c/709697/7/manila/scheduler/filters/create_from_snapshot.py@55 - if there's no snapshot_host, you should return True | 21:04 |
| *** jv has joined #openstack-manila | 21:05 | |
| dviroel | gouthamr: yes, there is no snapshot_host because we are not adding it when creating share-replicas. | 21:06 |
| gouthamr | dviroel: yes, so that filter must bail out and allow the host | 21:07 |
| dviroel | gouthamr: also works fine. I was considering make sure that is a replica, before return True | 21:07 |
| dviroel | gouthamr: yeah, was trying to complicate this | 21:08 |
| dviroel | gouthamr: ty | 21:08 |
| gouthamr | dviroel: you're welcome | 21:09 |
| *** jv has quit IRC | 21:30 | |
| *** jv has joined #openstack-manila | 21:30 | |
| dviroel | is there a way of limiting how many cores tox uses for running unit tests? | 21:44 |
| gouthamr | dviroel: you mean stestr? https://stestr.readthedocs.io/en/latest/MANUAL.html#cmdoption-stestr-run-concurrency | 22:03 |
| openstackgerrit | Merged openstack/manila master: Update hacking for Python3 https://review.opendev.org/716499 | 22:09 |
| *** carthaca has quit IRC | 22:11 | |
| openstackgerrit | Merged openstack/manila master: Hacking: Fix F632 https://review.opendev.org/716508 | 22:19 |
| openstackgerrit | Merged openstack/manila master: Hacking: Fix F841 https://review.opendev.org/716510 | 22:32 |
| openstackgerrit | Merged openstack/manila master: Hacking: Fix F601 https://review.opendev.org/716511 | 22:32 |
| openstackgerrit | Merged openstack/manila master: Hacking: Fix E241 https://review.opendev.org/716512 | 22:32 |
| dviroel | gouthamr: yeah, tks | 22:35 |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Create share from snapshot in another pool or backend https://review.opendev.org/709697 | 22:52 |
| *** tosky has quit IRC | 23:03 | |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: [NetApp] Improve create share from snapshot functionality https://review.opendev.org/712642 | 23:22 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!