opendevreview | James E. Blair proposed zuul/zuul-jobs master: Fix CORS and endpoint in AWS log upload https://review.opendev.org/c/zuul/zuul-jobs/+/858726 | 00:15 |
---|---|---|
opendevreview | James E. Blair proposed zuul/zuul-jobs master: Fix CORS and endpoint in AWS log upload https://review.opendev.org/c/zuul/zuul-jobs/+/858726 | 00:30 |
*** rlandy|rover|biab is now known as rlandy|rover | 00:31 | |
*** rlandy|rover is now known as rlandy|out | 00:45 | |
opendevreview | Merged opendev/system-config master: Make zk-ca role more generic https://review.opendev.org/c/opendev/system-config/+/858988 | 00:46 |
opendevreview | Merged opendev/system-config master: Export Zuul traces to Jaeger https://review.opendev.org/c/opendev/system-config/+/858989 | 00:46 |
*** ysandeep|out is now known as ysandeep | 01:47 | |
*** ysandeep is now known as ysandeep|afk | 03:15 | |
*** akahat|ruck is now known as akahat | 03:46 | |
*** ysandeep|afk is now known as ysandeep | 03:55 | |
*** ysandeep is now known as ysandeep|afk | 04:32 | |
*** chandankumar is now known as chkumar|rover | 04:32 | |
*** ysandeep|afk is now known as ysandeep | 05:07 | |
opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Added cloud-init growpart element https://review.opendev.org/c/openstack/diskimage-builder/+/855856 | 06:24 |
yoctozepto | clarkb: hi, I am not an OSA expert so cannot help, sorry (we are kolla-ansible) | 06:31 |
yoctozepto | (not sure what "my example" is being considered) | 06:31 |
*** frenzyfriday is now known as frenzyfriday|ruck | 07:01 | |
*** jpena|off is now known as jpena | 07:26 | |
*** ysandeep is now known as ysandeep|lunch | 08:44 | |
opendevreview | Artem Goncharov proposed openstack/project-config master: Add sdk/cli acceptance jobs https://review.opendev.org/c/openstack/project-config/+/859060 | 09:34 |
*** ysandeep|lunch is now known as ysandeep | 09:50 | |
*** rlandy|out is now known as rlandy | 10:38 | |
*** frenzyfriday|ruck is now known as frenzyfriday|ruck|food | 12:26 | |
*** frenzyfriday|ruck|food is now known as frenzyfriday|ruck | 13:11 | |
opendevreview | Merged opendev/system-config master: Re-expose our Mailman archives.yaml and robots.txt https://review.opendev.org/c/opendev/system-config/+/858913 | 13:19 |
*** pojadhav is now known as pojadhav|afk | 13:31 | |
*** dasm|off is now known as dasm | 14:02 | |
*** ysandeep is now known as ysandeep|out | 14:02 | |
*** pojadhav|afk is now known as pojadhav | 14:23 | |
opendevreview | James E. Blair proposed zuul/zuul-jobs master: Fix CORS and endpoint in AWS log upload https://review.opendev.org/c/zuul/zuul-jobs/+/858726 | 15:12 |
clarkb | yoctozepto: bah I'm sorry that ping was for noonedeadpunk but I mixed things up | 15:14 |
clarkb | apologies. That said I wonder if kolla has the same issue (I think I conflated them because tehy have similar needs) | 15:14 |
clarkb | it actually looks like kolla ansible does not have zuul clone the world even in its source jobs | 15:16 |
yoctozepto | clarkb: we obtain stuff from tarballs unless we have depends-on | 15:17 |
noonedeadpunk | but if you have depends-on - you should have all project in required for zuul? | 15:20 |
noonedeadpunk | So workspace preparpation would be exactly same? | 15:21 |
clarkb | noonedeadpunk: according to the job logs it isn't. I'm not sure how they manage it. In any case I did additional testing with devstack and compared to other osa changes and I think my change is unrelated to the problems osa is having. But the osa testing of my change does show an improvement for you and doesn't seem to break anything | 15:21 |
noonedeadpunk | clarkb: nah, it's not from the change - it's from my collections concern I raised earlier | 15:22 |
noonedeadpunk | Yes, we totally saving smth like 3 mins | 15:22 |
noonedeadpunk | so it's really good | 15:23 |
clarkb | its like half an hour for each osa change :) | 15:23 |
noonedeadpunk | yup | 15:23 |
noonedeadpunk | and if make that in multiprocessing.... :D | 15:23 |
noonedeadpunk | but yes, it's already big improvement, and wasn't breaking | 15:24 |
clarkb | I've removed my WIP from https://review.opendev.org/c/zuul/zuul-jobs/+/858961/ | 15:24 |
yoctozepto | noonedeadpunk, clarkb: depends-on bring the dependencies similarly like required-projects | 15:25 |
noonedeadpunk | but doesn't the project msut be in required to consume depends-on? | 15:26 |
yoctozepto | no | 15:26 |
noonedeadpunk | hm | 15:26 |
yoctozepto | it would be flaky to include all the openstack components this way | 15:26 |
clarkb | yoctozepto: and that is beacuse you're wanting to use known good points of time for the various components? | 15:27 |
clarkb | I thinks OSA does similar with a list of git hashes, but end result is similar | 15:27 |
yoctozepto | clarkb: no, we sometimes test with in-flight changes | 15:27 |
yoctozepto | I mean | 15:27 |
yoctozepto | I am not sure I am answering the right question | 15:27 |
yoctozepto | that was for using depends-on | 15:28 |
noonedeadpunk | yeah, it's slight difference in using only tags vs hashes | 15:28 |
yoctozepto | for using tarballs is because it is cheaper for end users | 15:28 |
clarkb | yoctozepto: mostly wondering why it is flaky to use openstack components from source | 15:28 |
clarkb | aha that explains it | 15:28 |
yoctozepto | clarkb: flaky to force us to list all components in zuul | 15:28 |
noonedeadpunk | flaky to add all components to required-projects | 15:28 |
yoctozepto | maybe "flaky" is not the best word here | 15:28 |
yoctozepto | more like "shame to do in the first place" ;-) | 15:29 |
clarkb | right it should never be flaky to have zuul configure git repos | 15:29 |
yoctozepto | ++ | 15:29 |
noonedeadpunk | yoctozepto: well, we build wheels, to make it cheaper for end users. So you still have tarballs, but locally hashed :) | 15:29 |
yoctozepto | zuul does great job | 15:29 |
clarkb | it might be slower (whcih is the issue we're trying to address here) but zuul goes to extensive effort to make sure it isn't flaky | 15:29 |
yoctozepto | noonedeadpunk: you prebuild wheels? and you cache them somewhere? | 15:29 |
noonedeadpunk | yup | 15:29 |
yoctozepto | interesting | 15:29 |
yoctozepto | we might want to reuse it | 15:30 |
noonedeadpunk | We use this thing https://opendev.org/openstack/ansible-role-python_venv_build | 15:30 |
yoctozepto | hmm | 15:30 |
noonedeadpunk | I do re-use this role without osa quite nicely | 15:30 |
yoctozepto | but does it mean each user does it for themselves? | 15:30 |
noonedeadpunk | We tried to make it as less opionated as possible | 15:30 |
noonedeadpunk | oh, yes, it does | 15:30 |
yoctozepto | for that we use entire built images so no need for that :D | 15:31 |
noonedeadpunk | But you build wheels for nova like once, and then deploy n computes from tarball | 15:31 |
yoctozepto | I mean, we build once and then just deploy | 15:31 |
yoctozepto | yeah | 15:31 |
yoctozepto | we build one nova-compute image | 15:31 |
yoctozepto | and deploy everywhere | 15:31 |
noonedeadpunk | and bind mount config? | 15:31 |
noonedeadpunk | or jsut change config from compute to compute inside image? | 15:32 |
noonedeadpunk | (sorry, that's offtopic) | 15:32 |
yoctozepto | we have a config-copying machinery | 15:33 |
yoctozepto | but it's pretty close to bind mounting | 15:33 |
yoctozepto | the image is config-less | 15:34 |
yoctozepto | except for defaults like policies and paste | 15:34 |
clarkb | its ok, I instigated the discussion (and I think understanding common use cases in the CI system we host is close enough to on topic :) ) | 15:34 |
yoctozepto | I am always happy to discuss openstack building and deployment :-) | 15:34 |
noonedeadpunk | Yeah, we just stick to SHAs, because project sometimes hesitate to release small, but quite important changes. So cunsuming SHAs seems to work better at the end. And we don't care if branch/tag/etc will be dropped from pypi or repo | 15:39 |
*** marios is now known as marios|out | 15:46 | |
opendevreview | Jeremy Stanley proposed opendev/engagement master: Initial commit https://review.opendev.org/c/opendev/engagement/+/729293 | 16:08 |
opendevreview | Merged opendev/engagement master: Initial commit https://review.opendev.org/c/opendev/engagement/+/729293 | 16:16 |
*** jpena is now known as jpena|off | 16:22 | |
*** rlandy is now known as rlandy|brb | 16:57 | |
*** rlandy|brb is now known as rlandy | 17:20 | |
*** dmsimard1 is now known as dmsimard | 17:43 | |
fungi | the ansible 2 removal finally merged a few minutes ago, so this weekend's restart will definitely include it | 17:49 |
clarkb | Gerrit stable-3.5 just updated mina sshd and jgit. I've asked that they backport the key extension handling fix too. This means that 1) our next gerrit update even on 3.5 will include a fairly large set of change sso be aware of that. and 2) it should fix the openssh rsa problems people have had | 17:55 |
clarkb | We should probably attempt to avoid updating gerrit until after the openstack release for this reason | 17:55 |
clarkb | (normally I odn't worry about it too much but mina and jgit are pretty big updates) | 17:56 |
slittle1 | were there concerns with https://review.opendev.org/c/openstack/project-config/+/858011 ? | 17:57 |
clarkb | slittle1: no, I think we're mostly just swamped with everything else going on. project-config is a repo we'd always appreciate more reviews in. | 18:00 |
fungi | slittle1: i've approved it now, thanks for bringing it to our attention! | 18:01 |
slittle1 | thanks | 18:06 |
opendevreview | Merged openstack/project-config master: Add oran-o2 app to StarlingX https://review.opendev.org/c/openstack/project-config/+/858011 | 18:08 |
clarkb | funny/odd/interesting observation: Gerrit shortens "Software Factory CI" to "Software" in the reviewers list in the change listing view | 18:37 |
fungi | slittle1: i added you as the initial member of the starlingx-app-oran-o2-core group in gerrit now that 858011 has deployed | 19:38 |
opendevreview | lotorev vitaly proposed openstack/project-config master: Update link to zuul gating docs https://review.opendev.org/c/openstack/project-config/+/859150 | 21:05 |
opendevreview | lotorev vitaly proposed openstack/project-config master: Update link to zuul gating docs https://review.opendev.org/c/openstack/project-config/+/859150 | 21:05 |
*** dasm is now known as dasm|off | 21:10 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!