| Clark[m] | fungi: looks like the storyboard fixup change passes | 00:10 |
|---|---|---|
| fungi | thanks! | 00:15 |
| opendevreview | Ian Wienand proposed opendev/lodgeit master: new paste: add maxlength to input box https://review.opendev.org/c/opendev/lodgeit/+/838418 | 00:29 |
| ianw | i guess our daily builds have been isolated from git updates until we merge a nodepool change that rebuilds the container? looks like the last was apr 13 | 00:40 |
| ianw | even then, i wonder what layer git comes in at; is it already in the published images? maybe we have to rebuild our python-builder environment too? | 00:40 |
| Clark[m] | Last I checked debian hadn't updated | 01:01 |
| Clark[m] | Which means we're isolated | 01:02 |
| ianw | oh, well that could explain it too :) i'd assumed they'd pulled the same update as ubuntu | 01:05 |
| Clark[m] | It's possible they have since I checked on Friday or that they do tomorrow | 01:09 |
| Clark[m] | But I decided to punt on that since debian packages weren't updated | 01:10 |
| Clark[m] | https://metadata.ftp-master.debian.org/changelogs//main/g/git/git_2.30.2-1_changelog | 01:10 |
| Clark[m] | I think that is the right changeling for bullseye | 01:10 |
| Clark[m] | Heh auto correct changelog to changeling | 01:11 |
| ianw | yeah https://security-tracker.debian.org/tracker/CVE-2022-24765 | 01:11 |
| opendevreview | Ian Wienand proposed opendev/lodgeit master: new paste: check input length https://review.opendev.org/c/opendev/lodgeit/+/838421 | 01:30 |
| opendevreview | Ian Wienand proposed opendev/lodgeit master: new paste: check input length https://review.opendev.org/c/opendev/lodgeit/+/838421 | 02:52 |
| *** pojadhav is now known as pojadhav|ruck | 03:59 | |
| *** chandankumar is now known as chkumar|ruck | 04:47 | |
| *** chkumar|ruck is now known as chandankumar | 04:51 | |
| *** ysandeep|out is now known as ysandeep | 05:49 | |
| dpawlik | gmann: hey, catch me when you are online | 06:14 |
| *** jpena|off is now known as jpena | 07:01 | |
| *** ysandeep is now known as ysandeep|lunch | 07:29 | |
| *** pojadhav|ruck is now known as pojadhav|lunch | 07:35 | |
| *** arxcruz|out is now known as arxcruz | 08:11 | |
| opendevreview | Alfredo Moralejo proposed zuul/zuul-jobs master: Update gpg key file for extras-common in CS9 https://review.opendev.org/c/zuul/zuul-jobs/+/838450 | 08:19 |
| *** pojadhav|lunch is now known as pojadhav|ruck | 08:30 | |
| *** ysandeep|lunch is now known as ysandeep | 09:50 | |
| *** icey_ is now known as icey | 09:53 | |
| *** dviroel|out is now known as dviroel | 11:31 | |
| *** ysandeep is now known as ysandeep|afk | 11:41 | |
| hrw | morning | 12:29 |
| hrw | can someone check https://mirror.regionone.osuosl.opendev.org/wheel/centos-8-aarch64/grpcio/ dir? 2 files, 0 bytes each == jobs fail | 12:30 |
| hrw | https://mirror.iad3.inmotion.opendev.org/wheel/centos-8-aarch64/grpcio/ has 0 bytes too | 12:33 |
| hrw | can those files be removed? | 12:33 |
| fungi | agreed, those files are 0 bytes in /afs/openstack.org/mirror/wheel/centos-8-aarch64/g/grpcio/ | 12:47 |
| fungi | with creation dates of august 13/14 of 2021 | 12:47 |
| fungi | need to dig out the docs on deleting files from afs | 12:48 |
| fungi | https://docs.opendev.org/opendev/system-config/latest/afs.html#deleting-files | 12:48 |
| hrw | and we did not built that old branch a while as newer versions are wheeled by upstream | 12:51 |
| *** ysandeep|afk is now known as ysandeep | 12:53 | |
| fungi | #status log Manually deleted /afs/openstack.org/mirror/wheel/centos-8-aarch64/g/grpcio as it contained only corrupt (zero-byte) packages | 12:55 |
| opendevstatus | fungi: finished logging | 12:55 |
| hrw | thanks | 12:55 |
| fungi | thanks for pointing it out! | 12:55 |
| hrw | I did not looked for other 0 ones - those were where builds failed | 12:56 |
| *** rcastillo|ruck is now known as rcastillo|rover | 13:47 | |
| *** dviroel is now known as dviroel|afk | 14:24 | |
| *** dviroel|afk is now known as dviroel|mtg | 14:39 | |
| *** ysandeep- is now known as ysandeep|out | 14:45 | |
| *** pojadhav|ruck is now known as pojadhav|out | 14:49 | |
| clarkb | hrw: as a side note the same filesystem servers all of the mirrors so you should see consistent info across them. If you don't that would be a bug | 14:50 |
| *** dviroel|mtg is now known as dviroel | 14:54 | |
| clarkb | dpawlik: gmann: re https://review.opendev.org/c/opendev/system-config/+/838324 and https://review.opendev.org/c/opendev/base-jobs/+/837619 can people use the new system today? If so can we announce it and start to shut down the old system? I really don't want to keep it around much longer since it is impacting jobs now | 15:02 |
| clarkb | and we're past the agreed upon transition point (yoga release) | 15:02 |
| gmann | dpawlik: clarkb yeah, so i am waiting for the redirect question on this https://review.opendev.org/c/openstack/governance-sigs/+/835838 | 15:03 |
| clarkb | kopecmartin: when your meeting is done did you want to review https://review.opendev.org/c/opendev/system-config/+/838344 which updates our refstack deployment images to python3.9? | 15:03 |
| gmann | I used the new dashbaord and I think it is good to start using. dpawlik can give more updates if anything we need to waity | 15:03 |
| gmann | wait | 15:03 |
| clarkb | gmann: dpawlik note we wouldn't host an http redirect. We can update dns to point logstash.openstack.org to a different name/ip though | 15:04 |
| kopecmartin | clarkb: sure | 15:04 |
| clarkb | a redirect implies an http server at the old name doing the redirect and we're saying we want to delete the host currently at that name | 15:04 |
| clarkb | gmann: dpawlik: but also updating the dns name can happen after the fact. I don't think that hsould hold anything up? | 15:06 |
| fungi | what i considered was cnaming the old name to the new dashboard, or to static.opendev.org and adding it as a redirect alias to either the new dashboard or the sig page with the conneciton info, but we could also just announce that the old name is no longer in use and tell people where to start going | 15:06 |
| *** dviroel is now known as dviroel|lunch | 15:06 | |
| clarkb | right the problem with just a CNAME or dns update is likely going to be ssl | 15:07 |
| clarkb | but I don't think opendev should host a redirect either | 15:07 |
| dpawlik | IMHO the new Elasticsearch is ready to use | 15:07 |
| clarkb | but also I really don't think that should hold up any change | 15:07 |
| clarkb | since you ahve to know how to login to the new thing anyway using a new url isn't that big of a deal | 15:07 |
| clarkb | if you do get redirected you'll just get stuck on logging in | 15:07 |
| clarkb | so.... | 15:07 |
| fungi | the old dashboard is broken since weeks at least, so i have doubts there are that many people who will be surprised if it just disappears | 15:07 |
| clarkb | dpawlik: that is greaet to hear :) | 15:08 |
| clarkb | dpawlik: maybe you can +1 those two changes I linked indicating you are ready for users to use the opensearch system now? that would be good record keeping | 15:08 |
| gmann | yeah, let me put the plan in TC meeting on thursday and we can discuss about communicating the new dashboard to community. | 15:08 |
| fungi | but communication planning shouldn't delay taking down an already broken service, right? | 15:08 |
| gmann | I also think we are good to go now. agree on redirect is not something blocker | 15:08 |
| fungi | nobody is able to use the old service now, and we have no plans to spend time fixing it | 15:09 |
| dpawlik | Last week I was trying to include the the openstack user into the private tenant, so user no matter where it connect to the opensearch dashboard will not force to change the tenant to global | 15:09 |
| dpawlik | clarkb: ok | 15:09 |
| clarkb | dpawlik: https://review.opendev.org/c/opendev/system-config/+/838324 and https://review.opendev.org/c/opendev/base-jobs/+/837619 are the two changes I referred to | 15:10 |
| gmann | i know things are broken now but waiting for communicating the new things does not harm which is just 2 days after or so | 15:11 |
| gmann | dpawlik: fungi so this is good to go right? I will +W https://review.opendev.org/c/openstack/governance-sigs/+/835838 | 15:12 |
| dpawlik | gmann: from my side yes | 15:13 |
| gmann | ok, approving that | 15:14 |
| gmann | dpawlik: clarkb fungi added it in TC thursday meeting agenda https://wiki.openstack.org/wiki/Meetings/TechnicalCommittee#Agenda_Suggestions | 15:14 |
| fungi | gmann: yes, that's what my code review +1 was meant to indicate. as tact sig chair, i'm in favor of approving it | 15:16 |
| dpawlik | cool. Glad that the topica is moving forward | 15:16 |
| fungi | mmm, tapioca | 15:16 |
| gmann | done | 15:17 |
| clarkb | apparently git-review's new get_git_version doesn't work on OSX because apple patches git to print different version info | 15:36 |
| clarkb | I'll work on a fix between meetings | 15:36 |
| fungi | oh fun :/ | 15:37 |
| clarkb | https://storyboard.openstack.org/#!/story/2010002 | 15:37 |
| fungi | i'm close to having the release doc builds i think, will push up some changes monentarily | 15:39 |
| opendevreview | Clark Boylan proposed opendev/git-review master: Fix get_git_version on OS X https://review.opendev.org/c/opendev/git-review/+/838523 | 15:47 |
| clarkb | something like ^ maybe. I'm trying to avoid having to write a super complicated parser that ends up breaking because it is complicated | 15:47 |
| clarkb | I suspect this is a good balance of simple and close enough. SHould work as long as we don't start prefixing the version with extra platform specific info too | 15:48 |
| *** marios is now known as marios|out | 15:48 | |
| clarkb | and now breakfast before the next meeting | 15:49 |
| hrw | clarkb: wanted to check in case one of mirrors went out of space or weird | 15:49 |
| opendevreview | Jeremy Stanley proposed opendev/base-jobs master: Add a opendev-publish-unversioned-tox-docs job https://review.opendev.org/c/opendev/base-jobs/+/838525 | 15:53 |
| opendevreview | Jeremy Stanley proposed opendev/base-jobs master: Rename opendev-zuul-docs to opendev-versioned-docs https://review.opendev.org/c/opendev/base-jobs/+/838526 | 15:53 |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Run opendev-publish-tox-docs on release https://review.opendev.org/c/opendev/git-review/+/838341 | 15:54 |
| opendevreview | Jeremy Stanley proposed opendev/bindep master: Publish documentation at every release https://review.opendev.org/c/opendev/bindep/+/838340 | 15:55 |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Publish documentation at every release https://review.opendev.org/c/opendev/git-review/+/838341 | 15:56 |
| mnaser | clarkb: you can kill those images, i don't think we've used them for a little bit | 16:06 |
| *** jpena is now known as jpena|off | 16:08 | |
| opendevreview | Jeremy Stanley proposed opendev/base-jobs master: Add a opendev-publish-unversioned-tox-docs job https://review.opendev.org/c/opendev/base-jobs/+/838525 | 16:13 |
| opendevreview | Jeremy Stanley proposed opendev/base-jobs master: Rename opendev-zuul-docs to opendev-versioned-docs https://review.opendev.org/c/opendev/base-jobs/+/838526 | 16:13 |
| *** dviroel|lunch is now known as dviroel | 16:14 | |
| clarkb | mnaser: thank you for confirming. This will help us trim down our image list quite a bit before adding python3.10 | 16:15 |
| clarkb | fungi: shoudl we approve https://review.opendev.org/c/opendev/git-review/+/838523 and do a quick release? | 16:17 |
| clarkb | kopecmartin: re https://review.opendev.org/c/opendev/system-config/+/838344 I checked and refstack runs tests on python3.9 now so I don't really expect any issues. But definitely point out if I've missed something | 16:18 |
| fungi | clarkb: yeah, though i'd like to try to get 838341 in too before we tag | 16:18 |
| fungi | since that will allow us to quickly test if it's a viable approach | 16:19 |
| clarkb | good point | 16:21 |
| clarkb | let me pull those up now | 16:21 |
| clarkb | fungi: looks like we set the branch and tag values to always be latest? that makes sense to me | 16:24 |
| clarkb | I've +2'd the two changes necessary for git-review to get that done. I guess we can alnd those then check it works and if so work to land the others | 16:24 |
| fungi | i left a comment with a caveat on 838525, i don't think it's a blocker but food for thought | 16:25 |
| clarkb | ya I think we can live with that | 16:25 |
| clarkb | it is a rare case, but the case of witing on doc updates is not rare | 16:25 |
| fungi | and i think it's solveable, but will need some edits to the ansible | 16:26 |
| fungi | also that job is probably not applicable for projects which tag releases on multiple branches, but again that's rare for opendev's own tool/utility projects | 16:27 |
| clarkb | I'm going to delete the held nodes that I set up to debug git problems last week | 16:27 |
| clarkb | looks like the gear tests failed on the dockerfile update too | 16:33 |
| clarkb | it was a job timeout so I'm rechecking to see if it is consistent but will try and dig into that a bit like I did with storyboard | 16:33 |
| clarkb | https://review.opendev.org/c/opendev/storyboard/+/838409 is the storyboard fix. Nothing too crazy in there and I tried to explain each chagne inthe commit message | 16:34 |
| opendevreview | Merged opendev/git-review master: Fix get_git_version on OS X https://review.opendev.org/c/opendev/git-review/+/838523 | 16:36 |
| opendevreview | Merged opendev/base-jobs master: Add a opendev-publish-unversioned-tox-docs job https://review.opendev.org/c/opendev/base-jobs/+/838525 | 16:40 |
| fungi | clarkb: should we be considering retiring gear now that we don't use it? | 16:48 |
| clarkb | fungi: its not a bad idea, though we may want to do that independent of this effort since I think some people have been using it | 16:49 |
| clarkb | basically take some time to communicate it? Though maybe no one is using it and I'm mistaken | 16:49 |
| clarkb | it does look like gear testing doesn't work with python3.9 without timing out. The other versions take just a few minutes to run | 16:50 |
| clarkb | in theory it did work on python3.9 though as testing would've had to pass to add the job | 16:50 |
| fungi | well, also if people are still using it and need new versions/fixes in it, we should perhaps encourage them to take it over | 16:50 |
| clarkb | ++ | 16:51 |
| clarkb | I cannot reproduce this timeout issue locally on tumbleweed with python3.9 either | 16:52 |
| fungi | once 838341 merges i'll prepare to tag a git-review 2.3.1 | 16:52 |
| fungi | clarkb: oh, hrm, 838523 did not include a release note. i expect we want one? | 16:53 |
| clarkb | oh ya. I always forget these small tools use reno | 16:54 |
| fungi | i'm about to push one now | 16:54 |
| clarkb | ok I can reproduce the issue if I update setuptools in my tox venv | 16:54 |
| clarkb | the issue is running python setup.py testr is broken with current setuptools. It warns you that it is depreacted but then it just hangs entirely | 16:56 |
| clarkb | I'll push up a gear change to fix that since I've sort it out anyway. Then we can land that change and the docker image update and then start planning to retire or hand it over | 16:56 |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Add release note for OSX Git version parsing fix https://review.opendev.org/c/opendev/git-review/+/838558 | 17:00 |
| fungi | clarkb: ^ | 17:00 |
| clarkb | approved thanks | 17:01 |
| fungi | i guess after that merges, i'll tag 2.3.1 | 17:01 |
| fungi | i need to disappear shortly, but can probably do that once i get where i'm headed in ~ an hour | 17:01 |
| opendevreview | Merged opendev/git-review master: Publish documentation at every release https://review.opendev.org/c/opendev/git-review/+/838341 | 17:06 |
| opendevreview | Merged opendev/git-review master: Add release note for OSX Git version parsing fix https://review.opendev.org/c/opendev/git-review/+/838558 | 17:23 |
| opendevreview | Clark Boylan proposed opendev/gear master: Fix python3.9 testing https://review.opendev.org/c/opendev/gear/+/838562 | 17:28 |
| opendevreview | Clark Boylan proposed opendev/gear master: Update the docker image to python3.9 and buster https://review.opendev.org/c/opendev/gear/+/838402 | 17:28 |
| opendevreview | Clark Boylan proposed opendev/storyboard master: Update container image to bullseye and python3.9 https://review.opendev.org/c/opendev/storyboard/+/838403 | 17:31 |
| fungi | okay, i'm back on for a bit, tagging | 17:40 |
| fungi | commit e2382507ed080556c70b5166ed195f2edc05eae4 (HEAD -> master, tag: 2.3.1, origin/master, origin/HEAD, gerrit/master) | 17:43 |
| fungi | oh, no that's not what we want. the other changes haven't merged yet | 17:43 |
| fungi | nevermind, it is what we want | 17:44 |
| fungi | i forgot the last one was the release note change | 17:44 |
| fungi | clarkb: ^ does that look right to you? | 17:45 |
| fungi | git log --oneline --no-merges 2.3.0..2.3.1 | 17:46 |
| fungi | 838558: Add release note for OSX Git version parsing fix | 17:46 |
| fungi | 838341: Publish documentation at every release | 17:46 |
| fungi | 838523: Fix get_git_version on OS X | 17:46 |
| fungi | that's what it covers | 17:46 |
| opendevreview | Clark Boylan proposed opendev/gear master: Fix python3.9 testing https://review.opendev.org/c/opendev/gear/+/838562 | 17:47 |
| clarkb | fungi: looking | 17:48 |
| clarkb | e2382507ed080556c70b5166ed195f2edc05eae4 as 2.3.1 lgtm | 17:48 |
| fungi | okay, pushing. fingers crossed we get a working doc update from it | 17:49 |
| clarkb | that new docs job is running now | 17:50 |
| fungi | yep | 17:50 |
| fungi | looks like it probably worked, need to wait for the timed vos release now | 17:55 |
| fungi | et voila! https://docs.opendev.org/opendev/git-review/latest/releasenotes.html | 17:55 |
| fungi | i suppose that means we can go ahead and approve 838340 now too, which will refresh the bindep docs when it hits promote | 17:56 |
| fungi | and then we'll have updated release notes published for both bindep and git-review and can send out release announcements | 17:56 |
| clarkb | I approved it and rechecked it | 17:57 |
| fungi | thanks! shouldn't need a recheck though | 17:57 |
| clarkb | I wasn't sure if zuul was smart enough to run it through check to get rid of the -1 | 17:57 |
| fungi | opendev tenant | 17:57 |
| clarkb | ah right | 17:57 |
| clarkb | -1 doest' matter | 17:58 |
| fungi | zactly | 17:58 |
| fungi | 99 problems, but a -1 ain't one | 17:58 |
| fungi | something like that | 17:58 |
| clarkb | https://review.opendev.org/q/topic:opendev-image-cleanup+status:open everything in that list has +1s now. There are fixups for storyboard and gear testing as well as removal of unneeded buster container iamges and our python3.7 bullseye image | 18:07 |
| clarkb | Once that has all landed I'll start to look at adding python3.10 images next | 18:07 |
| opendevreview | Merged opendev/bindep master: Publish documentation at every release https://review.opendev.org/c/opendev/bindep/+/838340 | 18:08 |
| clarkb | fungi: if you have time to review https://review.opendev.org/c/opendev/system-config/+/838344 that would be good (refstack on the python 3.9 image). I can approve it after lunch today and keep an eye on it | 18:19 |
| fungi | sure thing, i'll take a look once i'm done sending release announcements | 18:21 |
| fungi | #status log Released bindep 2.11.0 https://lists.opendev.org/pipermail/service-announce/2022-April/000035.html | 18:35 |
| opendevstatus | fungi: finished logging | 18:35 |
| fungi | #status log Released git-review 2.3.0 and 2.3.1 https://lists.opendev.org/pipermail/service-announce/2022-April/000036.html | 18:35 |
| opendevstatus | fungi: finished logging | 18:35 |
| fungi | clarkb: they all lgtm, approved the ones which could be | 18:46 |
| opendevreview | Merged opendev/gear master: Fix python3.9 testing https://review.opendev.org/c/opendev/gear/+/838562 | 18:48 |
| clarkb | thanks! | 18:48 |
| clarkb | fungi: what about https://review.opendev.org/c/opendev/system-config/+/838344 ? (I didn't put it on the same topic as it isn't really in the path of removing the buster images) | 18:48 |
| opendevreview | Merged opendev/gear master: Update the docker image to python3.9 and buster https://review.opendev.org/c/opendev/gear/+/838402 | 18:51 |
| fungi | ahh, missed that one, thanks | 18:52 |
| fungi | yep, that one also lgtm | 18:53 |
| opendevreview | Merged opendev/system-config master: Fixup some image dependencies in our zuul configs https://review.opendev.org/c/opendev/system-config/+/838345 | 18:53 |
| opendevreview | Merged opendev/system-config master: Switch Refstack image over to python3.9 https://review.opendev.org/c/opendev/system-config/+/838344 | 19:27 |
| clarkb | refstack's conatiner updated 44 minutes ago and https://refstack.openstack.org/#/ is still serving content | 20:26 |
| *** dviroel is now known as dviroel|out | 20:41 | |
| opendevreview | Merged opendev/storyboard master: Omnibus fixups for storyboard testing and CI https://review.opendev.org/c/opendev/storyboard/+/838409 | 21:34 |
| opendevreview | Merged opendev/storyboard master: Update container image to bullseye and python3.9 https://review.opendev.org/c/opendev/storyboard/+/838403 | 21:34 |
| ianw | if someone has a second to double-check my maths in https://review.opendev.org/q/topic:textarea-length that should stop 502 errors with long pastes on lodgeit | 21:57 |
| ianw | it's like working on an old car; somehow despite all the odds the thing keeps on trucking | 21:59 |
| clarkb | ianw: one thing I'm confused about is if it is 64k bytes or characters | 22:00 |
| clarkb | since they are different when it comes to unicode | 22:00 |
| clarkb | on the html side maxlength specifies characters not bytes | 22:00 |
| clarkb | then on the python side we seem to be checking the length of a unicode string which is again characters not bytes | 22:01 |
| clarkb | but on the mysql side I think it may be bytes? | 22:02 |
| clarkb | ya that seems to be the case. That means this can still fail, but possibly less often? | 22:02 |
| clarkb | I think the first change is fine we can limit the size to a reaosnable upper bound. But the second one that checks the size should probably check the byte size not unicode length | 22:03 |
| ianw | i guess just .encode(utf-8) | 22:06 |
| clarkb | ya I think so | 22:06 |
| clarkb | and then len() of that value will by byte length | 22:06 |
| opendevreview | Ian Wienand proposed opendev/lodgeit master: new paste: check input length https://review.opendev.org/c/opendev/lodgeit/+/838421 | 22:08 |
| ianw | thanks | 22:08 |
| ianw | clarkb: another if/when you have time is https://review.opendev.org/c/openstack/diskimage-builder/+/837248 which is intended to be a no-op refactor of installing rpm-ish release packages. over time that seems to have grown a bunch of add-on clauses that i think can be simplified | 22:09 |
| ianw | mostly looking for eyes on that to confirm it really is no-op :) | 22:09 |
| clarkb | can do. I'm going to do a short garage bike ride then will take a look | 22:16 |
| opendevreview | Merged openstack/project-config master: Remove tempest-lib from infra https://review.opendev.org/c/openstack/project-config/+/836703 | 22:31 |
| clarkb | ianw: I agree it is a noop other than the centos stream ncurses stuff. I don't understand why they are brought in now after your change if they weren't prior to the explicit insallations | 23:04 |
| clarkb | left a coupe of other minor things inline | 23:06 |
| ianw | i'm thinking because ncurses-libs wasn't pre-installed, so somehow ncurses-base gets chosen too because ... waves hands ... | 23:07 |
| ianw | great point, i was thinking the centos path above matched it, but you're right, 7 falls through there | 23:08 |
| opendevreview | Ian Wienand proposed openstack/diskimage-builder master: yum-minimal: clean up release package installs https://review.opendev.org/c/openstack/diskimage-builder/+/837248 | 23:11 |
| clarkb | I'm going to approve https://review.opendev.org/c/opendev/system-config/+/838347 first thing tomorrow unless someone else wants to watch for image problems nowish | 23:23 |
| clarkb | I'm happy to do it and don't expect any problems now that gear and storyboard are happy | 23:23 |
| clarkb | I should approve the child too and remove the python3.7 bullseye image too. I'll also work on getting python3.10 bullseye images going | 23:23 |
| opendevreview | Ian Wienand proposed opendev/statusbot master: Prefix twitter logs with emoji https://review.opendev.org/c/opendev/statusbot/+/838608 | 23:29 |
| ianw | i'm happy to watch this | 23:30 |
| opendevreview | Ian Wienand proposed opendev/statusbot master: Prefix twitter logs with emoji https://review.opendev.org/c/opendev/statusbot/+/838608 | 23:37 |
| ianw | sometimes that alert icon looks like a yellow alert icon, and other times it looks like a black-and-white outline | 23:48 |
| Clark[m] | It rendered properly for me in the commit message and the python file diff on gerrit | 23:54 |
| fungi | i could see it in gertty | 23:54 |
| fungi | my terminal font lacks the notice (pushpin) and ok (checkmark) glyphs though | 23:55 |
| ianw | ahhh, til | 23:56 |
| ianw | '\u26A0\ufe0f' is different to '\u26A0\ufe0e' | 23:56 |
| ianw | the last bit is a "selector" that says if it is a text or color version | 23:57 |
| ianw | if you leave that out, it may vary by platform and context which version is chosen | 23:57 |
| fungi | neat. i knew about compositors for things like diacritics, but didn't realize there were also some for color selection | 23:58 |
| opendevreview | Merged opendev/statusbot master: Prefix twitter logs with emoji https://review.opendev.org/c/opendev/statusbot/+/838608 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!