Thursday, 2022-04-21

opendevreviewGhanshyam proposed openstack/governance master: Remove TC Liaisons framework  https://review.opendev.org/c/openstack/governance/+/83789101:38
opendevreviewGhanshyam proposed openstack/governance master: Resolution to drop the lower constraints maintenance  https://review.opendev.org/c/openstack/governance/+/83800401:55
*** pojadhav is now known as pojadhav|ruck04:55
*** pojadhav|ruck is now known as pojadhav|lunch08:07
*** pojadhav|lunch is now known as pojadhav|ruck09:21
*** pojadhav|ruck is now known as pojadhav|brb11:22
*** pojadhav|brb is now known as pojadhav|ruck11:54
slaweqhi tc-members, I'm a bit sick today and I will not be able to attend tc meeting. Sorry for that. I will read logs from the meeting later14:47
gmannslaweq: np! I will catch up later if something for you. 14:48
mnaserslaweq: feel better! :)14:51
dansmithI suspect slaweq's own report shows him doing a bunch of blind rechecks and doesn't want to face the music, what do ya'll think?14:59
gmann:)15:00
gmann#startmeeting tc15:00
opendevmeetMeeting started Thu Apr 21 15:00:08 2022 UTC and is due to finish in 60 minutes.  The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'tc'15:00
jungleboyjo.15:00
dansmitho/15:00
jungleboyjo/15:00
gmann#topic Roll call15:00
gmanno/15:00
dpawlik\o15:01
gmannslaweq will not be present as he informed before meeting. kendall in Asia TZ so would not join.15:02
spotzo/15:02
gmannknikolla arne_wiebalck rosmaita meeting time15:02
rosmaitao/15:02
gmannToday agenda #link https://wiki.openstack.org/wiki/Meetings/TechnicalCommittee#Agenda_Suggestions15:02
gmann#topic Follow up on past action items15:03
gmannno action item as such, as this is first meeting after PTG15:03
gmann#topic Zed cycle Tracker15:03
gmann#link https://etherpad.opendev.org/p/tc-zed-tracker15:04
gmann#link http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028206.html15:04
gmannas you know, I have prepared the zed cycle tracker15:04
gmannplease check the items .15:04
gmanntwo items 2nd and 3rd need assignee 15:05
gmannI will wait for spotz arne_wiebalck Kendal to check first and then in next meeting we can discuss it15:06
gmannidea is we all in TC distribute the items we target for the cycle15:06
jungleboyjI added myself to the Documentation updates for release cadence.15:06
spotzI just added myself on to 215:07
spotzNot 2 or 3 though:(15:07
rosmaitaok, looks like me & jay for #215:08
jungleboyjAlso could work with rosmaita on the logging one.  Did logging stuff back in the day.15:08
gmann+1 15:08
jungleboyjShould not sign up for more as I know it is going to be a very busy 6 months for me.15:08
dansmith#3 is osc as in openstackclient yeah?15:08
arne_wiebalcko/15:08
gmannyes, that is important 15:08
gmannjungleboyj: np!. 15:09
dansmithI'm also signed up for multiple things, but I will be second fiddle on the osc one for sure15:09
dansmithI think glance might be a tough sell on that one, but important :/15:09
gmanndansmith: thanks15:09
jungleboyjI will be second fiddle on the Release Cadence.15:10
rosmaitai need to look into the claim of cinderclient feature parity15:10
jungleboyjThe user survey analysis should take toooo much more time.15:10
gmannanyone else for osc one as primary assignee ? 15:10
jungleboyjrosmaita:  ++ That does seem suspicious.15:10
gmannmainly to find someone ti drive the goal which is difficult so mainly "drive the goal"15:11
dansmithI thought keystone had already deprecated their CLI15:12
gmannI think so but not 100%15:12
gmannanyways I will keep it open if anyone want to help in this. dansmith is already signed up for help but need primary assignee 15:13
gmannanything else on tracker?15:13
spotzdmendiza[m]: Do you know if the Keystone CLI is deprecated?15:13
arne_wiebalckgmann: I am not signed up for anything yet, so ... 15:14
dansmitharne_wiebalck: osc!15:14
jungleboyj++ 15:14
spotz++:)15:15
dansmithwoooot15:15
gmannarne_wiebalck: great, thanks. 15:15
gmannand we will check the progress  in every alternate week on each item15:15
rosmaitaarne_wiebalck: you are the lucky winner!15:15
gmannmoving on..15:16
arne_wiebalck*gets a "hot ptotato" feeling*15:16
gmann:)15:16
gmann#topic Gate health check15:16
gmannany news on gate.15:16
dansmithso, I just heard in the glance meeting that their periodic cs9 fips job is failing off and on15:16
dansmithand a quick look showed another volume test15:16
dansmithlooks like cinder-volume is unable to allocate memory,15:17
dansmithbut I don't think there's an OOM15:17
gmannok may be we need to apply SSH-able workaround in more tests?15:17
gmannok15:17
dansmithno, I think it's memory-related15:17
gmannI see15:17
dansmithwhich is a great transition to....15:17
dansmithhttps://review.opendev.org/c/openstack/devstack/+/83713915:17
gmann+115:17
dansmiththis ^ might help us see that one service is really fat on cs9 or something and help explain the difference15:17
gmannin devstack we have c9 as voting, need to check if that is blcoked15:17
clarkbhttps://github.com/bloomberg/memray was getting traction on hacker news yesterday as a python memory profiler15:17
dansmithgmann: it's not 100% apparently, but yeah15:18
clarkbmight be a cood subsequent thing to look into once dansmith's tracking has landed and offenders are identified15:18
gmannk15:18
dansmithcool15:18
clarkbwas thinking we might be able to have a profiler like that as a non default devstack service that we can turn on for specific changes or something15:18
gmanndansmith: how we will track the some service going high in memory? compare with few previous run data?15:18
clarkbthen you don't really even need to run it locally if you don't want to15:18
dansmithclarkb: yeah15:18
gmannor capture the current usage somewhere and use that as threshold 15:19
dansmithgmann: still working on that for total automation, but I'm going to work on at least a simple tool that will let you compare two runs and highlight major differences15:19
gmanndansmith: great15:19
dansmithgmann: the easy thing to do is to just establish some baselines and then warn if you're 5% over that (per job type) or something15:19
gmannyeah, that will be good15:20
dansmithbut clarkb is also maybe going to help me with some grafana stuff and dpawlik is digesting the json file into opensearch15:20
gmann#link https://review.opendev.org/c/openstack/ci-log-processing/+/83865515:21
dpawlikdansmith: the logscraper patch is in review  https://review.opendev.org/c/openstack/ci-log-processing/+/83865515:21
dansmith++15:21
gmannthis is nice direction. 15:21
dansmithanyway, that's all I know of current state-of-the-gate15:21
gmanngit things are also merged in all supported branches and I think on EM also15:21
dpawlikwhen its merged, I will recreate container image and deploy latest version, so soon should be some results in the Opensearch15:22
dansmithI saw the recheck on T this morning so I assume that landed but didn't check and if so, all good on the git stuff yeah15:22
dansmithdpawlik: we have to merge the actual change to generate the file too, but yeah awesome :)15:22
gmannyeah all merged for git things- https://review.opendev.org/q/topic:bug%252F196879815:22
gmann#link https://review.opendev.org/q/topic:bug%252F196879815:22
dansmithsweet15:23
gmanndpawlik: +115:23
gmannanything else on gate?15:23
dansmithnot from me15:23
rosmaitamy, my, c-bak really is a memory hog15:23
clarkbas is privsep15:24
gmannwe were disabling that by default in devstack, is that merged?15:24
clarkbgmann: no the change is still WIP'd I think it failed beacuse there is some testing of c-bak in tempest?15:24
gmann#lin https://review.opendev.org/c/openstack/devstack/+/83763015:24
clarkbit will need more coordination to land that15:24
gmann#link https://review.opendev.org/c/openstack/devstack/+/83763015:24
gmannyeah, I think we can run those in separate job, I am sure cinder have few c-back specific job15:25
clarkbI removed my WIP feel free to update if we want ot keep pushing that15:25
gmann+115:25
dansmithrosmaita: yeah a gig is a lot :/15:25
gmannrosmaita: its on you I think ^^15:25
gmannwe are waiting for you opinion there but let's discuss it in gerrit15:26
rosmaitayeah, i will see if we can move the c-bak tests to cinder-tempest-plugin or something15:26
gmannor just run from tempest location.15:27
dansmithsee, performance.json is already helping and it isn't even merged. :P15:27
* dansmith whispers in kopecmartin's ear15:27
gmannlet's discuss that in review or in qa channel 15:27
gmann#topic Migration from old ELK service to new Dashboard15:27
gmannbefore we talk about the shutdown of old ELK server, let's check new dashboard15:28
clarkbI Guess I can just jump in?15:28
gmannCommunicating the new ELK service dashboard and login information15:28
clarkbah I'll wait15:28
gmannyeah15:28
gmanndpawlik: if you can update here what is state of new dashbaord15:28
dpawlikso the whole cluster is working normally without any issue so far15:29
dpawlikI create few simple visualization, one dashboard, but it just show basic information, nothing else15:29
dpawlikin the future I got a plan to create more visualization/better dashboard that will be helpful for developers15:30
gmann+115:30
dpawlikthe auto login is.... crazy to implement. I spend some time on doing it and I did not create a good configuration that will do autologin15:31
clarkbI would assume that is a lower priority since sharing credentials isn't the end of the world15:31
dpawlikalso backup/restore of kibana objects like visualization/dashboard should be done15:31
gmannyeah, autologin will be good to have but not blocker or priority thing. 15:32
dpawlikthe logstash service is not needed. The logscraper + logsender is working fine but will be good to have some feedback from the community if some builds are not there15:32
gmannand dpawlik has updated the login/url information in tact sig and p-t-g  15:33
gmann#link https://docs.openstack.org/project-team-guide/testing.html#checking-status-of-other-job-results15:34
gmann#link https://governance.openstack.org/sigs/tact-sig.html#opensearch15:34
gmannthese place we can use to communicate to community15:34
gmannI think next step is send it on ML and ask community to use. dpawlik ?15:34
dpawlikyup, will be good. I will write a message 15:35
fungi#link https://review.opendev.org/833264 also raises the question of whether opendev will be able to get rid of the status.o.o server as well, the main thing i'm wondering at this point is whether people are still actively using http://status.openstack.org/reviews/ (reviewday)15:35
gmannyeah, we will discuss next what all good to shutdown15:35
dansmithum,15:35
dansmithhow would we get rid of that.. we need it for the zuul dashboard at least right?15:36
dansmithstatus.o.o I mean15:36
gmanndpawlik: thanks, Please send and thanks again for your work.15:36
fungithe zuul dashboard is zuul.opendev.org15:36
gmann#action dpawlik to send the new dashboard communication on openstack-discuss ML15:36
dansmithoh, sorry I didn't realize it was redirecting there, I see15:36
dansmithI still hit status.o.o by habit15:36
fungistatus.openstack.org is what served the elastic recheck and openstack health dashboard, both of which are going away now15:36
gmannShutdown of old ELK service15:36
dpawliksame as dansmith15:36
gmannclarkb: your turn15:36
clarkbok so about a year ago the opendev team called out the concern that the ELK systems were under maintained and running on old distro releases. We asked for help but said without help we would have to shut it down. We also said we'd do our best to keep it up through the yoga release15:37
clarkbYoga has happend and while we didn't get direct help openstack took on the effort of using opensearch to replace this service themselves15:37
clarkbFor this reason I think we're ready to go ahead and remove the service and servers from opendev.15:38
clarkbThere is another pressure pushing this along which is that zuul is changing the way executors run playbooks which means we cannot rely on firewall rules to protect the submission of indexing jobs to the service with iptables any longer15:38
clarkbThis gives us a bit of a hard deadline for removing things: when those zuul updates land. But I'd like to go ahead and start shutting it down sooner (now)15:39
gmannAs new dashbaord is ready to use and we are going to communicate it on ML. we are good to shutdown the old server. 15:39
gmannany thoughts from other tc-members ?15:39
dansmithyeah, I guess so15:39
jungleboyjMakes sense.15:40
fungidpawlik's new design is also a much safer approach than how the old gearman submission solution worked, so doesn't suffer the risks clarkb mentioned15:40
clarkbthe first step is to remove the gearman job submissions from the base zuul job. Then we can remove our configuration management and finally delete the servers15:40
spotznice15:40
rosmaitai don't have any objection given dpawlik's work ... is there any big functionality we will lose?15:40
clarkbBut I expect things can move fairly quickly once we have a decision15:40
clarkbrosmaita: I'm not sure elastic-recheck is hooked up to the new system (and it currently runs on status.o.o which fungi would like to also cleanup)15:41
clarkbbut elatic-recheck could be made to run against opensearch. Maybe hosting it on the server doing the processing?15:41
fungiyes, part of the goal was for us to not be running elastic-recheck in opendev15:41
fungiit's part of that whole set15:42
rosmaitaok15:42
clarkbright basically all the related tools are 1) not maintained beacuse they lack maintainers and 2) running on old systems with old config management that need to be removed15:42
gmannso e-r is moved on opensearch or need to?15:42
fungi(logstash, kibana, elasticsearch, and so on)15:42
gmanndpawlik ^^15:42
clarkb2 +1 implies deletion. We started by asking for help to address 1 to address it without deleting thigns but didn't get that so here we are :/15:42
dpawlikrosmaita: there can be only one situation, that we need to ask Zuul if they can fix that. I mean: the logscraper is taking latest 1k job builds that has been done, but the results of that jobs can be executed long time ago, so later it will be not processed. I can explain that "bug" later15:43
rosmaitaok15:43
gmannso in opensearch, we will have log search and e-r right?15:44
dpawlikso the new deployment is not using any of Opendev services 15:44
dpawlikI mean logstash/gearman etc15:44
clarkbgmann: I think you have log search today but no e-r15:44
clarkbat least I'm unaware of a new e-r deployment talking to the new opensearch indexes15:44
gmannyeah, I am thinking what is plan before we agree on e-r shutdown15:45
fungispecifically, the component which creates and publishes these graphs: http://status.openstack.org/reviews/15:45
clarkbfrom opendev's side the plan is we are shutting it down ebcause it isn't sustainable or maintained :)15:45
fungier, http://status.openstack.org/elastic-recheck/15:45
gmann#link http://status.openstack.org/elastic-recheck/15:46
clarkbthe source code and the queries for e-r aren't going away. If/when the openstack opensearch deployment wants to deploy e-r that should be doable15:47
clarkband I think tripleo has a branch with updates to work against more modern elasticsearch as well15:47
gmannbecause I think we need e-r when we talked about opensearch. or I am missing things here?15:47
gmannyeah, source code is there15:48
clarkbgmann: I think there are two things. First is that yes e-r would be great to have. But second we literally cannot run this service any longer it must be shut down. That means someone somewhere somehow will need to address e-r15:48
gmannclarkb: so with the ELK shutdown, this also goes away http://status.openstack.org/elastic-recheck/15:48
fungiit looks like the opendev/elastic-recheck repository where people were curating the logstash queries hasn't been touched in a year either15:48
fungiso i don't think it's providing much value at this point15:48
clarkbwe have tried for quite some time to convince that someone somehow somewhere to help us and keep runnign the service but that did not happen. Instead a decision was made to address this without opendev :/15:48
fungiit's certainly not classifying any bugs which are new in the past two cycles15:49
clarkband yes e-r is in the group of tools that are under maintained15:49
clarkbtripleo decided that instead of maintaining it upstream they would essentialyl fork it on a branch dedicated to them15:49
clarkbits not ideal, but there isn't much that fungi and I can do about it considering the other demands on us15:49
dansmithfork what, e-r?15:49
gmannI agree on opendev perspective, just waiting for dpawlik if he has plan to integrate it in new system ?15:49
fungidansmith: elastic-recheck15:49
dansmithreally..15:49
clarkbdansmith: yes tripleo soft forked e-r15:49
fungi#link https://opendev.org/opendev/elastic-recheck/commits/branch/rdo15:50
gmannI thought they said they will maintain opendev/elastic-recheck repo only15:50
dpawlikthey have some improvements that will work with Opensearch15:50
fungimaybe the plan is to merge the rdo branch into master once we take down status.o.o15:50
gmannthey host it in their server15:51
fungiright, i'm saying the master branch is unmaintained and we're shutting down the instance we have of it, so what's in the rdo branch could realistically become the new master branch anyway15:51
spotzWe can ping and see15:52
gmannok, so back to shutdown of ELK15:52
dpawlikI need to check where it is hosted15:52
gmanndpawlik: I think in rdo servers. 15:52
dpawlikyes, but I need to check utilization of that server15:52
gmanndpawlik: but question is if openstack want to move e-r in new system opensearch, do we have server for that?15:52
gmannok15:52
gmannyeah, because whatever AWS servers/creds we have we need to see their consumption 15:53
dpawlikwe can replace the logstash with elasticsearch recheck and should be ok...15:54
gmannso if we shutdown the ELK now we loose the e-r service and dpawlik will check possibility to have to on new system. then is it ok for ELK shutdown?15:54
fungithe ci-log-processor system is running on a vm in one of opendev's donor clouds, we're just not managing it (dpawlik et all have root access)15:54
gmannok15:54
fungiso that's already not using aws credits15:54
dpawlikfungi: exactly. I added my team and attach the logscraper host into our monitoring system15:54
fungiso if they want to also integrate a new elastic-recheck replacement, they could do it on that server or another similar one15:55
dpawlikgmann: seems to be a good plan15:55
fungii expect elastic-recheck uses almost no system resources to run anyway15:55
fungiit's just hosting queries which link out to kibana anyway15:56
clarkbwhen it generates the graphs it can use some memory but ya most of the hard work is in elasticsearch15:56
gmannok, so as agreement 1. we are ok to shutdown ELK old server even e-r is not ready on new system 2. dpawlik to investigate on e-r on new system15:56
clarkbfungi: no those queries are not live. They are statically done every half hour15:56
fungiahh, okay15:56
gmanntc-members any objection on that ^^ plan15:56
dansmithno objection15:56
jungleboyjI think that sounds fine to me.15:57
rosmaitasounds good to me15:57
arne_wiebalcksounds good15:57
dpawlikok15:57
clarkbthanks! We'll probably remove the gearman job submissions today then and then work on systems cleanup tomorrow15:57
gmannok, clarkb we have agreement on ELK old server shutdown, please proceed on that15:57
gmannfungi: and on http://status.openstack.org/reviews/ and status.o.o let's continue the discussion in next meeting ?15:58
fungisure15:58
gmannfungi: but meanwhile if you can ask this in ML if anyone need it will be good input in that meeitng15:58
gmannfungi: thanks15:58
fungithat's my plan, just wanted to double-check if anyone here knew of teams still relying on it15:58
gmannsure.15:59
fungisince it's really the last thing on that site now15:59
gmannskipping open reivew but tc-member can check those and review15:59
gmann#topic Open Discussion15:59
gmanntick-tock release notes feedback (rosmaita)15:59
gmannrosmaita: not sure how much time we need on this but we can extend meeting lillte bit15:59
*** pojadhav|ruck is now known as pojadhav|out15:59
rosmaitayeah, the cinder team had some concerns about what was on the etherpad15:59
rosmaitai will be quick15:59
gmannsure,16:00
rosmaita1. don't like the "synopsis" idea ("Last week on OpenStack, ..."), because  it's really hard to know what is important and what's not (it's all  important -- that's why we wrote the release note in the first place!)16:00
rosmaita2. how about the beginning of the release notes having a section, something  like, "If you are upgrading from Tick, read these first (link to n-1  release notes)"16:00
rosmaita3. finally, important for all projects to do the same thing, or it will be a pain point for operators16:00
rosmaitathat's all16:00
jungleboyj:-)16:00
spotz:)16:03
gmannrosmaita: which one in https://etherpad.opendev.org/p/tc-zed-ptg ?16:03
gmannmay be we start tagging 'for-tick-too' while we do any release notes so we do not need to figure out at the end?16:04
dansmithI really don't want to make reading the last release notes required, and if we did, no point in bringing things forward16:04
rosmaitawell, the question is, aren't they all important?16:04
dansmithbut "recommending" that they read them is certainly good16:04
rosmaitawell, reading the release notes isnt' required16:04
rosmaita:D16:04
dansmithno, all our release notes are not important :)16:04
rosmaitamaybe in nova16:05
rosmaita:P16:05
dansmithand not even all notes we think are important are important to everyone,16:05
rosmaitawell, the problem is how do you tell the difference?16:05
jungleboyj:-)  I think recommending is good.  If we recommend and they don't do it ... well, we can't force them.16:05
rosmaitalet them read and decide for themselves16:05
dansmithso I would think any upgrade-related ones are critical to highlight, security maybe, but not every single bug16:05
gmann+1, upgraded and derpecation one if any16:05
dansmiththe important thing is that we don't *rely* on them reading them along with the current ones16:06
dansmithif they read none, then they're out of luck,16:06
gmannor we start not putting non-important (not recommended to read) in releasenotes :)16:06
rosmaitawhy are they going to read the current ones, though?16:06
dansmithbut we can't just say "we expect you've read about all the upgrade and deprecation things in the previous notes of the release you didn't run"16:06
rosmaitai mean, why are we writing them at all?16:06
dansmithespecially because sometimes those will be different in terms of what they need to do by the time the tick rolls around16:06
gmannyeah, with tick-tick. they need to know what things happened in between of these two release16:07
* arne_wiebalck needs to leave to pick up kids o/16:07
dansmithwe're over time and I have to get to another thing16:07
gmannadding previous tock release links for read does not harm16:07
dansmithmaybe we need a voice call to argue about this?16:07
gmannyeah, me too16:07
gmann+1, let's setup a call to figure out all these things16:08
rosmaitait's not a big deal until AA16:08
rosmaitaso we don't need to settle this now16:08
gmannbut we need to agree on things well before16:08
gmannlet me check about call later16:08
dansmithyeah, but still plenty of time16:08
dansmithreally CC is when we need to care16:08
gmannand end today meeting 16:08
jungleboyj++16:08
gmann#action gmann to schedule call on tick-tock release note question16:09
gmannthanks all for joining16:09
gmann#endmeeting16:09
opendevmeetMeeting ended Thu Apr 21 16:09:09 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:09
opendevmeetMinutes:        https://meetings.opendev.org/meetings/tc/2022/tc.2022-04-21-15.00.html16:09
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/tc/2022/tc.2022-04-21-15.00.txt16:09
opendevmeetLog:            https://meetings.opendev.org/meetings/tc/2022/tc.2022-04-21-15.00.log.html16:09
rosmaitabye!16:09
jungleboyjThank you!16:09
spotzBye arne_wiebalck16:09
spotzThanks all16:09
dpawlikthanks, bye16:10
opendevreviewGhanshyam proposed openstack/governance master: Resolution to drop the lower constraints maintenance  https://review.opendev.org/c/openstack/governance/+/83800418:56
gmannarne_wiebalck: replied on your comments in https://review.opendev.org/c/openstack/governance/+/83789119:00
opendevreviewGhanshyam proposed openstack/governance master: Fix keystone PTL model documentation  https://review.opendev.org/c/openstack/governance/+/83894019:03
gmannslaweq: rosmaita arne_wiebalck fixed you comment in this, please check https://review.opendev.org/c/openstack/governance/+/83800419:04
arne_wiebalckgmann: thanks, +1'ed (and apologies for the false positives!)19:12
gmannarne_wiebalck: thanks, np. you captured the keystone documentation issue there. 19:13
gmannarne_wiebalck: rosmaita  we need rollcall vote also in this https://review.opendev.org/c/openstack/governance/+/83789119:13
dansmithgmann: sorry I thought I had +1d that19:16
gmanndansmith: arne_wiebalck thanks19:17
gmanndansmith: arne_wiebalck as you are here, can you review this also. long pending one, I changed it to 'project stats tool' from 'project health tool' https://review.opendev.org/c/openstack/governance/+/81003719:18
gmanntc-members: in case you missed this email. "New CFN(Computing Force Network) SIG Proposal" http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028142.html19:26
gmannany opinion/feedback on ML will be helpful  to them19:26
gmannemail author reached out to me about feedback and after that discuss in TC meeting19:27
arne_wiebalckgmann: having a look19:31
gmannarne_wiebalck: thanks 19:32
arne_wiebalckgmann: for the project stats checker we may want to handle special characters in names (owners, l. 286): the tool fails for instance on ironic with a UnicodeEncodeError (on the latin-1 character set since we have two contributors from nordic countries with special characters in their names) ... or will the character set explicitly set when the tool is run automatically?20:06
fungimay want to force "utf-8 mode" or make sure to explicitly override locales in the script20:16
fungiwe take similar precautions in the election tooling because it needs to query all active change owners20:17
arne_wiebalckfungi: I suggested enforcing utf-8 on the change as well, not nice, but at least the tool does not fail.20:17
arne_wiebalckgmann: I added the failure I see to the change as well.20:18
fungiarne_wiebalck: out of curiosity, does exporting PYTHONUTF8=1 in the calling environment avoid that error?20:18
arne_wiebalckfungi: let me try ...20:18
fungithat's the new off-by-default toggle which is expected to eventually be on by default in future interpreters20:19
fungimay not be picked up by older interpreter versions though, depending on where you're testing20:19
fungirelated, exporting PYTHONWARNDFAULTENCODING=1 can be useful to find places where code is relying on implicit encoding defaults20:20
arne_wiebalckfungi: tried with py2.7 and py3 now, as well as '-X utf8': same error 20:23
fungiPYTHONUTF8 is available as far back as python 3.7, PYTHONWARNDFAULTENCODING is new in python 3.1020:23
arne_wiebalckheh, this is 3.6 ofc20:24
arne_wiebalcklet me move to another machine ...20:24
fungihttps://docs.python.org/3/using/windows.html#utf-8-mode (ignore that the section is about windows)20:24
fungiahh yeah https://docs.python.org/3/library/os.html#utf8-mode is the more general documentation for it20:25
arne_wiebalckfungi: 3.8 with 'export PYTHONUTF8=1' works20:30
arne_wiebalckfungi: breaks without the export20:31
arne_wiebalckfungi: and 'works' means it is not only not failing but correctly printing20:31
fungiarne_wiebalck: thanks for testing! i guess that means in future python interpreters (maybe 3.12? probably more like 3.13) this won't require changes. in the meantime we likely want to force utf-8 on stdout or just warn users to use a utf-8 capable locale?22:49

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