| opendevreview | Steve Baker proposed openstack/diskimage-builder master: Allow custom console=tty0 argument https://review.opendev.org/c/openstack/diskimage-builder/+/880008 | 03:52 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/project-config master: Retire openstack-health project: end project gating https://review.opendev.org/c/openstack/project-config/+/880013 | 04:52 |
| opendevreview | Steve Baker proposed openstack/diskimage-builder master: Allow custom console=tty0 argument https://review.opendev.org/c/openstack/diskimage-builder/+/880008 | 05:15 |
| opendevreview | Takashi Kajinami proposed openstack/project-config master: Retire puppet-rally - Step 5: Remove Project https://review.opendev.org/c/openstack/project-config/+/880019 | 06:14 |
| *** amoralej|off is now known as amoralej | 06:29 | |
| *** jpena|off is now known as jpena | 07:05 | |
| *** elodilles_pto is now known as elodilles | 07:24 | |
| *** Trevor is now known as Guest10611 | 08:42 | |
| *** amoralej is now known as amoralej|lunch | 11:57 | |
| *** dviroel_ is now known as dviroel | 12:14 | |
| *** amoralej|lunch is now known as amoralej | 13:01 | |
| corvus | infra-root: zuul-ci.org seems less than happy to me | 14:06 |
| corvus | #status log started apache on static01.opendev.org since it's serving zuul-ci.org | 14:07 |
| opendevstatus | corvus: finished logging | 14:07 |
| clarkb | corvus: zuul-ci.org shouldn't be served by static01 since friday. I only shutdown static01's apache yesterday after checking things looked good since friday | 14:41 |
| clarkb | corvus: is it possible you've got stale dns somehow? | 14:41 |
| clarkb | (I also spot checked apache logs on static01 before shutting down apache there and it looked like crawlers were generating the traffic) | 14:42 |
| clarkb | in any case if you have more info I'll happily run that down as I'd like to get rid of static01 in the near future | 14:55 |
| corvus | clarkb: i can look into the dns possibility | 14:57 |
| corvus | clarkb: hrm, all of my dns now says it's static02. so i'm sure it was a stale dns problem, but i no longer have the information needed to run it down (ie, i don't know if we had really long ttls, or my dns decided to serve a stale record) | 15:01 |
| corvus | #status log stopped apache on static01.opendev.org | 15:02 |
| opendevstatus | corvus: finished logging | 15:02 |
| clarkb | ttl should be 300 currently, however old ttl was 3600 and updated on friday | 15:02 |
| clarkb | I've got a couple changes proposed to bump the ttl back up again once we're happy with everything. I guess we can keep an eye on it for another day or two just to be extra careful given what you saw | 15:02 |
| corvus | it seems very likely that it was a local stale cache then (which is, weird). sorry i lost the thread so i can't pull in it further. :/ | 15:03 |
| clarkb | it happens. I'll keep an ear out for any further trouble | 15:03 |
| corvus | (i didn't suspect that initially because other static.opendev.org sites worked for me) | 15:04 |
| corvus | (and despite having 28374927349387 terminals open, none of them are the one where i ran dig) | 15:05 |
| clarkb | I'm glad I'm not the only one who ends up with way too many terminals | 15:07 |
| clarkb | the git 2.40 release notes taught me about git jump. I'm now hoping to find some time to figure that out as I think that may cut down on my terminal count | 15:09 |
| clarkb | the 2.40 release just added emacs support too so that may be useful to you as well | 15:10 |
| corvus | that looks pretty cool | 15:29 |
| clarkb | the deploy job for the etherpad refactor was skipped due to letsencrypt job failing | 16:17 |
| clarkb | the daily periodif job then succeeded. I'm going to double check on that before approving adding etherpad02 to the inventory | 16:17 |
| clarkb | ok the periodic daily run of infra-prod-letsencrypt succeeded so not a consistent error whatever it ends up being | 16:18 |
| clarkb | I'll look at the job failure as soon as local updates are done and I can reboot and reload keys | 16:20 |
| *** amoralej is now known as amoralej|off | 16:24 | |
| frickler | seems the new gerrit adds tags to links it includes in mails, like https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/879337?usp=email | 16:30 |
| clarkb | frickler: that might be editable in the email templates assuming they expose urls that don't have the tags? | 16:30 |
| frickler | do we evaluate that somewhere? otherwise maybe we can get rid of it to avoid giving people the impression we are tracking them somehow? | 16:30 |
| clarkb | we don't use that anywhere, but gerrit internally might? for example gerrit does allow you to comment on reviews now via email and maybe that plays into it somehow? | 16:31 |
| clarkb | (note we don't have the mail server set up for that to work anyway aiui so not a problem for us) | 16:31 |
| clarkb | frickler: maybe the first thing to do is ask upstream if that is important for something and see if maybe it can be changed upstream if not | 16:32 |
| clarkb | basically flip it around and force the one or two installs that use it (chrome and android I would assume) to update their templates rather than the other way around | 16:32 |
| frickler | clarkb: could you do that please? I've avoided the need to register in their bugtracker so far | 16:33 |
| clarkb | ya though it might not be until later today. Doing a quick grep in the repo shows: https://gerrit.googlesource.com/gerrit/+/refs/heads/stable-3.7/polygerrit-ui/app/models/views/change.ts#58 | 16:35 |
| clarkb | and elsewhere usp is set to service-worker-notification | 16:36 |
| clarkb | I suspect that generally you can't filter all usp parameters. But it may be the case that the email one isn't helpful | 16:36 |
| clarkb | sometimes it is set to dashboard as well | 16:37 |
| clarkb | I don't think we can change this via the templates because the java around the templating is adding this to the url not the templates themselves https://gerrit.googlesource.com/gerrit/+/refs/heads/stable-3.7/java/com/google/gerrit/server/mail/send/ChangeEmail.java#256 | 16:39 |
| clarkb | so I don | 16:41 |
| clarkb | er | 16:41 |
| clarkb | I don't think this is about tracking for nefarious reasons but it does look like elsewhere they use this to inform web state since it is a webapp interacting with the api | 16:41 |
| clarkb | I don't see anywhere where email is used on the web ui side though. Some of he other stuff seems to be used though | 16:42 |
| clarkb | I think instead of a bug to start I'll write an email to the list asking for some more explanations of how it is used | 16:43 |
| opendevreview | Merged openstack/diskimage-builder master: Rename openeuler mirror script https://review.opendev.org/c/openstack/diskimage-builder/+/878807 | 16:48 |
| *** jpena is now known as jpena|off | 16:52 | |
| clarkb | frickler: https://groups.google.com/g/repo-discuss/c/kkbyIKkzlAU/m/4t789yerBAAJ | 16:59 |
| clarkb | mirror01.regionone.linaro.opendev.org acme.sh execution returned -13 | 17:10 |
| clarkb | considering the next run was successful I think this is fine and I can probably approve the etherpad02 addition change | 17:11 |
| clarkb | https://review.opendev.org/c/opendev/system-config/+/879386 has been approved | 17:11 |
| opendevreview | Merged opendev/system-config master: Add etherpad02 to inventory https://review.opendev.org/c/opendev/system-config/+/879386 | 17:35 |
| ianw | i was thinking on the wiki/issue tracking etc. i wonder if with gitea we could keep the load-balancing for git requests, but then have one big server to enable issue tracking/wiki | 19:53 |
| ianw | to try and avoid having it be a distributed system | 19:53 |
| ianw | i'm hand-waving the login requirements too, and things like gerrit integration closing issues | 19:54 |
| clarkb | I think if we do that we should deploy it without git repo content too | 19:55 |
| clarkb | I think it is theoretically possible, but would require some thought | 19:55 |
| ianw | it seems like, in theory, haproxy could be made to do it | 19:56 |
| clarkb | oh i would decouple the url and fqdn too I think | 19:56 |
| clarkb | I mean yes what you are describing would be possible but we'd have to move all of the ssl termination into haproxy and I think it would make for a much more confusing setup operationally | 19:57 |
| clarkb | but maybe having the opendev.org issues link for $project link to tracker.opendev.org as another similar looking gitea instance would be too confusing | 19:57 |
| clarkb | er too confusing for endusers I mean | 19:58 |
| clarkb | definitely worthy of a spec if we go that route | 19:59 |
| ianw | i think a separate site would be painful if we did want to grow it into a cluster situation | 20:02 |
| ianw | it seems like proxying (HEAD|info/refs|objects/info/.*|git-upload-pack).* would be sufficient | 20:24 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove static01 from inventory https://review.opendev.org/c/opendev/system-config/+/880086 | 20:30 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove etherpad01 from inventory https://review.opendev.org/c/opendev/system-config/+/880087 | 20:30 |
| ianw | it also seems we could poc it on another port, although gitea-lb would need to get a cert | 20:30 |
| clarkb | I've WIP'd that second change but will leave the WIP off the first as I think we can remove that at any point once we are comfortable (tomorrow?) | 20:30 |
| clarkb | ianw: on the etherpad you've noted centos 8 stream removal. cs8 isn't EOL until next year. Is there a reason to do that more quickly? | 20:33 |
| ianw | just the practicality of keeping it going | 20:34 |
| opendevreview | Merged openstack/diskimage-builder master: Check and grow the GPT structure to the end of the volume https://review.opendev.org/c/openstack/diskimage-builder/+/827558 | 20:35 |
| ianw | but no, it doesn't need to be worked on right now | 20:35 |
| clarkb | ok I tried to organize by topic on that etherpad with the idea being that interests could be used to steer people to applicable tasks and then hopefully use that to grow greater understanding of the whole (I forget how ianw put it in the meeting and I've just butchered it) | 20:39 |
| ianw | thanks, yep sorry it was a bit of a dump of my todo as is | 20:40 |
| clarkb | ianw: heh re paste I think we were effectively upstream like 10 years ago | 20:54 |
| ianw | yeah, it hangs on, and i guess in some ways it's feature complete so it can just sit, but still | 20:56 |
| ianw | clarkb: can you make sense of this for NO_CHANGE matching? https://gerrit-review.googlesource.com/c/gerrit/+/64631 | 22:18 |
| ianw | if i'm understanding, you pushed a review for "refs/for/a-branch"; then "a-branch" merged back into master, then you rebase that change to master, but maintain your existing parent commit? | 22:27 |
| opendevreview | Ian Wienand proposed openstack/project-config master: wheel builds : move to individual releases https://review.opendev.org/c/openstack/project-config/+/879722 | 22:43 |
| ianw | ^ was the result of : git commit --amend --no-edit --date "$(date)" | 22:44 |
| ianw | on the push i got | 22:45 |
| ianw | "remote: warning: no changes between prior commit ff367dc and new commit be56227" | 22:45 |
| opendevreview | Ian Wienand proposed openstack/project-config master: wheel builds : move to individual releases https://review.opendev.org/c/openstack/project-config/+/879722 | 22:47 |
| opendevreview | Ian Wienand proposed openstack/project-config master: wheel builds : move to individual releases https://review.opendev.org/c/openstack/project-config/+/879722 | 22:48 |
| ianw | openstack/openstacksdk.config:[label "Allow-Post-Review"] is the only acl that didn't have a copycondition that got one | 23:19 |
| ianw | the way it checks for migration is looking for a commit with the message "Migrate label configs to copy conditions" | 23:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!