| openstackgerrit | Martin Kopec proposed opendev/system-config master: Deploy refstack with ansible docker https://review.opendev.org/c/opendev/system-config/+/705258 | 00:00 |
|---|---|---|
| clarkb | fungi: ok review-test:~clarkb/gerrit-consistency-notes/preferred-email-classifications updated with notes on those other preferred email issues. | 00:02 |
| clarkb | also high level notes here : https://etherpad.opendev.org/p/gerrit-user-consistency-2021 | 00:02 |
| clarkb | my brain is feeling overloaded after all that. I think I'll pick this back up in the morning and start looking at the email conflicts to see if I can classify those further | 00:21 |
| clarkb | anything I can review or look at before I call it a day? | 00:21 |
| *** tosky has quit IRC | 00:21 | |
| fungi | maybe you should just call it a day. i don't think there's anything that can't wait | 00:22 |
| ianw | ++ thanks for working through this! | 00:22 |
| clarkb | I'm beginning to think "data scientists" are super human and somehow have the ability to understand lots of characters on a screen | 00:23 |
| clarkb | at least for me I have a hard time writing any automation around this stuff iwthout first "seeing" it | 00:23 |
| clarkb | fungi: oh another thing I realized was that I bet (and I can test this) that we don't need to fix everything in one commit as long as we push a series of commits that results in a happy spot | 00:27 |
| clarkb | my hunch is that gerrit is only verifying the end state result (though it could be validating each commit) | 00:27 |
| clarkb | and even if it doesn't we can curate it that way then squash everything down if necessary | 00:28 |
| fungi | oh, yeah worth a shot | 00:30 |
| fungi | might at least make them easier to review | 00:30 |
| *** mlavalle has quit IRC | 00:57 | |
| *** DSpider has quit IRC | 01:05 | |
| *** priteau has quit IRC | 01:16 | |
| ianw | when i was at uni, there were stories about how people hadn't left enough digits for the DJI to go above 10,000 in their various code | 01:27 |
| ianw | i wonder how many people are going to be caught out when GME breaks through the 10,000 mark :) | 01:27 |
| *** hamalq has quit IRC | 02:10 | |
| kevinz | ianw: there? | 02:20 |
| *** zbr7 has joined #opendev | 02:20 | |
| ianw | kevinz: hey, yep | 02:20 |
| kevinz | Hi ianw: I find the gitweb link at opendev gerrit is 404 now. | 02:21 |
| kevinz | Do you know what happened to it? | 02:21 |
| *** portdirect has quit IRC | 02:21 | |
| *** mattmceuen has quit IRC | 02:21 | |
| kevinz | For example, https://review.opendev.org/c/openstack/nova/+/489951, click the "Links gitweb", then redirect to 404 | 02:22 |
| fungi | it hasn't worked since the upgrade, we need to work out how to properly link those to gitea ideally... i think mordred had a change up for that from some time back but it likely needs updating | 02:22 |
| ianw | yeah that is part of the gerrit upgrade, and i think we had a change somewhere to turn it into a gitea link | 02:22 |
| *** mattmceuen has joined #opendev | 02:22 | |
| *** zbr has quit IRC | 02:22 | |
| *** zbr7 is now known as zbr | 02:22 | |
| *** portdirect has joined #opendev | 02:22 | |
| ianw | jinx w/ fungi :) | 02:22 |
| kevinz | OK, thanks for that info | 02:23 |
| ianw | https://review.opendev.org/c/opendev/system-config/+/723526 is the change in questino | 02:25 |
| mordred | Yeah, that's the one | 02:52 |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: [dnm] fail to get a gerrit for testing pluging observer changes https://review.opendev.org/c/opendev/system-config/+/772823 | 02:53 |
| *** gouthamr has quit IRC | 04:18 | |
| *** ykarel has joined #opendev | 04:40 | |
| *** whoami-rajat__ has joined #opendev | 04:59 | |
| *** ykarel_ has joined #opendev | 05:49 | |
| *** ykarel has quit IRC | 05:52 | |
| *** ykarel_ is now known as ykarel | 05:53 | |
| *** marios has joined #opendev | 06:14 | |
| *** hemanth_n has joined #opendev | 06:42 | |
| *** zbr has quit IRC | 06:44 | |
| *** zbr has joined #opendev | 06:46 | |
| *** walshh__ has joined #opendev | 06:49 | |
| *** walshh_ has quit IRC | 06:49 | |
| *** walshh__ is now known as walshh_ | 06:49 | |
| *** redrobot1 has joined #opendev | 07:00 | |
| *** redrobot has quit IRC | 07:04 | |
| *** redrobot1 is now known as redrobot | 07:04 | |
| *** eolivare has joined #opendev | 07:43 | |
| *** akahat|rover is now known as akahat|lunch | 07:51 | |
| *** jpena|off is now known as jpena | 07:55 | |
| *** slaweq has joined #opendev | 07:58 | |
| *** ralonsoh has joined #opendev | 08:01 | |
| *** jpena is now known as jpena|off | 08:01 | |
| *** jpena|off is now known as jpena | 08:01 | |
| *** andrewbonney has joined #opendev | 08:14 | |
| *** rpittau|afk is now known as rpittau | 08:27 | |
| *** sboyron has joined #opendev | 08:40 | |
| *** tosky has joined #opendev | 08:45 | |
| *** hashar has joined #opendev | 08:48 | |
| *** ykarel is now known as ykarel|lunch | 08:51 | |
| *** tkajinam has quit IRC | 08:58 | |
| *** klonn has joined #opendev | 09:22 | |
| *** ykarel|lunch is now known as ykarel | 10:03 | |
| *** osmanlicilegi has quit IRC | 10:27 | |
| openstackgerrit | Martin Kopec proposed opendev/system-config master: Deploy refstack with ansible docker https://review.opendev.org/c/opendev/system-config/+/705258 | 10:39 |
| *** akahat|lunch is now known as akahat|rover | 10:44 | |
| *** osmanlicilegi has joined #opendev | 10:52 | |
| *** dtantsur|afk is now known as dtantsur | 10:55 | |
| *** osmanlicilegi has quit IRC | 11:01 | |
| *** calcmandan has quit IRC | 11:01 | |
| *** calcmandan has joined #opendev | 11:01 | |
| *** osmanlicilegi has joined #opendev | 11:06 | |
| *** ttx has quit IRC | 11:22 | |
| *** ttx has joined #opendev | 11:23 | |
| *** osmanlicilegi has quit IRC | 11:58 | |
| *** klonn has quit IRC | 12:11 | |
| *** dviroel_ is now known as dviroel | 12:16 | |
| *** ysandeep is now known as ysandeep|afk | 12:25 | |
| *** DSpider has joined #opendev | 12:26 | |
| *** hemanth_n has quit IRC | 12:26 | |
| *** jpena is now known as jpena|lunch | 12:31 | |
| *** priteau has joined #opendev | 12:41 | |
| *** klonn has joined #opendev | 12:43 | |
| *** osmanlicilegi has joined #opendev | 12:56 | |
| *** ysandeep|afk is now known as ysandeep | 13:21 | |
| *** DSpider has quit IRC | 13:24 | |
| *** jpena|lunch is now known as jpena | 13:24 | |
| *** redrobot5 has joined #opendev | 13:44 | |
| *** parallax_ has joined #opendev | 13:45 | |
| *** parallax has quit IRC | 13:45 | |
| *** redrobot has quit IRC | 13:45 | |
| *** tosky has quit IRC | 13:45 | |
| *** redrobot5 is now known as redrobot | 13:45 | |
| *** parallax_ is now known as parallax | 13:45 | |
| *** tosky_ has joined #opendev | 13:46 | |
| yoctozepto | morning infra; were there any answers to the tact call for help? | 14:02 |
| *** DSpider has joined #opendev | 14:27 | |
| fungi | yoctozepto: gmann offered to help out before we sent it, but no offers from anyone new yet since sending it to the ml | 14:29 |
| fungi | i have a feeling it'll percolate in people's minds and then they may start getting slowly involved | 14:29 |
| fungi | perhaps not formally at first, but that's fine | 14:29 |
| fungi | my ulterior motive with that post was to remind everyone that we all share responsibility for these tasks | 14:30 |
| yoctozepto | fungi: ack, I hope you are right, fingers crossed for TACTful future :D | 14:31 |
| ykarel | reviews please https://review.opendev.org/c/zuul/zuul-jobs/+/771980 | 14:31 |
| *** redrobot has quit IRC | 14:44 | |
| *** ysandeep is now known as ysandeep|away | 14:48 | |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/project-config master: CentOS 8 Stream initial enablement for AArch64 https://review.opendev.org/c/openstack/project-config/+/772887 | 14:50 |
| *** redrobot has joined #opendev | 14:57 | |
| *** sshnaidm|ruck is now known as sshnaidm|afk | 15:01 | |
| *** marios is now known as marios|call | 15:02 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix repo files for CentOS Stream https://review.opendev.org/c/zuul/zuul-jobs/+/771980 | 15:07 |
| *** gouthamr has joined #opendev | 15:14 | |
| *** redrobot has quit IRC | 15:27 | |
| *** klonn has quit IRC | 15:29 | |
| *** redrobot has joined #opendev | 15:34 | |
| *** mlavalle has joined #opendev | 15:44 | |
| *** klonn has joined #opendev | 15:48 | |
| *** sboyron has quit IRC | 15:49 | |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/project-config master: CentOS 8 Stream initial enablement for AArch64 https://review.opendev.org/c/openstack/project-config/+/772887 | 16:03 |
| *** tosky_ is now known as tosky | 16:06 | |
| *** mgagne has joined #opendev | 16:11 | |
| *** ykarel has quit IRC | 16:16 | |
| openstackgerrit | Riccardo Pittau proposed openstack/project-config master: Remove anachronistic job from scciclient https://review.opendev.org/c/openstack/project-config/+/772908 | 16:23 |
| *** sboyron has joined #opendev | 16:34 | |
| openstackgerrit | Clark Boylan proposed opendev/system-config master: Use patch instead of sed to update Gerrit's x/ route https://review.opendev.org/c/opendev/system-config/+/772923 | 16:52 |
| *** sshnaidm|afk is now known as sshnaidm|ruck | 16:58 | |
| *** jpena is now known as jpena|off | 17:02 | |
| *** marios|call is now known as marios | 17:03 | |
| *** tosky has quit IRC | 17:07 | |
| *** sboyron_ has joined #opendev | 17:07 | |
| *** tosky has joined #opendev | 17:07 | |
| *** sboyron has quit IRC | 17:07 | |
| *** marios is now known as marios|out | 17:49 | |
| clarkb | the above change is looking good so far in zuul. I'll update review-test with it to double check the sed to patch conversion still does what we want | 17:52 |
| clarkb | jsut need jobs to finish up first | 17:52 |
| *** eolivare has quit IRC | 17:54 | |
| *** rpittau is now known as rpittau|afk | 17:57 | |
| openstackgerrit | Clark Boylan proposed opendev/system-config master: Build Gerrit 3.3 images https://review.opendev.org/c/opendev/system-config/+/765021 | 18:20 |
| openstackgerrit | Clark Boylan proposed opendev/system-config master: Update Gerrit 3.2 plugin versions on image builds https://review.opendev.org/c/opendev/system-config/+/772931 | 18:20 |
| clarkb | those two changes occured to me while doing the x/ thing. I'll actually test the result of 772931 instead of 777923 | 18:20 |
| fungi | ahh, yep | 18:23 |
| *** dtantsur is now known as dtantsur|afk | 18:25 | |
| *** eolivare has joined #opendev | 18:25 | |
| *** andrewbonney has quit IRC | 18:26 | |
| sshnaidm|ruck | is there an option not to run specific jobs/queue/projects on a specific provider cloud? | 18:30 |
| sshnaidm|ruck | for example if I want to avoid running jobs on vexxhost ca-ymq-1 | 18:30 |
| fungi | sshnaidm|ruck: we don't have any means of expressing that, but also mnaser might be interested in addressing whatever makes your jobs not work in vexxhost ca-ymq-1 | 18:32 |
| sshnaidm|ruck | fungi, yep, we'll talk to him | 18:32 |
| openstackgerrit | Ghanshyam proposed openstack/project-config master: Remove Karbor projects from infra https://review.opendev.org/c/openstack/project-config/+/767057 | 18:34 |
| *** eolivare has quit IRC | 18:42 | |
| clarkb | ok I've got builds of that image now to test | 19:07 |
| clarkb | proceeding with that momentarily | 19:07 |
| *** diablo_rojo has joined #opendev | 19:11 | |
| diablo_rojo | I set up an etherpad for tracking the channel renames fungi: https://etherpad.opendev.org/p/openinfra-channel-renames | 19:11 |
| diablo_rojo | Not sure what else to include at this point. | 19:11 |
| clarkb | ok review-test has been updated. I am able to clone x/ansible-role-jenkins from it | 19:12 |
| clarkb | I think https://review.opendev.org/c/opendev/system-config/+/772923 and https://review.opendev.org/c/opendev/system-config/+/772931 are good to review | 19:13 |
| *** marios|out has quit IRC | 19:13 | |
| kopecmartin | clarkb: hi, it passed! \o/ https://review.opendev.org/c/opendev/system-config/+/705258 , what now? | 19:16 |
| *** portdirect has quit IRC | 19:17 | |
| clarkb | kopecmartin: next up we need to actually deploy a new server with it. The best thing for that may be to add it to our weekly meeting agenda and we can find volunteers to help and/or send email about it to service-discuss@lists.opendev.org | 19:17 |
| *** portdirect has joined #opendev | 19:17 | |
| fungi | diablo_rojo: ooh! i'll open a browser tab and... something, something, profit? but seriously i'll try to take a look, thanks for driving this | 19:17 |
| clarkb | kopecmartin: probably what we'll do is deploy it but not really expose it first, check it looks good, then schedule a downtime to cut over production to the new instance | 19:17 |
| diablo_rojo | fungi, no problem, maybe later today or tomorrow we can divvy up tasks or whatever. | 19:18 |
| fungi | yep, absotumitably | 19:19 |
| kopecmartin | clarkb: ok, sign me up, i'll help with testing | 19:20 |
| clarkb | kopecmartin: can you send the email and/or update the meeting agenda with info on the change and asking for help to get it deployed? | 19:21 |
| clarkb | there is a good chance I'll end up helping but getting it to a wider audience first is nice :) | 19:21 |
| kopecmartin | clarkb: sure, can you please point me to the agenda? | 19:22 |
| clarkb | kopecmartin: https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting | 19:23 |
| kopecmartin | clarkb: thanks | 19:23 |
| *** d34dh0r53 has quit IRC | 19:25 | |
| *** d34dh0r53 has joined #opendev | 19:30 | |
| openstackgerrit | Radosław Piliszek proposed opendev/system-config master: ask.o.o: Fix two issues with ro message https://review.opendev.org/c/opendev/system-config/+/772937 | 19:33 |
| rm_work | hey, have been on a bit of a hiatus and came back to this new gerrit interface... hoping someone here has some good greasemonkey scripts to bring back something more usable? | 19:53 |
| rm_work | IE, something akin to the old greasemonkey script that gave us realtime zuul statuses (which seem to now be missing altogether?!) | 19:54 |
| clarkb | rm_work: the zuul status are there as a static table which is how we've always shipped them | 19:55 |
| clarkb | there is a openstack-discuss thread on that subject though and you may find useful info there | 19:55 |
| rm_work | where? | 19:55 |
| rm_work | it seems to be completely missing from the new gerrit interface | 19:56 |
| clarkb | "Zuul Summary" | 19:56 |
| clarkb | ^F that string will get you to it | 19:56 |
| rm_work | ah it's tabbed, doesn't just show up | 19:56 |
| rm_work | i couldn't even tell those were clickable | 19:56 |
| rm_work | yeah ok, can probably just make a greasemonkey script to expose that at a higher level, will check the ML first tho to see if someone already fixed it, thanks :) | 19:57 |
| fungi | rm_work: if you're looking for the status for in-progress builds for a change, that's what's currently being discussed yeah. there's an available gerrit plugin which we may be able to use or at least adapt to work in our deployment | 19:58 |
| *** whoami-rajat__ has quit IRC | 19:58 | |
| fungi | the current zuul report tab you're seeing is just for completed buidls | 19:59 |
| fungi | builds | 19:59 |
| rm_work | right, but previously it was higher level, not buried | 19:59 |
| rm_work | and yeah there was that great greasemonkey script before that had it fully realtime, was amazingly useful | 19:59 |
| fungi | well, we were adding it with a hacky custom javascript overlay which depended on implementation details of their ui | 20:00 |
| rm_work | right, and it worked brilliantly :D | 20:00 |
| fungi | it wasn't really part of gerrit | 20:00 |
| fungi | and the ui uses a totally different framework now, but is also pluggable, so we can actually upstream our plugins | 20:00 |
| rm_work | sweet | 20:01 |
| rm_work | yeah, would love to see it back at top-level | 20:01 |
| rm_work | of course I also wish they hadn't removed all of the lines/borders/colors such that everything all just blends into a giant mess, but i guess eventually i'll figure out how to parse this | 20:01 |
| rm_work | in the meantime I bet I can fix it with some custom CSS :D | 20:01 |
| *** d34dh0r53 has quit IRC | 20:04 | |
| clarkb | if you're really interested in helping with upstream gerrit ui design they have a thread on their repo-discuss mailing list soliciting ui feedback nits and such | 20:05 |
| openstackgerrit | Guillaume Chauvel proposed opendev/system-config master: Increase autogenerated comment width to avoid line wrap https://review.opendev.org/c/opendev/system-config/+/771445 | 20:06 |
| openstackgerrit | Guillaume Chauvel proposed opendev/system-config master: [DNM] test comment width: review without autogenerated tag https://review.opendev.org/c/opendev/system-config/+/771798 | 20:06 |
| clarkb | I think it is worth pointing out that a ton of effort has gone into this and while it isn't perfect it is a compromise for the better. We have asked for help pretty regularly since like september? This shouldn't be a surprise and the best way to make things better is to help | 20:08 |
| rm_work | right like I said, I've just come back from hiatus | 20:09 |
| rm_work | I think the last time I really did a review was ... october | 20:09 |
| rm_work | getting up to speed | 20:10 |
| clarkb | rm_work: https://groups.google.com/g/repo-discuss/c/hbvSHHM8sVQ that is the upstream thread if you are interested | 20:11 |
| rm_work | i am very interested, thanks | 20:12 |
| rm_work | other than the fact that I HATE mailing lists T_T | 20:12 |
| fungi | hopefully once you gain some familiarity with the new gerrit interface, you'll find it an improvement over the previous one, even if there are still some rough edges and missing feature parity with what we had (still trying to close the gap on some of it) | 20:12 |
| clarkb | ya there are a lot of new great features like the single page diff viewing of all files | 20:13 |
| fungi | the easiest way to deal with mailing lists is to use an mua which will let you just delete everything from the list ;) | 20:13 |
| fungi | then read from the archives if someone points you to a message which was important | 20:13 |
| clarkb | its fairly low volume on that one (maybe one or two messages a day max). But I think google groups also allows you to just respond directly via the web view | 20:14 |
| clarkb | (I've never done that though) | 20:14 |
| openstackgerrit | Merged opendev/system-config master: ask.o.o: Fix two issues with ro message https://review.opendev.org/c/opendev/system-config/+/772937 | 20:15 |
| *** d34dh0r53 has joined #opendev | 20:15 | |
| rm_work | it supports themes, right? | 20:15 |
| rm_work | but I guess they have to be installed on the server side? | 20:15 |
| fungi | yup, there are a couple already | 20:15 |
| fungi | if you go into your preferences there's a "dark mode" theme you can switch to | 20:16 |
| rm_work | honestly i think it's entirely just theming issues for me | 20:16 |
| fungi | also theming is pluggable, that's how we do the opendevification for our deployment. example here: https://review.opendev.org/765422 | 20:16 |
| rm_work | yeah, i guess for now I'll see if the existing greasemonkey script can be easily adapted | 20:18 |
| *** ralonsoh has quit IRC | 20:48 | |
| ianw | speaking of, can some people click around on the two changes @ https://23.253.159.205 and make sure the summary tab is updating for them | 21:19 |
| ianw | the change is rather simple; https://gerrit-review.googlesource.com/c/plugins/zuul-results-summary/+/294862 but just want to make sure it works for !me too :) | 21:20 |
| corvus | ianw: when i click between changes, it goes back to the files tab, then i click on zuul summary again, and the contents are different for the 2 different changes. iiuc that is expected behavior and your patch is gtg | 22:26 |
| ianw | corvus: thanks, yep that's expected behaviour | 22:26 |
| corvus | ianw: i left a V+1 based on that | 22:27 |
| *** klonn has quit IRC | 22:47 | |
| *** sboyron_ has quit IRC | 23:05 | |
| *** slaweq has quit IRC | 23:11 | |
| *** mlavalle has quit IRC | 23:17 | |
| clarkb | ianw: do the borg things need reviews or were we still waiting on a new ps? | 23:19 |
| ianw | clarkb: i got a bit distracted, again, sorry | 23:20 |
| clarkb | k no worries | 23:20 |
| * clarkb has been finding distractions too | 23:20 | |
| ianw | i think that ask.o.o is one of the bigger users | 23:20 |
| clarkb | huh it should be fairly static at this point? | 23:20 |
| ianw | yeah, rolling over and gzipping the db , it must have some sort of timestamps that change it just enough | 23:21 |
| clarkb | and then gzip compounds that "makes sense" | 23:21 |
| ianw | i'm thinking that maybe we just take one manual backup snapshot of it and then disable further periodic backups | 23:21 |
| fungi | i suppose with it being in read-only mode we could stop the db dumps yeah | 23:22 |
| *** diablo_rojo has quit IRC | 23:22 | |
| fungi | or end backups for it entirely, sure | 23:22 |
| clarkb | ya I think that is a reasonable next step | 23:22 |
| fungi | server snapshot seems plenty sufficient to preserve it until we turn it off | 23:22 |
| ianw | i will get to mordred's comment on https://review.opendev.org/c/opendev/system-config/+/771738 to fix up the scripts to make even better raw dump outputs | 23:22 |
| ianw | maybe if you want to review that WRT having the raw streams, that would be good | 23:23 |
| ianw | wasn't 100% clear if we're all on board with that approach | 23:23 |
| clarkb | I think I'm willing to accept it :) | 23:23 |
| clarkb | having docs about it too would be good | 23:23 |
| clarkb | so that we don't confuse ourselves in the future when we wonder why it is split that way | 23:24 |
| ianw | sure, i added some stuff to the readme for the roles, but can also expand in our docs/* | 23:24 |
| clarkb | I think docs was the only major thing missing after my previous look at it | 23:25 |
| clarkb | oh and do we need to squash the child into the parent? | 23:26 |
| clarkb | to avoid issues with pruning once that change lands if there is a gap? | 23:26 |
| clarkb | (also it isn't clear to me how we migrate the existing backups to the newly named backps or maybe we don/t) | 23:27 |
| clarkb | so ya mostly just docs :) | 23:27 |
| ianw | yeah, i can, i was expecting to merge them both close, but can squash if we like | 23:27 |
| ianw | yeah, it's going to be a bit manual swapping over, i've still got to clear space on vexxhost | 23:27 |
| clarkb | if we don't squash them I think we should land them together | 23:27 |
| clarkb | ianw: also should we set a reserved space in the borg configs? | 23:27 |
| ianw | i feel like 1tb should be enough for our working set of backups, that's why i'm trying to get us within that | 23:28 |
| clarkb | this will keep it from completely filling the disk | 23:28 |
| ianw | we can, i've not played with that option. i guess the backup fails one way or the other | 23:28 |
| clarkb | yup it will still fail but ensures you can do other operations I think | 23:29 |
| ianw | sorry i've just freaking notcied that the zuul summary fix doesn't seem to work on chrome | 23:34 |
| ianw | of course it gives no error or other indication, other than not being there :( | 23:35 |
| clarkb | mmm browsers | 23:35 |
| clarkb | unfortunately I'm not a ton of help in that arean | 23:35 |
| *** knikolla has quit IRC | 23:36 | |
| *** knikolla has joined #opendev | 23:36 | |
| *** ildikov has quit IRC | 23:36 | |
| *** zaro_ has joined #opendev | 23:37 | |
| *** dviroel_ has joined #opendev | 23:37 | |
| *** aprice has quit IRC | 23:37 | |
| *** zaro has quit IRC | 23:37 | |
| *** aprice has joined #opendev | 23:37 | |
| *** dviroel has quit IRC | 23:37 | |
| *** dviroel_ is now known as dviroel | 23:37 | |
| *** zaro_ is now known as zaro | 23:37 | |
| *** ildikov has joined #opendev | 23:38 | |
| ianw | now it does, and i did nothing but copy the file? | 23:40 |
| ianw | maybe it's a caching thing | 23:40 |
| ianw | if someone else with google chrome could validate https://23.253.159.205/c/test-project/+/3 that would be great | 23:41 |
| ianw | it seems to work from my personal laptop & phone | 23:41 |
| mordred | chrome won't let me load that | 23:42 |
| clarkb | ianw: what do you want me to verify there? | 23:42 |
| mordred | because of certs | 23:42 |
| ianw | clarkb: just that the zuul summary tab is there and looks like it's working | 23:42 |
| clarkb | ianw: it does on chrome for me | 23:42 |
| mordred | ianw: fwiw: http://paste.openstack.org/show/802114/ | 23:43 |
| mordred | is the error I get | 23:43 |
| mordred | and chrome will not let me say "yeah, I don't care" | 23:43 |
| ianw | mordred: interesting, i got told to "click here to enable enhanced protection" but was allowed to accept the self-signed | 23:44 |
| clarkb | ya I could accept the risk in chrome too | 23:44 |
| mordred | ianw: I think I updated chrome a couple of days ago | 23:44 |
| mordred | 88.0.4324.96 | 23:44 |
| mordred | so - maybe I'm a glimpse of the future? | 23:45 |
| mordred | oh wow | 23:45 |
| ianw | clarkb: thanks, i don't think it's anything related to javascript then. when i re-copied the file it popped back up, so i guess it must have been something to do with serving it, being stale or something | 23:45 |
| mordred | it's a mac thing | 23:45 |
| ianw | my phone is 88.0.4324.93 | 23:46 |
| mordred | if I click on the warning page and type "thisisunsafe" it proceeds | 23:46 |
| mordred | I see the zuul summary tab | 23:46 |
| clarkb | "its a mac thing" <- I feel left out :P | 23:46 |
| ianw | haha | 23:46 |
| mordred | and clicking through the changes the zuul summary tabs change | 23:47 |
| ianw | It's a UNIX system! I know this! | 23:47 |
| ianw | except these days i think it's less and less a unix system | 23:47 |
| mordred | yah. srrsly | 23:47 |
| mordred | of course, I could say the same of linux too ... | 23:48 |
| ianw | thanks for checking it. i will presume nobody has an issue with the row highlight or left padding i added | 23:48 |
| mordred | ianw: I think your upstream patch is ready to land, yes? | 23:48 |
| ianw | mordred: yeah, i stacked a couple of UI bits on too if you'd like to look | 23:48 |
| clarkb | it looked fine to me | 23:49 |
| clarkb | but I tend to not be super particular | 23:49 |
| mordred | those are all applied to the test instance, right? | 23:50 |
| ianw | there was a proposal to shrink the columns, but i wanted to make sure across different CI's (which are separate tables) the columns remained a consistent width | 23:50 |
| ianw | i.e. all the times line up | 23:50 |
| mordred | ianw: I landed the first one, and have +2+V'd the rest of them | 23:50 |
| ianw | i think the row highlight makes it easier to pick out the info, and is consistent with the file picker, which is 100% and uses a row highlight | 23:51 |
| ianw | thanks; i want to update the zuul job to add some more changes to the test gerrit, it will mean next time to debug similar you won't have to manually inject sample changes | 23:52 |
| clarkb | ianw: one thing that occured to me is that it might be good to have a x/foo repo in the test and have a test that clones it (to confirm our workaround for that continues to function) | 23:53 |
| clarkb | ianw: is there already code there to create a random repo with some content? or would I need to build that up? | 23:53 |
| clarkb | particularly as the 3.3 image spinup happens I think confirming that for 3.3 too would be good | 23:54 |
| ianw | clarkb: yeah, i think all that needs is a x/foo project create like ! https://opendev.org/opendev/system-config/src/branch/master/playbooks/test-review.yaml#L32 | 23:58 |
| ianw | @ even | 23:58 |
| clarkb | excellent, I'll take a look at that tomorrow as I think that will help avoid us regressing in that spot | 23:59 |
| ianw | ++ | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!