| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Get executor job params https://review.opendev.org/c/zuul/zuul/+/607078 | 00:16 |
|---|---|---|
| *** jamesmcarthur has joined #zuul | 00:16 | |
| *** jamesmcarthur has quit IRC | 00:20 | |
| *** jamesmcarthur has joined #zuul | 00:20 | |
| *** cloudnull has quit IRC | 00:27 | |
| *** cloudnull has joined #zuul | 00:27 | |
| *** tosky has quit IRC | 00:51 | |
| *** vishalmanchanda has joined #zuul | 03:27 | |
| *** ianychoi_ has joined #zuul | 03:27 | |
| *** ianychoi__ has quit IRC | 03:30 | |
| *** jamesmcarthur has quit IRC | 04:33 | |
| *** jamesmcarthur has joined #zuul | 04:44 | |
| *** ykarel has joined #zuul | 04:50 | |
| *** jfoufas1 has joined #zuul | 04:58 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:33 | |
| *** saneax has joined #zuul | 05:38 | |
| *** ykarel_ has joined #zuul | 06:08 | |
| *** ykarel has quit IRC | 06:08 | |
| *** ykarel_ has quit IRC | 06:31 | |
| *** ykarel has joined #zuul | 06:32 | |
| *** saneax is now known as saneax_AFK | 06:51 | |
| *** jamesmcarthur has quit IRC | 06:58 | |
| *** jamesmcarthur has joined #zuul | 06:59 | |
| *** jcapitao has joined #zuul | 07:02 | |
| *** jamesmcarthur has quit IRC | 07:05 | |
| *** jamesmcarthur has joined #zuul | 07:29 | |
| *** hashar has joined #zuul | 07:48 | |
| *** ianychoi_ has quit IRC | 08:32 | |
| *** jamesmcarthur has quit IRC | 08:40 | |
| *** tosky has joined #zuul | 08:51 | |
| zbr | tobiash: can you help with https://review.opendev.org/c/zuul/zuul-jobs/+/773245 ? | 08:55 |
| *** jamesmcarthur has joined #zuul | 08:55 | |
| *** jpena|off is now known as jpena | 08:58 | |
| *** jamesmcarthur has quit IRC | 09:02 | |
| *** nils has joined #zuul | 09:16 | |
| *** jamesmcarthur has joined #zuul | 09:34 | |
| *** jamesmcarthur has quit IRC | 09:43 | |
| openstackgerrit | Albin Vass proposed zuul/zuul master: WIP: Filter events on event connection https://review.opendev.org/c/zuul/zuul/+/760907 | 09:44 |
| openstackgerrit | Albin Vass proposed zuul/zuul master: WIP: Filter events on event connection https://review.opendev.org/c/zuul/zuul/+/760907 | 09:47 |
| openstackgerrit | Albin Vass proposed zuul/zuul master: Filter events on event connection https://review.opendev.org/c/zuul/zuul/+/760907 | 09:49 |
| *** jamesmcarthur has joined #zuul | 09:56 | |
| *** jamesmcarthur has quit IRC | 10:01 | |
| openstackgerrit | Albin Vass proposed zuul/zuul master: Filter events on event connection https://review.opendev.org/c/zuul/zuul/+/760907 | 10:11 |
| avass | added a releasenote for the bugfix ^ | 10:11 |
| *** saneax_AFK is now known as saneax | 10:14 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/c/zuul/zuul/+/734082 | 10:31 |
| *** jamesmcarthur has joined #zuul | 10:32 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add authentication-realm attribute to tenants https://review.opendev.org/c/zuul/zuul/+/735586 | 10:32 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/c/zuul/zuul/+/734850 | 10:33 |
| openstackgerrit | Albin Vass proposed zuul/zuul master: Filter events on event connection https://review.opendev.org/c/zuul/zuul/+/760907 | 10:36 |
| *** jamesmcarthur has quit IRC | 10:39 | |
| *** jangutter has joined #zuul | 10:42 | |
| *** jangutter_ has quit IRC | 10:45 | |
| *** jamesmcarthur has joined #zuul | 10:56 | |
| *** bodgix has quit IRC | 10:59 | |
| *** bodgix_ has joined #zuul | 10:59 | |
| *** jamesmcarthur has quit IRC | 11:01 | |
| *** Phoenikzz has joined #zuul | 11:02 | |
| *** jangutter_ has joined #zuul | 11:06 | |
| *** jangutter_ has quit IRC | 11:07 | |
| *** jangutter_ has joined #zuul | 11:08 | |
| *** jangutter has quit IRC | 11:09 | |
| *** jcapitao is now known as jcapitao_lunch | 11:44 | |
| openstackgerrit | Merged zuul/zuul-jobs master: bindep.txt: skip python-devel for el8 platform https://review.opendev.org/c/zuul/zuul-jobs/+/780050 | 11:47 |
| *** hashar is now known as hasharLunch | 12:10 | |
| dry | covrus: I've seen your change 778814 in nodepool, to use own Azul client for Azure API. I have to access Azure API by HTTP proxy, and configuring it in current client is PITA. Are there any reasons not to merge this soon? | 12:21 |
| *** jpena is now known as jpena|lunch | 12:32 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Separate out executor server from runner https://review.opendev.org/c/zuul/zuul/+/607079 | 12:45 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Move repository preparation into common class https://review.opendev.org/c/zuul/zuul/+/648642 | 12:45 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Separate out executor concerns from AnsibleJob https://review.opendev.org/c/zuul/zuul/+/648643 | 12:48 |
| tristanC | zuul-maint: i'm working on implementing the zuul-runner spec, it seems like the above refactoring patch are correct and i would really appreciate some reviews or estimate when this should be merged. If there are other pressing work, i'd be happy to review them first and rebase the zuul-runner on it | 12:52 |
| *** jamesmcarthur has joined #zuul | 12:57 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: implement prep-workspace https://review.opendev.org/c/zuul/zuul/+/607082 | 12:57 |
| *** hasharLunch is now known as hashar | 13:00 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add configuration schema https://review.opendev.org/c/zuul/zuul/+/640672 | 13:02 |
| *** jamesmcarthur has quit IRC | 13:06 | |
| *** ykarel has quit IRC | 13:08 | |
| *** ykarel has joined #zuul | 13:09 | |
| *** jcapitao_lunch is now known as jcapitao | 13:11 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] run TestSchedulerSSL test with pending gear change https://review.opendev.org/c/zuul/zuul/+/780261 | 13:31 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] run TestSchedulerSSL test with pending gear change https://review.opendev.org/c/zuul/zuul/+/780261 | 13:39 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] run TestSchedulerSSL test with pending gear change https://review.opendev.org/c/zuul/zuul/+/780261 | 13:40 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] run TestSchedulerSSL test with pending gear change https://review.opendev.org/c/zuul/zuul/+/780261 | 13:42 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: implement prep-workspace https://review.opendev.org/c/zuul/zuul/+/607082 | 13:44 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add configuration schema https://review.opendev.org/c/zuul/zuul/+/640672 | 13:44 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add initial command line https://review.opendev.org/c/zuul/zuul/+/644770 | 13:44 |
| *** jangutter has joined #zuul | 13:45 | |
| *** jangutter_ has quit IRC | 13:48 | |
| *** jpena|lunch is now known as jpena | 13:49 | |
| mordred | corvus: I did the second +2 on the azul change but didn't +A - not sure if we're waiting on anything - if not, feel free to +A | 13:50 |
| mordred | dry: ^^ | 13:50 |
| *** hashar is now known as hasharAway | 14:20 | |
| Open10K8S | corvus: regarding https://review.opendev.org/c/zuul/zuul-jobs/+/776677, | 14:23 |
| Open10K8S | https://review.opendev.org/c/opendev/base-jobs/+/777087 is for 3rd. | 14:23 |
| Open10K8S | and how can we do 4th task? I am not sure how to Announce on the zuul-announce list about the change. OR are zuul dev guys responsible for this ? | 14:25 |
| *** hasharAway is now known as hashar | 14:49 | |
| avass | mordred, dry: I'll take a look at it as well since we're going to use it. | 14:55 |
| avass | dry: I think we're going to have the same problem with proxy | 14:56 |
| fungi | why do web proxies cause problems for azure's api? | 14:57 |
| avass | fungi: usually it's the library causing problems for some reason. the kubernetes package has a problem like that as well where it ignores the proxy settings | 14:58 |
| avass | fungi: unless you do this... : https://github.com/kubernetes-client/python/issues/333#issuecomment-398087826 | 14:59 |
| fungi | ahh, that seems more tractable at least. if we're implementing with the requests library, i think it even has its own support for standard proxy envvars? | 14:59 |
| avass | yeah requests library just works | 14:59 |
| fungi | so why will we have the same problem with proxies? or by "we" you meant your use of the original azure client, not "we" in the sense of the new azure client we're implementing in zuul? | 15:01 |
| avass | I mean we as in volvo will have proxy problems as will if we started using the azure driver right now. but if that uses requests I don't think azul will have any problems | 15:02 |
| mordred | I was reading "we" to mean volvo - and that avass would review the azul patch as it would also potentially alleviate such an issue for them as well | 15:02 |
| * mordred read right! | 15:02 | |
| corvus | dry, avass, mordred: yeah, i think we can merge that first change; i'm continuing work on the state machine driver which should continue to improve efficiency and features; no rush on that | 15:02 |
| avass | yeah that was ambiguous :) | 15:02 |
| corvus | it would be good to get early feedback on any problems from switching to azul, before we do the state machine stuff | 15:02 |
| corvus | (i am live-testing all of these changes, so have high confidence they work, but it's still not exactly stress-testing) | 15:03 |
| *** jamesmcarthur has joined #zuul | 15:04 | |
| corvus | Open10K8S: thanks, and you can just send an email to zuul-announce. it will go into the moderation queue, and i can approve it at the appropriate time | 15:04 |
| corvus | dry: we probably won't make a nodepool release until all the azure work is finished -- are you able to run from master, or do you run releases? | 15:05 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Separate out executor concerns from AnsibleJob https://review.opendev.org/c/zuul/zuul/+/648643 | 15:05 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: implement prep-workspace https://review.opendev.org/c/zuul/zuul/+/607082 | 15:07 |
| avass | corvus: I think one thing that could be missing from azul is the possibility to use azure managed service identity. if i understand that correctly it works like instance profiles in aws. I don't think we're gonna use it but if someone is (and the azure library supports it) that could break something | 15:09 |
| *** jamesmcarthur has quit IRC | 15:09 | |
| corvus | avass: do you mean nodepool using a managed service identity to launch nodes? (or do you mean launching nodes with service identities attached to them for use in jobs?) | 15:10 |
| avass | the first | 15:10 |
| corvus | we can probably support that in azul -- it's not actually clear to me that it's really supported in the current driver | 15:11 |
| corvus | (to support it in the current driver, we would need to plumb through some options in nodepool; with azul, we would need to do that and also implement the token fetching) | 15:12 |
| avass | yeah just saw the get_client_from_auth_file so I guess that requires an auth file. boto just works and grabs any credentials it can find :) | 15:13 |
| corvus | avass: yeah -- the current azure nodepool driver also hardcodes the file requirement too | 15:13 |
| corvus | avass: but i'd like it to not only support msi but also certs | 15:14 |
| corvus | usually the process is basically you ask a well-known metadata endpoint for an oauth token | 15:14 |
| corvus | i haven't tried it with azure yet, but it looks similar | 15:14 |
| avass | yeah | 15:15 |
| corvus | this does it for google cloud: https://opendev.org/zuul/gcp-authdaemon/src/branch/master/authdaemon.py#L38-L73 | 15:15 |
| dry | covrus: I run on master with few patches | 15:16 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/c/zuul/zuul/+/734850 | 15:16 |
| corvus | but nexit on my list is ipv6 support, then quota | 15:16 |
| *** jfoufas1 has quit IRC | 15:17 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/c/zuul/zuul/+/736772 | 15:17 |
| corvus | dry: great, any patches in review upstream? or just local patches? | 15:17 |
| dry | I've seen your work on state machine driver, and wondered if this first change is intended to work alone | 15:18 |
| corvus | dry: it is | 15:18 |
| corvus | the state machine changes after it should be considered together, but the initial azul change is stand-alone | 15:19 |
| avass | corvus: yeah aws/azure uses 169.254.169.254 for that | 15:20 |
| dry | just local patches: local docker registry in Dockerfile, support for Sentry, patched DIB dependency, local CA in trusted certs | 15:20 |
| corvus | cool, sounds like it shouldn't be too hard to try it out then | 15:21 |
| *** jamesmcarthur has joined #zuul | 15:22 | |
| avass | corvus: lgtm | 15:25 |
| dry | fungi: I'm adding proxy setting for Azure in providers, envvars would make all HTTPS calls use proxy, and i need it just for Azure | 15:26 |
| corvus | dry: sounds like a good change | 15:26 |
| avass | dry: unless you specify no_proxy. but yeah being able to specify proxy for the azure driver only is good. | 15:27 |
| corvus | i approved the change; dry you should be ready to put this into production right before you go home for the weekend, i assume. ;) | 15:28 |
| *** jamesmcarthur has quit IRC | 15:28 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/c/zuul/zuul/+/734082 | 15:39 |
| *** jamesmcarthur has joined #zuul | 15:40 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add authentication-realm attribute to tenants https://review.opendev.org/c/zuul/zuul/+/735586 | 15:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/c/zuul/zuul/+/734850 | 15:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/c/zuul/zuul/+/736772 | 15:41 |
| *** jamesmcarthur has quit IRC | 15:47 | |
| *** jangutter has quit IRC | 15:51 | |
| *** jangutter has joined #zuul | 15:51 | |
| dry | covrus: Thanks! actually deploying changes in Zuul on Friday is safer - this way only few people working on weekend are affected, not over 400 of them :-) | 15:52 |
| corvus | dry: yeah, i'm actually planning on restarting opendev's zuul today for that reason :) | 15:53 |
| avass | hehe we do usually do the same :D | 15:58 |
| corvus | i'm looking forward to v5 when we can just do rolling restarts all the time :) | 15:59 |
| *** jamesmcarthur has joined #zuul | 15:59 | |
| *** hashar is now known as hasharAway | 16:06 | |
| *** jamesmcarthur has quit IRC | 16:06 | |
| *** hasharAway is now known as hashar | 16:11 | |
| fungi | dry: oh, yep, i just meant proxy configuration features in nodepool notwithstanding, you could even just set the relevant envvar in the calling environment for the launcher and requests would respect that | 16:19 |
| fungi | but i agree supporting proxies in configuration makes sense too | 16:19 |
| *** jamesmcarthur has joined #zuul | 16:20 | |
| *** jamesmcarthur has quit IRC | 16:25 | |
| clarkb | and if you need it only for azure you could run an azure specific launcher | 16:26 |
| *** jamesmcarthur has joined #zuul | 16:37 | |
| *** jamesmcarthur has quit IRC | 16:43 | |
| clarkb | I'm starting to look at that launcher issue with deleting instances. The first thing I notice is we do have a number of valid "stuck" deleting instances. The instances are in an error state according to the cloud but do exist. For those I would expect the records to stick around until the instances go away somehow | 16:51 |
| clarkb | the other are the ones I called out yesterday where the cloud says the instance does not exist, but we somehow keep a dummy record around for it. I am going to dig into this second set | 16:51 |
| corvus | yep, reflects reality, makes quota calcs work better, and keeps us deleting them | 16:51 |
| *** bridgefan has joined #zuul | 16:54 | |
| clarkb | corvus: in zk I see /nodepool/nodes/0000318268/lock/data__lock_otherdata | 16:54 |
| clarkb | how do I map that data/otherdata to something useful? | 16:54 |
| clarkb | looks like this may come from kazoo /me finds kazoo docs | 16:56 |
| *** jamesmcarthur has joined #zuul | 16:57 | |
| bridgefan | Hi! I have a question on zuul's tenant config file. I am looking to add many git repos with similar name: ABC/name1 ABC/name2 ABC/name3, can I use ABC/** or ABC/*? | 16:57 |
| bridgefan | or do I need to list them all out? | 16:57 |
| corvus | bridgefan: i think they all need to be listed | 16:58 |
| bridgefan | ok thanks | 16:58 |
| corvus | bridgefan: if you're doing *a lot* you might consider generating the file automatically (you can also have zuul call a script to output the data) | 16:58 |
| corvus | (the option for that is called tenant_config_script or something like that) | 16:58 |
| *** hashar has quit IRC | 16:59 | |
| clarkb | we don't seem to set the Lock identifier argument on node locks | 16:59 |
| avass | yup | 16:59 |
| openstackgerrit | Merged zuul/nodepool master: Azure: switch to Azul https://review.opendev.org/c/zuul/nodepool/+/778814 | 16:59 |
| corvus | clarkb: we don't seem to have 'dump' and 'stat' in our zk whitelist :( | 16:59 |
| avass | we use it to serve the tenant config on an http endpoint to get around configmap update delays and not need to restart the scheduler. | 17:00 |
| corvus | avass: you shouldn't need to restart the scheduler to update the tenant config regardless of whether it's a file or a script (but yes, configmaps can take a while to update) | 17:01 |
| avass | yeah but a restart can be quicker for some services to update their configmaps. zuul-scheduler is not one of those | 17:01 |
| *** jamesmcarthur has quit IRC | 17:02 | |
| clarkb | corvus: exists (stat is an alias for this) does seem to work for me but ya dump does not | 17:02 |
| corvus | avass: in gerrit's zuul, i put the config in a secret because it is not subject to that delay | 17:02 |
| avass | corvus: oh I didn't know that | 17:03 |
| clarkb | oh I see the stat exists alias thing is a zk-shell ism? | 17:04 |
| clarkb | if I try to stat directly with nc then I get the whitelist problem | 17:04 |
| avass | corvus: according to the docs it sounds like the delay is the same as configmaps so I suppose it's depending on the cluster config | 17:06 |
| *** jcapitao has quit IRC | 17:06 | |
| clarkb | corvus: ZOO_4LW_COMMANDS_WHITELIST seems to control this on the docker images | 17:07 |
| corvus | avass: hrm :/ oh! i'm wrong, sorry! i put in a wait | 17:07 |
| clarkb | though we also mount in a zoo.cfg | 17:08 |
| corvus | avass: https://gerrit.googlesource.com/zuul/ops/+/refs/heads/master/playbooks/deploy.yaml#141 | 17:08 |
| corvus | clarkb: i think it's worth enabling those | 17:08 |
| avass | corvus: I believe that can take up to 2min to update in worst case. | 17:08 |
| corvus | avass: good, i set the timeout for 5m :) | 17:09 |
| corvus | avass: take a look -- it waits for the content to match, so it'll run as soon as it's actually updated | 17:09 |
| corvus | seems to have worked pretty well so far | 17:09 |
| corvus | but if the script option is easy in your env, that's even better/faster | 17:10 |
| avass | corvus: yeah it's just checking out a git repo and reads a tenant config file and serves that on an endpoint. so far it's been working great :) | 17:11 |
| *** jamesmcarthur has joined #zuul | 17:11 | |
| corvus | avass: maybe we should add a tenant_config_url option | 17:12 |
| avass | and it's all in the helm chart which makes it a bit easier to deploy in a local test environment | 17:12 |
| avass | corvus: ++ | 17:12 |
| *** jpena is now known as jpena|brb | 17:12 | |
| avass | corvus: I had an idea is to be able to point to single tenant configs and merge them as well. but we haven't had any use for that yet | 17:12 |
| avass | so the tenants can handle their own config | 17:13 |
| *** vishalmanchanda has quit IRC | 17:26 | |
| *** jangutter has quit IRC | 17:26 | |
| *** jangutter has joined #zuul | 17:27 | |
| *** stevthedev has left #zuul | 17:43 | |
| clarkb | I have been able to check ephemeral node session ids to confirm that the right launcher is locking the ephemeral zk node for the cloud nodes that do not exist | 17:46 |
| clarkb | I'm fairly confident that the zk stuff is all correct (eg no stale launchers somewhere hanging on to a lock) | 17:47 |
| clarkb | this led me back to the nodepool service logs to see if I could find any more info there about failures doing the deletion and that led me to http://paste.openstack.org/show/1aUgmWSyjF6bBrUNsBX1/ | 17:47 |
| clarkb | I suspect that nodepool is asking the cloud to delete the instance but openstacksdk is raising an exception because the instance does not exist anymore? | 17:47 |
| clarkb | I need to go read that code next | 17:48 |
| clarkb | oh interesting, no we've set a 10 minute timeout in nodepool to wait for the deletion | 17:49 |
| clarkb | which means openstacksdk isn't returning a result within 10 minutes? | 17:50 |
| mordred | clarkb: openstacksdk should never raise an exception if you ask to delete something that's not there | 17:52 |
| clarkb | this gets more interesting because the code that triggers that event seems to do a server listing then for every server in the cleanup queue that is not in the cloud server listing the event is set. That would imply the server that is failign to delete is somehow still in that list but if I do a show on it it doesn't show up | 17:52 |
| mordred | deleting a non-existent thing is considered a success - you asked for it to not exist, it does not exist, win! | 17:52 |
| clarkb | mordred: ya I thought the exception was bubbling up from sdk but it isn't, it is a t imeout internal to nodepool | 17:52 |
| mordred | cool | 17:52 |
| * clarkb tries to list servers instead of doing a show | 17:53 | |
| mordred | maybe the remote codepaths are different for list than show | 17:53 |
| mordred | yeah | 17:53 |
| clarkb | wow yes list and show produce different results | 17:54 |
| clarkb | the uuid shows up in list as stuck in BUILD | 17:54 |
| clarkb | if I copy the uuid from the list output and give it to show I get told the server does not exist | 17:54 |
| *** jpena|brb is now known as jpena | 17:54 | |
| clarkb | since the bulk of these are in ovh and ovh is currently very distracted by other more improtant problems I think I'll call this debugged for now | 17:55 |
| clarkb | nodepool is doing the right thing based on its use of list | 17:55 |
| corvus | clarkb: wow nice find | 17:55 |
| clarkb | I get to add a new item to my list of weird openstack behaviors today :) | 17:56 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/c/zuul/zuul/+/736772 | 18:02 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Web UI: allow a privileged user to request autohold https://review.opendev.org/c/zuul/zuul/+/768115 | 18:02 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | 18:03 |
| *** ykarel has quit IRC | 18:04 | |
| mhu | hi folks, I've reworked the admin web ui patch chain at https://review.opendev.org/q/topic:%22fffaff%22+(status:open%20OR%20status:merged) | 18:07 |
| mhu | I'm going to be on paternity leave pretty soon so I'd like to get feedback while I'm still not sleep deprived or covered in baby barf | 18:07 |
| mhu | I think Clément's patch @ https://review.opendev.org/c/zuul/zuul/+/767691 would be a good addition, but it hasn't been touched in a month - I'll have a look at it on monday | 18:09 |
| mhu | Unless someone knows if he is on IRC and I can ping him | 18:09 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add configuration schema https://review.opendev.org/c/zuul/zuul/+/640672 | 18:12 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add initial command line https://review.opendev.org/c/zuul/zuul/+/644770 | 18:12 |
| *** hamalq has joined #zuul | 18:14 | |
| *** jangutter_ has joined #zuul | 18:30 | |
| *** nils has quit IRC | 18:32 | |
| *** jangutter has quit IRC | 18:34 | |
| *** jpena is now known as jpena|off | 18:44 | |
| *** jamesmcarthur has quit IRC | 19:11 | |
| *** jamesmcarthur has joined #zuul | 19:16 | |
| *** jamesmcarthur has quit IRC | 19:39 | |
| *** jamesmcarthur has joined #zuul | 19:39 | |
| *** hashar has joined #zuul | 20:07 | |
| hamalq | hi i have a question about thid PR https://review.opendev.org/c/openstack/designate-tempest-plugin/+/755876 which depends on another change and i followed the https://zuul-ci.org/docs/zuul/discussion/gating.html#cross-project-dependencies to add that change but still not ttaken | 20:14 |
| hamalq | so basically depends-on not working for mee | 20:15 |
| fungi | hamalq: so it's a openstack/designate-tempest-plugin change declaring a cross-project dependency on a openstack/designate change | 20:17 |
| fungi | which of the jobs which ran for the openstack/designate-tempest-plugin change did you expect to consume openstack/designate from source? | 20:17 |
| fungi | the failing designate-tempest-plugin-split-horizon job? | 20:18 |
| hamalq | yes | 20:18 |
| hamalq | that one | 20:18 |
| fungi | we should be able to tell from its logs where it got a copy of openstack/designate from | 20:18 |
| hamalq | should take the change https://review.opendev.org/c/openstack/designate/+/748285 | 20:18 |
| fungi | hamalq: https://zuul.opendev.org/t/openstack/build/9e955193ee114062a218cec2c5111d9c/log/controller/logs/devstacklog.txt#63 | 20:20 |
| fungi | 2021-03-12 19:19:10.454 | fccff4a7 Merge commit 'refs/changes/85/748285/40' of ssh://review.opendev.org:29418/openstack/designate into HEAD | 20:21 |
| fungi | so devstack should be deploying that from source | 20:21 |
| hamalq | but this https://review.opendev.org/c/openstack/designate/+/748285 deepends also on https://review.opendev.org/c/openstack/designate/+/755379/6 | 20:22 |
| hamalq | should i add 755379 as depends-on | 20:23 |
| hamalq | ? | 20:24 |
| fungi | this indicates it should have been included in that ref zuul created: https://zuul.opendev.org/t/openstack/build/9e955193ee114062a218cec2c5111d9c/log/zuul-info/inventory.yaml#150 | 20:24 |
| hamalq | so all the changes is already there right? | 20:25 |
| fungi | that's what the log indicates | 20:35 |
| fungi | hamalq: the fccff4a7 ref checked out by devstack was constructed by zuul to include changes 755379, 748285 and 755876 | 20:36 |
| fungi | merged on to the (then current) state of the openstack/designate master branch | 20:38 |
| *** jamesmcarthur has quit IRC | 20:41 | |
| hamalq | fungi: thanks i learned allot on how to debug zuul | 21:02 |
| hamalq | thanks | 21:02 |
| fungi | hamalq: you're welcome, let us know if you have more questions | 21:02 |
| fungi | for future reference, that zuul inventory should be archived by all jobs which run in opendev (it's a zuul-jobs feature we incorporate into our base jobs), and that reports all the state information zuul used to set up the build | 21:04 |
| fungi | probably the single most useful bit of user-facing data for debugging zuul jobs | 21:05 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: implement local job execution https://review.opendev.org/c/zuul/zuul/+/630944 | 21:20 |
| hamalq | devstack_local_conf: | 21:30 |
| hamalq | post-config: | 21:30 |
| hamalq | "$DESIGNATE_CONFIG": | 21:30 |
| hamalq | "service:mdns": | 21:30 |
| hamalq | external_view_tsigkeys: external | 21:30 |
| hamalq | fungi: i added the following config above how to know its taken in desingnate conf | 21:31 |
| clarkb | hamalq: the job should log the service configs and you can confirm it that way | 21:32 |
| fungi | hamalq: that's probably more of a question for the #openstack-qa channel, they're the ones maintaining the job which consumes that configuration | 21:32 |
| hamalq | ok thanks | 21:32 |
| *** hashar has quit IRC | 21:37 | |
| *** jamesmcarthur has joined #zuul | 21:48 | |
| *** jamesmcarthur has quit IRC | 22:04 | |
| *** jamesmcarthur has joined #zuul | 22:07 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Component Registry in ZooKeeper https://review.opendev.org/c/zuul/zuul/+/759187 | 22:11 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Move management and result events to model https://review.opendev.org/c/zuul/zuul/+/761163 | 22:13 |
| openstackgerrit | Merged zuul/zuul master: Instantiate executor client, merger, nodepool and app within Scheduler https://review.opendev.org/c/zuul/zuul/+/779087 | 22:47 |
| *** jamesmcarthur has quit IRC | 22:48 | |
| *** saneax has quit IRC | 22:49 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Implementation of Zookeeper backed event queues https://review.opendev.org/c/zuul/zuul/+/761170 | 22:59 |
| *** jamesmcarthur has joined #zuul | 23:00 | |
| *** saneax has joined #zuul | 23:11 | |
| *** jamesmcarthur has quit IRC | 23:13 | |
| *** jamesmcarthur has joined #zuul | 23:14 | |
| *** sanjayu_ has joined #zuul | 23:15 | |
| *** saneax has quit IRC | 23:16 | |
| *** sanjayu__ has joined #zuul | 23:16 | |
| *** jamesmcarthur has quit IRC | 23:19 | |
| *** sanjayu_ has quit IRC | 23:20 | |
| *** jamesmcarthur has joined #zuul | 23:29 | |
| *** jamesmcarthur has quit IRC | 23:32 | |
| *** jamesmcarthur has joined #zuul | 23:36 | |
| *** jamesmcarthur has quit IRC | 23:45 | |
| *** jamesmcarthur has joined #zuul | 23:47 | |
| *** jamesmcarthur has quit IRC | 23:58 | |
| *** jamesmcarthur has joined #zuul | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!