| openstackgerrit | Ian Wienand proposed openstack-infra/zuul feature/zuulv3: Ensure the merger prunes on update https://review.openstack.org/520855 | 02:51 |
|---|---|---|
| *** jesusaur has quit IRC | 03:36 | |
| *** jesusaur has joined #zuul | 04:24 | |
| *** jesusaurum has joined #zuul | 05:01 | |
| *** jesusaur has quit IRC | 05:02 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/jobs route https://review.openstack.org/503270 | 06:37 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/jobs route https://review.openstack.org/503270 | 07:17 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/builds route https://review.openstack.org/466561 | 07:17 |
| *** kmalloc has quit IRC | 07:54 | |
| rcarrillocruz | jlk: hope to catch you later, cannot make the recheck working on GitHub pipeline, not sure if I'm doing something wrong | 08:21 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: mqtt: Add basic reporter https://review.openstack.org/518279 | 08:21 |
| rcarrillocruz | oh neat ^ | 08:22 |
| rcarrillocruz | that for firehose in mind, right tristanC ? | 08:22 |
| tristanC | rcarrillocruz: yes, mosquitto | 08:24 |
| SpamapS | rcarrillocruz: I have no problems with rechecks on my github enterprise. | 08:24 |
| tristanC | on the other hand, having the mqtt notification in a post job task would be better, it would be published immediately instead of at the end of the buildset | 08:27 |
| tristanC | perhaps the zuul mqtt driver should only publish buildset result... | 08:27 |
| rcarrillocruz | oh, SpamapS , you are awake | 08:28 |
| rcarrillocruz | let me link you | 08:29 |
| rcarrillocruz | https://github.com/rcarrillocruz-org/zuul-tested-repo/pull/10 | 08:29 |
| rcarrillocruz | ist that comment what is supposed to trigger things | 08:29 |
| rcarrillocruz | or actually a comment in the code | 08:29 |
| rcarrillocruz | like | 08:29 |
| rcarrillocruz | 'start review' thing on GH | 08:29 |
| rcarrillocruz | tristanC: keep on dude, love the feature stuff you're pushing on nodepool/zuul | 08:30 |
| tristanC | rcarrillocruz: heh, you're welcome :-) | 08:34 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Refactor provider config to driver module https://review.openstack.org/488384 | 08:36 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement a static driver for Nodepool https://review.openstack.org/468624 | 08:36 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement an OpenContainer driver https://review.openstack.org/468753 | 08:36 |
| tristanC | rcarrillocruz: actually we are about to put those features in production with software-factory 2.7, i'm looking forward moving most of our job to OCI slaves | 08:46 |
| *** hashar has joined #zuul | 09:34 | |
| *** nguyentrihai has joined #zuul | 12:45 | |
| tobiash | tristanC: so you're using nodepool itself as build node? | 13:57 |
| *** jkilpatr has quit IRC | 14:03 | |
| tristanC | tobiash: dedicated node, the driver connect to a remote node to create containerized sshd process | 14:07 |
| tobiash | ah, ok | 14:08 |
| *** jkilpatr has joined #zuul | 14:16 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add ensure-reno and ensure-babel roles https://review.openstack.org/521106 | 15:19 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 15:19 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 15:19 |
| *** hashar has quit IRC | 16:14 | |
| rcarrillocruz | folks, isee there's an implicit (and automatic?) setup role on zuul runs that effectively just gather facts on nodes | 16:39 |
| rcarrillocruz | can that be disabled on a job by job basis? | 16:39 |
| pabelanger | rcarrillocruz: yah, it is called setup_playbooks playbooks, IIRC | 16:42 |
| pabelanger | i don't think you can opt out | 16:42 |
| rcarrillocruz | uh | 16:42 |
| rcarrillocruz | that's a problem | 16:42 |
| rcarrillocruz | network operating systems have their own facts modules | 16:42 |
| rcarrillocruz | like | 16:42 |
| rcarrillocruz | gather_facts is just a thing on servers | 16:43 |
| pabelanger | yah, I think we did it because of SSH issue | 16:43 |
| pabelanger | let me fine commit | 16:43 |
| rcarrillocruz | if so, i'll have to think a patch controlling that | 16:44 |
| rcarrillocruz | in other news | 16:44 |
| pabelanger | rcarrillocruz: https://review.openstack.org/507933/ | 16:44 |
| rcarrillocruz | http://38.145.34.35/logs/ | 16:45 |
| rcarrillocruz | super janky | 16:45 |
| odyssey4me | of course you could rather implement the use of the setup task, perhaps with a configurable fact subset (using the default of all) | 16:45 |
| rcarrillocruz | but i got my zuul v3 + github + nodepool to scp logs | 16:45 |
| pabelanger | rcarrillocruz: woot | 16:45 |
| odyssey4me | jobs would continue to work, but could change the subset via a var | 16:45 |
| rcarrillocruz | http://38.145.34.35/logs/ | 16:46 |
| pabelanger | rcarrillocruz: I plan to work on github testing again soon, maybe this weekend | 16:47 |
| rcarrillocruz | you'll realize that the role doesn't install the editable package for some reason | 16:47 |
| rcarrillocruz | i had to pip uninstall | 16:47 |
| rcarrillocruz | and pip install manually | 16:47 |
| rcarrillocruz | the sigmavirus commit from requirements | 16:48 |
| rcarrillocruz | pabelanger: https://github.com/rcarrillocruz-org/ansible-fork/pull/2 | 16:48 |
| rcarrillocruz | PR i'm using to trigger things | 16:48 |
| pabelanger | rcarrillocruz: yah, had that issue before | 16:48 |
| pabelanger | we need to fix that or get a proper release | 16:48 |
| rcarrillocruz | i think a release | 16:48 |
| rcarrillocruz | since the thing got merged | 16:48 |
| rcarrillocruz | it didn't get a release in quite a few | 16:49 |
| rcarrillocruz | someone should ping sigmavirus | 16:49 |
| pabelanger | yah, I tried asking fr a release, but was ignored | 16:49 |
| pabelanger | but ya, we should ping again | 16:49 |
| rcarrillocruz | odyssey4me: there's ongoing work to plugin-ize gather_facts | 16:49 |
| rcarrillocruz | iirc by giving hints to executor | 16:49 |
| rcarrillocruz | if ansible_network_os blah | 16:49 |
| rcarrillocruz | then use blah_facts for gather_facts | 16:50 |
| rcarrillocruz | and the likes | 16:50 |
| rcarrillocruz | i think alikins was on it, need to cehck with him | 16:50 |
| odyssey4me | rcarrillocruz ah, that would be nice too - but perhaps my suggestion would be an acceptable workaround for now? | 16:50 |
| rcarrillocruz | but i'm not sure what you mean, you mean overriding setup module by bundling it or.. | 16:51 |
| rcarrillocruz | like, from what isee | 16:52 |
| rcarrillocruz | the implicit setup_playbooks role | 16:52 |
| rcarrillocruz | does ansible -m setup | 16:52 |
| rcarrillocruz | but that will die as soon as it tries to connect to a network OS node | 16:52 |
| rcarrillocruz | as there's no python or even posix shell there | 16:52 |
| pabelanger | rcarrillocruz: if was a quick way to test networking, if python isn't on remote node, thoughts on another method? | 16:53 |
| rcarrillocruz | shell ping, dunno | 16:54 |
| rcarrillocruz | ansible ping wont work | 16:54 |
| rcarrillocruz | as it's a python call in the end | 16:54 |
| pabelanger | raw: something? | 16:55 |
| pabelanger | raw: noop (eg) | 16:55 |
| pabelanger | wonder if paramiko has something | 16:56 |
| pabelanger | the idea was just to see if we could setup SSH connection, then start into ansible | 16:56 |
| rcarrillocruz | sure, that could work | 16:57 |
| SpamapS | rcarrillocruz: sorry, I made that comment, and then fell asleep ;) | 17:04 |
| SpamapS | rcarrillocruz: I'd need to see your pipeline config to know if that recheck should work. | 17:05 |
| rcarrillocruz | i'll pastebin it in a bit | 17:05 |
| rcarrillocruz | http://paste.openstack.org/show/626639/ | 17:06 |
| rcarrillocruz | SpamapS: ^ | 17:06 |
| rcarrillocruz | thing is i don't get a GH webhook event on scheduler | 17:07 |
| rcarrillocruz | so if anything, doesn't appear a regex issue | 17:07 |
| rcarrillocruz | is just is not triggered for whatever reason | 17:07 |
| rcarrillocruz | maybe i need to tweak something on the GH app to get those events? | 17:07 |
| SpamapS | rcarrillocruz: yeah that should work. | 17:07 |
| SpamapS | rcarrillocruz: it's identical to the one I have. | 17:08 |
| SpamapS | rcarrillocruz: are you using webhooks or a GH app? | 17:08 |
| rcarrillocruz | GH app | 17:08 |
| SpamapS | rcarrillocruz: ok, do you get anything in the scheduler log when you make the comment? | 17:09 |
| rcarrillocruz | nope | 17:10 |
| rcarrillocruz | which is why i think must be something on webhook/GH app | 17:10 |
| rcarrillocruz | i must be missing some perms | 17:10 |
| rcarrillocruz | anyway, if you tell me it works, then is no zuul GH side , but my env and probably the GH app...will check | 17:11 |
| rcarrillocruz | thx | 17:11 |
| pabelanger | I think there is an event to send comments in GH app settings? | 17:12 |
| pabelanger | I'd check openstack-zuul, but don't know where the admin page is | 17:15 |
| rcarrillocruz | yah, i don't have perms for that | 17:16 |
| rcarrillocruz | anyway, on to other tihng | 17:16 |
| rcarrillocruz | sooo | 17:16 |
| rcarrillocruz | the network integration tests of ansible | 17:16 |
| rcarrillocruz | we run it from a bastion | 17:16 |
| rcarrillocruz | like | 17:16 |
| rcarrillocruz | ansible-test network-integration ios-.* | 17:16 |
| pabelanger | k, found it | 17:17 |
| rcarrillocruz | that runs all tests (which are roles under test/integration/targets) for ios platform | 17:17 |
| rcarrillocruz | so i tried to run this by invoking it from executor | 17:17 |
| rcarrillocruz | but ofc | 17:17 |
| rcarrillocruz | "msg": "Executing local code is prohibited" | 17:17 |
| pabelanger | rcarrillocruz: we are subscribed to 9 events | 17:17 |
| rcarrillocruz | so i guess i'm bound to having two nodes jobs | 17:18 |
| rcarrillocruz | one as bastion | 17:18 |
| pabelanger | rcarrillocruz: pr review comments, have that? | 17:18 |
| rcarrillocruz | the other as the platform to test (ios, vyos, ) | 17:18 |
| pabelanger | rcarrillocruz: is your job trusted? | 17:18 |
| rcarrillocruz | so that means i should clone the change in quesiton on the bastion node | 17:18 |
| rcarrillocruz | i guess i need zuul-cloner for that? | 17:18 |
| rcarrillocruz | pabelanger: no, it's on untristed project | 17:19 |
| rcarrillocruz | https://github.com/rcarrillocruz-org/ansible-fork/pull/2 | 17:19 |
| rcarrillocruz | ansible-fork is a copy of ansible/ansible on my own org to simulate how i'd do this on ansible/ansible, when iroll out the 'official' GH app | 17:19 |
| pabelanger | rcarrillocruz: yah, you'll likely need to be trusted to run things from executor | 17:19 |
| rcarrillocruz | so you mean | 17:19 |
| rcarrillocruz | if i move those jobs | 17:19 |
| rcarrillocruz | to 'common-config' | 17:19 |
| rcarrillocruz | which is myu config-project then i'll be golden? | 17:19 |
| rcarrillocruz | woot | 17:19 |
| pabelanger | rcarrillocruz: yah, move them into config-project, and try | 17:19 |
| pabelanger | but, you will lose dynamic reconfiguration support | 17:20 |
| rcarrillocruz | oterwise, how hard would be to pull everything for a change with zuul-cloner | 17:20 |
| rcarrillocruz | that's what i did not want to put that as config project in first instance, yikes | 17:20 |
| rcarrillocruz | cos making a job as part of a change is great | 17:20 |
| rcarrillocruz | rather than doing two steps merges | 17:20 |
| rbergeron | pabelanger / mordred: did you both submit the things for devconf? | 17:20 |
| pabelanger | rbergeron: I submitted 2 things, JJB -> ansile talk and installfest hands-on | 17:21 |
| pabelanger | rcarrillocruz: confused by zuul-cloner question | 17:21 |
| rcarrillocruz | pabelanger: my point is, if i wanted to not have the jobs in config-project i need to have a bastion to run the job from (this would need the change in question as zuul prepares it) and the target network operating system i want to test | 17:22 |
| rcarrillocruz | thus | 17:22 |
| rcarrillocruz | i need a way to have the change on the bastion | 17:23 |
| rcarrillocruz | isn't zuul cloner supposed to do that? | 17:23 |
| pabelanger | rcarrillocruz: oh, so look at prepare-workspace role in zuul-jobs | 17:25 |
| rcarrillocruz | neat, th | 17:25 |
| rcarrillocruz | x | 17:25 |
| pabelanger | rcarrillocruz: we no longer use zuul-cloner, but push the git repos onto disk via syncronize | 17:25 |
| pabelanger | synchronize* | 17:25 |
| rcarrillocruz | oh | 17:26 |
| rcarrillocruz | a mere rsync | 17:26 |
| rcarrillocruz | cool | 17:26 |
| rcarrillocruz | hmm | 17:27 |
| rcarrillocruz | yeah | 17:28 |
| rcarrillocruz | i better not put this up in the inheritance tree | 17:28 |
| rcarrillocruz | cos otherwise | 17:28 |
| rcarrillocruz | it will die if this is attempted on network nodes | 17:28 |
| rcarrillocruz | i just need bastion | 17:28 |
| rbergeron | pabelanger: ossum, thx | 17:29 |
| pabelanger | rcarrillocruz: yah, bastion might be a good first step | 17:29 |
| rbergeron | rcarrillocruz: if there's anything you want to submit (DCI, ansible-networking anything, whatever) be suer to get it in ;) | 17:29 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 17:41 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Update ensure-reno to use requirements files https://review.openstack.org/521141 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 17:42 |
| *** jesusaurum has quit IRC | 17:54 | |
| SpamapS | rcarrillocruz: why can't you use a static node from nodepool? | 17:59 |
| SpamapS | rcarrillocruz: isn't that what they're for? | 17:59 |
| *** jesusaurum has joined #zuul | 18:13 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add ensure-reno and ensure-babel roles https://review.openstack.org/521106 | 18:17 |
| *** jesusaurum has quit IRC | 18:22 | |
| *** jesusaurum has joined #zuul | 18:24 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add compress capabilities to stage artifacts https://review.openstack.org/509234 | 18:33 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add a generic process-test-results role https://review.openstack.org/509459 | 18:33 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: WIP: Convert from legacy to native devstack job https://review.openstack.org/520664 | 19:05 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Update ensure-reno to use requirements files https://review.openstack.org/521141 | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 19:34 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 19:34 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 19:34 |
| *** haint has joined #zuul | 19:55 | |
| *** nguyentrihai has quit IRC | 19:58 | |
| *** jesusaurum has quit IRC | 20:15 | |
| *** jesusaurum has joined #zuul | 20:17 | |
| *** jesusaurum is now known as jesusaur | 20:17 | |
| *** jesusaur has quit IRC | 20:20 | |
| *** jesusaur has joined #zuul | 20:22 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 20:31 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 20:35 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 20:52 |
| *** ianw has quit IRC | 21:02 | |
| *** ianw has joined #zuul | 21:09 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:09 |
| *** tobiash has quit IRC | 21:13 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:17 |
| *** tobiash has joined #zuul | 21:18 | |
| *** tobiash has quit IRC | 21:19 | |
| *** tobiash has joined #zuul | 21:21 | |
| *** tobiash has quit IRC | 21:21 | |
| *** tobiash has joined #zuul | 21:30 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:32 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:50 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:15 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:27 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:43 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:48 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 23:00 |
| dmsimard | go mordred you can do it | 23:00 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 23:06 |
| mordred | dmsimard: yah man. I am CERTAIN that I can solve this simple task in at least another 100 patches | 23:07 |
| mordred | dmsimard: HOLY CRAP IT ACTUALLY WORKED THAT LAST TIME | 23:11 |
| dmsimard | I can help review if I have a chance this weekend, otherwise monday :) | 23:13 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 23:31 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!