| *** mtreinish_ is now known as mtreinish | 00:59 | |
| opendevreview | daniel.pawlik proposed openstack/ci-log-processing master: Change test assertion date https://review.opendev.org/c/openstack/ci-log-processing/+/869067 | 10:13 |
|---|---|---|
| opendevreview | daniel.pawlik proposed openstack/ci-log-processing master: Add Opensearch dashboards backup script; add dashboards objects https://review.opendev.org/c/openstack/ci-log-processing/+/860811 | 10:14 |
| opendevreview | daniel.pawlik proposed openstack/ci-log-processing master: Split custom_ca_crt per service https://review.opendev.org/c/openstack/ci-log-processing/+/869070 | 10:36 |
| opendevreview | Merged openstack/ci-log-processing master: Change test assertion date https://review.opendev.org/c/openstack/ci-log-processing/+/869067 | 10:43 |
| opendevreview | Merged openstack/project-config master: Fix tox4 error https://review.opendev.org/c/openstack/project-config/+/868526 | 10:52 |
| *** rlandy|out is now known as rlandy | 10:53 | |
| opendevreview | Merged openstack/ci-log-processing master: Split custom_ca_crt per service https://review.opendev.org/c/openstack/ci-log-processing/+/869070 | 11:19 |
| *** dviroel_ is now known as dviroel | 11:39 | |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Tie recursion calls to Dist object, not module https://review.opendev.org/c/openstack/pbr/+/869082 | 13:35 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Tie recursion calls to Dist object, not module https://review.opendev.org/c/openstack/pbr/+/869082 | 13:47 |
| *** dasm|off is now known as dasm | 14:06 | |
| *** dviroel is now known as dviroel|lunch | 15:32 | |
| *** dansmith_ is now known as dansmith | 16:24 | |
| dansmith | it seems like I have lost syntax highlighting in gerrit, perhaps related to the upgrade? is this a known issue? | 16:29 |
| fungi | i haven't heard it mentioned yet, but i wouldn't be surprised if it's a bug in the new version | 16:30 |
| fungi | infra-root: ^ has anyone else reported it that i'd missed? | 16:31 |
| dansmith | I confirmed with one other person | 16:31 |
| dansmith | so it appears not to just be me | 16:32 |
| fungi | i don't review with the webui, so am not surprised i didn't notice | 16:32 |
| dansmith | it took me a minute to even realize.. I was thinking I had forgotten how to read python code after my extended absence :) | 16:33 |
| fungi | i'm quite certain i have | 16:33 |
| fungi | too much eggnog or something | 16:33 |
| *** dviroel|lunch is now known as dviroel | 16:45 | |
| frickler | seems to still be working for me | 16:52 |
| fungi | agreed, i just pulled up https://review.opendev.org/c/openstack/pbr/+/869082/2/pbr/core.py at random and see syntax highlighting | 16:59 |
| fungi | there's a brief delay when it loads where all text is the default color and then the colorization appears | 16:59 |
| fungi | maybe something is breaking that step in some browsers? testing in ff 108 on debian/unstable | 17:00 |
| dansmith | so, it seems like I see it on some files and not others | 17:00 |
| dansmith | I have no highlighting here: https://review.opendev.org/c/openstack/nova/+/867832/2/nova/compute/manager.py | 17:00 |
| dansmith | but I do here: https://review.opendev.org/c/openstack/tempest/+/822318/5/tempest/lib/common/dynamic_creds.py | 17:01 |
| fungi | agreed, maybe it's related to file size? | 17:01 |
| fungi | can you tell if it's all longish files where you don't get highlighting? | 17:02 |
| dansmith | meaning you also don't see it on files like that first one? | 17:02 |
| fungi | i also don't see highlighting on nova/compute/manager.py right | 17:02 |
| dansmith | okay | 17:03 |
| dansmith | looking at a file with only 7k lines I don't have syntax, but do on one with 3k | 17:03 |
| dansmith | but do in another 7k line one | 17:04 |
| fungi | the reason i mention file size is it's possible this is a "feature" where for performance reasons it only tries to highlight in files under a certain size | 17:04 |
| dansmith | perhaps it's actual file size and not line count I guess | 17:04 |
| fungi | could be | 17:04 |
| fungi | i have no idea if it's even related to size/length at all with the number of data points so far | 17:05 |
| fungi | so just a guess | 17:05 |
| dansmith | I don't see any user preference for changing that, is it possibly a server-side thing? | 17:15 |
| dansmith | that would go in line with it having changed recently... | 17:15 |
| fungi | entirely possible it's a hard-coded limit even | 17:38 |
| *** dviroel is now known as dviroe|out | 21:17 | |
| *** rlandy is now known as rlandy|out | 23:38 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!