Monday, 2019-11-18

*** bjackman has quit IRC00:11
*** jamesmcarthur has quit IRC00:14
*** jamesmcarthur has joined #zuul00:18
*** jamesmcarthur has quit IRC00:20
*** jamesmcarthur has joined #zuul00:39
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Fix zuul-stream-functional tests  https://review.opendev.org/69461900:41
Shrewsdmsimard: let us know when that dot release drops so we can get ^^ in.00:45
*** jamesmcarthur has quit IRC00:45
*** tosky has quit IRC00:46
*** jamesmcarthur has joined #zuul00:52
*** jamesmcarthur has quit IRC00:58
ianwargh, the shadow user creation doesn't actually work :/01:10
ianwin the dockerfile01:10
*** jamesmcarthur has joined #zuul01:14
*** jamesmcarthur has joined #zuul01:15
*** jamesmcarthur has quit IRC01:33
dmsimardShrews: the fix has landed but I want another patch in before tagging. If it's blocking something, maybe we can temporarily merge it with ignore_errors until the dot release is out01:36
dmsimardor install from git01:36
*** jamesmcarthur has joined #zuul02:17
*** bhavikdbavishi has joined #zuul02:30
*** bhavikdbavishi1 has joined #zuul02:33
*** bhavikdbavishi has quit IRC02:34
*** bhavikdbavishi1 is now known as bhavikdbavishi02:34
*** jamesmcarthur has quit IRC02:36
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: add user to shadow file too  https://review.opendev.org/69364702:51
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: create APP_DIR  https://review.opendev.org/69364602:51
openstackgerritIan Wienand proposed zuul/nodepool master: docker: Base nodepool images on diskimage-builder base image  https://review.opendev.org/69417702:51
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346402:51
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile : install sudo for nodepool-builder  https://review.opendev.org/69470902:51
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346403:49
*** bjackman has joined #zuul04:18
*** raukadah is now known as chandankumar05:44
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346405:45
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346406:12
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346406:18
openstackgerritIan Wienand proposed zuul/nodepool master: docker: Base nodepool images on diskimage-builder base image  https://review.opendev.org/69417706:31
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile : install sudo for nodepool-builder  https://review.opendev.org/69470906:31
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346406:32
*** yolanda has quit IRC06:45
*** bjackman has quit IRC07:02
*** themroc has joined #zuul07:27
*** gtema has joined #zuul07:31
*** jamesmcarthur has joined #zuul07:39
*** sileht has quit IRC07:53
*** sileht has joined #zuul07:53
*** gtema has quit IRC07:56
*** igordc has joined #zuul07:58
*** igordc has quit IRC08:02
*** tosky has joined #zuul08:20
*** jamesmcarthur has quit IRC08:21
*** jamesmcarthur has joined #zuul08:23
*** jamesmcarthur has quit IRC08:28
*** saneax has joined #zuul08:28
*** jamesmcarthur has joined #zuul08:38
*** jpena|off is now known as jpena08:49
*** jamesmcarthur has quit IRC09:07
*** jamesmcarthur has joined #zuul09:12
*** jamesmcarthur has quit IRC09:17
*** yolanda has joined #zuul09:22
*** jamesmcarthur has joined #zuul09:38
*** jamesmcarthur has quit IRC09:42
*** mhu has joined #zuul09:47
*** lennyb has quit IRC10:06
*** pcaruana has joined #zuul10:10
*** jamesmcarthur has joined #zuul10:12
*** jamesmcarthur has quit IRC10:17
*** sshnaidm|off is now known as sshnaidm|ruck10:20
*** sanjayu_ has joined #zuul10:51
*** saneax has quit IRC10:53
*** jamesmcarthur has joined #zuul10:59
*** jamesmcarthur has quit IRC11:04
*** sanjayu__ has joined #zuul11:42
*** sanjayu_ has quit IRC11:44
*** rfolco_pto has joined #zuul11:50
*** rfolco_pto is now known as rfolco11:58
*** jamesmcarthur has joined #zuul12:00
*** jamesmcarthur has quit IRC12:05
*** jpena is now known as jpena|lunch12:26
*** rlandy has joined #zuul12:56
*** rlandy is now known as rlandy|rover12:56
*** jamesmcarthur has joined #zuul13:02
*** jamesmcarthur has quit IRC13:08
*** jamesmcarthur_ has joined #zuul13:08
*** lennyb has joined #zuul13:09
*** jpena|lunch is now known as jpena13:25
*** jamesmcarthur_ has quit IRC13:33
*** jamesmcarthur has joined #zuul13:50
*** Goneri has joined #zuul13:57
*** lennyb has quit IRC14:14
pabelangerfungi: where can I find your script again, that generated the enqueue command for zuul cli, from failed post jobs?  IIRC, you use it when release jobs fail for openstack14:15
*** rlandy|rover is now known as rlandy|rover|mtg14:20
fungipabelanger: https://review.opendev.org/61367614:25
pabelangertyty14:25
pabelangerrechecking to see what angered the pep8 gods14:26
fungiit may need tweaking i'm not sure14:26
fungiahh, no, the version there is still identical to what i've been running for zuul.opendev.org14:29
fungiinvocation is currently like: python3 reenqueue.py https://174ab73dc2f37b91ee54-e84e491efa8a2348e120c66acb6b4680.ssl.cf1.rackcdn.com/da345a0810d85501366686ca56d2527cd306e6bf/release-post/publish-tox-docs-static/71c8b17/14:30
*** jamesmcarthur has quit IRC14:53
*** rlandy|rover|mtg is now known as rlandy|rover15:00
*** jamesmcarthur has joined #zuul15:05
*** jamesmcarthur has quit IRC15:08
*** jamesmcarthur has joined #zuul15:08
*** sanjayu__ has quit IRC15:28
*** Kepioo49 has joined #zuul15:33
corvuso/15:45
corvusi'm back at work this week -- though i have a bit of catch-up to do first15:45
mordredcorvus: welcome back! there are things for you to look at from when you were other places15:48
mordredyou know - whenever you get there and stuff15:48
corvusif there's anything urgent, let me know and i'll put it at the top of the list15:48
Shrewscorvus: welcome back. nothing too urgent atm, i believe15:51
*** bhavikdbavishi has quit IRC15:51
mordredcorvus: nothing urgent from my end15:53
openstackgerritPaul Belanger proposed zuul/zuul master: Add reenqueue utility  https://review.opendev.org/61367615:59
*** Kepioo49 has quit IRC16:09
*** jamesmcarthur_ has joined #zuul16:20
*** jamesmcarthur has quit IRC16:23
fungiif anyone has any zuul-related updates for this month's osf newsletter, i'm happy to get them included... just let me know16:28
*** vivobg has joined #zuul16:30
pabelangercorvus: the only thing we experenced in zuul.a.c, was getting rate limited by github ( 2 times for 1 hour each) api for unauthenticated search requests. This resulted in zuul being blocked for an extended period of time. Talking with tobiash, https://review.opendev.org/694390/ was suggested (needs testing)16:31
vivobgHello. I am trying to set up a Windows static node in nodepool, following the documentation in https://zuul-ci.org/docs/nodepool/configuration.html#value-providers.[static].pools.nodes.connection-type.winrm16:31
vivobgbut i don't know where to specify the password for the winrm user16:31
vivobgwhere does nodepool get the windows password from? Any help will be greatly appreciated. Thanks16:33
pabelangervivobg: I don't believe winrm uses passwords, you need to setup public / private key. tobiash may know more16:33
*** jamesmcarthur has joined #zuul16:53
*** jamesmcarthur_ has quit IRC16:56
*** jamesmcarthur has quit IRC17:01
*** jamesmcarthur has joined #zuul17:01
*** jamesmcarthur has quit IRC17:01
tobiashvivobg: zuul uses cert auth, nodepool specifies only port, conn type and user. The certs are needed on the executor and need to be mentioned in the (un)trusted_ro_paths of the executor config17:19
*** vivobg has quit IRC17:23
*** jamesmcarthur has joined #zuul17:32
ShrewsI've been fighting this feeling like I'm getting sick ever since yesterday afternoon. I think I have to surrender to it now and lie down for a while. Will check in later.17:36
AJaegerShrews: get well soon!17:37
*** jpena is now known as jpena|off17:47
*** sshnaidm|ruck is now known as sshnaidm|afk18:05
*** tjgresha has quit IRC18:18
*** gtema has joined #zuul18:24
*** igordc has joined #zuul18:46
*** gtema has quit IRC18:47
*** gouthamr_ is now known as gouthamr18:51
*** dmyrhorodskyi has joined #zuul19:24
dmsimardShrews, fungi: ara 0.16.6 was just tagged, it includes the fix for the binary output issue we found in zuul-stream-functional19:25
dmsimardwill be in a pypi mirror near you soon19:25
*** dmyrhorodskyi has quit IRC19:28
*** tosky has quit IRC19:34
*** michael-beaver has joined #zuul19:41
*** hogepodge has joined #zuul20:29
hogepodgeclarkb: hey, where do third party zuul connectors usually live? I had someone asking me about how to set up a node pool to connect to GitLab and I wanted to point them to sample source to get started20:29
clarkbhogepodge: depends on what you are trying to integrate. Zuul talks to code review systems like gerrit and gitlab and the code for that lives in zuul/zuul's tree (there is a change up to start a gitlab driver too). Nodepool talks to cloud resources and those drivers live in the zuul/nodepool tree. Things like openstack and kubernetes and azure drivers.20:33
fungihogepodge: like to have nodepool hand off resources from gitlab-ci?20:33
clarkbhogepodge: https://review.opendev.org/#/c/685682/ is the start of Zuul's gitlab driver20:34
fungialso not sure what you mean by "third party connectors" but assuming drivers20:34
clarkboh ya if you want what fungi describes that all happens via zookeeper and at least so far I only know of zuul and nodepool implementing that interface but other tools could implement it too20:34
fungiand they're all in-tree as far as i know, i don't think zuul has a pluggable driver interface (yet anyway)20:35
* hogepodge fungi: by connector I was just using the language here https://zuul-ci.org/docs/zuul/admin/connections.html20:36
clarkbhogepodge: for that https://review.opendev.org/#/c/685682/ is a good start for gitlab20:36
hogepodgebut someone is on GitLab, and they want to spin up K8s clusters from a nodepool to run CI jobs on. Sounded like a perfect place to integrate zuul so I wanted to be sure they had a good start20:37
hogepodgethanks clarkb and fungi, that's exactly what I was looking for20:39
fungiaha, yep, in that case what clarkb is talking about, plus the kubernetes driver in nodepool20:44
fungijust be aware the gitlab driver isn't finished yet, though i get the impression fbo would appreciate help trying it out20:45
fungithe use of "third party" there was throwing me off ;)20:46
hogepodgeanything not gerrit and opendev ;-)20:47
openstackgerritMerged zuul/zuul-jobs master: Role to install podman  https://review.opendev.org/69345520:52
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile : install sudo for nodepool-builder  https://review.opendev.org/69470921:03
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346421:03
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: add an ENABLE_DEBUG build arg/environment  https://review.opendev.org/69484521:03
*** michael-beaver has quit IRC21:56
*** rfolco has quit IRC22:04
*** mhu has quit IRC22:06
*** mhu has joined #zuul22:07
*** mhu has quit IRC22:09
*** mhu has joined #zuul22:10
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: add an ENABLE_DEBUG build arg/environment  https://review.opendev.org/69484522:20
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346422:20
openstackgerritDavid Moreau Simard proposed zuul/zuul master: DNM: Test zuul-stream-functional with ara 1.2  https://review.opendev.org/69462222:21
corvusfbo: is it worth adding a test that covers https://review.opendev.org/691254 ?22:22
*** tjgresha has joined #zuul22:25
*** tosky has joined #zuul22:26
*** pcaruana has quit IRC22:26
openstackgerritDavid Moreau Simard proposed zuul/zuul master: DNM: Test zuul-stream-functional with ara 1.2  https://review.opendev.org/69462222:32
openstackgerritTristan Cacqueray proposed zuul/zuul master: config: blacklist pipeline names that can not be used in template  https://review.opendev.org/69396122:40
openstackgerritDavid Moreau Simard proposed zuul/zuul master: DNM: Test zuul-stream-functional with ara 1.2  https://review.opendev.org/69462222:49
openstackgerritTristan Cacqueray proposed zuul/zuul master: config: blacklist pipeline names that can not be used in template  https://review.opendev.org/69396122:55
openstackgerritMerged zuul/zuul-registry master: Disable namespacing  https://review.opendev.org/68921922:57
openstackgerritMerged zuul/zuul master: Fix issue search in FakeGithubClient  https://review.opendev.org/69455323:04
corvusmordred, tristanC: i'm working on adding podman support to use-buildset-registry.  i need to edit the containers.conf file (which may already exist) which is in toml format.  my current best idea is to vendor https://github.com/uiri/toml/tree/master/toml and https://github.com/dbohdan/remarshal/blob/master/remarshal.py into the role and wrap them in an ansible module.  that sounds like a terrible idea (i23:15
corvusthought everyone said dealing with this was supposed to be easier than yaml/json?).  do you have any others?23:15
mordredcorvus: I believe "easier to deal with" in this context means "it's easier to vendor the library"23:15
mordredcorvus: because pip works via vendoring dependencies, and pyyaml is a harder sell for that23:16
clarkbis the configparser ini format close enough that you can write out ini that is also valid toml?23:17
mordredcorvus: I agree - it sounds like a terrible idea - but it might be a better idea than the other ideas23:18
openstackgerritMerged zuul/zuul master: Add propagate=0 for loggers in logging.conf-sample  https://review.opendev.org/69417023:18
mordredclarkb: yes, but the problem is that someone else may have already written toml that's not configparser compat that has to be dealt with23:18
clarkbanother option would be to supply our own containers.conf and template it directly23:19
corvus(it's unfortunate that ansible can't natively modify podman's config files :/)23:19
clarkbrather than editing an in place file though that is far less flexible23:19
tristanCcorvus: what kind of edit do you need to make? I can't find a containers.conf in my f31 system...23:19
corvusclarkb: i think we need to maintain the existing content23:20
corvustristanC: i need to add a mirror23:20
tristanCcorvus: shouldn't that goes in /etc/containers/registries.d/default.yaml ?23:20
mordredhrm. dropping things into /etc/containers/registries.d in yaml format seems much more pleasing if that's an option23:21
openstackgerritDavid Moreau Simard proposed zuul/zuul master: DNM: Test zuul-stream-functional with ara 1.2  https://review.opendev.org/69462223:23
corvussorry, i meant registries.conf23:23
tristanCcorvus: then it seems like you could edit registries.conf using configparser23:23
corvuswhat is the relationship/difference between registries.conf and registries.d/default.yaml?23:23
tristanCor just template the file, there is only one option set on a default f31 install, the default list of registries23:25
mordredcorvus: I cannot find any docs explaining that question23:25
corvusi'm using the v2 version of registries.conf, which is more toml-like than the v1 format23:25
corvustristanC: i can't assume the default file23:26
corvuswe have to handle it being empty, missing, or having an arbitrary and unknown set of existing options23:26
mordredyup.23:27
mordredI'd love it if registries.d is just a dir of yaml format stuff that can contain the same as registries.conf23:27
corvus(the first and last are the most likely situations, and i know we will encounter both of those)23:27
mordredif it is, then it would be super easy to always just drop in a file23:27
corvusis registries.d only for setting the sigstore location for registries?23:30
mordredcorvus: https://github.com/containers/image/blob/master/docs/containers-registries.d.5.md and https://github.com/containers/image/blob/master/docs/containers-registries.conf.5.md23:30
mordred(finally found the docs)23:30
corvusmordred: yeah, the only thing i see in that first link is stuff about sigstore23:31
mordredyeah. I agree.23:32
openstackgerritMerged zuul/zuul master: Revert "Revert "Add the process environment to zuul.conf parser""  https://review.opendev.org/69246923:33
mordredSpamapS: ^^23:34
corvusclarkb: i tried to load http://paste.openstack.org/show/786310/ with configparser and it broke on the 1st line :(23:34
mordredcorvus: I think that we might be at "vendor toml lib because the entire ecosystem is unavoidably broken"23:36
tristanCclarkb: here is what is in my registries.conf: http://paste.openstack.org/show/786311/23:38
corvusyeah, that's the v1 format; which i guess we also need to handle23:39
corvus(we're going to have to promote it to v2)23:39
corvusthe docker version of this is like 3 tasks in ansible; the podman version is going to be several thousand lines of python :(23:40
mordredand a vendoered library23:40
openstackgerritMerged zuul/zuul master: executor: add merge_jobs options to disable gearman merge jobs  https://review.opendev.org/67697423:41
*** jamesmcarthur has quit IRC23:46
*** jamesmcarthur has joined #zuul23:46
*** jamesmcarthur has quit IRC23:48

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!