Friday, 2020-06-05

openstackgerritMerged zuul/zuul-jobs master: download-artifact: cleanup long when statement  https://review.opendev.org/73344600:00
*** DSpider has quit IRC00:17
*** gtema has joined #opendev00:34
*** gtema has quit IRC00:39
*** factor has joined #opendev00:44
*** Meiyan has joined #opendev00:54
ianwyay, centos8 rpms published, something up with centos700:57
ianwoh dear, i wonder if this doesn't work with ovh ...01:00
ianwthat doesn't make sense though, as zuul generates the index.html01:03
ianw./RPMS/index.html: gzip compressed data, last modified: Fri Jun  5 00:44:47 2020, max compression, original size modulo 2^32 106601:16
ianwarrggghhh mimetype issues i guess01:16
ianwit works with compression=auto01:18
*** ysandeep|away is now known as ysandeep01:29
*** xiaolin has joined #opendev01:30
ianwgreat, it's basically going to be impossible to generically do this i think01:30
ianwi'll put in a fix but i think i give up01:31
openstackgerritIan Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression  https://review.opendev.org/73372801:33
ianwif that works on the executor, it's probably enough to keep it as an option01:34
ianwotherwise, i guess we should re-implement wget as custom python to read the manifest, and pull all files under a given path01:34
ianwyep, not present on the executor01:41
*** xiaolin has quit IRC01:45
*** xiaolin has joined #opendev01:51
openstackgerritIan Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression  https://review.opendev.org/73372802:28
*** smcginnis has quit IRC02:28
*** gtema has joined #opendev02:35
*** gtema has quit IRC02:39
*** kevinz has joined #opendev02:51
kevinzianw: there?02:52
ianwkevinz: hey, yep02:52
*** hashar has joined #opendev02:52
kevinzianw: Could you tell me the default tempest concurrency of "tempest-full-py3"?02:52
ianwummm, not off the top of my head but i bet we can find out :)02:53
ianwkevinz: https://opendev.org/openstack/tempest/src/branch/master/roles/run-tempest/tasks/main.yaml#L2102:56
ianwi'm going to say that it would be the number of cores by default02:57
kevinzianw: Thanks for clarification, I will try to reduce the concurrency to see if there is different02:57
openstackgerritIan Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression  https://review.opendev.org/73372802:58
kevinzianw: as some of the test are succeed to run when they are not concurrency, and I saw some rabbitmq connection to neutron in the log02:58
kevinzand we are using 16C, so the concurrency would be 8, it maybe overload for aarch64 I guess02:59
ianwahhh, yes, that's an interesting point ... we did up it to 16 over 8 (hrw was driving that one?) so the arm64 nodes would be doing twice as much concurrency as the x86 nodes i guess03:05
openstackgerritIan Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression  https://review.opendev.org/73372803:12
*** tkajinam is now known as tkajinam|lunch03:19
*** tkajinam|lunch is now known as tkajinam03:19
openstackgerritIan Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression  https://review.opendev.org/73372803:21
*** priteau has joined #opendev03:39
openstackgerritIan Wienand proposed zuul/zuul-jobs master: Revert "download-artifact : support recursive download"  https://review.opendev.org/73373903:43
*** priteau has quit IRC03:46
openstackgerritMerged zuul/zuul-jobs master: Revert "download-artifact : support recursive download"  https://review.opendev.org/73373904:08
*** ysandeep is now known as ysandeep|off04:27
*** gtema has joined #opendev04:36
*** ykarel|away is now known as ykarel04:38
*** gtema has quit IRC04:40
*** hashar has quit IRC05:15
openstackgerritIan Wienand proposed zuul/zuul-jobs master: download-artifact: fix missed part of revert  https://review.opendev.org/73374405:25
openstackgerritMerged zuul/zuul-jobs master: download-artifact: fix missed part of revert  https://review.opendev.org/73374405:35
*** gtema has joined #opendev05:45
openstackgerritMerged openstack/project-config master: Prepare to retire swift-specs  https://review.opendev.org/73299905:45
ianwclarkb: could you look at https://review.opendev.org/#/c/731719 and merge if you don't have any other ideas about virtualenv detection in dib?  cc: dtantsur|afk05:51
ianwarrrggghhh the tar extractor doesn't make directories05:55
*** gtema has quit IRC06:49
*** lpetrut has joined #opendev06:50
*** gtema has joined #opendev06:50
openstackgerritIan Wienand proposed opendev/system-config master: [wip] arm64 role tests  https://review.opendev.org/73303406:52
ianwit has been a rather long path getting the arm64 rpms published but they are now ^ if anyone feel like further debugging the installs feel free! :) otherwise i'll get to it on tuesday, mon is holiday here06:55
ianwactually they may not be published till the next release pulse, but give it 20 mins or so06:55
*** hashar has joined #opendev07:04
*** rpittau|afk is now known as rpittau07:04
*** gtema has quit IRC07:05
*** priteau has joined #opendev07:10
*** slaweq has joined #opendev07:15
zbrianw: morning!07:22
*** tosky has joined #opendev07:31
openstackgerritSorin Sbarnea (zbr) proposed zuul/zuul-jobs master: tox: allow tox to be upgraded  https://review.opendev.org/69005707:34
AJaegerianw: do we need to revert https://review.opendev.org/#/c/733442/ as well?07:35
*** priteau has quit IRC07:35
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367507:40
fricklerAJaeger: iiuc it will be broken either way for now, so we could leave it like this as well until the recursion thing is fixed07:44
AJaegerindeed07:51
*** DSpider has joined #opendev07:58
*** moppy has quit IRC08:01
*** moppy has joined #opendev08:01
*** priteau has joined #opendev08:02
openstackgerritFelix Edel proposed zuul/zuul-jobs master: Return upload_results in upload-logs-swift role  https://review.opendev.org/73356408:06
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support  https://review.opendev.org/73376208:08
openstackgerritFelix Edel proposed zuul/zuul-jobs master: Return upload_results in upload-logs-swift role  https://review.opendev.org/73356408:14
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support  https://review.opendev.org/73376208:14
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Drop mock, six usage  https://review.opendev.org/73376508:14
openstackgerritIan Wienand proposed opendev/system-config master: [wip] arm64 role tests  https://review.opendev.org/73303408:16
*** ravsingh has joined #opendev08:19
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367508:21
*** priteau has quit IRC08:22
*** priteau has joined #opendev08:24
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support  https://review.opendev.org/73376208:25
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Drop mock, six usage  https://review.opendev.org/73376508:25
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs  https://review.opendev.org/73376808:25
AJaegerianw: FYI, I'm cleaning up the dib py27 support ^08:26
*** tkajinam has quit IRC08:31
*** dtantsur|afk is now known as dtantsur08:34
openstackgerritFelix Edel proposed zuul/zuul-jobs master: Return upload_results in upload-logs-swift role  https://review.opendev.org/73356408:39
*** roman_g has joined #opendev08:39
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs  https://review.opendev.org/73376808:50
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support  https://review.opendev.org/73376208:50
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Drop six usage  https://review.opendev.org/73376508:50
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Add libffi-dev to bindep  https://review.opendev.org/73377608:50
openstackgerritIan Wienand proposed opendev/system-config master: [wip] arm64 role tests  https://review.opendev.org/73303409:11
hrwianw: and iirc we later moved back to 809:14
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs  https://review.opendev.org/73376809:27
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support  https://review.opendev.org/73376209:27
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Drop six usage  https://review.opendev.org/73376509:27
*** gtema has joined #opendev09:33
ianwhrw: ahh, yeah maybe we just kept the large format https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.openstack.org.yaml#L43909:40
ianwkevinz: ^ you could try that, see if it works better?  would be a data point09:41
kevinzianw: yes, already use this to run devstack tempest09:47
kevinzianw: will continue check the log, it seems reducing the concurrency didn't help09:48
*** Meiyan has quit IRC09:57
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs  https://review.opendev.org/73376810:04
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support  https://review.opendev.org/73376210:04
openstackgerritAndreas Jaeger proposed openstack/diskimage-builder master: Drop six usage  https://review.opendev.org/73376510:04
*** rpittau is now known as rpittau|bbl10:08
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367510:16
*** hashar has quit IRC10:50
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367511:03
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Add support for overriding parameters  https://review.opendev.org/73380911:04
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367511:11
*** gtema has quit IRC11:32
*** rpittau|bbl is now known as rpittau11:59
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367512:08
*** gtema has joined #opendev12:13
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367512:16
*** gtema has joined #opendev12:27
*** tosky_ has joined #opendev12:35
*** tosky has quit IRC12:37
*** tosky has joined #opendev12:39
*** tosky_ has quit IRC12:40
openstackgerritPierre Riteau proposed openstack/project-config master: Add Backport-Candidate label for Kolla deliverables  https://review.opendev.org/73324312:41
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Add support for overriding parameters  https://review.opendev.org/73380912:41
*** tosky_ has joined #opendev12:41
*** tosky has quit IRC12:43
*** gtema has quit IRC12:45
*** tosky_ is now known as tosky12:50
*** smcginnis has joined #opendev13:04
*** sgw has quit IRC13:04
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367513:11
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Add support for overriding parameters  https://review.opendev.org/73380913:11
*** avass has joined #opendev13:19
*** smcginnis has quit IRC13:34
*** smcginnis has joined #opendev13:35
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367513:35
*** smcginnis has quit IRC13:40
*** auristor has quit IRC13:40
mordredcorvus, clarkb: woot. the latest iteration of https://review.opendev.org/#/c/733409 failed correctly!  https://zuul.opendev.org/t/openstack/build/b38f84ad5cf545638d7ff7b3cb8d0ab313:41
*** auristor has joined #opendev13:41
openstackgerritMonty Taylor proposed opendev/system-config master: Stop using backend hostname in zuul testinfra tests  https://review.opendev.org/73340913:41
mordredclarkb, corvus ^^ that fixes the todo, so that should go green13:41
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367513:43
openstackgerritMonty Taylor proposed opendev/system-config master: Stop cloning a bunch of puppet modules we don't use  https://review.opendev.org/72089213:44
openstackgerritMonty Taylor proposed opendev/system-config master: Stop cloning more puppet modules  https://review.opendev.org/72932113:44
mordredinfra-root: I'd like to land those 2 ^^ so that we don't keep merge-conflicting13:44
*** smcginnis has joined #opendev13:46
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367513:53
fungii'll take a peek in a sec13:57
AJaegermordred: let me help with retiring the repos, I'll push everything else in shape...13:58
fungimordred: i've approved those but can't keep an eye on them just yet to make sure they merge14:01
fungiwe've got an hvac service rep visiting to size a replacement system for us14:02
fungiso will likely still be tied up for a few more minutes14:02
mordredfungi: thanks! I'll keep an eye on them14:04
mordredAJaeger: did you see the other patch with removals?14:04
mordredAJaeger: https://review.opendev.org/#/c/731947/ - oh, you did, you +1'd it :)14:04
AJaegermordred: yeah, saw both - and will check what's in flight and needs updating and what's missing...14:05
AJaegermordred: I'll send an email to service-discuss for the retirement...14:09
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules  https://review.opendev.org/73385014:10
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules  https://review.opendev.org/73385014:11
AJaegermordred, fungi, ^14:11
openstackgerritAndreas Jaeger proposed opendev/puppet-ansible master: Retire repo  https://review.opendev.org/73385114:16
openstackgerritAndreas Jaeger proposed opendev/puppet-openafs master: Retire repo  https://review.opendev.org/73385214:19
openstackgerritAndreas Jaeger proposed opendev/puppet-openafs master: Retire repo  https://review.opendev.org/73385214:19
openstackgerritAndreas Jaeger proposed opendev/puppet-snmpd master: Retire repo  https://review.opendev.org/73385314:20
openstackgerritAndreas Jaeger proposed opendev/puppet-drupal master: Retire repo  https://review.opendev.org/73385514:21
*** lpetrut has quit IRC14:23
corvusmordred: \o/14:24
openstackgerritMerged opendev/system-config master: Stop cloning a bunch of puppet modules we don't use  https://review.opendev.org/72089214:25
openstackgerritCarlos Goncalves proposed openstack/diskimage-builder master: Remove doc files and autoremove packages  https://review.opendev.org/67554614:28
AJaegermordred: you did not retire puppet-{jenkins,nodepool,openstackci}, please abandon https://review.opendev.org/722012 , https://review.opendev.org/722011 https://review.opendev.org/72201014:35
*** gtema has joined #opendev14:36
AJaegermordred: and puppet-zuul is not retired, so abandon https://review.opendev.org/#/c/722016/1 as well, please14:39
mordredAJaeger: kk14:40
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire unused puppet modules  https://review.opendev.org/72090114:48
AJaegermordred, corvus, fungi: please +2  https://review.opendev.org/733850 - so that we can remove the extra repos.14:49
AJaegermordred: https://review.opendev.org/#/c/722018/ accessbot is also not retired yet14:50
* AJaeger will be back later14:51
corvusi still see puppet-openafs in modules.env14:52
*** hashar has joined #opendev14:53
corvusAJaeger, fungi: left a -1 for that -- can you double check me?14:55
fungilookin14:55
fungii thought mordred's change removed it14:55
fungihrm14:56
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367514:57
AJaegercorvus: did 729321 merge? That should remove it15:13
AJaegermordred, https://review.opendev.org/#/c/729321/ failed tests and thus is not merged yet ;(15:14
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles  https://review.opendev.org/73367515:14
AJaegercorvus: I think we can merge 733850 already - but we can wait for 729321 as well...15:15
AJaegermordred: puppet-accessbot is good to go, I double checked...15:21
*** ykarel is now known as ykarel|away15:23
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules  https://review.opendev.org/73385015:25
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire unused puppet modules  https://review.opendev.org/72090115:25
openstackgerritAndreas Jaeger proposed openstack/project-config master: Finish retiring unused puppet modules  https://review.opendev.org/72090115:26
AJaegerfungi, corvus, I think 733850 can merge directly - it just adds the noop-jobs. We need t owait with 720901 until everything is merged...15:29
openstackgerritMonty Taylor proposed opendev/system-config master: Stop cloning more puppet modules  https://review.opendev.org/72932115:30
mordredAJaeger, corvus: fungi: ^^ that should fix the puppet-afs removal patch (missed a ine in the previous rev)15:30
corvusAJaeger: i think 321 only removes ::client usage, not server15:30
mordredcorvus: AH15:30
corvus(which is correct -- i think our servers are still all run via puppet)15:31
mordredcorvus: *duh* sorry - let me fix that15:32
corvuswe could merge 850 on the assumption that we probably won't touch puppet-openafs to make any changes to servers, but i don't know that ansiblifying those is super high on the list, so we might need to make an emergency change still.15:32
corvusmordred: fix what?15:32
corvusto be clear, i believe our afs servers are configured via puppet, using the puppet-openafs module, and therefore we should leave the puppet-openafs repo active15:34
openstackgerritMonty Taylor proposed opendev/system-config master: Stop cloning more puppet modules  https://review.opendev.org/72932115:34
mordredcorvus: this ^^15:34
mordredcorvus: it erroneously removed puppet-openafs - I have fixed15:34
corvusmordred: cool, i agree with that fix :)15:34
* mordred got too eager15:34
AJaegerOk, will readd puppet-openafs...15:35
AJaeger(in the project-config changes)15:35
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs.  https://review.opendev.org/73367515:42
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules  https://review.opendev.org/73385015:45
openstackgerritAndreas Jaeger proposed openstack/project-config master: Finish retiring unused puppet modules  https://review.opendev.org/72090115:45
AJaegerupdated - added openafs back in ^15:45
openstackgerritAndreas Jaeger proposed openstack/project-config master: Finish retiring unused puppet modules  https://review.opendev.org/72090115:45
AJaegerSo, full list of reviews: https://review.opendev.org/#/q/status:open+topic:retire-opendev-puppet plus https://review.opendev.org/72932115:47
AJaegerinfra-root, if the list above is correct, please approve - and abandon open reviews for these repos ^15:50
AJaegerArgh, 321 is still failing16:01
openstackgerritMerged openstack/project-config master: Retire more opendev puppet modules  https://review.opendev.org/73385016:07
*** tkajinam has joined #opendev16:08
*** dtantsur is now known as dtantsur|afk16:10
*** rpittau is now known as rpittau|afk16:10
fungiharumph16:13
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs.  https://review.opendev.org/73367516:25
*** tkajinam has quit IRC16:49
openstackgerritMonty Taylor proposed opendev/puppet-openafs master: Stop requiring openafs::client  https://review.opendev.org/73389016:50
mordredfungi, AJaeger, corvus: ^^ that should fix 321 - once it lands we can recheck - or I could add a depends-on if we want16:50
*** roman_g has quit IRC16:50
clarkbmordred: do we install the client on the servers with ansible?16:51
clarkbinventory/groups.yaml says we do cool16:52
mordredclarkb: yah16:53
clarkbinfra-root now that the ptg is over I'm thinking we should maybe shutdown jvb02-04 then clean those up if demand remains low?17:05
clarkbbased on cacti data we probably didn't need 03 and 04 during the ptg either (but better to have them and not need them than otherwise)17:05
fungitc is still wrapping up, but they're using zoom17:06
clarkbgood point there could be stragglers today. Maybe do that Monday then17:06
AJaegermordred: want to add the depends-on to speed up the check?17:06
fungiclarkb: even just give it an hour17:08
fungii mean, it's friday, nobody really want to run over17:08
fungithe tc just finished their session a minute ago17:09
mordredAJaeger: sure17:09
*** gtema has left #opendev17:09
openstackgerritMonty Taylor proposed opendev/system-config master: Stop cloning more puppet modules  https://review.opendev.org/72932117:09
fungiclarkb: i bet there's a way we could see if a jvb is still in use and take it offline if not?17:09
mordredAJaeger: there we go17:09
mordredfungi, clarkb, corvus : ^^ mind re-+Aing that?17:10
clarkbfungi: there is, basically check jicofo logs for added participants as well as cpu use of the jvb process (it idles at a percent or two but in use climbs to about 10 easily)17:12
clarkbfungi: on meetpad01 do `sudo docker logs jitsi-meet-docker_jicofo_1 2>&1 | grep 'Added participant'` then on jvb0X do top basically17:14
clarkblooks like meetpad01 is in use, but the others haveb't had any new participants in a while17:15
fungiyeah, so looks like we can shut down as many of them as we like for now17:19
fungii agree, we likely want to leave jvb01 running in addition to meetpad01 mainly so we don't go back to an aio deployment and possibly regress some of the cluster communications (though our testing should also catch that)17:20
AJaegermordred: the lint job fails your openafs change ;(17:21
AJaegerhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_840/733890/1/check/legacy-puppet-lint/840571a/17:21
*** priteau has quit IRC17:21
fungiin other jitsi-meet news, today i decided to try the zoom html5 webclient on my workstation with webcam turned on... performance was basically the same for me as meetpad or maybe even a bit worse (ate all my cpu, system load skyrocketed, audio started cutting out to the point where it became unusable)17:22
fungithat was with the same browser17:23
fungiso from a system resource consumption perspective, they're quite comparable17:23
fungiwith a room topping out around 17 participants, it was really only usable for me with my camera turned off and video streaming of other participants disabled17:24
*** sgw has joined #opendev17:24
*** mlavalle has joined #opendev17:26
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs.  https://review.opendev.org/73367517:30
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs.  https://review.opendev.org/73367517:42
openstackgerritMerged openstack/project-config master: Retire swift-specs  https://review.opendev.org/73300317:56
mordredAJaeger: SIGH17:59
mordredAJaeger: how about we remove puppet-lint from that repo17:59
AJaegermordred: cheating ;)17:59
mordredAJaeger: I guess I can fix it :)18:00
AJaegerLet me send a patch ;) Let's see what's easier...18:00
mordredAJaeger: it's an easy fix18:01
openstackgerritMonty Taylor proposed opendev/puppet-openafs master: Stop requiring openafs::client  https://review.opendev.org/73389018:01
openstackgerritMonty Taylor proposed opendev/puppet-openafs master: Fix puppet lint  https://review.opendev.org/73390318:01
openstackgerritAndreas Jaeger proposed openstack/project-config master: Remove lint job from puppet-openafs  https://review.opendev.org/73390418:02
AJaegermordred: and you're faster ;)18:02
AJaegerok, will abandon mine18:03
*** DSpider has quit IRC18:12
*** hashar is now known as hasharAway18:14
*** DSpider has joined #opendev18:15
fungiclarkb: so should we just shutdown jvb02-04 manually first? and then push a patch to remove from the inventory and delete the servers once it merges?18:26
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs.  https://review.opendev.org/73367518:47
openstackgerritMerged opendev/puppet-openafs master: Fix puppet lint  https://review.opendev.org/73390318:49
clarkbfungi: ya I think so18:51
openstackgerritMerged opendev/puppet-openafs master: Stop requiring openafs::client  https://review.opendev.org/73389018:51
openstackgerritMerged opendev/system-config master: Stop cloning more puppet modules  https://review.opendev.org/72932118:51
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs.  https://review.opendev.org/73367518:56
AJaegeryeah, system-config change merged \o/18:58
AJaegerinfra-root, full list of reviews for these puppet modules: https://review.opendev.org/#/q/status:open+topic:retire-opendev-puppet plus abandon of all open reviews for them18:58
*** DSpider has quit IRC18:59
*** DSpider has joined #opendev19:00
fungiclarkb: cool, shutting down the servers now19:04
fungi#status log powered off jvb02.opendev.org, jvb03.opendev.org and jvb04.opendev.org in preparation for their removal19:06
openstackstatusfungi: finished logging19:06
openstackgerritMerged opendev/puppet-cgit master: Retire repo  https://review.opendev.org/72200119:06
openstackgerritMerged opendev/puppet-etherpad_lite master: Retire repo  https://review.opendev.org/72200219:06
openstackgerritMerged opendev/puppet-exim master: Retire repo  https://review.opendev.org/72200319:06
openstackgerritMerged opendev/puppet-gerrit master: Retire repo  https://review.opendev.org/72200419:06
openstackgerritMerged opendev/puppet-gerritbot master: Retire repo  https://review.opendev.org/72200519:06
openstackgerritMerged opendev/puppet-github master: Retire repo  https://review.opendev.org/72200619:06
openstackgerritMerged opendev/puppet-infracloud master: Retire repo  https://review.opendev.org/72200719:06
openstackgerritMerged opendev/puppet-ipsilon master: Retire repo  https://review.opendev.org/72200819:07
openstackgerritMerged opendev/puppet-iptables master: Retire repo  https://review.opendev.org/72200919:07
openstackgerritMerged opendev/puppet-os_client_config master: Retire repo  https://review.opendev.org/72201319:07
openstackgerritMerged opendev/puppet-packagekit master: Retire repo  https://review.opendev.org/72201419:07
openstackgerritMerged opendev/puppet-phabricator master: Retire repo  https://review.opendev.org/72201519:07
openstackgerritMerged opendev/puppet-ansible master: Retire repo  https://review.opendev.org/73385119:07
openstackgerritMerged opendev/puppet-snmpd master: Retire repo  https://review.opendev.org/73385319:07
openstackgerritMerged opendev/puppet-drupal master: Retire repo  https://review.opendev.org/73385519:07
corvusfungi: want to +3 https://review.opendev.org/722018 ? :)19:08
fungioh do i ever19:09
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs.  https://review.opendev.org/73367519:09
openstackgerritMerged opendev/puppet-accessbot master: Retire repo  https://review.opendev.org/72201819:09
fungii think when i first skimmed the set that one had failed a job19:10
corvusi think a solution to that was implemented :)19:10
*** ravsingh has quit IRC19:11
fungiindeed19:11
AJaegercorvus, fungi, we need to abandon a few reviews first: Please abandon:19:14
AJaegerhttps://review.opendev.org/69758819:14
AJaeger(more coming in a bit)19:14
fungican do19:15
corvusdid https://review.opendev.org/697588 make it into ansible?19:15
AJaegerabandon: https://review.opendev.org/374124 https://review.opendev.org/589001 https://review.opendev.org/295331 https://review.opendev.org/363533 https://review.opendev.org/548933 https://review.opendev.org/45092719:15
fungiAJaeger: are we still retiring puppet-openafs?19:15
AJaegerabandon: https://review.opendev.org/34436819:16
AJaegerfungi: no, we did not - sorry!19:16
AJaegerfungi: good catch19:16
clarkbcorvus: https://review.opendev.org/#/c/697589/19:16
corvusexcellent; lets abandon 588 then19:16
corvus(even though it's not necessary for retirement; save us some time later)19:17
AJaegerto abandon: https://review.opendev.org/610750 https://review.opendev.org/330260 https://review.opendev.org/#/q/project:opendev/puppet-gerrit+is:open19:17
AJaegerto abandon: https://review.opendev.org/#/q/project:opendev/puppet-etherpad_lite+is:open https://review.opendev.org/444309 https://review.opendev.org/21395219:18
AJaegerthat's it...19:18
* AJaeger added a WIP on the project-config change, let's first abandon these ^19:20
AJaegerfungi, tell me when you're done and I'll remove the WIP.19:20
fungiwill do, just a sec19:21
AJaegertake your time, just some cleanup ;)19:21
fungiokay, i think i got them all19:25
fungiAJaeger: ^19:25
fungithanks19:25
AJaegerfungi: let me refresh my views, I still see them.19:26
AJaegerfungi: did gertty finish pushing? I still see them19:27
AJaegerfungi: https://review.opendev.org/344368 https://review.opendev.org/548933 https://review.opendev.org/450927 https://review.opendev.org/36353319:28
AJaegerthe rest is gone...19:28
*** slaweq has quit IRC19:29
clarkbAJaeger: those 4 need to be abandoned?19:38
clarkbI can do that now if that is what you need19:38
AJaegerclarkb: yes, fungi missed those - that's the last for the puppet retirement19:39
AJaegerclarkb: once those are retired, we can merge https://review.opendev.org/#/c/72090119:39
fungisorry, in and out of the kitchen19:40
fungiwill take another look19:40
clarkbcool abandoning now19:40
AJaegerfungi: food or coffee is crucial ;)19:40
clarkbdone19:41
AJaegerthanks!19:41
AJaegerapproved 720901...19:41
fungiokay, got those four stragglers abandoned now. missed them in the original list19:42
AJaegerfungi: all good now19:42
AJaegerthanks clarkb and fungi19:42
* AJaeger signs off now and wishes everybody a great weekend19:43
mordredclarkb: what's the deal with centos8 there?19:48
clarkbmordred: those jobs test system-config/roles/* contents19:49
clarkbwhich includes openafs-client19:49
clarkbmordred: I think we should recheck and see if the problem persists19:49
clarkbI've done the recheck toget logs and we can go from there19:50
clarkb(there is a centos8 package for openafs that we build so it should work, but logs will tell us more)19:50
openstackgerritMerged openstack/project-config master: Finish retiring unused puppet modules  https://review.opendev.org/72090119:50
mordredclarkb: excting19:55
*** hasharAway is now known as hashar19:57
*** xiaolin has quit IRC20:06
openstackgerritMerged opendev/system-config master: Increase udp buffer sizes when installing afs client  https://review.opendev.org/69758920:43
fungii don't think i've spotted any change to remove jvb02-04 in the earlier flurry of changes, so i'll push that up now while i have a few minutes20:44
openstackgerritJeremy Stanley proposed opendev/system-config master: Revert "Add three more JVB servers"  https://review.opendev.org/73391620:47
fungiand there it is20:47
fungiconvenient those three were added separately to begin with20:47
corvusfungi: want to revert out the dns change too?21:16
corvusfungi: (should probably depends-on that one)21:16
corvusi mean, the dns removal change should depends-on the system-config removal21:16
fungioh, right-o21:18
fungion it21:18
fungiand yes on the depends-on21:18
openstackgerritJeremy Stanley proposed opendev/zone-opendev.org master: Revert "Add three more jvbs to DNS"  https://review.opendev.org/73392321:21
fungiand there's the rest21:21
fungithanks for the reminder!!!21:21
clarkbI've approved both21:30
*** DSpider has quit IRC21:39
fungidomo21:43
openstackgerritMerged opendev/system-config master: Revert "Add three more JVB servers"  https://review.opendev.org/73391621:45
*** hashar has quit IRC22:36
*** tosky has quit IRC23:56

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