| *** pojadhav is now known as pojadhav|ruck | 01:53 | |
| *** lajoskatona_ is now known as lajoskatona | 03:17 | |
| *** pojadhav is now known as pojadhav|ruck | 04:18 | |
| *** pojadhav|ruck is now known as pojadhav|lunch | 07:33 | |
| *** pojadhav|lunch is now known as pojadhav|ruck | 08:30 | |
| opendevreview | Dan Smith proposed openstack/devstack master: Write safe.directory items to system git config https://review.opendev.org/c/openstack/devstack/+/837759 | 13:57 |
|---|---|---|
| opendevreview | Dan Smith proposed openstack/devstack master: WIP: Gather performance data after tempest https://review.opendev.org/c/openstack/devstack/+/837139 | 14:00 |
| clarkb | frickler: ++ to interested parties taking it on. I think those EM branches need dansmith's --system fix instead though | 15:16 |
| clarkb | dansmith: ^ re the --system fix do you know if that has landed or been backported etc? I'm happy to help push that along if necessary | 15:17 |
| dansmith | clarkb: not landed, addressed a comment this morning and it's almost done in check | 15:17 |
| dansmith | https://review.opendev.org/c/openstack/devstack/+/837759 | 15:17 |
| clarkb | I should rereview then | 15:17 |
| opendevreview | Dan Smith proposed openstack/tempest master: Add test to rebuild volume backed instance https://review.opendev.org/c/openstack/tempest/+/831018 | 15:23 |
| dansmith | clarkb: includes rabbit and etcd now: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_343/837139/14/check/tempest-full-py3/343a1ce/controller/logs/performance.json | 15:50 |
| dansmith | I must not be grabbing the right (or all the) apache logs as I'm only getting identity api calls | 15:50 |
| dansmith | or.. hmm, maybe this isn't a tls-proxy job, that must be it | 15:50 |
| dansmith | gmann: kopecmartin: could you glance over this and see if there's anything major you want changed? https://review.opendev.org/c/openstack/devstack/+/837139 | 15:53 |
| clarkb | I thought tls-proxy was enabled by default, but maybe some jobs toggle it off? | 15:53 |
| dansmith | gmann: kopecmartin: sometimes it takes a couple cycles to test these and I'd like to iterate on anything major with the other jobs disabled before I un-un-comment those | 15:54 |
| dansmith | clarkb: I don't see it enabled in stackrc by default, so I'm not sure | 15:54 |
| dansmith | I know yoctozepto was working towards that as a default, but not sure it ever happened | 15:55 |
| dansmith | clarkb: oh yeah, this has enable_service tls-proxy, hmm | 15:57 |
| dansmith | ah, I need tls-proxy access log | 15:58 |
| opendevreview | Clark Boylan proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 16:14 |
| clarkb | sean-k-mooney: ^ I apparently couldn't help myself. That is a rebase with edits to more modern devsatck. The depends on isn't enough to make grenade work so those jobs shoudl fail but the regular tempest job will be good feedback | 16:15 |
| opendevreview | Dan Smith proposed openstack/devstack master: Gather performance data after tempest https://review.opendev.org/c/openstack/devstack/+/837139 | 16:25 |
| dansmith | clarkb: how do you test little snippets of ansible in the larger scheme of things without doing a full run in zuul? | 16:28 |
| dansmith | I needed to test this: https://review.opendev.org/c/openstack/devstack/+/837139/15/roles/capture-performance-data/tasks/main.yaml | 16:28 |
| dansmith | I was able to comment out some stuff from post.yml and run that as a playbook with "-i localhost," but it feels like there's got to be a better way | 16:29 |
| clarkb | ya I have a local virtuaklenv with ansible installed and a file with localhost localconnection in it that I use | 16:29 |
| clarkb | then I do ansible-playbook -i localfile path/to/some/test/playbook.yaml | 16:30 |
| dansmith | okay, I guess if I had a little helper playbook I could also preload the vars I need instead of trying to run some of the other roles to load up things like those log files bits | 16:31 |
| gmann | dansmith: ack, I will check | 16:38 |
| dansmith | gmann: thanks, I see I need to document my new role, so doing that now | 16:41 |
| gmann | dansmith: on --system. +W, you are planning to backport this until stable/stein (last branch on bionic) ? https://review.opendev.org/c/openstack/devstack/+/837759 | 16:45 |
| dansmith | gmann: I can if you/we want to .. it sounded like we didn't need to be running grenade voting there anyway, so I'm not sure it really matters, but... I can | 16:45 |
| opendevreview | Clark Boylan proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 16:48 |
| gmann | dansmith: but without that stable/ussuri or < still broken right? as failure in https://review.opendev.org/c/openstack/devstack/+/837749 | 16:48 |
| dansmith | gmann: it's broken even with it, IIRC, I forget the other reason.. clarkb ^ | 16:48 |
| dansmith | gmann: btw, I added a README.rst to my roles/foo/ but the docs target isn't finding it like the other roles (just doesn | 16:49 |
| clarkb | dansmith: sort of. The issue here is there are people who want EM grenade to keep working but don't necessarily put effort into making sure it does at all times for the non EM stable branches | 16:49 |
| dansmith | 't show up in html) | 16:49 |
| clarkb | what EM branches ahev apparently donesi sm ake grenade non voting and then its best effort. I think that the EM branch folks can backport dansmith's change themselves if ethy want it? | 16:50 |
| dansmith | clarkb: there was something else broken other than just this I thought | 16:50 |
| gmann | clarkb: dansmith not greande even tempest job failure | 16:50 |
| gmann | this one https://zuul.opendev.org/t/openstack/build/d64bbaed5c1447dba87d72dea5704920 | 16:50 |
| dansmith | gmann: that will be fixed by the system patch I think | 16:51 |
| gmann | yeah, so we need 837749 and your --system one until stable/stein to fix normal tempest job too | 16:51 |
| gmann | and we need to squash both to get them merge in stable/ussuri -> stable/stein | 16:53 |
| dansmith | oh, yeah I guess that's the other half.. getting it to the pre-ussuri devstack | 16:53 |
| dansmith | well, anyway, if that's what we want then fine, but at some point "no longer supported" has to be accurate :) | 16:53 |
| gmann | :) I always advocate of that and really concern on keeping that many EM branches which we always endup fixing for things like this case :) | 16:54 |
| clarkb | what I would do is backport to the supported versions for consistency | 16:55 |
| gmann | and not sure which EM maintainers team we can reach on that except pinging elodilles always :) | 16:55 |
| clarkb | then tell the people interested in EM that they can backport further if they wish | 16:55 |
| gmann | clarkb: yeah that is one way. and we will get to know which EM branch people care about | 16:55 |
| gmann | but honestly saying, either we need to do or it end up elodilles to do everything as he is not fan of EOL branch anymore :) | 16:56 |
| clarkb | then just don't do it | 16:57 |
| gmann | IMO, till stable/stein we can EOL those and keep only stable/train to stable/victoria(EMing soon) as EM | 16:57 |
| gmann | sounds good. so we just need to backport --system one till stable/victoria. | 16:58 |
| gmann | dansmith: on README.rst, not sure about issue. may be if you push it up then we can see if that is not happening ? | 17:01 |
| dansmith | gmann: yeah I will after the current run finishes.. adding the readme makes the docs target not fail, but just doesn't include my role in the rendered list | 17:01 |
| gmann | ok | 17:02 |
| opendevreview | Clark Boylan proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 17:27 |
| opendevreview | Clark Boylan proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 17:52 |
| opendevreview | Dan Smith proposed openstack/devstack master: Gather performance data after tempest https://review.opendev.org/c/openstack/devstack/+/837139 | 18:12 |
| opendevreview | Clark Boylan proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 18:18 |
| opendevreview | Merged openstack/devstack master: Write safe.directory items to system git config https://review.opendev.org/c/openstack/devstack/+/837759 | 18:24 |
| opendevreview | Clark Boylan proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 18:42 |
| dansmith | gmann: well, it worked in CI so I dunno why not locally but I guess I'm sorted :) | 18:51 |
| clarkb | any idea why the venv change would cause https://zuul.opendev.org/t/openstack/build/a6bd68698c4740b3ab5cced05cedc455/log/job-output.txt#13912 ? | 19:36 |
| clarkb | not urgent and I'm about to context switch away from that but I didn't expect errors like that | 19:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!