openstackgerrit | Merged zuul/zuul-jobs master: download-artifact: cleanup long when statement https://review.opendev.org/733446 | 00:00 |
---|---|---|
*** DSpider has quit IRC | 00:17 | |
*** gtema has joined #opendev | 00:34 | |
*** gtema has quit IRC | 00:39 | |
*** factor has joined #opendev | 00:44 | |
*** Meiyan has joined #opendev | 00:54 | |
ianw | yay, centos8 rpms published, something up with centos7 | 00:57 |
ianw | oh dear, i wonder if this doesn't work with ovh ... | 01:00 |
ianw | that doesn't make sense though, as zuul generates the index.html | 01:03 |
ianw | ./RPMS/index.html: gzip compressed data, last modified: Fri Jun 5 00:44:47 2020, max compression, original size modulo 2^32 1066 | 01:16 |
ianw | arrggghhh mimetype issues i guess | 01:16 |
ianw | it works with compression=auto | 01:18 |
*** ysandeep|away is now known as ysandeep | 01:29 | |
*** xiaolin has joined #opendev | 01:30 | |
ianw | great, it's basically going to be impossible to generically do this i think | 01:30 |
ianw | i'll put in a fix but i think i give up | 01:31 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression https://review.opendev.org/733728 | 01:33 |
ianw | if that works on the executor, it's probably enough to keep it as an option | 01:34 |
ianw | otherwise, i guess we should re-implement wget as custom python to read the manifest, and pull all files under a given path | 01:34 |
ianw | yep, not present on the executor | 01:41 |
*** xiaolin has quit IRC | 01:45 | |
*** xiaolin has joined #opendev | 01:51 | |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression https://review.opendev.org/733728 | 02:28 |
*** smcginnis has quit IRC | 02:28 | |
*** gtema has joined #opendev | 02:35 | |
*** gtema has quit IRC | 02:39 | |
*** kevinz has joined #opendev | 02:51 | |
kevinz | ianw: there? | 02:52 |
ianw | kevinz: hey, yep | 02:52 |
*** hashar has joined #opendev | 02:52 | |
kevinz | ianw: Could you tell me the default tempest concurrency of "tempest-full-py3"? | 02:52 |
ianw | ummm, not off the top of my head but i bet we can find out :) | 02:53 |
ianw | kevinz: https://opendev.org/openstack/tempest/src/branch/master/roles/run-tempest/tasks/main.yaml#L21 | 02:56 |
ianw | i'm going to say that it would be the number of cores by default | 02:57 |
kevinz | ianw: Thanks for clarification, I will try to reduce the concurrency to see if there is different | 02:57 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression https://review.opendev.org/733728 | 02:58 |
kevinz | ianw: as some of the test are succeed to run when they are not concurrency, and I saw some rabbitmq connection to neutron in the log | 02:58 |
kevinz | and we are using 16C, so the concurrency would be 8, it maybe overload for aarch64 I guess | 02:59 |
ianw | ahhh, 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 guess | 03:05 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression https://review.opendev.org/733728 | 03:12 |
*** tkajinam is now known as tkajinam|lunch | 03:19 | |
*** tkajinam|lunch is now known as tkajinam | 03:19 | |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] download-artifact: fix compression https://review.opendev.org/733728 | 03:21 |
*** priteau has joined #opendev | 03:39 | |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Revert "download-artifact : support recursive download" https://review.opendev.org/733739 | 03:43 |
*** priteau has quit IRC | 03:46 | |
openstackgerrit | Merged zuul/zuul-jobs master: Revert "download-artifact : support recursive download" https://review.opendev.org/733739 | 04:08 |
*** ysandeep is now known as ysandeep|off | 04:27 | |
*** gtema has joined #opendev | 04:36 | |
*** ykarel|away is now known as ykarel | 04:38 | |
*** gtema has quit IRC | 04:40 | |
*** hashar has quit IRC | 05:15 | |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: download-artifact: fix missed part of revert https://review.opendev.org/733744 | 05:25 |
openstackgerrit | Merged zuul/zuul-jobs master: download-artifact: fix missed part of revert https://review.opendev.org/733744 | 05:35 |
*** gtema has joined #opendev | 05:45 | |
openstackgerrit | Merged openstack/project-config master: Prepare to retire swift-specs https://review.opendev.org/732999 | 05:45 |
ianw | clarkb: 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|afk | 05:51 |
ianw | arrrggghhh the tar extractor doesn't make directories | 05:55 |
*** gtema has quit IRC | 06:49 | |
*** lpetrut has joined #opendev | 06:50 | |
*** gtema has joined #opendev | 06:50 | |
openstackgerrit | Ian Wienand proposed opendev/system-config master: [wip] arm64 role tests https://review.opendev.org/733034 | 06:52 |
ianw | it 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 here | 06:55 |
ianw | actually they may not be published till the next release pulse, but give it 20 mins or so | 06:55 |
*** hashar has joined #opendev | 07:04 | |
*** rpittau|afk is now known as rpittau | 07:04 | |
*** gtema has quit IRC | 07:05 | |
*** priteau has joined #opendev | 07:10 | |
*** slaweq has joined #opendev | 07:15 | |
zbr | ianw: morning! | 07:22 |
*** tosky has joined #opendev | 07:31 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 07:34 |
AJaeger | ianw: do we need to revert https://review.opendev.org/#/c/733442/ as well? | 07:35 |
*** priteau has quit IRC | 07:35 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 07:40 |
frickler | AJaeger: iiuc it will be broken either way for now, so we could leave it like this as well until the recursion thing is fixed | 07:44 |
AJaeger | indeed | 07:51 |
*** DSpider has joined #opendev | 07:58 | |
*** moppy has quit IRC | 08:01 | |
*** moppy has joined #opendev | 08:01 | |
*** priteau has joined #opendev | 08:02 | |
openstackgerrit | Felix Edel proposed zuul/zuul-jobs master: Return upload_results in upload-logs-swift role https://review.opendev.org/733564 | 08:06 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support https://review.opendev.org/733762 | 08:08 |
openstackgerrit | Felix Edel proposed zuul/zuul-jobs master: Return upload_results in upload-logs-swift role https://review.opendev.org/733564 | 08:14 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support https://review.opendev.org/733762 | 08:14 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Drop mock, six usage https://review.opendev.org/733765 | 08:14 |
openstackgerrit | Ian Wienand proposed opendev/system-config master: [wip] arm64 role tests https://review.opendev.org/733034 | 08:16 |
*** ravsingh has joined #opendev | 08:19 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 08:21 |
*** priteau has quit IRC | 08:22 | |
*** priteau has joined #opendev | 08:24 | |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support https://review.opendev.org/733762 | 08:25 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Drop mock, six usage https://review.opendev.org/733765 | 08:25 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs https://review.opendev.org/733768 | 08:25 |
AJaeger | ianw: FYI, I'm cleaning up the dib py27 support ^ | 08:26 |
*** tkajinam has quit IRC | 08:31 | |
*** dtantsur|afk is now known as dtantsur | 08:34 | |
openstackgerrit | Felix Edel proposed zuul/zuul-jobs master: Return upload_results in upload-logs-swift role https://review.opendev.org/733564 | 08:39 |
*** roman_g has joined #opendev | 08:39 | |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs https://review.opendev.org/733768 | 08:50 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support https://review.opendev.org/733762 | 08:50 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Drop six usage https://review.opendev.org/733765 | 08:50 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Add libffi-dev to bindep https://review.opendev.org/733776 | 08:50 |
openstackgerrit | Ian Wienand proposed opendev/system-config master: [wip] arm64 role tests https://review.opendev.org/733034 | 09:11 |
hrw | ianw: and iirc we later moved back to 8 | 09:14 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs https://review.opendev.org/733768 | 09:27 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support https://review.opendev.org/733762 | 09:27 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Drop six usage https://review.opendev.org/733765 | 09:27 |
*** gtema has joined #opendev | 09:33 | |
ianw | hrw: ahh, yeah maybe we just kept the large format https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.openstack.org.yaml#L439 | 09:40 |
ianw | kevinz: ^ you could try that, see if it works better? would be a data point | 09:41 |
kevinz | ianw: yes, already use this to run devstack tempest | 09:47 |
kevinz | ianw: will continue check the log, it seems reducing the concurrency didn't help | 09:48 |
*** Meiyan has quit IRC | 09:57 | |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Add back pep8 and tarball jobs https://review.opendev.org/733768 | 10:04 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Cleanup py27 and docs support https://review.opendev.org/733762 | 10:04 |
openstackgerrit | Andreas Jaeger proposed openstack/diskimage-builder master: Drop six usage https://review.opendev.org/733765 | 10:04 |
*** rpittau is now known as rpittau|bbl | 10:08 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 10:16 |
*** hashar has quit IRC | 10:50 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 11:03 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add support for overriding parameters https://review.opendev.org/733809 | 11:04 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 11:11 |
*** gtema has quit IRC | 11:32 | |
*** rpittau|bbl is now known as rpittau | 11:59 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 12:08 |
*** gtema has joined #opendev | 12:13 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 12:16 |
*** gtema has joined #opendev | 12:27 | |
*** tosky_ has joined #opendev | 12:35 | |
*** tosky has quit IRC | 12:37 | |
*** tosky has joined #opendev | 12:39 | |
*** tosky_ has quit IRC | 12:40 | |
openstackgerrit | Pierre Riteau proposed openstack/project-config master: Add Backport-Candidate label for Kolla deliverables https://review.opendev.org/733243 | 12:41 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add support for overriding parameters https://review.opendev.org/733809 | 12:41 |
*** tosky_ has joined #opendev | 12:41 | |
*** tosky has quit IRC | 12:43 | |
*** gtema has quit IRC | 12:45 | |
*** tosky_ is now known as tosky | 12:50 | |
*** smcginnis has joined #opendev | 13:04 | |
*** sgw has quit IRC | 13:04 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 13:11 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add support for overriding parameters https://review.opendev.org/733809 | 13:11 |
*** avass has joined #opendev | 13:19 | |
*** smcginnis has quit IRC | 13:34 | |
*** smcginnis has joined #opendev | 13:35 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 13:35 |
*** smcginnis has quit IRC | 13:40 | |
*** auristor has quit IRC | 13:40 | |
mordred | corvus, clarkb: woot. the latest iteration of https://review.opendev.org/#/c/733409 failed correctly! https://zuul.opendev.org/t/openstack/build/b38f84ad5cf545638d7ff7b3cb8d0ab3 | 13:41 |
*** auristor has joined #opendev | 13:41 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Stop using backend hostname in zuul testinfra tests https://review.opendev.org/733409 | 13:41 |
mordred | clarkb, corvus ^^ that fixes the todo, so that should go green | 13:41 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 13:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Stop cloning a bunch of puppet modules we don't use https://review.opendev.org/720892 | 13:44 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Stop cloning more puppet modules https://review.opendev.org/729321 | 13:44 |
mordred | infra-root: I'd like to land those 2 ^^ so that we don't keep merge-conflicting | 13:44 |
*** smcginnis has joined #opendev | 13:46 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 13:53 |
fungi | i'll take a peek in a sec | 13:57 |
AJaeger | mordred: let me help with retiring the repos, I'll push everything else in shape... | 13:58 |
fungi | mordred: i've approved those but can't keep an eye on them just yet to make sure they merge | 14:01 |
fungi | we've got an hvac service rep visiting to size a replacement system for us | 14:02 |
fungi | so will likely still be tied up for a few more minutes | 14:02 |
mordred | fungi: thanks! I'll keep an eye on them | 14:04 |
mordred | AJaeger: did you see the other patch with removals? | 14:04 |
mordred | AJaeger: https://review.opendev.org/#/c/731947/ - oh, you did, you +1'd it :) | 14:04 |
AJaeger | mordred: yeah, saw both - and will check what's in flight and needs updating and what's missing... | 14:05 |
AJaeger | mordred: I'll send an email to service-discuss for the retirement... | 14:09 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules https://review.opendev.org/733850 | 14:10 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules https://review.opendev.org/733850 | 14:11 |
AJaeger | mordred, fungi, ^ | 14:11 |
openstackgerrit | Andreas Jaeger proposed opendev/puppet-ansible master: Retire repo https://review.opendev.org/733851 | 14:16 |
openstackgerrit | Andreas Jaeger proposed opendev/puppet-openafs master: Retire repo https://review.opendev.org/733852 | 14:19 |
openstackgerrit | Andreas Jaeger proposed opendev/puppet-openafs master: Retire repo https://review.opendev.org/733852 | 14:19 |
openstackgerrit | Andreas Jaeger proposed opendev/puppet-snmpd master: Retire repo https://review.opendev.org/733853 | 14:20 |
openstackgerrit | Andreas Jaeger proposed opendev/puppet-drupal master: Retire repo https://review.opendev.org/733855 | 14:21 |
*** lpetrut has quit IRC | 14:23 | |
corvus | mordred: \o/ | 14:24 |
openstackgerrit | Merged opendev/system-config master: Stop cloning a bunch of puppet modules we don't use https://review.opendev.org/720892 | 14:25 |
openstackgerrit | Carlos Goncalves proposed openstack/diskimage-builder master: Remove doc files and autoremove packages https://review.opendev.org/675546 | 14:28 |
AJaeger | mordred: you did not retire puppet-{jenkins,nodepool,openstackci}, please abandon https://review.opendev.org/722012 , https://review.opendev.org/722011 https://review.opendev.org/722010 | 14:35 |
*** gtema has joined #opendev | 14:36 | |
AJaeger | mordred: and puppet-zuul is not retired, so abandon https://review.opendev.org/#/c/722016/1 as well, please | 14:39 |
mordred | AJaeger: kk | 14:40 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Retire unused puppet modules https://review.opendev.org/720901 | 14:48 |
AJaeger | mordred, corvus, fungi: please +2 https://review.opendev.org/733850 - so that we can remove the extra repos. | 14:49 |
AJaeger | mordred: https://review.opendev.org/#/c/722018/ accessbot is also not retired yet | 14:50 |
* AJaeger will be back later | 14:51 | |
corvus | i still see puppet-openafs in modules.env | 14:52 |
*** hashar has joined #opendev | 14:53 | |
corvus | AJaeger, fungi: left a -1 for that -- can you double check me? | 14:55 |
fungi | lookin | 14:55 |
fungi | i thought mordred's change removed it | 14:55 |
fungi | hrm | 14:56 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 14:57 |
AJaeger | corvus: did 729321 merge? That should remove it | 15:13 |
AJaeger | mordred, https://review.opendev.org/#/c/729321/ failed tests and thus is not merged yet ;( | 15:14 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: WIP: Add terraform roles https://review.opendev.org/733675 | 15:14 |
AJaeger | corvus: I think we can merge 733850 already - but we can wait for 729321 as well... | 15:15 |
AJaeger | mordred: puppet-accessbot is good to go, I double checked... | 15:21 |
*** ykarel is now known as ykarel|away | 15:23 | |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules https://review.opendev.org/733850 | 15:25 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Retire unused puppet modules https://review.opendev.org/720901 | 15:25 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Finish retiring unused puppet modules https://review.opendev.org/720901 | 15:26 |
AJaeger | fungi, 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 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Stop cloning more puppet modules https://review.opendev.org/729321 | 15:30 |
mordred | AJaeger, corvus: fungi: ^^ that should fix the puppet-afs removal patch (missed a ine in the previous rev) | 15:30 |
corvus | AJaeger: i think 321 only removes ::client usage, not server | 15:30 |
mordred | corvus: AH | 15:30 |
corvus | (which is correct -- i think our servers are still all run via puppet) | 15:31 |
mordred | corvus: *duh* sorry - let me fix that | 15:32 |
corvus | we 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 |
corvus | mordred: fix what? | 15:32 |
corvus | to 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 active | 15:34 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Stop cloning more puppet modules https://review.opendev.org/729321 | 15:34 |
mordred | corvus: this ^^ | 15:34 |
mordred | corvus: it erroneously removed puppet-openafs - I have fixed | 15:34 |
corvus | mordred: cool, i agree with that fix :) | 15:34 |
* mordred got too eager | 15:34 | |
AJaeger | Ok, will readd puppet-openafs... | 15:35 |
AJaeger | (in the project-config changes) | 15:35 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 15:42 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Retire more opendev puppet modules https://review.opendev.org/733850 | 15:45 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Finish retiring unused puppet modules https://review.opendev.org/720901 | 15:45 |
AJaeger | updated - added openafs back in ^ | 15:45 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Finish retiring unused puppet modules https://review.opendev.org/720901 | 15:45 |
AJaeger | So, full list of reviews: https://review.opendev.org/#/q/status:open+topic:retire-opendev-puppet plus https://review.opendev.org/729321 | 15:47 |
AJaeger | infra-root, if the list above is correct, please approve - and abandon open reviews for these repos ^ | 15:50 |
AJaeger | Argh, 321 is still failing | 16:01 |
openstackgerrit | Merged openstack/project-config master: Retire more opendev puppet modules https://review.opendev.org/733850 | 16:07 |
*** tkajinam has joined #opendev | 16:08 | |
*** dtantsur is now known as dtantsur|afk | 16:10 | |
*** rpittau is now known as rpittau|afk | 16:10 | |
fungi | harumph | 16:13 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 16:25 |
*** tkajinam has quit IRC | 16:49 | |
openstackgerrit | Monty Taylor proposed opendev/puppet-openafs master: Stop requiring openafs::client https://review.opendev.org/733890 | 16:50 |
mordred | fungi, AJaeger, corvus: ^^ that should fix 321 - once it lands we can recheck - or I could add a depends-on if we want | 16:50 |
*** roman_g has quit IRC | 16:50 | |
clarkb | mordred: do we install the client on the servers with ansible? | 16:51 |
clarkb | inventory/groups.yaml says we do cool | 16:52 |
mordred | clarkb: yah | 16:53 |
clarkb | infra-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 |
clarkb | based 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 |
fungi | tc is still wrapping up, but they're using zoom | 17:06 |
clarkb | good point there could be stragglers today. Maybe do that Monday then | 17:06 |
AJaeger | mordred: want to add the depends-on to speed up the check? | 17:06 |
fungi | clarkb: even just give it an hour | 17:08 |
fungi | i mean, it's friday, nobody really want to run over | 17:08 |
fungi | the tc just finished their session a minute ago | 17:09 |
mordred | AJaeger: sure | 17:09 |
*** gtema has left #opendev | 17:09 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Stop cloning more puppet modules https://review.opendev.org/729321 | 17:09 |
fungi | clarkb: i bet there's a way we could see if a jvb is still in use and take it offline if not? | 17:09 |
mordred | AJaeger: there we go | 17:09 |
mordred | fungi, clarkb, corvus : ^^ mind re-+Aing that? | 17:10 |
clarkb | fungi: 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 |
clarkb | fungi: on meetpad01 do `sudo docker logs jitsi-meet-docker_jicofo_1 2>&1 | grep 'Added participant'` then on jvb0X do top basically | 17:14 |
clarkb | looks like meetpad01 is in use, but the others haveb't had any new participants in a while | 17:15 |
fungi | yeah, so looks like we can shut down as many of them as we like for now | 17:19 |
fungi | i 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 |
AJaeger | mordred: the lint job fails your openafs change ;( | 17:21 |
AJaeger | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_840/733890/1/check/legacy-puppet-lint/840571a/ | 17:21 |
*** priteau has quit IRC | 17:21 | |
fungi | in 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 |
fungi | that was with the same browser | 17:23 |
fungi | so from a system resource consumption perspective, they're quite comparable | 17:23 |
fungi | with 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 disabled | 17:24 |
*** sgw has joined #opendev | 17:24 | |
*** mlavalle has joined #opendev | 17:26 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 17:30 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 17:42 |
openstackgerrit | Merged openstack/project-config master: Retire swift-specs https://review.opendev.org/733003 | 17:56 |
mordred | AJaeger: SIGH | 17:59 |
mordred | AJaeger: how about we remove puppet-lint from that repo | 17:59 |
AJaeger | mordred: cheating ;) | 17:59 |
mordred | AJaeger: I guess I can fix it :) | 18:00 |
AJaeger | Let me send a patch ;) Let's see what's easier... | 18:00 |
mordred | AJaeger: it's an easy fix | 18:01 |
openstackgerrit | Monty Taylor proposed opendev/puppet-openafs master: Stop requiring openafs::client https://review.opendev.org/733890 | 18:01 |
openstackgerrit | Monty Taylor proposed opendev/puppet-openafs master: Fix puppet lint https://review.opendev.org/733903 | 18:01 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Remove lint job from puppet-openafs https://review.opendev.org/733904 | 18:02 |
AJaeger | mordred: and you're faster ;) | 18:02 |
AJaeger | ok, will abandon mine | 18:03 |
*** DSpider has quit IRC | 18:12 | |
*** hashar is now known as hasharAway | 18:14 | |
*** DSpider has joined #opendev | 18:15 | |
fungi | clarkb: 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 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 18:47 |
openstackgerrit | Merged opendev/puppet-openafs master: Fix puppet lint https://review.opendev.org/733903 | 18:49 |
clarkb | fungi: ya I think so | 18:51 |
openstackgerrit | Merged opendev/puppet-openafs master: Stop requiring openafs::client https://review.opendev.org/733890 | 18:51 |
openstackgerrit | Merged opendev/system-config master: Stop cloning more puppet modules https://review.opendev.org/729321 | 18:51 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 18:56 |
AJaeger | yeah, system-config change merged \o/ | 18:58 |
AJaeger | infra-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 them | 18:58 |
*** DSpider has quit IRC | 18:59 | |
*** DSpider has joined #opendev | 19:00 | |
fungi | clarkb: cool, shutting down the servers now | 19:04 |
fungi | #status log powered off jvb02.opendev.org, jvb03.opendev.org and jvb04.opendev.org in preparation for their removal | 19:06 |
openstackstatus | fungi: finished logging | 19:06 |
openstackgerrit | Merged opendev/puppet-cgit master: Retire repo https://review.opendev.org/722001 | 19:06 |
openstackgerrit | Merged opendev/puppet-etherpad_lite master: Retire repo https://review.opendev.org/722002 | 19:06 |
openstackgerrit | Merged opendev/puppet-exim master: Retire repo https://review.opendev.org/722003 | 19:06 |
openstackgerrit | Merged opendev/puppet-gerrit master: Retire repo https://review.opendev.org/722004 | 19:06 |
openstackgerrit | Merged opendev/puppet-gerritbot master: Retire repo https://review.opendev.org/722005 | 19:06 |
openstackgerrit | Merged opendev/puppet-github master: Retire repo https://review.opendev.org/722006 | 19:06 |
openstackgerrit | Merged opendev/puppet-infracloud master: Retire repo https://review.opendev.org/722007 | 19:06 |
openstackgerrit | Merged opendev/puppet-ipsilon master: Retire repo https://review.opendev.org/722008 | 19:07 |
openstackgerrit | Merged opendev/puppet-iptables master: Retire repo https://review.opendev.org/722009 | 19:07 |
openstackgerrit | Merged opendev/puppet-os_client_config master: Retire repo https://review.opendev.org/722013 | 19:07 |
openstackgerrit | Merged opendev/puppet-packagekit master: Retire repo https://review.opendev.org/722014 | 19:07 |
openstackgerrit | Merged opendev/puppet-phabricator master: Retire repo https://review.opendev.org/722015 | 19:07 |
openstackgerrit | Merged opendev/puppet-ansible master: Retire repo https://review.opendev.org/733851 | 19:07 |
openstackgerrit | Merged opendev/puppet-snmpd master: Retire repo https://review.opendev.org/733853 | 19:07 |
openstackgerrit | Merged opendev/puppet-drupal master: Retire repo https://review.opendev.org/733855 | 19:07 |
corvus | fungi: want to +3 https://review.opendev.org/722018 ? :) | 19:08 |
fungi | oh do i ever | 19:09 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 19:09 |
openstackgerrit | Merged opendev/puppet-accessbot master: Retire repo https://review.opendev.org/722018 | 19:09 |
fungi | i think when i first skimmed the set that one had failed a job | 19:10 |
corvus | i think a solution to that was implemented :) | 19:10 |
*** ravsingh has quit IRC | 19:11 | |
fungi | indeed | 19:11 |
AJaeger | corvus, fungi, we need to abandon a few reviews first: Please abandon: | 19:14 |
AJaeger | https://review.opendev.org/697588 | 19:14 |
AJaeger | (more coming in a bit) | 19:14 |
fungi | can do | 19:15 |
corvus | did https://review.opendev.org/697588 make it into ansible? | 19:15 |
AJaeger | abandon: 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/450927 | 19:15 |
fungi | AJaeger: are we still retiring puppet-openafs? | 19:15 |
AJaeger | abandon: https://review.opendev.org/344368 | 19:16 |
AJaeger | fungi: no, we did not - sorry! | 19:16 |
AJaeger | fungi: good catch | 19:16 |
clarkb | corvus: https://review.opendev.org/#/c/697589/ | 19:16 |
corvus | excellent; lets abandon 588 then | 19:16 |
corvus | (even though it's not necessary for retirement; save us some time later) | 19:17 |
AJaeger | to abandon: https://review.opendev.org/610750 https://review.opendev.org/330260 https://review.opendev.org/#/q/project:opendev/puppet-gerrit+is:open | 19:17 |
AJaeger | to abandon: https://review.opendev.org/#/q/project:opendev/puppet-etherpad_lite+is:open https://review.opendev.org/444309 https://review.opendev.org/213952 | 19:18 |
AJaeger | that's it... | 19:18 |
* AJaeger added a WIP on the project-config change, let's first abandon these ^ | 19:20 | |
AJaeger | fungi, tell me when you're done and I'll remove the WIP. | 19:20 |
fungi | will do, just a sec | 19:21 |
AJaeger | take your time, just some cleanup ;) | 19:21 |
fungi | okay, i think i got them all | 19:25 |
fungi | AJaeger: ^ | 19:25 |
fungi | thanks | 19:25 |
AJaeger | fungi: let me refresh my views, I still see them. | 19:26 |
AJaeger | fungi: did gertty finish pushing? I still see them | 19:27 |
AJaeger | fungi: https://review.opendev.org/344368 https://review.opendev.org/548933 https://review.opendev.org/450927 https://review.opendev.org/363533 | 19:28 |
AJaeger | the rest is gone... | 19:28 |
*** slaweq has quit IRC | 19:29 | |
clarkb | AJaeger: those 4 need to be abandoned? | 19:38 |
clarkb | I can do that now if that is what you need | 19:38 |
AJaeger | clarkb: yes, fungi missed those - that's the last for the puppet retirement | 19:39 |
AJaeger | clarkb: once those are retired, we can merge https://review.opendev.org/#/c/720901 | 19:39 |
fungi | sorry, in and out of the kitchen | 19:40 |
fungi | will take another look | 19:40 |
clarkb | cool abandoning now | 19:40 |
AJaeger | fungi: food or coffee is crucial ;) | 19:40 |
clarkb | done | 19:41 |
AJaeger | thanks! | 19:41 |
AJaeger | approved 720901... | 19:41 |
fungi | okay, got those four stragglers abandoned now. missed them in the original list | 19:42 |
AJaeger | fungi: all good now | 19:42 |
AJaeger | thanks clarkb and fungi | 19:42 |
* AJaeger signs off now and wishes everybody a great weekend | 19:43 | |
mordred | clarkb: what's the deal with centos8 there? | 19:48 |
clarkb | mordred: those jobs test system-config/roles/* contents | 19:49 |
clarkb | which includes openafs-client | 19:49 |
clarkb | mordred: I think we should recheck and see if the problem persists | 19:49 |
clarkb | I've done the recheck toget logs and we can go from there | 19:50 |
clarkb | (there is a centos8 package for openafs that we build so it should work, but logs will tell us more) | 19:50 |
openstackgerrit | Merged openstack/project-config master: Finish retiring unused puppet modules https://review.opendev.org/720901 | 19:50 |
mordred | clarkb: excting | 19:55 |
*** hasharAway is now known as hashar | 19:57 | |
*** xiaolin has quit IRC | 20:06 | |
openstackgerrit | Merged opendev/system-config master: Increase udp buffer sizes when installing afs client https://review.opendev.org/697589 | 20:43 |
fungi | i 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 minutes | 20:44 |
openstackgerrit | Jeremy Stanley proposed opendev/system-config master: Revert "Add three more JVB servers" https://review.opendev.org/733916 | 20:47 |
fungi | and there it is | 20:47 |
fungi | convenient those three were added separately to begin with | 20:47 |
corvus | fungi: want to revert out the dns change too? | 21:16 |
corvus | fungi: (should probably depends-on that one) | 21:16 |
corvus | i mean, the dns removal change should depends-on the system-config removal | 21:16 |
fungi | oh, right-o | 21:18 |
fungi | on it | 21:18 |
fungi | and yes on the depends-on | 21:18 |
openstackgerrit | Jeremy Stanley proposed opendev/zone-opendev.org master: Revert "Add three more jvbs to DNS" https://review.opendev.org/733923 | 21:21 |
fungi | and there's the rest | 21:21 |
fungi | thanks for the reminder!!! | 21:21 |
clarkb | I've approved both | 21:30 |
*** DSpider has quit IRC | 21:39 | |
fungi | domo | 21:43 |
openstackgerrit | Merged opendev/system-config master: Revert "Add three more JVB servers" https://review.opendev.org/733916 | 21:45 |
*** hashar has quit IRC | 22:36 | |
*** tosky has quit IRC | 23:56 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!