| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 00:17 |
|---|---|---|
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add inline comment support to nox https://review.opendev.org/c/zuul/zuul-jobs/+/868134 | 00:17 |
| *** rlandy|bbl is now known as rlandy | 00:17 | |
| ianw | ... http://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5fc/868054/3/check/system-config-run-review-3.6/5fccf0c/bridge99.opendev.org/screenshots/gerrit-main-page.png | 00:35 |
| ianw | this *does* show the RP column | 00:35 |
| ianw | http://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_873/868054/2/check/system-config-run-review-3.6/873fa28/bridge99.opendev.org/screenshots/gerrit-main-page.png | 00:35 |
| clarkb | ianw: whats the difference? | 00:36 |
| ianw | from yesterday, doesn't | 00:36 |
| clarkb | oh heh did gerrit change something? | 00:36 |
| ianw | no i don't think so | 00:36 |
| ianw | i think with that one, we have 3 changes and i set the votes to -1, +1 and +2 ... none of them have a zero vote | 00:36 |
| ianw | i wonder if its something to do with *all* changes having to have a vote for the column to show? a thread to pull on | 00:37 |
| ianw | on the change before, i think i had a -2, 0 and +2 vote on the three test changes | 00:38 |
| ianw | ok i think i replicated it on https://213.32.72.249/dashboard/self | 00:43 |
| ianw | it seems the column only appears if a change has a negative vote | 00:43 |
| ianw | interesting. there i've setup "review priority" which goes from -1 -> +2. i wonder if it's negative, or "max negative" -- i.e. if i change that to have -2 votes, will a -1 make the column appear | 00:47 |
| ianw | ... ok it's a "max negative" that makes it appear. i.e. i added -2 votes ^, and a -1 vote does not make the RP column appear, but a -2 does | 00:49 |
| ianw | i feel like the old behviour was the column appears if any vote is made | 00:50 |
| ianw | knowing that ... i think https://gerrit.googlesource.com/gerrit/+/refs/heads/master/polygerrit-ui/app/utils/label-util.ts is where it's all computed | 01:15 |
| *** rlandy is now known as rlandy|out | 01:30 | |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 02:09 |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add inline comment support to nox https://review.opendev.org/c/zuul/zuul-jobs/+/868134 | 02:09 |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 02:41 |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add inline comment support to nox https://review.opendev.org/c/zuul/zuul-jobs/+/868134 | 02:41 |
| *** yadnesh|away is now known as yadnesh | 04:10 | |
| ianw | so i know generally what's going on -- the thing that lays out the changes is looking at the change and essentially looking at the .submit_requirements field, and showing a column for each of those | 05:20 |
| ianw | i think this is a implementation detail of the "anyWithBlock" config item (lowest vote blocks change) -- only if the change has a lowest vote for one of our labels does it's .submit_requirements field have an entry | 05:21 |
| ianw | { | 05:22 |
| ianw | "name": "Code-Review", | 05:22 |
| ianw | "status": "UNSATISFIED", | 05:22 |
| ianw | "is_legacy": true, | 05:22 |
| ianw | "submittability_expression_result": { | 05:22 |
| ianw | "expression": "label:Code-Review=MAX -label:Code-Review=MIN", | 05:22 |
| ianw | "fulfilled": false, | 05:22 |
| ianw | "passing_atoms": [], | 05:23 |
| ianw | "failing_atoms": [ | 05:23 |
| ianw | "label:Code-Review=MAX", | 05:23 |
| ianw | "-label:Code-Review=MIN" | 05:23 |
| ianw | ] | 05:23 |
| ianw | } | 05:23 |
| ianw | } | 05:23 |
| ianw | oops, didn't mean to paste that ... but anyway ... that is the submit_requirements object in the change | 05:23 |
| ianw | ok, https://213.32.72.249/dashboard/self is clearly what you're supposed to do. use submitRequirementColumns to statically set that column as showing up | 05:54 |
| ianw | the problem is that doesn't really work for us that well where projects may or may not use this, or may have their own versions of tags. | 05:55 |
| ianw | anyway, i've got enough insight now to discuss this properly with upstream. | 05:56 |
| ianw | this is lightly discussed in https://groups.google.com/g/repo-discuss/c/9QM7d1Cw2tM ... but doesn't quite cover our usage | 05:56 |
| *** pojadhav- is now known as pojadhav|rover | 06:04 | |
| *** akahat|ruck is now known as akahat|ruck|lunch | 09:42 | |
| *** dviroel|out is now known as dviroel | 10:58 | |
| *** akahat|ruck|lunch is now known as akahat|ruck | 11:05 | |
| *** rlandy|out is now known as rlandy | 11:19 | |
| *** yadnesh is now known as yadnesh|away | 12:39 | |
| fungi | yeah, that seems to be a site-wide list of which labels to show, while we want to get the dynamic inclusion back | 13:12 |
| *** dasm|off is now known as dasm | 13:43 | |
| *** cloudnull5 is now known as cloudnull | 14:31 | |
| *** jpena|off is now known as jpena | 14:40 | |
| *** dviroel is now known as dviroel|lunch | 15:43 | |
| fungi | okay, i've managed to confirm on the held node that `docker exec mailman-web ./manage.py runjobs hourly` does indeed exit nonzero (137) when SITE_ID=0 in the settings.py and not when i switch it to SITE_ID=1, but then rerunning it succeeds when switching it back to SITE_ID=0 after that | 16:06 |
| fungi | i wonder if simple lineinfile tasks to undo/redo that edit around the manage.py call would suffice | 16:06 |
| opendevreview | Jeremy Stanley proposed opendev/zone-opendev.org master: Dummy mailman hostname to house the list of lists https://review.opendev.org/c/opendev/zone-opendev.org/+/867981 | 16:14 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Use magic domain guessing in Mailman 3 https://review.opendev.org/c/opendev/system-config/+/867987 | 16:14 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: DNM force mm3 failure to hold the node https://review.opendev.org/c/opendev/system-config/+/855292 | 16:14 |
| fungi | okay, new autohold in place for 855292 on the assumption that 867987 will pass this time. also 867981 was a rebase to resolve the serial conflict from the iweb mirror server cleanup | 16:19 |
| clarkb | fungi: and you're still aiming to test that on he hold before landing anything? | 16:22 |
| fungi | yeah, though i did test that the temporary edit should solve the warning from the faq about initial migrations run breaking (what i think the job failure on the previous revision demonstrated too) | 16:22 |
| fungi | i wonder if we should be running migrations earlier in the job though, like as part of the server deploy | 16:23 |
| fungi | since right now it's only getting exercised in the test | 16:23 |
| clarkb | are we automating any migrations? | 16:24 |
| fungi | these are database migrations | 16:24 |
| fungi | which run from the hourly cronjobs on the server | 16:24 |
| clarkb | oh db migrations. They are run when the service starts | 16:25 |
| clarkb | pretty sure if you look at the service logs part of the startup is the migrations | 16:25 |
| clarkb | and part of what we wait for on setting up the admin user is sufficient db content | 16:25 |
| fungi | except it's only `./manage.py runjobs hourly` which seems to fail with a migrations problem, container startup works fine | 16:25 |
| clarkb | ya that is odd. Maybe they don't run all of the migrations on startup just a minimal set | 16:26 |
| clarkb | (an interesting design feature...) | 16:26 |
| fungi | so if we stand up a new server with SITE_ID=0 then the hourly cronjobs in the mailman-web container will break until we run them at least once with SITE_ID=1 temporarily | 16:27 |
| fungi | not sure how exactly to encode that into ansible for actual server deployment | 16:27 |
| fungi | (for the test it seems easy enough) | 16:28 |
| clarkb | I think you can start the deployment with site id = 1. Then run the hourly cron manually if site id = 1 and if that succeeds flip side id to 0 | 16:28 |
| clarkb | then you won't run the migrations explicitly in the future because you've flipped the switch on them | 16:28 |
| fungi | oh, interesting approach. so pseudocode "if SITE_ID=1 exec hourly cron and set SITE_ID=0" | 16:29 |
| clarkb | yup | 16:29 |
| fungi | how do we reconcile that with deploying new copies of the settings.py file though? | 16:30 |
| fungi | since that would undo the SITE_ID=0 | 16:30 |
| clarkb | we would probably need to template the file with a default value of 1, but then set SITE_ID to the value that already exists in the file maybe? You're right that this is clunky (and largely a side effect of mailman not being able to run under its expected config from the start) | 16:31 |
| clarkb | or use versions of the file and select the appropriate file version based on where you are in that conditional logic | 16:32 |
| clarkb | this might be simpler to implement in ansible but result in copying extra content around | 16:32 |
| fungi | we could drop a flagfile to track whether the initial exec has happened and if that file isn't present then switch to SITE_ID=1 temporarily and run the exec and switch back to SITE_ID=0 to match the copy of the file we deploy? | 16:33 |
| fungi | worst case someone blows away the flagfile and ansible reruns the cron once | 16:34 |
| fungi | that avoids templating or having multiple copies of the config | 16:34 |
| clarkb | ya that might work | 16:34 |
| clarkb | and it might be useful to be able to delete that file and have it rerun | 16:35 |
| fungi | i'll try to work on that if i get another break between meetings today | 16:35 |
| opendevreview | Clark Boylan proposed opendev/bindep master: Use nox https://review.opendev.org/c/opendev/bindep/+/868004 | 16:37 |
| *** marios is now known as marios|out | 16:46 | |
| *** dviroel|lunch is now known as dviroel | 16:59 | |
| opendevreview | James E. Blair proposed opendev/base-jobs master: Add nox-docs base jobs https://review.opendev.org/c/opendev/base-jobs/+/868226 | 17:08 |
| opendevreview | James E. Blair proposed opendev/base-jobs master: Add nox-docs base jobs https://review.opendev.org/c/opendev/base-jobs/+/868226 | 17:15 |
| opendevreview | James E. Blair proposed opendev/base-jobs master: Add nox-docs base jobs https://review.opendev.org/c/opendev/base-jobs/+/868226 | 17:17 |
| *** jpena is now known as jpena|off | 17:20 | |
| fungi | opendev holiday party is happening in https://meetpad.opendev.org/opendev-holiday-party and #opendev-meeting and on https://boardgamearena.com/ | 19:07 |
| fungi | latest revision to 867987 still didn't work, i wonder if we need to down and up the mailman-web container with each change to settings.py | 20:30 |
| fungi | i hadn't thought about it, but i did that when i was testing on the held node so maybe that's why it worked for me | 20:30 |
| Clark[m] | Or maybe hup it at least | 20:31 |
| ianw | fungi/clarkb : https://etherpad.opendev.org/p/-MyiqZX4dijrluhkNuXx is my draft email about the column display. it does tie in with the submitrequirements, but one thing at a time :) i'm having trouble ending it with a concrete suggestion | 21:17 |
| rgunasekaran | Hello All, | 21:19 |
| rgunasekaran | new to this group & the community so bear with me while i get my footing | 21:19 |
| rgunasekaran | I want to contribute, can you help me picking out something to work on? please suggest on accurate successful documentation for setting up the environment and start contributing! | 21:19 |
| clarkb | rgunasekaran: is there anything in particular that interests you? Right now the main things we've got going on are a few conversions of services to containers (from puppet), sorting out Gerrit ugprades (what ianw is working on above), dealing with the recent tox v4 release (it isn't backward compatible), and updating our existing containers to newer versions of python. | 21:23 |
| clarkb | Oh and finally some server upgrades from ubuntu bionic to jammy | 21:23 |
| clarkb | ianw: couple of notes on your etherpad draft but lgtm | 21:25 |
| rgunasekaran | @clarkb: i am interested to work with containers & kubernetes! please let me know how i can contribute for the same! | 21:27 |
| clarkb | we don't currently run any kubernetes. All of our containers are run by docker-compose managed by ansible. Most of that can be found in https://opendev.org/opendev/system-config under the docker/ and playbooks/roles/ dirs | 21:28 |
| clarkb | One example activity that would be helpful is updating our images like the refstack image (https://opendev.org/opendev/system-config/src/branch/master/docker/refstack/Dockerfile) to python3.10 or 3.11 depending on compatibility. With the eventual goal that if we get everything off of python3.9 and can stop building those images | 21:30 |
| ianw | clarkb: thanks! i'll come back it and let it percolate, see if i have any other ideas now i understand what's happening | 21:30 |
| clarkb | https://codesearch.opendev.org/?q=opendevorg%2Fpython-builder%3A3.9-bullseye&i=nope&literal=nope&files=&excludeFiles=&repos= shows you what is still on 3.9 and if you look at the git history for those files you'll see their evolution from 3.6 -> 3.9 or so | 21:30 |
| rgunasekaran | clarkb: okay i can start contribute for containers run by docker-compose managed by ansible | 21:33 |
| clarkb | rgunasekaran: another good place to start to get familiar with things is to look at the existing changes that have been proposed | 21:34 |
| clarkb | rgunasekaran: https://review.opendev.org/q/status:open+project:opendev/system-config for example | 21:35 |
| clarkb | that will show you what others are currently work on to get a sense for what changes are happening | 21:35 |
| rgunasekaran | sure @clarkb i am there on the opendev changes | 21:38 |
| rgunasekaran | looking at the open changes! | 21:38 |
| *** dviroel is now known as dviroel|out | 21:42 | |
| clarkb | infra-root: I'm working on collecting possible content for the OpenDev Foundation Annual Report section. Currently plan on calling out the Gerrit upgrades, work with Zuul to get to fully redundant deployment and automated upgrades, (slow) transition to Jammy, and python3.10/3.11 bootstrapping. I'm sure there is moreand I still need to look at git logs and my historical notes file for | 21:55 |
| clarkb | ideas. Anything you'd like to see get called out? | 21:55 |
| clarkb | oh the centos streaming happened | 21:56 |
| clarkb | We fixed the Gerrit ssh rsa problem | 21:57 |
| clarkb | We shutdown the ELK system | 21:57 |
| clarkb | And ethercalc | 21:57 |
| ianw | ansible upgrades across zuul maybe? | 21:57 |
| clarkb | ++ | 21:57 |
| clarkb | The bastion updates should be mentioned too | 21:58 |
| clarkb | We deployed a keycloak and mm3 | 21:59 |
| clarkb | The jitsi meet upgrades | 22:00 |
| ianw | not sure how to phrase it but things that we noticed/did that went back into zuul upstream; e.g. streaming console, quite a few UI/PF4 upgrade things, other bugs i've forgotten | 22:00 |
| clarkb | In the past I think I've descirbed how we have a feedback loop between zuul and opendev. We dogfood zuul and help make it better. I think we should continue to cal lthat out | 22:01 |
| fungi | revisiting the non-submit-requirement labels in gerrit, i wouldn't be surprised if they come back with the suggestion to use hashtags instead | 22:01 |
| clarkb | fungi: thats a good point. I don't think hashtags get a summary on listings though so may not fully solve the problem. But I'm willing to try it if that is their response | 22:02 |
| ianw | fungi: yeah; although we do have the ones that are setup to block on negative votes too. with 3.6 they're doing the weird thing of any blocking vote will make the summary column appear, but non-blocking ones won't | 22:03 |
| fungi | in a way, these use cases do seem like overloading the label/vote mechanism for off-label (pun intended) purposes | 22:03 |
| ianw | yeah, that was what i was trying to get at with the "gerrit -> developer" communication, versus "developer -> developer" communication | 22:04 |
| *** JasonF is now known as JayF | 22:39 | |
| ianw | i've uploaded a ubuntu base image to the new linaro cloud; it sort of seems to work but has no console. not 100% sure what's going on. probably best to trying bringup of a mirror node as first thing | 22:49 |
| ianw | i've created some users and projects that should hopefully wokr | 22:49 |
| opendevreview | Ian Wienand proposed opendev/system-config master: Linaro: initial cloud config https://review.opendev.org/c/opendev/system-config/+/868247 | 23:19 |
| clarkb | that looks good to me. I think you can probabl approve it once the groupvars/hostvars are sorted | 23:33 |
| ianw | thanks, yeah i want to watch the cloud launcher run closely, i may well have stuffed something up on the cloud side | 23:37 |
| opendevreview | Ian Wienand proposed opendev/system-config master: Linaro: initial cloud config https://review.opendev.org/c/opendev/system-config/+/868247 | 23:41 |
| ianw | project_name/project_id typo. not sure why we have both | 23:41 |
| clarkb | I think thats a long term holdover from some clouds only using ids and not names? | 23:54 |
| clarkb | its probably slightly faster to use ids too | 23:55 |
| clarkb | but I like names for human interaction | 23:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
