| *** thorst has joined #openstack-powervm | 00:16 | |
| *** openstackgerrit has quit IRC | 00:41 | |
| *** thorst has quit IRC | 00:54 | |
| *** thorst has joined #openstack-powervm | 01:53 | |
| *** thorst has quit IRC | 01:53 | |
| *** efried has quit IRC | 02:31 | |
| *** efried has joined #openstack-powervm | 02:32 | |
| *** thorst has joined #openstack-powervm | 02:53 | |
| *** thorst has quit IRC | 02:59 | |
| *** thorst has joined #openstack-powervm | 03:55 | |
| *** thorst has quit IRC | 03:59 | |
| *** thorst has joined #openstack-powervm | 04:55 | |
| *** thorst has quit IRC | 05:00 | |
| *** tjakobs has joined #openstack-powervm | 05:22 | |
| *** tjakobs has quit IRC | 05:47 | |
| *** thorst has joined #openstack-powervm | 05:56 | |
| *** thorst has quit IRC | 06:01 | |
| *** thorst has joined #openstack-powervm | 06:57 | |
| *** thorst has quit IRC | 07:02 | |
| *** ChanServ sets mode: +o openstack | 07:15 | |
| *** thorst has joined #openstack-powervm | 07:58 | |
| *** thorst has quit IRC | 08:02 | |
| *** thorst has joined #openstack-powervm | 08:59 | |
| *** thorst has quit IRC | 09:03 | |
| *** edmondsw has joined #openstack-powervm | 09:15 | |
| *** edmondsw has quit IRC | 09:20 | |
| *** thorst has joined #openstack-powervm | 10:00 | |
| *** thorst has quit IRC | 10:04 | |
| *** smatzek has joined #openstack-powervm | 10:40 | |
| *** smatzek has quit IRC | 10:41 | |
| *** smatzek has joined #openstack-powervm | 10:41 | |
| *** thorst has joined #openstack-powervm | 11:00 | |
| *** thorst has quit IRC | 11:05 | |
| *** thorst has joined #openstack-powervm | 11:59 | |
| *** edmondsw has joined #openstack-powervm | 12:13 | |
| *** kylek3h has quit IRC | 12:31 | |
| *** esberglu has joined #openstack-powervm | 13:01 | |
| esberglu | efried: edmondsw: We're gonna have to force one of these through. They are dependent on each other to pass jenkins tox | 13:07 |
|---|---|---|
| esberglu | https://review.openstack.org/#/c/505843/ | 13:07 |
| esberglu | https://review.openstack.org/#/c/504946/ | 13:07 |
| edmondsw | esberglu please remove the -W on the latter | 13:09 |
| esberglu | Also OOT CI is toast until those both get in | 13:09 |
| esberglu | edmondsw: Done | 13:09 |
| edmondsw | esberglu and we might not have to force anything (how would that work?) if we can get them to merge https://review.openstack.org/#/c/505831/ | 13:10 |
| esberglu | edmondsw: I was gonna ask you that. If we W+1 I think the jenkins -1 would still prevent the merge | 13:11 |
| esberglu | edmondsw: Do you know if the jenkins tox supports depends-on? | 13:11 |
| edmondsw | esberglu it does to my knowledge | 13:12 |
| edmondsw | talking about vnc in nova now | 13:12 |
| esberglu | edmondsw: The latter is passing PowerVM CI. So if we put the depends-on the former it might also pass jenkins and unblock us | 13:12 |
| *** kylek3h has joined #openstack-powervm | 13:15 | |
| *** openstackgerrit has joined #openstack-powervm | 13:16 | |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: Rename dac_admin to path https://review.openstack.org/504946 | 13:16 |
| edmondsw | esberglu I may have misunderstood your question there... I thought you were asking if nova's jenkins supported depends-on | 13:19 |
| edmondsw | I guess it's worth a shot | 13:19 |
| efried | Nova's does, for certain projects. What are we talking about here? | 13:21 |
| efried | You can't have a circular Depends-On - I know that. | 13:22 |
| edmondsw | efried the last link above, esberglu added depends-on in the hopes of getting that to pass so we can merge it and unblock CI | 13:22 |
| efried | We don't have Depends-On support in nova-powervm at all. | 13:22 |
| edmondsw | if that doesn't work, and I'm not so sure it will, we'll have to wait for nova's vnc conf fix to merge | 13:23 |
| edmondsw | efried yeah, that's what I thought | 13:23 |
| efried | So I take it they merged the privsep rename patch in nova? | 13:23 |
| edmondsw | yup | 13:23 |
| edmondsw | bad timing | 13:23 |
| edmondsw | efried we should try to hurry along the nova vnc fix to get us unblocked | 13:24 |
| esberglu | edmondsw: efried: I think the jenkins checks support depends-on for nova-powervm | 13:26 |
| esberglu | http://zuul.openstack.org/ | 13:26 |
| efried | If so, cool, but I thought we'd tried it recently and it didn't work. | 13:26 |
| esberglu | PowerVM CI doesn't | 13:26 |
| efried | Yeah, but there may be some configuration you need to do to make it work. | 13:26 |
| efried | oh, gotcha. | 13:26 |
| esberglu | You can see them linked together on the zuul status page anyways | 13:27 |
| efried | Okay, I know I've run up against the situation in nova where a Depends-On didn't work because the target was in a project that wasn't hooked in somehow. Not sure how those connections are controlled. | 13:27 |
| * edmondsw crosses fingers | 13:28 | |
| esberglu | IIRC depends-on <change in different project> doesn't work. One of the things that is supposed to come with zuul v3 | 13:28 |
| esberglu | Even if it passes jenkins It might block the merge since the patch it depends on won't have merged yet | 13:30 |
| esberglu | Not sure how that works | 13:30 |
| *** chhavi has joined #openstack-powervm | 13:31 | |
| efried | Depends-On definitely blocks merge until the dependency merges. | 13:36 |
| efried | Swhy you can't have circular. | 13:36 |
| efried | Depends-On <change in different project> definitely does work in nova for *some* values of <different project> | 13:36 |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Rename [vnc]vncserver_proxyclient_address https://review.openstack.org/505843 | 13:53 |
| efried | I rebased ^ onto the dac rename one. So ^ will pass, but the dep won't until the nova side merges. What a tangled web we weave. | 13:55 |
| *** tjakobs has joined #openstack-powervm | 14:24 | |
| *** tjakobs has quit IRC | 14:28 | |
| *** openstackgerrit has quit IRC | 14:33 | |
| edmondsw | esberglu efried please look at http://morpheus.pok.stglabs.ibm.com/#/c/5937 | 14:36 |
| edmondsw | drat... 5937 | 14:36 |
| efried | nova vnc opt snafu fix just merged. Who wants to post rechecks? | 16:33 |
| edmondsw | on it | 16:37 |
| edmondsw | efried actually, we need to wait for https://review.openstack.org/#/c/504946/ to merge as well | 16:37 |
| efried | Yeah, I was going to start rebasing everything on top of that, but... | 16:38 |
| edmondsw | efried actually, we either need to remove the depends-on there or get https://review.openstack.org/#/c/505843 merged | 16:40 |
| edmondsw | speaking of which... why no jenkins or ci results there? rebase was hours ago... | 16:40 |
| edmondsw | esberglu ^ | 16:40 |
| efried | possibly because the dep didn't verify | 16:41 |
| edmondsw | efried what dep? | 16:41 |
| efried | it's sitting on top of the dac_admin rename | 16:41 |
| edmondsw | efried ohhhh... circular dependency | 16:41 |
| edmondsw | nice | 16:42 |
| efried | yup | 16:42 |
| edmondsw | I'll remove the depends-on from https://review.openstack.org/#/c/504946/ | 16:42 |
| efried | k | 16:42 |
| *** openstackgerrit has joined #openstack-powervm | 16:42 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova-powervm master: Rename dac_admin to path https://review.openstack.org/504946 | 16:42 |
| openstackgerrit | Merged openstack/nova-powervm master: Rename dac_admin to path https://review.openstack.org/504946 | 17:10 |
| *** smatzek has quit IRC | 17:11 | |
| *** smatzek has joined #openstack-powervm | 17:17 | |
| esberglu | edmondsw: efried: Looks like you guys got everything figured out while I was at lunch. The PowerVM CI run is going now that the depends on is removed | 17:39 |
| efried | With the above merge, everything should be cleared out now. | 17:41 |
| efried | esberglu is CI running on https://review.openstack.org/#/c/505843/ ? | 17:42 |
| esberglu | efried: Yep | 17:42 |
| efried | coo | 17:43 |
| *** chhavi has quit IRC | 17:59 | |
| *** esberglu has quit IRC | 18:07 | |
| *** esberglu has joined #openstack-powervm | 18:08 | |
| *** esberglu has quit IRC | 18:12 | |
| -openstackstatus- NOTICE: Gerrit OpenIDs have been accidentally overwritten and are in the process of being restored | 18:18 | |
| *** esberglu has joined #openstack-powervm | 18:18 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly | 18:23 | |
| *** ChanServ changes topic to "The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly" | 18:23 | |
| edmondsw | is that max_virtual_slots bug we fixed in 1.1.8 only present in 1.1.7? I.e., 1.1.6 is fine, right? | 18:29 |
| edmondsw | efried ^ | 18:30 |
| efried | Correct. | 18:30 |
| efried | edmondsw ^ | 18:30 |
| edmondsw | efried remind me why we only bumped u-c to 1.1.8, not g-r? | 18:33 |
| efried | edmondsw Because we don't care about the max_virtual_slots thing in the community case. | 18:33 |
| efried | according to thorst ^^ | 18:33 |
| thorst | correct, it always goes to 64 I think | 18:34 |
| *** smatzek has quit IRC | 18:37 | |
| *** ChanServ changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 19:21 | |
| -openstackstatus- NOTICE: OpenIDs for the Gerrit service have been restored from a recent backup and the service is running again; before/after table states are being analyzed now to identify any remaining cleanup needed for changes made to accounts today | 19:21 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Add context to attach/detach_interface https://review.openstack.org/505374 | 19:33 |
| openstackgerrit | Merged openstack/nova-powervm master: Add context to attach/detach_interface https://review.openstack.org/505374 | 19:54 |
| *** thorst has quit IRC | 19:58 | |
| *** thorst has joined #openstack-powervm | 20:03 | |
| *** thorst has quit IRC | 20:07 | |
| *** tjakobs has joined #openstack-powervm | 20:21 | |
| openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm master: [WIP] ISCSI volume attachment fixes https://review.openstack.org/467599 | 20:35 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: ci check https://review.openstack.org/328315 | 20:36 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: CI Check2 https://review.openstack.org/328317 | 20:36 |
| *** thorst has joined #openstack-powervm | 20:40 | |
| *** edmondsw has quit IRC | 20:51 | |
| *** edmondsw has joined #openstack-powervm | 20:57 | |
| *** edmondsw has quit IRC | 21:01 | |
| *** thorst has quit IRC | 21:46 | |
| *** efried has quit IRC | 21:47 | |
| *** adi_____ has quit IRC | 21:48 | |
| *** efried has joined #openstack-powervm | 22:03 | |
| *** kylek3h has quit IRC | 22:06 | |
| *** tjakobs has quit IRC | 22:33 | |
| *** edmondsw has joined #openstack-powervm | 22:58 | |
| *** edmondsw has quit IRC | 23:02 | |
| *** thorst has joined #openstack-powervm | 23:46 | |
| *** adi_____ has joined #openstack-powervm | 23:48 | |
| *** thorst has quit IRC | 23:50 | |
| *** thorst has joined #openstack-powervm | 23:50 | |
| *** thorst has quit IRC | 23:53 | |
| *** chhavi has joined #openstack-powervm | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!