| *** jesusaur has quit IRC | 00:12 | |
| *** jesusaur has joined #zuul | 00:19 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: prometheus: add options to start the server and process collector https://review.openstack.org/599209 | 00:25 |
|---|---|---|
| *** zigo has joined #zuul | 01:09 | |
| *** threestrands has joined #zuul | 02:22 | |
| *** rlandy|bbl is now known as rlandy | 02:45 | |
| openstackgerrit | Merged openstack-infra/nodepool master: uncap sphinx https://review.openstack.org/599742 | 05:16 |
| *** shachar is now known as snapiri | 05:16 | |
| openstackgerrit | Merged openstack-infra/nodepool master: Add list of metrics provided to statsd https://review.openstack.org/590233 | 05:41 |
| *** jiapei has joined #zuul | 05:48 | |
| openstackgerrit | Merged openstack-infra/nodepool master: Update static nodes in Zookeeper on config change https://review.openstack.org/598156 | 06:23 |
| *** pcaruana has joined #zuul | 06:23 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul-jobs master: Add a role to upload logs to Amazon's S3 service https://review.openstack.org/599931 | 06:30 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: prometheus: add options to start the server and process collector https://review.openstack.org/599209 | 06:40 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: prometheus: add existing metrics https://review.openstack.org/599210 | 06:40 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: prometheus: add decorators to measure scheduler latency https://review.openstack.org/599211 | 06:40 |
| *** rlandy has quit IRC | 06:45 | |
| openstackgerrit | Markus Hosch proposed openstack-infra/nodepool master: Add metric for image build result https://review.openstack.org/590412 | 06:49 |
| *** hashar has joined #zuul | 06:58 | |
| SpamapS | fascinating.. | 07:15 |
| SpamapS | if you put cloudflare in front of your zuul API, it will deny encrypt_secret.py | 07:15 |
| SpamapS | TIL | 07:15 |
| SpamapS | guessing it's the generic/lame user agent | 07:16 |
| openstackgerrit | Markus Hosch proposed openstack-infra/nodepool master: Add metric for image build result https://review.openstack.org/590412 | 07:39 |
| *** jpena|off is now known as jpena | 07:45 | |
| *** goern has joined #zuul | 07:57 | |
| *** corvus has quit IRC | 08:05 | |
| *** electrofelix has joined #zuul | 08:13 | |
| *** threestrands has quit IRC | 08:26 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped, JWT-protected actions https://review.openstack.org/576907 | 08:27 |
| *** sshnaidm|afk is now known as sshnaidm | 08:36 | |
| *** pcaruana has quit IRC | 08:50 | |
| *** hashar is now known as hasharAway | 09:43 | |
| *** pcaruana has joined #zuul | 09:58 | |
| *** Diabelko has joined #zuul | 11:04 | |
| *** jpena is now known as jpena|lunch | 11:36 | |
| *** sshnaidm has quit IRC | 11:43 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Do not abort node launch if failed node cannot be deleted https://review.openstack.org/589854 | 11:44 |
| *** zigo has quit IRC | 11:45 | |
| *** sshnaidm has joined #zuul | 11:46 | |
| *** odyssey4me has quit IRC | 11:53 | |
| *** odyssey4me has joined #zuul | 11:55 | |
| *** zigo has joined #zuul | 12:08 | |
| *** rlandy has joined #zuul | 12:33 | |
| *** jpena|lunch is now known as jpena | 12:41 | |
| *** ssbarnea has quit IRC | 12:59 | |
| *** bhavikdbavishi has joined #zuul | 13:00 | |
| *** ssbarnea has joined #zuul | 13:01 | |
| *** ssbarnea has quit IRC | 13:01 | |
| *** samccann has joined #zuul | 13:03 | |
| *** hasharAway is now known as hashar | 13:07 | |
| *** bhavikdbavishi1 has joined #zuul | 13:11 | |
| *** bhavikdbavishi has quit IRC | 13:12 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 13:12 | |
| *** bhavikdbavishi has quit IRC | 13:17 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-website master: Add a "start" redirect https://review.openstack.org/599699 | 13:19 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Do not abort node launch if failed node cannot be deleted https://review.openstack.org/589854 | 13:54 |
| *** hashar is now known as hasharAway | 14:06 | |
| tobiash | clarkb, corvus: the cherrypy fix has been merged so the next cherrypy release should be fine again | 14:14 |
| Shrews | So, https://review.openstack.org/597302 and children have multiple +2's all the way through the stack (I just approved the parent of that one). 302 has one -1 from jhesketh about variable naming, but is willing to defer to the group. What do we want to do with that? | 14:33 |
| Shrews | We can always approve the stack and fix vars after, if we want. | 14:34 |
| Shrews | I think corvus wanted these in before ptg, iirc | 14:35 |
| Shrews | and he is gleefully holidaying :) | 14:35 |
| * Shrews super jealous | 14:35 | |
| Shrews | i think i may just approve the stack and leave the var name bikeshedding for later | 14:37 |
| tobiash | Shrews: just wanted to suggest that | 14:38 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Improve static provider to check non-ssh hosts https://review.openstack.org/600079 | 14:38 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Fix missing node state refresh in static driver https://review.openstack.org/600080 | 14:38 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Fix use of wrong label static test fixture https://review.openstack.org/600081 | 14:38 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Worker and interfaces for checking node liveness https://review.openstack.org/600082 | 14:38 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Implement liveness probe for static driver https://review.openstack.org/600083 | 14:38 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Cleanup of leaked resource for static driver https://review.openstack.org/600084 | 14:39 |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Reload tenant in case of new project branches https://review.openstack.org/600088 | 14:48 |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Reload tenant in case of new project branches https://review.openstack.org/600088 | 14:49 |
| openstackgerrit | Merged openstack-infra/zuul master: Add private key storage migration https://review.openstack.org/592213 | 14:51 |
| *** ssbarnea|bkp2 has joined #zuul | 14:53 | |
| ssbarnea|bkp2 | clarkb fungi : https://review.openstack.org/#/c/598222/ should take 10s to read and review. | 14:55 |
| fungi | ssbarnea|bkp2: indeed, http://logs.openstack.org/22/598222/3/check/zuul-build-dashboard/c4c9c24/npm/html/status.html looks like it dtrt | 14:58 |
| fungi | though to be fair, it took me more than 10s to pull up the draft rendering alone ;) | 14:59 |
| ssbarnea|bkp2 | fungi cool,i didn't know there was a way to check the UI by using the logs. | 14:59 |
| ssbarnea|bkp2 | i tested using a browser styler exptension only. | 14:59 |
| openstackgerrit | Merged openstack-infra/zuul master: Change project.private_key to private_secrets_key https://review.openstack.org/597302 | 15:00 |
| fungi | ssbarnea|bkp2: yeah, it's the zuul-build-dashboard job result link (obviously) | 15:01 |
| *** corvus has joined #zuul | 15:02 | |
| openstackgerrit | Merged openstack-infra/zuul master: Create per-project ssh keys https://review.openstack.org/597711 | 15:03 |
| openstackgerrit | Merged openstack-infra/zuul master: Provide per-project ssh key to executor https://review.openstack.org/597727 | 15:04 |
| openstackgerrit | Merged openstack-infra/zuul master: Serve project SSH keys and document https://review.openstack.org/599063 | 15:07 |
| openstackgerrit | Merged openstack-infra/zuul master: Display correct cursor when hovering over patchset header https://review.openstack.org/598222 | 15:21 |
| *** pcaruana has quit IRC | 15:25 | |
| tristanC | SpamapS: could you please comment on https://review.openstack.org/599209 ? | 15:40 |
| openstackgerrit | Merged openstack-infra/zuul master: Add container spec https://review.openstack.org/560136 | 15:42 |
| *** hasharAway is now known as hashar | 15:59 | |
| *** jiapei has quit IRC | 16:22 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: add-build-sshkey: Remove only the master key https://review.openstack.org/599073 | 16:26 |
| *** hashar has quit IRC | 16:48 | |
| corvus | tristanC: where's the data storage for grafana + prometheus? | 16:53 |
| tristanC | corvus: prometheus comes with it's own timeserie database, but it can export to other backend like gnocchi for example | 16:54 |
| corvus | tristanC: internally in a prometheus server then? | 16:55 |
| tristanC | corvus: yes, it's a service that polls data from target | 16:56 |
| corvus | tristanC: so statsd client -> prometheus endpoint; statsd+graphite -> prometheus server. | 16:56 |
| tristanC | corvus: the prometheus_client is just a server to serve the current metrics, it does not accumulate or do any computation | 16:56 |
| corvus | tristanC: have you used https://github.com/prometheus/statsd_exporter ? | 16:58 |
| tristanC | corvus: no i haven't, it seems like it would act as middleware between statsd push and prometheus pull | 16:59 |
| tristanC | corvus: which seems less efficient than having zuul services expose the metrics directly | 17:00 |
| corvus | tristanC: i think you could point zuul at it and get all the data into prometheus today; might let you work with the full stack before comitting to it | 17:00 |
| bramwelt | irc | 17:01 |
| corvus | tristanC: there's no reason it should be inefficient. it's not very different than statsd itself. it's basically a prometheus backend for statsd. it offloads the http serving from the application. | 17:02 |
| *** hashar has joined #zuul | 17:03 | |
| corvus | tristanC: but i hear your point that prometheus as a system is simpler to set up, while that's adding another component. | 17:04 |
| corvus | i mostly suggest it as a way to start using prometheus now | 17:04 |
| tristanC | corvus: it's a good suggestion, i actually didn't knew statsd_exporter. the other thing i like with poll is that you get a nice of running service and a red light on the one unreachable | 17:08 |
| corvus | tristanC: yeah, that's certainly the one thing you don't get with push (unless you have a heartbeat) | 17:08 |
| tristanC | a nice list of* | 17:08 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Add missing README.rst for roles https://review.openstack.org/600128 | 17:10 |
| *** hashar has quit IRC | 17:10 | |
| *** hashar has joined #zuul | 17:11 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Turn of missing docs warnings temporarily https://review.openstack.org/600132 | 17:14 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Add missing README.rst for roles https://review.openstack.org/600128 | 17:16 |
| *** hashar has quit IRC | 17:17 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Add missing README.rst for roles https://review.openstack.org/600128 | 17:19 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-sphinx master: Ignore non-directory entries in roles/ https://review.openstack.org/600135 | 17:19 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Revert "Turn of missing docs warnings temporarily" https://review.openstack.org/600137 | 17:20 |
| *** sshnaidm is now known as sshnaidm|afk | 17:32 | |
| *** electrofelix has quit IRC | 17:32 | |
| *** jpena is now known as jpena|off | 17:40 | |
| *** pcaruana has joined #zuul | 17:52 | |
| tobiash | corvus, tristanC: we're using statsd_exporter with prometheus and it kinda works | 17:55 |
| tobiash | but it has its glitches | 17:55 |
| bramwelt | So I've got Zuul running, and a Gerrit instance, and I'm trying to figure out how to debug my zuul.yaml configuration as nothing shows up in the web-ui | 17:55 |
| tobiash | we need a ton of mapping rules to map the statsd metrics to something useful in prometheus | 17:55 |
| tobiash | that also involves complex regex matching due to a non-optimal structure of the nodepool metrics | 17:56 |
| corvus | bramwelt: can you be more specific about what's missing in the web-ui? | 17:56 |
| bramwelt | If I have a simple zuul.yaml file with just '- topic:\nname: quickstart', I can see it listed in the ui, but as soon as I add the source with gerrit and set config-projects nothing it all disappears. | 17:56 |
| corvus | tobiash: this is good info :) | 17:56 |
| bramwelt | corvus: the tenents.html page displays, but the table is empty | 17:57 |
| SpamapS | tristanC: commented | 17:58 |
| tobiash | corvus: so it is usable but with quite some effort to get it running | 17:58 |
| bramwelt | As a side note zuul-ci.org is down for me... | 17:58 |
| corvus | bramwelt: can you paste your main.yaml? (the zuul tenant config file) | 17:58 |
| bramwelt | Yep | 17:58 |
| corvus | bramwelt: zuul-ci.org seems to be working for me | 17:59 |
| bramwelt | https://paste.fedoraproject.org/paste/1HFP8wRAcgKrG-FXnThH9Q | 17:59 |
| tobiash | so I personally would love to have a pluggable metrics system in zuul (similar to the drivers interface) to be able to use native metrics for prometheus | 17:59 |
| bramwelt | corvus: It may be my dns resolver. | 17:59 |
| tobiash | corvus: but I also can understand your position and could live without it | 18:00 |
| SpamapS | tobiash: I'm coming to this late but.. https://github.com/prometheus/statsd_exporter should give you that, no? | 18:00 |
| *** openstackgerrit has quit IRC | 18:00 | |
| tobiash | SpamapS: that's what we're using | 18:00 |
| SpamapS | so just have zuul send the statsd packets it already knows how to send, to that. | 18:00 |
| corvus | bramwelt: so the 'workshop' tenant doesn't show up in the tenant list.... is there any useful information in the scheduler's log file? | 18:01 |
| tobiash | SpamapS: it's not *that* easy | 18:01 |
| *** hashar has joined #zuul | 18:02 | |
| tobiash | SpamapS: you need a mapping rule for every metric you want in prometheus that parses values out of parts of the metric name | 18:02 |
| tobiash | SpamapS: we even contributed to it to make it possible to use it with nodepool: https://github.com/prometheus/statsd_exporter/pull/136 | 18:02 |
| bramwelt | corvus: Hmm, this wasn't happening before but now I see it says unknown host key for Gerrit | 18:02 |
| clarkb | bramwelt: corvus: if this is the opendaylight server the sshd there is broken | 18:03 |
| clarkb | you hvae to manually add the rsa host key to known hosts yourself so that zuul will talk to it as the ecdsa or e25519 key is broken on that version of gerrit | 18:03 |
| bramwelt | clarkb: Ah, no, but that could be related. | 18:04 |
| *** hashar is now known as hasharAway | 18:04 | |
| bramwelt | I've changed some config in my testing..so let me get back to you all where I've gotten this to a sane state. | 18:04 |
| corvus | bramwelt: great, let us know if we can help more | 18:05 |
| bramwelt | Thanks! | 18:05 |
| bramwelt | Will do | 18:05 |
| corvus | bramwelt: ("ssh -p 29418 $gerrit_hostname -i /path/to/key ls-projects" run as the zuul user is usually a good connectivity test) | 18:06 |
| corvus | though if you run into the problem clarkb describes, that won't show it since openssh and paramiko have different key preferences | 18:07 |
| bramwelt | right, and there doesn't seem to be a host_key setting like there is for nodepool | 18:07 |
| bramwelt | Perhaps this is a better question. What is the minimum configuration needed in a 'config-projects' repo? A single zuul.yaml file with a job definition? | 18:28 |
| bramwelt | I haven't been able to find the docs on standing up a zuul project. There's quite a bit on running and configuring zuul though :) | 18:29 |
| corvus | bramwelt: technically... nothing i think :) a reasonable potentially useful minimum might be a pipeline definition. | 18:29 |
| corvus | bramwelt: yes... unfortunately, no one has written that yet | 18:29 |
| bramwelt | That's okay, I'm hoping to contribute it :) | 18:29 |
| corvus | (that's what's supposed to come next in the zuul-from-scratch document) | 18:29 |
| corvus | bramwelt: that would be great :) | 18:29 |
| bramwelt | Okay, so just '---\n- pipeline:\ncheck ...'? | 18:30 |
| corvus | bramwelt: yep. you can omit the '---' bit. | 18:30 |
| bramwelt | Right. i'll give that a shot | 18:30 |
| corvus | bramwelt: https://zuul-ci.org/docs/zuul/user/config.html#pipeline is a good base | 18:31 |
| bramwelt | Yeah, that's what I'm using. :) | 18:31 |
| tobiash | tristanC, corvus: added my point of view to 599209 | 18:33 |
| corvus | tobiash: it's a shame statsd_exporter isn't more automatic. i don't understand why it can't just handle the metrics just like statsd does (it prefixes the different metric names to different hierarchies, like "statsd_timers") | 18:36 |
| bramwelt | Huh, I crashed gearman | 18:36 |
| bramwelt | (not the internal one) | 18:36 |
| tobiash | corvus: because the data models are different. By default it creates a metric per statsd metric | 18:37 |
| bramwelt | https://paste.fedoraproject.org/paste/oQi5EtLhOEEh21mlKP1YyA | 18:37 |
| tobiash | but in zuul use case this doesn't get us anywhere because many parts of the metric name actually need to be labels in prometheus | 18:37 |
| corvus | tobiash: i see | 18:38 |
| tobiash | and it has no possibility to know which elements of the path is just a metric name and which element should be a label | 18:38 |
| bramwelt | For context, I'm trying to get Zuul spun up with docker-compose. Had to use an external german container as I believe either forking wasn't happening in the process, or my configuration was off. :) | 18:38 |
| tobiash | that's highly dependent on the metrics and data itself | 18:38 |
| tobiash | e.g. in nodepool you have nodepool.launch.provider.<provider>.<az>.<result> | 18:39 |
| tobiash | that needs to be mapped to nodepool_launch_by_provider with the labels <provider>, <az> and result | 18:39 |
| tobiash | to make that more complicated nodepool emits ready or error.foo as <result> which breaks simple globbing | 18:40 |
| corvus | tobiash: i'm not sure it's worth our time to make an abstraction layer for stats. that sounds like a lot of work for little gain. it may be better to pick one and stick with it. | 18:40 |
| tobiash | corvus: I meant *if* we go and support statsd and prometheus we should not mix it but create an abstraction layer | 18:41 |
| corvus | tobiash: yeah, i agree we shouldn't mix it; i'm just saying i don't think supporting both is worthwhile. we have better things to do than split our development and review effort supporting multiple systems there. | 18:42 |
| tobiash | corvus: so If the decision is to stick with statsd I can live with it, but in that case we should have some reference mapping rules in zuul | 18:42 |
| tobiash | I'd be happy to provide what we have in mapping rules | 18:43 |
| corvus | and, ftr, i'm willing to consider switching if it seems like a good idea | 18:43 |
| tobiash | and we should think about some little restructure of the metric names to better support the mapping | 18:43 |
| corvus | bramwelt: i haven't used docker-compose, but i'm not sure why there would be a problem forking... any ideas tobiash? | 18:44 |
| * tobiash reads backscroll | 18:44 | |
| mordred | I feel like jlk or spamaps had used docker-compose successfully in the past | 18:44 |
| clarkb | I've got nginx running at least one forked process in a docker compose managed container here | 18:45 |
| tobiash | we used docker-compose too back with zuulv2 | 18:45 |
| jlk | I did, a while ago. It's been a while though. | 18:45 |
| tobiash | bramwelt which gearman server do you use? | 18:46 |
| bramwelt | Right, I figure it was just my config then, as I'm using the same zuul.conf for all the services. | 18:46 |
| tobiash | there are a ton of them out there | 18:46 |
| bramwelt | just kendu/gearman | 18:46 |
| bramwelt | I believe it installs and runs whatever package is in Ubuntu | 18:47 |
| pabelanger | one reason to consider switching away from statsd is https://github.com/etsy/statsd doesn't really seem to be active any more | 18:47 |
| pabelanger | eg: is this project dead: https://github.com/etsy/statsd/issues/646 | 18:47 |
| corvus | pabelanger: in many cases, that's a feature, not a bug :) | 18:47 |
| tobiash | corvus: I think if the decision is to use *one* metric system we should stick with statsd. I personally would love to switch to prometheus but that would close the door for everything else. While using prometheus with statsd is possible with some effort. | 18:47 |
| pabelanger | corvus: agree. I just think in the neat future, with dependency changes to npm, it might become harder to install statsd. So far, it does still work | 18:48 |
| corvus | bramwelt: it should be okay to use the same zuul.conf for all services | 18:49 |
| tobiash | bramwelt: back then I used gearmand but it's really much easier to use start the gearman server together with the zuul-scheduler | 18:49 |
| tobiash | bramwelt: further only the python gear server supports the ssl encryption and client certificates afaik | 18:49 |
| mordred | I thought Shrews added ssl support to the gearmand server a while back | 18:50 |
| Shrews | i seem to recall doing that for geard (not gearmand) | 18:51 |
| Shrews | also, hi mordred | 18:51 |
| mordred | Shrews: ohai | 18:51 |
| Shrews | oh, did i also add it to gearmand??? i forget, but that may be why i did it for geard | 18:52 |
| tobiash | at least I couldn't get ssl to work with zuul3 and gearmand so I switched to geard with zuul3 | 18:52 |
| mordred | Shrews: I thought that's why you added it to the python ... but we'd be talking 6 years ago or so IIRC | 18:53 |
| Shrews | mordred: i don't remember what i did yesterday | 18:53 |
| mordred | also - there was a long while before SpamapS took it over that gearmand had a bunch of unrelased changed | 18:54 |
| mordred | changes | 18:54 |
| tobiash | while we're talking about gearman, I created a change to gear that solves a infinite merger freeze in case the scheduler vm crashed and the scheduler gets rescheduled on a different vm | 18:54 |
| tobiash | https://review.openstack.org/599567 | 18:54 |
| tobiash | and https://review.openstack.org/599573 would be the zuul part of it | 18:55 |
| Shrews | mordred: ah, i did the cli part: https://bazaar.launchpad.net/~tangent-trunk/gearmand/1.2/revision/819 | 18:57 |
| Shrews | pretty sure brian added wolfssl support | 18:58 |
| mordred | Shrews: so long ago | 18:59 |
| tobiash | corvus, tristanC: these are our current zuul statsd_exporter mappings: http://paste.openstack.org/show/729542/ | 19:10 |
| tobiash | and these for nodepool: http://paste.openstack.org/show/729541/ | 19:11 |
| tobiash | so if our decision will be to stick with statsd I'd like to have them maintained in zuul so we can have them in parallel and run a statsd_exporter as a sidecar container to zuul with the upstream maintained mappings | 19:12 |
| clarkb | tobiash: mordred pabelanger can we get a second review on https://review.openstack.org/#/c/600135/1 to fix a bug with the wraning that was added there | 19:19 |
| mordred | tobiash: that seems like a reasonable possibility to me | 19:19 |
| mordred | clarkb: done | 19:20 |
| clarkb | tyty | 19:20 |
| *** openstackgerrit has joined #zuul | 19:28 | |
| openstackgerrit | Merged openstack-infra/zuul-sphinx master: Ignore non-directory entries in roles/ https://review.openstack.org/600135 | 19:28 |
| clarkb | https://review.openstack.org/#/c/600137/1 should be mergeable now that ^ is in | 19:29 |
| clarkb | nevermind apparenrly we need to release zuul-sphinx first | 19:30 |
| corvus | clarkb: how about i tag that as 0.2.5 | 19:34 |
| *** AJaeger has joined #zuul | 19:34 | |
| clarkb | wfm ajaeger asking about in in #openstack-infra too | 19:34 |
| clarkb | I think if we make the release then we don't need 600137 and the change it reverts at all | 19:34 |
| corvus | oh that hasn't merged yet? :) | 19:35 |
| clarkb | indeed | 19:35 |
| corvus | tag pushed | 19:35 |
| AJaeger | corvus: no, failed first time ;( | 19:35 |
| AJaeger | ok, I'll rebase and abandon... | 19:35 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Add missing README.rst for roles https://review.openstack.org/600128 | 19:35 |
| AJaeger | until 600128 starts testing, we should be good - zuul is very busy ;( | 19:36 |
| SpamapS | bramwelt: regarding the forked geard.. I had similar issues but it's probably just that you don't have a logging config for it. | 19:37 |
| Shrews | mordred: oh! i have a present for you: https://hub.docker.com/u/zuul/dashboard/ | 19:37 |
| SpamapS | gearmand proper I believe is still missing native TLS support and thus I prefer the internal geard. | 19:37 |
| mordred | SpamapS: that's super cool! | 19:38 |
| mordred | gah | 19:38 |
| mordred | Shrews: that's super cool! | 19:38 |
| SpamapS | mordred: and no, I haven't used docker compose.. but I'm learning it right now for a side project that I want to convert to kubernetes. :) | 19:38 |
| mordred | Shrews: we're pushing those images on comits now? | 19:39 |
| Shrews | mordred: yup | 19:39 |
| mordred | \o/ | 19:39 |
| tobiash | mordred: I get 404 on that | 19:39 |
| mordred | tobiash: you get 404 on the dockerhub link? | 19:40 |
| Shrews | mordred: but we could not use protected/final because of the job var. we use allowed-projects based on corvus' suggestion | 19:40 |
| tobiash | yes | 19:40 |
| mordred | tobiash: what about for https://hub.docker.com/r/zuul/ ? | 19:40 |
| corvus | Shrews, mordred: yeah, we need to rework that job, but it's good enough for nodepool/zuul now | 19:40 |
| tobiash | mordred: that works | 19:41 |
| tobiash | mordred: is that dashboard link some owner stuff? | 19:41 |
| mordred | might be - yeah | 19:41 |
| Shrews | tobiash: probably (i'm logged in as the owner now) | 19:41 |
| mordred | me too | 19:41 |
| tobiash | I don't even have an account there ;) | 19:42 |
| Shrews | i don't think you need one to pull an image | 19:42 |
| clarkb | ya they have anonymous downloads | 19:43 |
| clarkb | openstack infra relies on that with our proxies caching the images all over | 19:44 |
| tobiash | yes, pulling works | 19:46 |
| AJaeger | zuul-sphinx 0.2.5 is on pypi now... | 19:48 |
| pabelanger | oh, speaking of zuul containers, I think https://review.openstack.org/597224/ removes some left over apk indexes in images | 19:52 |
| pabelanger | Shrews: mordred: clarkb: ^ | 19:52 |
| mordred | pabelanger: +A | 19:53 |
| pabelanger | Yay | 19:53 |
| *** hasharAway has quit IRC | 20:00 | |
| dmsimard | mordred: welcome back? :) | 20:02 |
| mordred | dmsimard: thanks! :) | 20:04 |
| openstackgerrit | Merged openstack/pbrx master: Don't store apk indexes in cache https://review.openstack.org/597224 | 20:13 |
| corvus | bramwelt: are you using the images on dockerhub? https://hub.docker.com/r/zuul/ | 20:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Remove restriction on add_host https://review.openstack.org/600205 | 20:35 |
| mordred | corvus, clarkb: ^^ | 20:35 |
| *** pcaruana has quit IRC | 20:40 | |
| *** sshnaidm|afk has quit IRC | 20:54 | |
| *** sshnaidm has joined #zuul | 21:00 | |
| *** sshnaidm has quit IRC | 21:01 | |
| *** samccann has quit IRC | 21:14 | |
| *** jesusaur has quit IRC | 21:22 | |
| *** jesusaur has joined #zuul | 21:23 | |
| *** rfolco has quit IRC | 21:52 | |
| bramwelt | corvus: Uh..no. I looked and those never showed up. | 21:59 |
| bramwelt | Oh, they just got built huh? :) | 21:59 |
| corvus | bramwelt: they're brand new, we just started publishing them. but they should be published on every commit. they may be helpful (or they may throw a wrench in your plans). depending on how adventurous you are feeling. :) | 21:59 |
| bramwelt | Haha, awesome! Can you point me to the Dockerfile for them? | 22:00 |
| bramwelt | Er wait, I think I see they're built through Python. :) | 22:02 |
| *** sshnaidm has joined #zuul | 22:06 | |
| *** sshnaidm is now known as sshnaidm|off | 22:08 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add missing README.rst for roles https://review.openstack.org/600128 | 23:34 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!