| openstackgerrit | Merged openstack-infra/zuul-jobs master: document that the build-reno-releasenotes job accepts sphinx_python https://review.openstack.org/561041 | 00:39 |
|---|---|---|
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Fix typo in build-sphinx-docs docstring https://review.openstack.org/561040 | 00:39 |
| mnaser | so either we did something terribly wrong in puppet-openstack | 00:42 |
| mnaser | or for some reason, a certain job is running as non voting when its not | 00:42 |
| mnaser | https://github.com/openstack/puppet-openstack-integration/blob/master/.zuul.yaml .. yet notice .. https://review.openstack.org/#/c/560628/ puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial is non-voting | 00:43 |
| pabelanger | mnaser: did you back port change to stable branches? | 00:45 |
| mnaser | maybe not, im thinking that might be it | 00:45 |
| pabelanger | http://logs.openstack.org/28/560628/1/check/puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial/ade36fd/zuul-info/inventory.yaml is the branch matchers that were applied | 00:45 |
| EmilienM | yeah but zuul should check master no? | 00:45 |
| mnaser | we did not for the set to voting | 00:45 |
| pabelanger | it will check all branches | 00:46 |
| EmilienM | https://github.com/openstack/puppet-openstack-integration/blob/stable/queens/.zuul.yaml#L139 | 00:46 |
| EmilienM | this is in queens, non voting | 00:46 |
| mnaser | its interesting that only scenario4 is the one going non voting | 00:46 |
| EmilienM | but puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial as well | 00:46 |
| EmilienM | yeah | 00:46 |
| mnaser | https://github.com/openstack/puppet-openstack-integration/blob/stable/queens/.zuul.yaml#L88 | 00:46 |
| mnaser | yeah except the rest are okay and not this one | 00:46 |
| EmilienM | yeah | 00:46 |
| EmilienM | there is a nice debug option | 00:46 |
| EmilienM | we can use it | 00:46 |
| pabelanger | the inventory file I linked shows how the branch matcher was applied | 00:47 |
| EmilienM | mnaser: do that: https://review.openstack.org/#/c/529846/1/.zuul.yaml | 00:47 |
| EmilienM | so we can debug | 00:47 |
| *** harlowja has quit IRC | 00:52 | |
| *** JasonCL has joined #zuul | 00:56 | |
| *** JasonCL has quit IRC | 00:57 | |
| *** gouthamr has quit IRC | 00:59 | |
| *** gouthamr has joined #zuul | 01:05 | |
| *** gouthamr has quit IRC | 01:37 | |
| *** gouthamr_ is now known as gouthamr | 01:37 | |
| *** rlandy|afk has quit IRC | 01:53 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: mqtt: add basic reporter https://review.openstack.org/535543 | 01:54 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: mqtt: add basic reporter https://review.openstack.org/535543 | 01:56 |
| *** JasonCL has joined #zuul | 01:57 | |
| *** JasonCL has quit IRC | 01:58 | |
| *** JasonCL has joined #zuul | 02:01 | |
| *** JasonCL has quit IRC | 02:03 | |
| *** Wei_Liu has joined #zuul | 02:08 | |
| *** JasonCL has joined #zuul | 02:13 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: mqtt: add basic reporter https://review.openstack.org/535543 | 02:30 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: reporter: do not expect branch attribute in Tag object https://review.openstack.org/560700 | 02:31 |
| *** xinliang has quit IRC | 02:47 | |
| *** xinliang has joined #zuul | 02:58 | |
| *** xinliang has quit IRC | 02:58 | |
| *** xinliang has joined #zuul | 02:58 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: reporter: do not expect branch attribute in Tag object https://review.openstack.org/560700 | 03:40 |
| *** riex7HX4KJ has joined #zuul | 03:45 | |
| *** riex7HX4KJ has quit IRC | 03:47 | |
| *** Dagger44EKP1 has joined #zuul | 04:02 | |
| *** Dagger44EKP1 has quit IRC | 04:04 | |
| *** gouthamr_ has joined #zuul | 04:09 | |
| *** gouthamr has quit IRC | 04:11 | |
| *** gouthamr_ is now known as gouthamr | 04:12 | |
| *** gouthamr_ has joined #zuul | 04:12 | |
| *** gouthamr has quit IRC | 04:20 | |
| *** gouthamr_ has quit IRC | 04:20 | |
| *** gouthamr has joined #zuul | 04:21 | |
| *** harlowja has joined #zuul | 04:45 | |
| *** mordred has quit IRC | 05:29 | |
| *** harlowja has quit IRC | 05:39 | |
| *** mordred has joined #zuul | 05:41 | |
| *** hashar has joined #zuul | 06:53 | |
| *** jpena|off is now known as jpena | 07:49 | |
| *** electrofelix has joined #zuul | 07:58 | |
| *** sshnaidm|afk is now known as sshnaidm|off | 09:10 | |
| *** sshnaidm|off has quit IRC | 09:14 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Align github status tests with real behavior https://review.openstack.org/561176 | 09:33 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Support regex matching of github status https://review.openstack.org/561177 | 09:33 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Don't create ci test admin login unencrypted in postgres https://review.openstack.org/561179 | 09:45 |
| *** xinliang has quit IRC | 09:45 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Support regex matching of github status https://review.openstack.org/561177 | 09:55 |
| *** xinliang has joined #zuul | 09:57 | |
| *** xinliang has quit IRC | 09:57 | |
| *** xinliang has joined #zuul | 09:57 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Support regex matching of github status https://review.openstack.org/561177 | 10:08 |
| *** JasonCL has quit IRC | 10:19 | |
| *** JasonCL has joined #zuul | 10:31 | |
| *** tflink has quit IRC | 10:34 | |
| *** tflink has joined #zuul | 10:35 | |
| *** snapiri has quit IRC | 10:43 | |
| *** xinliang has quit IRC | 10:51 | |
| *** xinliang has joined #zuul | 11:05 | |
| *** xinliang has quit IRC | 11:05 | |
| *** xinliang has joined #zuul | 11:05 | |
| *** jpena is now known as jpena|lunch | 11:41 | |
| *** tflink has quit IRC | 11:56 | |
| *** tflink has joined #zuul | 11:58 | |
| *** tflink has quit IRC | 12:05 | |
| *** tflink has joined #zuul | 12:19 | |
| mrhillsman | are there any known reasons for an executor to stop polling | 12:26 |
| *** jpena|lunch is now known as jpena | 12:29 | |
| *** rlandy has joined #zuul | 12:31 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Support regex matching of github status https://review.openstack.org/561177 | 12:33 |
| *** xinliang has quit IRC | 12:49 | |
| *** xinliang has joined #zuul | 12:50 | |
| *** xinliang has quit IRC | 12:50 | |
| *** xinliang has joined #zuul | 12:50 | |
| *** tflink has quit IRC | 12:54 | |
| *** pwhalen has quit IRC | 13:03 | |
| *** pwhalen has joined #zuul | 13:08 | |
| *** pwhalen has joined #zuul | 13:08 | |
| *** sshnaidm|off has joined #zuul | 13:32 | |
| *** dkranz has joined #zuul | 13:38 | |
| corvus | mrhillsman: stop polling for what? (i'm not sure the executor polls anything) | 13:45 |
| mrhillsman | jobs from gearman | 13:46 |
| mrhillsman | thought i was seeing it 'Polling 1 connection' or 'Polling for 1 connection' in debug before but no longer | 13:46 |
| mrhillsman | i pushed a change and see it just sit for +10 minutes and not pick up the job | 13:47 |
| mrhillsman | then i restart it and it does its thing | 13:48 |
| mrhillsman | after some time it does it, just sits and i don't see it checking for jobs when iirc it was before | 13:48 |
| corvus | mrhillsman: ah yep, that's a gear log message. it's slightly misleading in that it's right before a call to the "poll" syscall, which is why it says "polling", but it's not a traditional "poll after interval" sort of operation. it actually just sits there indefinitely waiting for a packet over the gearman connection. once it gets a packet, it exits the loop, does work, then enters again and emits the | 13:52 |
| corvus | "polling" message. | 13:52 |
| pabelanger | mrhillsman: zuul-executor has a governor for load and memory, when hit the limits of one, it will stop launching jobs | 13:52 |
| corvus | mrhillsman: so, all that said, that means that: it's normal for it to not emit that message if nothing's happening | 13:53 |
| corvus | mrhillsman: but it's not normal for it to not do any work if it's otherwise idle and there's a job waiting. | 13:53 |
| mrhillsman | yep | 13:53 |
| corvus | mrhillsman: the most likely cause is a firewall between the executor and the scheduler which has decided to drop the connection between the two | 13:53 |
| mrhillsman | forgive me if i get the seq wrong | 13:54 |
| corvus | mrhillsman: not at all -- i'm probably overexplaining as is my habit :) | 13:54 |
| mrhillsman | but i recall seeing some gearman variables or whatever like NOOP, PRE_SLEEP, etc | 13:54 |
| corvus | mrhillsman: we had some ideas about tcp keepalives... let me see if they made it into the code | 13:54 |
| fbo_ | hi, is there any plan to externalize tools/encrypt_secret.py in its own repo ? It could be convenient for a zuul user to just pip install zuul-secret (whatever the name) | 13:55 |
| mrhillsman | that firewall could be the problem | 13:55 |
| corvus | mrhillsman: yep, executor says "PRE_SLEEP", then goes to idle. scheduler says "NOOP" when there's a job to wake the executor up. executor then says "GRAB_JOB_UNIQ" | 13:55 |
| mrhillsman | yeah | 13:55 |
| corvus | fbo_: not at the moment, but it's intended to work with standard python lib so you can just "curl" it out of git.zuul-ci.org | 13:56 |
| mrhillsman | i think it could be firewall -_- | 13:56 |
| mrhillsman | i increased the traffic quite a bit over the last week and maybe it triggered something or someone | 13:56 |
| fbo_ | corvus: ok | 13:56 |
| mrhillsman | to start killing the connection after some time | 13:56 |
| corvus | mrhillsman: oh, does this happen when things are busy, or when it's idle? | 13:57 |
| corvus | mrhillsman: if it is the firewall/network, and it happens when it's idle, then adding keepalive support may prevent it from happening. if it happens when it's busy, then it may not prevent it, but it could allow zuul to recover automatically. | 13:58 |
| corvus | mrhillsman: this is the change we need: https://review.openstack.org/425248 | 13:58 |
| mrhillsman | i have seen it happen when i am starting and stopping jobs trying to troubleshoot something via pr updates | 13:59 |
| mrhillsman | which i thought may be just me being the problem | 13:59 |
| mrhillsman | but when i see it sit for quite a while after i know it ran a job before and it does not is what triggered my question | 14:00 |
| mrhillsman | the former did not really concern me as i thought i may have hit some github ratelimit or something else | 14:01 |
| mrhillsman | and just restarted the executor | 14:01 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Tell geard to use keepalives https://review.openstack.org/425248 | 14:02 |
| mrhillsman | i'll pull that in and see if it helps, thx corvus | 14:02 |
| corvus | mrhillsman: ^ you'll need that updated version | 14:02 |
| *** elyezer has quit IRC | 14:05 | |
| *** elyezer has joined #zuul | 14:08 | |
| corvus | jlk: i +2d but did not approve https://review.openstack.org/561177 in case you want to take a look | 14:11 |
| corvus | SpamapS: ^ | 14:11 |
| *** tflink has joined #zuul | 14:12 | |
| openstackgerrit | Merged openstack-infra/zuul master: Don't create ci test admin login unencrypted in postgres https://review.openstack.org/561179 | 14:30 |
| corvus | tobiash, tristanC, mordred: if you have time today to update https://etherpad.openstack.org/p/zuul-update-email with what you did this week and/or plan to next week that would be great. | 14:36 |
| tobiash | corvus: on my way home, will do later this evening | 14:37 |
| corvus | anyone else too -- i just distinctly remember reviewing changes from those folks :) | 14:38 |
| *** acozine1 has joined #zuul | 14:41 | |
| *** openstackgerrit has quit IRC | 14:50 | |
| mordred | corvus, Shrews, tobiash: in reviewing Shrews docs patches, I realized that the static driver is written to be ssh-specific | 14:57 |
| mordred | feels like we need a connection-type config option (defaulting to ssh) and to rename 'ssh-port' to 'port', right? | 14:58 |
| tobiash | mordred: there is a change in review about connection type | 15:00 |
| mordred | awesome | 15:00 |
| tobiash | mordred: ah, that's probably openstack specific | 15:01 |
| tobiash | Or was that merged already? Don't remember | 15:02 |
| mordred | tobiash: there's one for diskimages | 15:03 |
| mordred | tobiash: I'll put up one for static | 15:03 |
| dmsimard | The superuser post about Zuul, OpenStack and ARA is live on superuser: http://superuser.openstack.org/articles/scaling-ara-ansible/ | 15:14 |
| dmsimard | s/superuser// | 15:15 |
| Shrews | this seemingly random zuul test failure might be concerning, yes? http://logs.openstack.org/00/558600/4/check/zuul-tox-py35/5307042/job-output.txt.gz#_2018-04-12_19_11_20_800803 | 15:20 |
| *** openstackgerrit has joined #zuul | 15:20 | |
| openstackgerrit | Merged openstack-infra/nodepool master: Don't gather host keys for non ssh connections https://review.openstack.org/535685 | 15:20 |
| openstackgerrit | Merged openstack-infra/nodepool master: Add connection-port to provider diskimage https://review.openstack.org/535686 | 15:21 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Support winrm hosts in static driver https://review.openstack.org/561256 | 15:21 |
| mordred | tobiash: ^^ | 15:21 |
| rcarrillocruz | mordred: would it make sense to do 'if node['connetion-type'] != ssh': return | 15:26 |
| rcarrillocruz | like i'm thinking | 15:26 |
| rcarrillocruz | if we had nodes with exotic transports that are non ssh | 15:26 |
| rcarrillocruz | along with winrm | 15:26 |
| mordred | rcarrillocruz: yah- tat's probably better | 15:27 |
| rcarrillocruz | we have in-house eapi/nxapi (http based connections), and i think netconf can be run on non-ssh | 15:28 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Support winrm hosts in static driver https://review.openstack.org/561256 | 15:28 |
| mordred | rcarrillocruz: done! | 15:28 |
| rcarrillocruz | sweet, thx | 15:28 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul master: Add static driver doc to Zuul From Scratch https://review.openstack.org/558802 | 15:30 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul master: Add sample systemd service files. https://review.openstack.org/558830 | 15:30 |
| tobiash | mordred: just found a small issue | 15:34 |
| tobiash | mordred: re https://review.openstack.org/561256 | 15:34 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Support winrm hosts in static driver https://review.openstack.org/561256 | 15:45 |
| mordred | tobiash: good catch | 15:45 |
| tobiash | mordred: actually just a nit but could lead to confusion if someone looks at the tests as a reference | 15:46 |
| mordred | yah - and there's no need to have that possible confusion there | 15:49 |
| mordred | :) | 15:49 |
| mordred | rcarrillocruz: ^^ patch updated | 15:49 |
| tobiash | corvus: is a zuul-sphinx release needed to pick up 559963? | 15:50 |
| corvus | tobiash: yes! | 15:50 |
| corvus | oh, that reminds me, we should send out the release announcement for 301 | 15:50 |
| corvus | yay! https://zuul-ci.org/docs/zuul/releasenotes.html#release-notes | 15:51 |
| corvus | oh, hrm | 15:51 |
| corvus | the anchor is "id1" | 15:51 |
| corvus | i kind of worry that when we release 3.0.2, *that* anchor will be id1, and 3.0.1 will be id2. | 15:52 |
| corvus | i guess we'll see, and if so, maybe we can update reno to produce stable anchors | 15:52 |
| *** sshnaidm|off has quit IRC | 15:52 | |
| corvus | anyway, that's what we've got for now, so i'll write up an announcement | 15:52 |
| corvus | do we want just a short announcement with download/release note links, or do we want to copy the relnotes into the email? | 15:53 |
| corvus | https://etherpad.openstack.org/p/51gI38SIyu | 15:54 |
| mordred | corvus: yah - I tink we should update reno to produce stable anchors - id_3_0_1 would be a nicer anchor - and id_unreleased for the unreleased bits | 15:54 |
| corvus | zuul-sphinx 0.2.2 pushed | 15:55 |
| corvus | tobiash: ^ | 15:55 |
| mordred | corvus: I think copying the relnotes into the email would be nice ... it tends to irk me when I get an email and the only thing in it is a link | 15:56 |
| tobiash | corvus: thanks :) | 15:56 |
| mordred | corvus: but - if we're going to do that, we might want to automate (or partially automated) it in some way - like, it would be neat if we could generate a plain-text form of those release notes with the links in text-email footnote form | 15:57 |
| corvus | mordred: yeah, we may want to look into the release-announce emails | 15:58 |
| corvus | mordred: in the interim, i updated the etherpad | 15:58 |
| mordred | corvus: ++ | 16:00 |
| corvus | okay, i'll send that now | 16:01 |
| SpamapS | corvus: tobiash I really love the regex status matching. I wonder if we could get it in behind the re2 switch, and never support backrefs in those regexes? | 16:02 |
| SpamapS | Like you could even modify the patch to pull in re2 and just use it for this. | 16:03 |
| tobiash | SpamapS: ok, sounds good | 16:03 |
| SpamapS | I will say this is less dangerous than the file matcher. | 16:04 |
| SpamapS | Since only write users can make statuses. | 16:04 |
| tobiash | SpamapS, corvus: re2 is decided for the future? | 16:04 |
| corvus | tobiash: i think so -- i haven't heard anything against it -- we're just going to need a transition period once the branch matcher patches are fixed up and ready to go | 16:06 |
| tobiash | corvus: so it's logical to start right with re2 for any new regex feature | 16:06 |
| corvus | tobiash: ++ but it's worth thinking about whether lack of negative matchers will be a problem here, and if we need to solve it by adding more options (like "not-status:") | 16:09 |
| corvus | also, "not-branches" and "not-files" is really growing on me, i have to say. | 16:09 |
| tobiash | corvus: at least for the status we have already reject which would enable probably most of these not-usecases for status | 16:10 |
| corvus | makes sense | 16:10 |
| tobiash | corvus: so should I change it to re2? | 16:11 |
| SpamapS | tobiash: https://review.openstack.org/#/c/536389/ has the details | 16:11 |
| SpamapS | but yeah, the interface for matchers is also not quite right yet | 16:11 |
| corvus | tobiash: that sounds like a good idea to me | 16:11 |
| SpamapS | corvus: I like not-* too. | 16:11 |
| tobiash | SpamapS: thanks, have it open already | 16:11 |
| SpamapS | makes it read like natural language. | 16:12 |
| corvus | SpamapS: and still doesn't violate the descriptive property we like. | 16:13 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Support regex matching of github status https://review.openstack.org/561177 | 16:16 |
| tobiash | SpamapS, corvus: ^ | 16:16 |
| tobiash | mordred: do you have time for an easy review? https://review.openstack.org/553702 | 16:23 |
| *** hashar is now known as hasharAway | 16:24 | |
| mordred | tobiash: done | 16:25 |
| tobiash | :) | 16:25 |
| *** JasonCL has quit IRC | 16:26 | |
| tobiash | corvus: shall I also change 535713 to re2? | 16:27 |
| *** JasonCL has joined #zuul | 16:28 | |
| *** sshnaidm|off has joined #zuul | 16:33 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Support winrm hosts in static driver https://review.openstack.org/561256 | 16:37 |
| openstackgerrit | Merged openstack-infra/zuul master: Align github status tests with real behavior https://review.openstack.org/561176 | 17:00 |
| *** jpena is now known as jpena|off | 17:07 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Don't count non-live queue items in tenant list https://review.openstack.org/553702 | 17:26 |
| tobiash | corvus, mordred: this re-adds the json import in test_web.py which was missing now on master and broke gate ^ | 17:27 |
| *** electrofelix has quit IRC | 17:38 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Don't count non-live queue items in tenant list https://review.openstack.org/553702 | 17:48 |
| openstackgerrit | Merged openstack-infra/zuul master: Remove zuul-tox-py35 https://review.openstack.org/560983 | 18:21 |
| *** smyers has quit IRC | 18:30 | |
| *** smyers has joined #zuul | 18:33 | |
| openstackgerrit | Merged openstack-infra/nodepool master: Use AFS mirrors for debian-stretch testing https://review.openstack.org/558633 | 18:36 |
| pabelanger | Shrews: left review on 560993 | 18:46 |
| mordred | tobiash: +2 | 18:50 |
| *** harlowja has joined #zuul | 19:10 | |
| mrhillsman | did something change re accessing project public key for encryption? | 19:19 |
| mrhillsman | with the changes to zuul-web? | 19:19 |
| clarkb | mrhillsman: I think the api paths may have changed but other than that I don't know of any changes | 19:20 |
| *** acozine1 has quit IRC | 19:20 | |
| clarkb | mrhillsman: its now /api/tenent/stuff iirc. Unsure if the encryption stuff goes through that api | 19:20 |
| mrhillsman | nah, the tool is not using /api | 19:21 |
| mrhillsman | req = Request("%s/key/%s.pub" % (args.url.rstrip('/'), args.project)) | 19:21 |
| mrhillsman | you provide url and project | 19:22 |
| mrhillsman | but manually it is no bueno | 19:22 |
| mrhillsman | i am probably typing it wrong | 19:22 |
| clarkb | right I'm not sure if that moved under /api | 19:22 |
| Shrews | pabelanger: the default IS actually nodepool.pid | 19:24 |
| Shrews | pabelanger: that threw me too at first | 19:24 |
| Shrews | pabelanger: we can change the default, but that should probably be a separate change | 19:25 |
| pabelanger | Shrews: Ah, I see now. nodepool.pid and nodepool-builder.pid | 19:27 |
| Shrews | pabelanger: yup | 19:28 |
| pabelanger | yah, maybe a follow change it to nodepool-launcher makes sense | 19:28 |
| pabelanger | I'll update my vote | 19:28 |
| Shrews | tobiash: ^^^ | 19:28 |
| clarkb | mrhillsman: ('GET', '/api/tenant/{tenant}/key/{project:.*}.pub' is the route in zuul web | 19:29 |
| mrhillsman | ugh | 19:29 |
| mrhillsman | thx | 19:29 |
| clarkb | mrhillsman: so ya should be /api/tenant/yourtenanthere/key/projectname.pub | 19:29 |
| mrhillsman | documentation needs some updating ;) | 19:29 |
| clarkb | mrhillsman: and the script too | 19:29 |
| mrhillsman | ++ | 19:29 |
| clarkb | mrhillsman: areyou going to test that and push up an update? | 19:29 |
| mrhillsman | sure can ;) | 19:29 |
| Shrews | pabelanger: tobiash: one of you care to add the +A to that? | 19:31 |
| tobiash | mrhillsman, clarkb: ya one of my customers also had problems with that | 19:31 |
| pabelanger | Shrews: no issue with nodepool starting using it? I haven't tested myself | 19:32 |
| tobiash | Encryptsecret works if you put api/tenant into the url before your tenant name | 19:33 |
| Shrews | pabelanger: i didn't test these, just pulled them from what was already merged | 19:33 |
| tobiash | But that's awkward | 19:33 |
| pabelanger | Shrews: yah, i think we might want to consider a test before +A. I can give a try here in a bit and update to +A if working, unless somebody else beats me | 19:34 |
| clarkb | mrhillsman: tobiash cool happy to review it since I've already done the investigating of how it should work now | 19:35 |
| clarkb | *review the fixes | 19:35 |
| mrhillsman | ++ | 19:35 |
| mrhillsman | for some reason mine is taking forever to respond | 19:35 |
| mrhillsman | once i confirm it is working will push up docs and tool patch unless someone beats me to it | 19:35 |
| tobiash | mrhillsman: zuul-web might be hanging | 19:36 |
| mrhillsman | i'm thinking that may be it with the changes | 19:36 |
| mrhillsman | which brought me to ask initially with the tool not working out the box | 19:36 |
| tobiash | Now that the url is more complex we might want to set the tenant via an argument instead of encoding it into the url on the commandline | 19:37 |
| mrhillsman | AttributeError: 'NoneType' object has no attribute 'public_key' | 19:38 |
| tobiash | clarkb, mordred what do you think about a javascript based variant of the encrypt script? | 19:38 |
| openstackgerrit | Merged openstack-infra/nodepool master: Add example nodepool-launcher systemd service file https://review.openstack.org/560993 | 19:39 |
| mrhillsman | scheduler crying | 19:39 |
| mrhillsman | https://www.irccloud.com/pastebin/YDgHVccF/ | 19:39 |
| tobiash | I think that would greatly increase usability for non zuul devs | 19:39 |
| clarkb | tobiash: with a text box that you can drop the secret into and have it spit out yaml to copy and paste? | 19:39 |
| clarkb | tobiash: probably not a bad idea especially if we can keep all of that in the browser | 19:40 |
| clarkb | (so we never see the secret unencrypted) | 19:40 |
| tobiash | Yes, like zerobin | 19:40 |
| tobiash | mrhillsman: in that case the project is not found | 19:41 |
| tobiash | Maybe missing tenant information | 19:41 |
| mrhillsman | hrm, let me try again | 19:41 |
| mrhillsman | zuul-web went away as well | 19:42 |
| mrhillsman | let me just restart all the things real quick | 19:42 |
| mrhillsman | i was working on some other stuff | 19:42 |
| mrhillsman | stupid internet went down | 20:27 |
| mrhillsman | or rather my shit isp did | 20:28 |
| openstackgerrit | Merged openstack-infra/zuul master: Don't count non-live queue items in tenant list https://review.openstack.org/553702 | 20:37 |
| *** JasonCL has quit IRC | 20:38 | |
| *** JasonCL has joined #zuul | 20:57 | |
| *** evrardjp has quit IRC | 21:40 | |
| *** rlandy has quit IRC | 22:09 | |
| *** hasharAway has quit IRC | 22:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!