| *** cknight has quit IRC | 00:03 | |
| *** yangyapeng has quit IRC | 00:11 | |
| openstackgerrit | Merged openstack/manila: Fix for incorrect LVMMixin exception message https://review.openstack.org/294481 | 00:12 |
|---|---|---|
| *** cknight has joined #openstack-manila | 00:14 | |
| *** ChanServ changes topic to "4 bugs left until RC1" | 00:16 | |
| bswartz | ameade gouthamr: ganso says he found a failure in https://review.openstack.org/#/c/292659/10 | 00:16 |
| *** tpsilva has quit IRC | 00:17 | |
| bswartz | Indeed there were several failures in the LVM job but I didn't see the error he referred to | 00:17 |
| bswartz | ganso: if you're still around can you point to where you saw that message? | 00:17 |
| ameade | its in m-shr logs, looks unrelated to me. Do is this a flakey test? | 00:18 |
| bswartz | ameade: LVM has a pretty solid record | 00:19 |
| bswartz | I haven't seen it fail in a long time | 00:19 |
| bswartz | is there a backtrace that points to where it happened? can we rule out the snapshot instances patch? | 00:19 |
| bswartz | ameade: every patch that's a child of snapshot instances fails LVM | 00:22 |
| ameade | lookin | 00:22 |
| ameade | yeah same error it seems | 00:23 |
| bswartz | if we merge now we'll have to fix that in RC2 | 00:23 |
| bswartz | and we'd rather avoid RC2 | 00:23 |
| bswartz | so let's spend a least a little bit hunting down the root cause | 00:24 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 00:24 |
| *** cknight has quit IRC | 00:25 | |
| *** gouthamr_ is now known as gouthamr | 00:25 | |
| bswartz | gouthamr_ gouthamr: ^ | 00:26 |
| gouthamr | bswartz: yep looking, nick issues with my ill mannered drop offs | 00:26 |
| ameade | bswartz: figured it out | 00:27 |
| bswartz | I wondered how people get names like nick_ nick__ and nick___ | 00:27 |
| ameade | it's def due to instances | 00:27 |
| bswartz | easy to fix? | 00:27 |
| ameade | yeah i think so | 00:27 |
| * bswartz hopes.... | 00:27 | |
| ameade | incoming | 00:27 |
| gouthamr | ameade: the lvm driver was expecting something from the snapshot model? | 00:28 |
| gouthamr | ameade: ugh | 00:28 |
| ameade | we added the share instance details ot the snapshot | 00:28 |
| gouthamr | share size | 00:28 |
| ameade | but didnt says with_share_data | 00:28 |
| gouthamr | i didn't refactor the create shapshot method, | 00:29 |
| bswartz | could other drivers be affected similarly? we've seen some CI failures but ignored most of them due to normal CI instability | 00:29 |
| gouthamr | snapshot_instance = self.db.share_snapshot_instance_get( | 00:29 |
| gouthamr | context, snapshot_ref.instance['id'], with_share_data=True | 00:29 |
| gouthamr | ) | 00:29 |
| gouthamr | omg | 00:30 |
| gouthamr | :P | 00:30 |
| gouthamr | i know what happened there | 00:30 |
| ameade | yeh | 00:31 |
| bswartz | what happened? | 00:31 |
| gouthamr | i wonder how that test ever passed | 00:31 |
| gouthamr | (5th stage of denial) | 00:32 |
| *** abhi has quit IRC | 00:36 | |
| gouthamr | bswartz: other drivers seem to use snapshot['size'], the lvm driver is using snapshot['share']['size'] <-- where share's the share instance object and we weren't setting the share model data in it. | 00:37 |
| openstackgerrit | Alex Meade proposed openstack/manila: NetApp cDOT: Handle replicated snapshots https://review.openstack.org/293243 | 00:37 |
| openstackgerrit | Alex Meade proposed openstack/manila: Data Replication: Ensure Snapshots across replicas https://review.openstack.org/292659 | 00:37 |
| gouthamr | thanks ameade | 00:37 |
| ameade | trying to run unittests atm to make sure thats good | 00:38 |
| ameade | looks like our ci 7mode fell over | 00:38 |
| ameade | lol | 00:38 |
| ameade | fixing that too | 00:38 |
| gouthamr | yah that's how we roll on a friday night 8) | 00:38 |
| *** gouthamr is now known as gouthamr_away_fo | 00:44 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 00:45 |
| *** eharney has quit IRC | 00:48 | |
| openstackgerrit | Merged openstack/python-manilaclient: Revert fix access rules functional tests https://review.openstack.org/293392 | 01:03 |
| *** esker has quit IRC | 01:08 | |
| *** gouthamr_away_fo is now known as gouthamr | 01:15 | |
| gouthamr | bswartz: fyi, ZFS tests are passing on the latest replicated snapshot tests -> https://jenkins01.openstack.org/job/gate-manila-tempest-dsvm-postgres-zfsonlinux/25/console | 01:24 |
| *** tpsilva has joined #openstack-manila | 01:26 | |
| *** ganso has quit IRC | 01:29 | |
| *** yangyapeng has joined #openstack-manila | 01:46 | |
| *** yangyapeng has quit IRC | 01:51 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 01:53 |
| gouthamr | bswartz: ping | 01:54 |
| ameade | whoa didnt realize how late it was | 01:54 |
| gouthamr | yeah its pretty late | 01:55 |
| ameade | i had fallen asleep on the couch earlier | 01:55 |
| gouthamr | i'm still hopeful of making it to RC1 though :P | 01:55 |
| *** akshai has joined #openstack-manila | 02:00 | |
| bswartz | gouthamr: pong | 02:18 |
| gouthamr | bswartz: hi! we're all good on all three patches | 02:18 |
| gouthamr | 4* | 02:18 |
| bswartz | awesome | 02:19 |
| bswartz | who's awake to do the merge? | 02:19 |
| bswartz | xyang1? markstur? | 02:19 |
| bswartz | does zfs require a rebase? | 02:21 |
| bswartz | gouthamr: https://review.openstack.org/#/c/292659 failed LVM -- seemingly due to timeout | 02:22 |
| gouthamr | bswartz: i just posted the log on the latest recheck | 02:23 |
| gouthamr | https://jenkins01.openstack.org/job/gate-manila-tempest-minimal-dsvm-lvm/30/console | 02:23 |
| gouthamr | :) | 02:23 |
| bswartz | oh you already rechecked | 02:23 |
| bswartz | I didn't see that | 02:23 |
| bswartz | yes I see it | 02:23 |
| bswartz | good | 02:23 |
| bswartz | gouthamr: where is the change that shortens the ZFS update interval? | 02:26 |
| gouthamr | https://review.openstack.org/#/c/293113 | 02:26 |
| gouthamr | bswartz ^ | 02:26 |
| bswartz | gouthamr: vponoparyov is going to -1 that when he sees it, lol | 02:27 |
| bswartz | I know he won't like it | 02:27 |
| gouthamr | bswartz: why? | 02:27 |
| bswartz | but I can't think of a better way | 02:27 |
| gouthamr | bswartz: 60 seconds is perfect for tempest tests that create 1 gig shares :) | 02:28 |
| bswartz | yeah -- but this change will do it for everyone who installs devstack | 02:28 |
| bswartz | why did you use DEFAULT and not $BE? | 02:28 |
| gouthamr | bswartz: could it be done in the backend section? | 02:29 |
| bswartz | I don't know | 02:29 |
| bswartz | but if it should be in DEFAULT then it should be outside the loop | 02:29 |
| bswartz | the loop iterates once per backend | 02:29 |
| gouthamr | i am expecting the option in the default group .. | 02:29 |
| gouthamr | ohh | 02:29 |
| gouthamr | sure. will fix that | 02:29 |
| bswartz | and can we wrap it with a "is tempest enabled" or something | 02:30 |
| bswartz | or wait a minute.... | 02:30 |
| bswartz | can we set it to $MANILA_ZFS_REPLICA_STATE_UPDATE_INTERVAL and then set that value in the pre test hook? | 02:30 |
| bswartz | that would ensure that it's only set to 60 when run inside of devstack gate | 02:31 |
| bswartz | ordinary devstack users would get the ordinary value | 02:31 |
| gouthamr | bswartz: yes that seems better than overriding it for unsuspecting users that use devstack/zfs | 02:32 |
| gouthamr | will do | 02:32 |
| *** baojg has joined #openstack-manila | 02:42 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 02:59 |
| bswartz | gouthamr: typo on that patch | 02:59 |
| bswartz | missed a $ in plugin.sh | 03:00 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 03:00 |
| gouthamr | :) | 03:00 |
| gouthamr | thanks fixed! | 03:00 |
| bswartz | also | 03:01 |
| bswartz | misspelled the option name in settings | 03:01 |
| bswartz | you must be tired man | 03:01 |
| bswartz | working so hard | 03:01 |
| gouthamr | settings as well? | 03:02 |
| gouthamr | argh :P | 03:02 |
| bswartz | in pre_test_hook.sh you might want to include a comment explaining why 60 | 03:03 |
| bswartz | that's all I can see | 03:04 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 03:04 |
| gouthamr | bswartz: it was a change of mind about the name | 03:04 |
| bswartz | okay this looks good | 03:04 |
| bswartz | well you're right we could use a non-ZFS specific option and set it outside the ZFS-specific code in plugin.sh | 03:05 |
| gouthamr | bswartz: =) wanted to see if i can make it generic if our CI wants to reuse MANILA_REPLICA_STATE_UPDATE_INTERVAL in their hooks.. | 03:05 |
| bswartz | yes we could do that | 03:05 |
| bswartz | why did you decide against that? | 03:05 |
| bswartz | it seems better | 03:05 |
| gouthamr | okay, will revert to that then. | 03:06 |
| gouthamr | akerr may thank me | 03:06 |
| bswartz | when you do it include a comment where you set it to 60 explaining why | 03:06 |
| gouthamr | sure will do | 03:07 |
| *** cknight has joined #openstack-manila | 03:08 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 03:13 |
| bswartz | doh! | 03:14 |
| bswartz | forgot $ again | 03:14 |
| gouthamr | o.O | 03:15 |
| bswartz | devstack/plugin.sh | 03:15 |
| bswartz | I dont' see any other issues | 03:15 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 03:15 |
| gouthamr | sorry about that | 03:15 |
| gouthamr | :) fixed | 03:16 |
| bswartz | okay | 03:16 |
| bswartz | finally this looks good | 03:16 |
| bswartz | thanks for staying up | 03:16 |
| gouthamr | np! :) | 03:16 |
| bswartz | hopefully vponomaryov sees this tomorrow morning and workflows | 03:16 |
| gouthamr | are these the final patches? | 03:17 |
| bswartz | as far as I'm concerned | 03:18 |
| bswartz | but we have 2 cores to merge stuff for a reason | 03:18 |
| bswartz | ganso caught that LVM issue which was lucky | 03:18 |
| bswartz | vponomaryov might catch something else | 03:18 |
| gouthamr | yes.. | 03:18 |
| bswartz | nothing to do now but sleep.... | 03:19 |
| bswartz | get some rest man | 03:19 |
| gouthamr | i was so sure i didn't change the existing snapshot behavior. | 03:19 |
| bswartz | thanks for all your hard work and I hope you enjoy your weekend | 03:19 |
| gouthamr | and there had to be one driver that was doing snapshot['share']['size'] :D | 03:19 |
| gouthamr | hey np! | 03:20 |
| gouthamr | i'll drop off now. thanks for helping me fix the test patch. I learnt the hook stuff new today! | 03:20 |
| *** gouthamr has quit IRC | 03:30 | |
| *** tpsilva has quit IRC | 03:37 | |
| *** cknight1 has joined #openstack-manila | 04:03 | |
| *** cknight has quit IRC | 04:05 | |
| *** baojg has quit IRC | 04:39 | |
| *** yangyapeng has joined #openstack-manila | 04:50 | |
| *** yangyapeng has quit IRC | 04:55 | |
| *** baojg has joined #openstack-manila | 04:59 | |
| *** lpetrut has joined #openstack-manila | 06:18 | |
| *** david-lyle has quit IRC | 06:22 | |
| *** david-lyle has joined #openstack-manila | 06:23 | |
| *** ociuhandu has quit IRC | 06:50 | |
| *** ociuhandu has joined #openstack-manila | 07:13 | |
| *** lpetrut has quit IRC | 07:18 | |
| *** lpetrut has joined #openstack-manila | 07:27 | |
| *** lpetrut has quit IRC | 07:31 | |
| *** yangyapeng has joined #openstack-manila | 08:27 | |
| *** baojg has quit IRC | 08:27 | |
| *** baojg has joined #openstack-manila | 08:29 | |
| *** baojg has quit IRC | 08:31 | |
| *** yangyapeng has quit IRC | 08:31 | |
| *** baojg has joined #openstack-manila | 08:42 | |
| openstackgerrit | Merged openstack/manila: Data Replication: Ensure Snapshots across replicas https://review.openstack.org/292659 | 09:15 |
| *** cfey has quit IRC | 10:58 | |
| *** cfey has joined #openstack-manila | 11:01 | |
| *** cfey has quit IRC | 11:10 | |
| *** cfey has joined #openstack-manila | 11:15 | |
| *** cfey has quit IRC | 11:20 | |
| *** cfey has joined #openstack-manila | 11:23 | |
| *** cfey has quit IRC | 11:25 | |
| *** cfey has joined #openstack-manila | 11:28 | |
| *** cfey has quit IRC | 11:31 | |
| *** cfey has joined #openstack-manila | 11:32 | |
| *** ociuhandu has quit IRC | 11:34 | |
| *** cfey has quit IRC | 11:34 | |
| openstackgerrit | Merged openstack/manila: NetApp cDOT: Handle replicated snapshots https://review.openstack.org/293243 | 11:35 |
| *** cfey has joined #openstack-manila | 11:37 | |
| *** cfey has quit IRC | 11:50 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 11:53 |
| *** cfey has joined #openstack-manila | 11:53 | |
| *** lpetrut has joined #openstack-manila | 11:54 | |
| *** cfey has quit IRC | 11:59 | |
| *** cfey has joined #openstack-manila | 12:02 | |
| *** ganso has joined #openstack-manila | 12:11 | |
| ganso | bswartz, vponomaryov: ping | 12:12 |
| *** lpetrut has quit IRC | 12:14 | |
| bswartz | ganso: thank goodness you're here | 12:27 |
| ganso | bswartz: good morning | 12:27 |
| ganso | bswartz: what's up? | 12:28 |
| bswartz | https://review.openstack.org/#/c/293377/ needs workflow | 12:28 |
| ganso | bswartz: I suspect RC1 has not been cut, there are 2 changes still to merge | 12:28 |
| bswartz | vponomyov can't do that one | 12:28 |
| ganso | bswartz: aren't you waiting on https://review.openstack.org/#/c/293113/ ? | 12:28 |
| bswartz | everything was passing last night | 12:29 |
| bswartz | not sure what gouthamr is up to this morning | 12:29 |
| bswartz | looks like simple rebase | 12:30 |
| ganso | bswartz: no change is expected on 377? | 12:30 |
| bswartz | nope | 12:30 |
| ganso | bswartz: ok, will take a last look at it | 12:30 |
| bswartz | after these 2 merge I'll cut the RC and then we can start looking for quality issues worthy to an RC2 | 12:31 |
| ganso | bswartz: feel free to workflow it | 12:48 |
| ganso | bswartz: btw, generic-no-share-servers is failing consistently after something from yesterday has merged | 12:50 |
| ganso | bswartz: my patch was the last one that passed | 12:53 |
| bswartz | you think it's related? | 12:59 |
| bswartz | I noticed it but did not investigate | 13:00 |
| ganso | bswartz: https://review.openstack.org/#/c/290511/ | 13:01 |
| ganso | bswartz: that patch broke generic-no-share-servers CI | 13:01 |
| bswartz | ah | 13:01 |
| bswartz | any idea why? | 13:01 |
| ganso | bswartz: either devstack script or changes in nova client | 13:02 |
| bswartz | well it would be nice to fix that before we cut the RC | 13:03 |
| bswartz | if we can get to the bottom of it | 13:03 |
| ganso | bswartz: I am looking | 13:03 |
| bswartz | I won't have much time to spend on it today | 13:03 |
| bswartz | maybe tonight I can | 13:03 |
| *** yangyapeng has joined #openstack-manila | 13:03 | |
| bswartz | if you see what hte problem is go ahead and file the bug in LP so we can target it | 13:04 |
| *** yangyapeng has quit IRC | 13:08 | |
| ganso | bswartz: I found the problem, I don't know what is causing it | 13:12 |
| ganso | bswartz: instances are taking very long time to boot | 13:12 |
| ganso | bswartz: tempest is starting tests before servers are up | 13:12 |
| ganso | bswartz: this causes most of the tests to fail | 13:13 |
| ganso | bswartz: but then later everything runs fine | 13:13 |
| bswartz | is there a mechanism to make tempest wait for the VM to boot? | 13:13 |
| bswartz | I would have assumed a wait loop already existed | 13:13 |
| ganso | bswartz: I assumed as well | 13:14 |
| ganso | I don't understand why that patch broke it | 13:14 |
| ganso | bswartz: https://bugs.launchpad.net/manila/+bug/1559456 | 13:18 |
| openstack | Launchpad bug 1559456 in Manila "tests run before instance is ready in generic-no-share-servers CI " [Undecided,New] | 13:18 |
| bswartz | I don't want to target it unless we know what exactly needs to be fixed | 13:18 |
| ganso | bswartz: I am not familiar with how it worked before, if there was a wait loop | 13:19 |
| bswartz | let me take a quick look | 13:19 |
| ganso | bswartz: vponomaryov probably knows this | 13:19 |
| bswartz | but then I have to run | 13:19 |
| bswartz | https://github.com/openstack/manila/blob/master/contrib/ci/common.sh#L35 | 13:21 |
| bswartz | this? | 13:21 |
| ganso | bswartz: nice | 13:22 |
| bswartz | looks like it waits for "ping" to succeed | 13:23 |
| bswartz | maybe it needs to wait for SSH to also be up | 13:23 |
| ganso | bswartz: yes | 13:23 |
| bswartz | or for some additional services | 13:23 |
| ganso | bswartz: on a succeeded run, it took 2 minutes to be SSH'able | 13:23 |
| ganso | bswartz: on the failed runs, it took 5 minutes | 13:23 |
| bswartz | until socat /dev/null TCP:$HOST:22,connect-timeout=1 2> /dev/null ; do sleep 1 ; done | 13:24 |
| bswartz | that's how my scripts wait for SSH to be up | 13:24 |
| *** yangyapeng has joined #openstack-manila | 13:24 | |
| bswartz | let me throw up a fix for this | 13:25 |
| ganso | bswartz: replace a ping by that | 13:25 |
| bswartz | k 1 sec | 13:25 |
| ganso | bswartz: that could work | 13:25 |
| *** yangyapeng has quit IRC | 13:26 | |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: In CI, wait for SSH instead of ping https://review.openstack.org/294910 | 13:29 |
| *** ChanServ changes topic to "2 bugs left until RC1" | 13:30 | |
| bswartz | Okay I'll bbl | 13:30 |
| ganso | bswartz: | 13:30 |
| ganso | bswartz: ok | 13:30 |
| *** lpetrut has joined #openstack-manila | 13:38 | |
| openstackgerrit | Merged openstack/manila: Make ZFSonLinux driver handle snapshots of replicated shares properly https://review.openstack.org/293377 | 13:54 |
| *** ChanServ changes topic to "1 bug left until RC1" | 13:58 | |
| *** lpetrut has quit IRC | 14:00 | |
| openstackgerrit | Merged openstack/manila: Add tests to ensure snapshots across replicas https://review.openstack.org/293113 | 14:01 |
| *** lpetrut has joined #openstack-manila | 14:04 | |
| bswartz | ganso: I see some fails early on https://jenkins01.openstack.org/job/gate-manila-tempest-dsvm-generic-no-share-servers/33/console | 14:08 |
| ganso | bswartz: I see them too | 14:08 |
| ganso | bswartz: I cannot see anything other than console... waiting for test to finish | 14:08 |
| bswartz | is this how the other failure looked? | 14:08 |
| bswartz | or is this better | 14:08 |
| ganso | bswartz: looks exactly the same | 14:08 |
| bswartz | crud | 14:08 |
| bswartz | so maybe this sleep isn't big enough? https://github.com/openstack/manila/blob/master/contrib/ci/common.sh#L75 | 14:09 |
| ganso | bswartz: I don't know, if in m-shr I can see that it is still trying to SSH and instance is not ready, then your fix probably did not work | 14:09 |
| bswartz | ok so we have to wait for log upload :-/ | 14:10 |
| ganso | bswartz: yup | 14:10 |
| *** lpetrut has quit IRC | 14:11 | |
| ganso | bswartz: your fix probably did not work, I will test on my lab | 14:17 |
| bswartz | well it's what I use in my testing to wait for servers to be up and it's rock solid | 14:19 |
| bswartz | so either these VMs are missing the socat binary, or it's not SSH that's the problem | 14:20 |
| *** vponomaryov1 has joined #openstack-manila | 14:20 | |
| vponomaryov1 | bswartz, ganso: any logs of finished job is available right away | 14:21 |
| vponomaryov1 | http://logs.openstack.org/10/294910/1/check/gate-manila-tempest-dsvm-generic-no-share-servers/fc87b69/ | 14:21 |
| ganso | vponomaryov1: yea, I have look over them | 14:21 |
| bswartz | http://paste.openstack.org/show/491183/ | 14:21 |
| bswartz | ^ This looks suspicious | 14:21 |
| bswartz | It appears my fix never even got invoked because something went wrong beforehand | 14:22 |
| vponomaryov1 | oh, problem is simple )) | 14:23 |
| vponomaryov1 | problems appeared after switch to v3 | 14:23 |
| vponomaryov1 | and v2 endpoint not installed | 14:23 |
| vponomaryov1 | so, all client calls in post_test_hook fails now | 14:24 |
| vponomaryov1 | we need just to impropert env vars properly now in post_test_hook | 14:24 |
| vponomaryov1 | s/impropert/import/ | 14:24 |
| vponomaryov1 | import + proper = impropert )) | 14:24 |
| ganso | vponomaryov1: are you able to upload the necessary fix? | 14:25 |
| vponomaryov1 | need to find that "impropert" )) | 14:26 |
| ganso | vponomaryov1: lol | 14:26 |
| bswartz | vponomaryov1: do you want to push a patch? | 14:28 |
| bswartz | then ganso and I can merge it and we can be done with RC1 | 14:28 |
| bswartz | indeed ganso noted that it started failing after keystonev3 merged | 14:29 |
| vponomaryov1 | first guess, admin user stopped being related to project we use | 14:30 |
| vponomaryov1 | or domain | 14:30 |
| bswartz | vponomaryov1: if you have a fix, use this bug: https://bugs.launchpad.net/manila/+bug/1559456 | 14:31 |
| openstack | Launchpad bug 1559456 in Manila "tests run before instance is ready in generic-no-share-servers CI " [High,In progress] - Assigned to Ben Swartzlander (bswartz) | 14:31 |
| vponomaryov1 | I saw | 14:31 |
| bswartz | I'm leaving here in a moment but I'll bring a laptop so I can check in after lunch | 14:31 |
| bswartz | cknight1 maybe be around this morning too to help merge something | 14:33 |
| cknight1 | bswartz: I'm here for a bit | 14:34 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Fix call of clients in post_test_hook.sh https://review.openstack.org/294918 | 14:46 |
| ganso | vponomaryov1: are you saying we need a fix for manilaclient as well? | 14:50 |
| bswartz | vponomaryov1: you sure those exports won't overwrite anything important? | 14:50 |
| bswartz | I might have used a subshell for the wait for drivers init | 14:50 |
| bswartz | so the env of the devstack gate shell doesn't get altered | 14:51 |
| vponomaryov1 | ganso: vice versa | 14:51 |
| vponomaryov1 | ganso: manilaclient smarter than novaclient | 14:51 |
| vponomaryov1 | ganso: )) | 14:52 |
| ganso | vponomaryov1: great :D | 14:52 |
| bswartz | well I'm going to optimistically +2 this before I leave | 14:52 |
| ganso | I am watching the job | 14:53 |
| vponomaryov1 | bswartz: debug print of opts rules | 14:53 |
| ganso | bswartz: I can workflow it later if it passes | 14:53 |
| bswartz | if anyone sees an EMC person, please let them know the ISILON CI is hosed badly | 14:53 |
| vponomaryov1 | PS2 will be uploaded, mistake sneaked in | 14:56 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Fix call of clients in post_test_hook.sh https://review.openstack.org/294918 | 14:58 |
| bswartz | Okay I'll be AFK for a few hours | 15:01 |
| bswartz | I'll try to checkin from laptop after lunch | 15:02 |
| vponomaryov1 | fix works - https://jenkins04.openstack.org/job/gate-manila-tempest-dsvm-generic-no-share-servers/27/console | 15:33 |
| ganso | vponomaryov1: yup! :D | 15:42 |
| ganso | vponomaryov1: waiting for job to be finished | 15:42 |
| vponomaryov1 | ganso: fix is proven, no need to wait | 15:45 |
| ganso | vponomaryov1: yes, but job must finish before going to the gate | 15:46 |
| vponomaryov1 | and? )) | 15:46 |
| ganso | vponomaryov1: I really don't like workflowing before jenkins | 15:47 |
| vponomaryov1 | ganso: if you workflow it now, you can driver beer already, if not you should keep an eye on job and be sober | 15:48 |
| vponomaryov1 | s/driver/drink/ | 15:48 |
| ganso | vponomaryov1: I got a flu... I can't drink :( | 15:48 |
| ganso | vponomaryov1: not today... | 15:49 |
| vponomaryov1 | ganso: ok, then you can get some rest earlier )) | 15:49 |
| vponomaryov1 | ganso: and I could recheck myself | 15:49 |
| vponomaryov1 | ganso: )) | 15:49 |
| ganso | vponomaryov1: ok ok you win :P | 15:50 |
| ganso | vponomaryov1: I am already "resting". Can't really enjoy my saturday :( | 15:51 |
| vponomaryov1 | ganso: but you have good reason to be lazy whole day! | 15:52 |
| vponomaryov1 | ganso: to watch some cool TV series | 15:52 |
| vponomaryov1 | ganso: for which you haven't time for months | 15:54 |
| ganso | vponomaryov1: not as fun as drinking though :\ | 15:54 |
| ganso | vponomaryov1: going to need a recheck | 16:04 |
| *** pcaruana has quit IRC | 16:15 | |
| *** pcaruana has joined #openstack-manila | 16:28 | |
| *** ociuhandu has joined #openstack-manila | 17:07 | |
| *** gouthamr has joined #openstack-manila | 17:35 | |
| *** gouthamr has quit IRC | 17:38 | |
| openstackgerrit | Merged openstack/manila: Fix call of clients in post_test_hook.sh https://review.openstack.org/294918 | 18:14 |
| *** pcaruana has quit IRC | 18:18 | |
| ganso | bswartz: ^ | 18:23 |
| *** pcaruana has joined #openstack-manila | 18:32 | |
| *** amit213 has quit IRC | 19:16 | |
| *** amit213 has joined #openstack-manila | 19:17 | |
| *** gouthamr has joined #openstack-manila | 19:33 | |
| *** gouthamr has quit IRC | 19:39 | |
| *** gouthamr has joined #openstack-manila | 19:39 | |
| *** esker has joined #openstack-manila | 19:59 | |
| *** jasonsb has quit IRC | 20:07 | |
| *** ganso has quit IRC | 20:29 | |
| *** vponomaryov1 has left #openstack-manila | 20:50 | |
| *** jasonsb has joined #openstack-manila | 21:13 | |
| *** dalgaaf has joined #openstack-manila | 21:43 | |
| *** gouthamr has quit IRC | 21:59 | |
| -openstackstatus- NOTICE: Gerrit is restarting to increase performance issues | 22:24 | |
| *** cfey has quit IRC | 22:27 | |
| *** cfey has joined #openstack-manila | 22:30 | |
| *** cfey has quit IRC | 23:21 | |
| *** cfey has joined #openstack-manila | 23:23 | |
| *** ChanServ changes topic to "RC1 complete!" | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!