| @fajfer:reszka.org | yeah just got it yesterday, no worries. I'll ping you in 2 weeks to merge the changes | 07:49 |
|---|---|---|
| -@gerrit:opendev.org- Damian Fajfer proposed: [zuul/zuul] 958645: Remove Swagger/OpenAPI components https://review.opendev.org/c/zuul/zuul/+/958645 | 12:37 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 958647: Executor: allow configuring termination signal https://review.opendev.org/c/zuul/zuul/+/958647 | 13:14 | |
| @mhuin:matrix.org | here's a funny one, we're experimenting with gracefully stopping executors running on openshift when performing scaledowns, and we observed that the SIGTERM signal sent by kubernetes in that context is propagated to everything within the container | 13:16 |
| @mhuin:matrix.org | at least AFAICT. For example, if a job in progress just as the graceful stop is initiated needs to git clone a repo, the subprocess call to git fails with a -15 error code | 13:17 |
| -@gerrit:opendev.org- Damian Fajfer proposed wip: [zuul/zuul] 958645: Remove Swagger/OpenAPI components https://review.opendev.org/c/zuul/zuul/+/958645 | 13:19 | |
| @mhuin:matrix.org | also, the log streamer started as a child process by the executor seems to just die, and we lose access to the consoles of the jobs that are running prior to the executor shutdown (but this might be another issue) | 13:19 |
| @mhuin:matrix.org | so basically in that context the graceful stop is not so graceful | 13:19 |
| -@gerrit:opendev.org- Damian Fajfer proposed wip: [zuul/zuul] 957273: Temporary set node_version var for nodejs jobs https://review.opendev.org/c/zuul/zuul/+/957273 | 13:23 | |
| -@gerrit:opendev.org- Damian Fajfer proposed: [zuul/zuul] 956167: Update React version to 17.0.2 https://review.opendev.org/c/zuul/zuul/+/956167 | 13:23 | |
| -@gerrit:opendev.org- Damian Fajfer marked as active: [zuul/zuul] 958645: Remove Swagger/OpenAPI components https://review.opendev.org/c/zuul/zuul/+/958645 | 13:23 | |
| -@gerrit:opendev.org- Damian Fajfer proposed: | 13:28 | |
| - [zuul/zuul] 956167: Update React version to 17.0.2 https://review.opendev.org/c/zuul/zuul/+/956167 | ||
| - [zuul/zuul] 958645: Remove Swagger/OpenAPI components https://review.opendev.org/c/zuul/zuul/+/958645 | ||
| -@gerrit:opendev.org- Damian Fajfer proposed wip: [zuul/zuul] 957273: Temporary set node_version var for nodejs jobs https://review.opendev.org/c/zuul/zuul/+/957273 | 13:28 | |
| -@gerrit:opendev.org- Jan Gutter proposed: [zuul/zuul] 958656: Update implicit default in sphinx config https://review.opendev.org/c/zuul/zuul/+/958656 | 13:47 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 958553: Remove model backwards compat https://review.opendev.org/c/zuul/zuul/+/958553 | 14:26 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 958667: zuul-executor: add jobs-stats subcommand https://review.opendev.org/c/zuul/zuul/+/958667 | 15:10 | |
| -@gerrit:opendev.org- Damian Fajfer proposed: [zuul/zuul] 958645: Remove Swagger/OpenAPI components https://review.opendev.org/c/zuul/zuul/+/958645 | 15:19 | |
| @fajfer:reszka.org | I've checked Gerrit and it has X-Frame-Options header by default, it also disallows loading in frame "for security reasons" and jumps out of it as documented here: https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#gerrit (gerrit.canLoadInIFrame, gerrit.xframeOption) | 15:40 |
| @fajfer:reszka.org | I've successfully removed swagger and looks like it works | 15:41 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 958667: zuul-executor: add jobs-stats subcommand https://review.opendev.org/c/zuul/zuul/+/958667 | 16:04 | |
| -@gerrit:opendev.org- Damian Fajfer proposed: [zuul/zuul] 958645: Remove Swagger/OpenAPI components https://review.opendev.org/c/zuul/zuul/+/958645 | 17:18 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!