Friday, 2022-09-23

opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Fix CORS and endpoint in AWS log upload  https://review.opendev.org/c/zuul/zuul-jobs/+/85872600:15
opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Fix CORS and endpoint in AWS log upload  https://review.opendev.org/c/zuul/zuul-jobs/+/85872600:30
*** rlandy|rover|biab is now known as rlandy|rover00:31
*** rlandy|rover is now known as rlandy|out00:45
opendevreviewMerged opendev/system-config master: Make zk-ca role more generic  https://review.opendev.org/c/opendev/system-config/+/85898800:46
opendevreviewMerged opendev/system-config master: Export Zuul traces to Jaeger  https://review.opendev.org/c/opendev/system-config/+/85898900:46
*** ysandeep|out is now known as ysandeep01:47
*** ysandeep is now known as ysandeep|afk03:15
*** akahat|ruck is now known as akahat03:46
*** ysandeep|afk is now known as ysandeep03:55
*** ysandeep is now known as ysandeep|afk04:32
*** chandankumar is now known as chkumar|rover04:32
*** ysandeep|afk is now known as ysandeep05:07
opendevreviewRafal Lewandowski proposed openstack/diskimage-builder master: Added cloud-init growpart element  https://review.opendev.org/c/openstack/diskimage-builder/+/85585606:24
yoctozeptoclarkb: 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|ruck07:01
*** jpena|off is now known as jpena07:26
*** ysandeep is now known as ysandeep|lunch08:44
opendevreviewArtem Goncharov proposed openstack/project-config master: Add sdk/cli acceptance jobs  https://review.opendev.org/c/openstack/project-config/+/85906009:34
*** ysandeep|lunch is now known as ysandeep09:50
*** rlandy|out is now known as rlandy10:38
*** frenzyfriday|ruck is now known as frenzyfriday|ruck|food12:26
*** frenzyfriday|ruck|food is now known as frenzyfriday|ruck13:11
opendevreviewMerged opendev/system-config master: Re-expose our Mailman archives.yaml and robots.txt  https://review.opendev.org/c/opendev/system-config/+/85891313:19
*** pojadhav is now known as pojadhav|afk13:31
*** dasm|off is now known as dasm14:02
*** ysandeep is now known as ysandeep|out14:02
*** pojadhav|afk is now known as pojadhav14:23
opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Fix CORS and endpoint in AWS log upload  https://review.opendev.org/c/zuul/zuul-jobs/+/85872615:12
clarkbyoctozepto: bah I'm sorry that ping was for noonedeadpunk but I mixed things up15:14
clarkbapologies. That said I wonder if kolla has the same issue (I think I conflated them because tehy have similar needs)15:14
clarkbit actually looks like kolla ansible does not have zuul clone the world even in its source jobs15:16
yoctozeptoclarkb: we obtain stuff from tarballs unless we have depends-on15:17
noonedeadpunkbut if you have depends-on - you should have all project in required for zuul?15:20
noonedeadpunkSo workspace preparpation would be exactly same?15:21
clarkbnoonedeadpunk: 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 anything15:21
noonedeadpunkclarkb: nah, it's not from the change - it's from my collections concern I raised earlier 15:22
noonedeadpunkYes, we totally saving smth like 3 mins15:22
noonedeadpunkso it's really good15:23
clarkbits like half an hour for each osa change :)15:23
noonedeadpunkyup15:23
noonedeadpunkand if make that in multiprocessing.... :D15:23
noonedeadpunkbut yes, it's already big improvement, and wasn't breaking15:24
clarkbI've removed my WIP from https://review.opendev.org/c/zuul/zuul-jobs/+/858961/15:24
yoctozeptonoonedeadpunk, clarkb: depends-on bring the dependencies similarly like required-projects15:25
noonedeadpunkbut doesn't the project msut be in required to consume depends-on?15:26
yoctozeptono15:26
noonedeadpunkhm15:26
yoctozeptoit would be flaky to include all the openstack components this way15:26
clarkbyoctozepto: and that is beacuse you're wanting to use known good points of time for the various components?15:27
clarkbI thinks OSA does similar with a list of git hashes, but end result is similar15:27
yoctozeptoclarkb: no, we sometimes test with in-flight changes15:27
yoctozeptoI mean15:27
yoctozeptoI am not sure I am answering the right question15:27
yoctozeptothat was for using depends-on15:28
noonedeadpunkyeah, it's slight difference in using only tags vs hashes15:28
yoctozeptofor using tarballs is because it is cheaper for end users15:28
clarkbyoctozepto: mostly wondering why it is flaky to use openstack components from source15:28
clarkbaha that explains it15:28
yoctozeptoclarkb: flaky to force us to list all components in zuul15:28
noonedeadpunkflaky to add all components to required-projects15:28
yoctozeptomaybe "flaky" is not the best word here15:28
yoctozeptomore like "shame to do in the first place" ;-)15:29
clarkbright it should never be flaky to have zuul configure git repos15:29
yoctozepto++15:29
noonedeadpunkyoctozepto: well, we build wheels, to make it cheaper for end users. So you still have tarballs, but locally hashed :)15:29
yoctozeptozuul does great job15:29
clarkbit 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 flaky15:29
yoctozeptonoonedeadpunk: you prebuild wheels? and you cache them somewhere?15:29
noonedeadpunkyup15:29
yoctozeptointeresting15:29
yoctozeptowe might want to reuse it15:30
noonedeadpunkWe use this thing https://opendev.org/openstack/ansible-role-python_venv_build15:30
yoctozeptohmm15:30
noonedeadpunkI do re-use this role without osa quite nicely15:30
yoctozeptobut does it mean each user does it for themselves?15:30
noonedeadpunkWe tried to make it as less opionated as possible15:30
noonedeadpunkoh, yes, it does15:30
yoctozeptofor that we use entire built images so no need for that :D15:31
noonedeadpunkBut you build wheels for nova like once, and then deploy n computes from tarball15:31
yoctozeptoI mean, we build once and then just deploy15:31
yoctozeptoyeah15:31
yoctozeptowe build one nova-compute image15:31
yoctozeptoand deploy everywhere15:31
noonedeadpunkand bind mount config?15:31
noonedeadpunkor jsut change config from compute to compute inside image?15:32
noonedeadpunk(sorry, that's offtopic)15:32
yoctozeptowe have a config-copying machinery15:33
yoctozeptobut it's pretty close to bind mounting15:33
yoctozeptothe image is config-less15:34
yoctozeptoexcept for defaults like policies and paste15:34
clarkbits 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
yoctozeptoI am always happy to discuss openstack building and deployment :-)15:34
noonedeadpunkYeah, 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 repo15:39
*** marios is now known as marios|out15:46
opendevreviewJeremy Stanley proposed opendev/engagement master: Initial commit  https://review.opendev.org/c/opendev/engagement/+/72929316:08
opendevreviewMerged opendev/engagement master: Initial commit  https://review.opendev.org/c/opendev/engagement/+/72929316:16
*** jpena is now known as jpena|off16:22
*** rlandy is now known as rlandy|brb16:57
*** rlandy|brb is now known as rlandy17:20
*** dmsimard1 is now known as dmsimard17:43
fungithe ansible 2 removal finally merged a few minutes ago, so this weekend's restart will definitely include it17:49
clarkbGerrit 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 had17:55
clarkbWe should probably attempt to avoid updating gerrit until after the openstack release for this reason17:55
clarkb(normally I odn't worry about it too much but mina and jgit are pretty big updates)17:56
slittle1were there concerns with https://review.opendev.org/c/openstack/project-config/+/858011 ?17:57
clarkbslittle1: 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
fungislittle1: i've approved it now, thanks for bringing it to our attention!18:01
slittle1thanks18:06
opendevreviewMerged openstack/project-config master: Add oran-o2 app to StarlingX  https://review.opendev.org/c/openstack/project-config/+/85801118:08
clarkbfunny/odd/interesting observation: Gerrit shortens "Software Factory CI" to "Software" in the reviewers list in the change listing view18:37
fungislittle1: i added you as the initial member of the starlingx-app-oran-o2-core group in gerrit now that 858011 has deployed19:38
opendevreviewlotorev vitaly proposed openstack/project-config master: Update link to zuul gating docs  https://review.opendev.org/c/openstack/project-config/+/85915021:05
opendevreviewlotorev vitaly proposed openstack/project-config master: Update link to zuul gating docs  https://review.opendev.org/c/openstack/project-config/+/85915021:05
*** dasm is now known as dasm|off21:10

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!