*** markvoelker has joined #openstack-infra | 00:02 | |
paladox | mordred you may want to switch off autoReload at https://review.opendev.org/#/c/691173/3/playbooks/roles/gerrit/templates/replication.config.j2 | 00:03 |
---|---|---|
paladox | there's a bug that when the config is touched whilst gerrit is running, it'll just stop all replications until gerrit's restarted | 00:03 |
paladox | that kept hitting us until i switched it off :P | 00:06 |
fungi | i think what we observed was it cancelled any queued replications | 00:06 |
fungi | but i thought we did turn it back off | 00:06 |
fungi | though maybe we left it active on review-dev? | 00:06 |
*** weifan has joined #openstack-infra | 00:06 | |
*** markvoelker has quit IRC | 00:06 | |
paladox | oh | 00:07 |
*** weifan has quit IRC | 00:07 | |
*** weifan has joined #openstack-infra | 00:07 | |
paladox | All we realised is that it wipped the config (by that i mean it didn't really load the config that was on disk). It was almost like a empty config. | 00:08 |
paladox | Since replicating through ssh failed too (e.g forcing the replication using the ssh command) | 00:08 |
*** weifan has quit IRC | 00:08 | |
*** weifan has joined #openstack-infra | 00:08 | |
*** weifan has quit IRC | 00:09 | |
*** weifan has joined #openstack-infra | 00:09 | |
*** weifan has quit IRC | 00:10 | |
*** weifan has joined #openstack-infra | 00:10 | |
paladox | i think luca was fixing it. Not sure if there was a patch already. | 00:10 |
*** weifan has quit IRC | 00:10 | |
*** weifan has joined #openstack-infra | 00:10 | |
*** weifan has quit IRC | 00:11 | |
*** weifan has joined #openstack-infra | 00:11 | |
*** weifan has quit IRC | 00:12 | |
*** weifan has joined #openstack-infra | 00:12 | |
*** STEWIE925 has quit IRC | 00:12 | |
*** weifan has quit IRC | 00:13 | |
*** weifan has joined #openstack-infra | 00:13 | |
*** weifan has quit IRC | 00:13 | |
*** weifan has joined #openstack-infra | 00:14 | |
*** xek has quit IRC | 00:14 | |
*** jamesmcarthur has quit IRC | 00:14 | |
*** weifan has quit IRC | 00:14 | |
*** weifan has joined #openstack-infra | 00:14 | |
*** weifan has quit IRC | 00:15 | |
*** weifan has joined #openstack-infra | 00:15 | |
clarkb | ya its been working fine on vurrent gerrit | 00:16 |
*** weifan has quit IRC | 00:16 | |
*** weifan has joined #openstack-infra | 00:16 | |
fungi | uh, not "fine" as far as i'm aware | 00:16 |
clarkb | oh? | 00:16 |
fungi | it ended up causing missed commits if a replication event was still queued when the config changed | 00:16 |
*** weifan has quit IRC | 00:17 | |
*** weifan has joined #openstack-infra | 00:17 | |
fungi | observed if we altered the config on disk while a lengthy full replication was underway (for example following a restart) | 00:17 |
*** weifan has quit IRC | 00:17 | |
*** weifan has joined #openstack-infra | 00:18 | |
fungi | any of the pending replication tasks in the queue were simply dropped | 00:18 |
clarkb | The only missed commits I remember were due to broken filesystem on the remote | 00:18 |
paladox | fungi does replication still replicate new changes/ | 00:18 |
paladox | *? | 00:18 |
*** weifan has quit IRC | 00:18 | |
clarkb | paladox: yes | 00:18 |
paladox | oh | 00:18 |
paladox | then this bug is in 2.15 | 00:18 |
paladox | i dunno about 2.14 | 00:18 |
fungi | paladox: yeah, i don't think we observed it failing to replicate new commits | 00:19 |
paladox | since as soon as the replication config is reloaded, all replications is stopped | 00:19 |
fungi | well, regardless, it sounds fairly unsafe | 00:19 |
clarkb | note the replication on start is its own anti feature but enabaled by d efault which we have disabled | 00:20 |
paladox | heh we enabled that | 00:20 |
paladox | to prevent things getting out of sync | 00:20 |
paladox | more importantly our replica | 00:20 |
clarkb | there is ttpically nothing newto replicate if gerrit is off though | 00:20 |
clarkb | becausegit repos arentbeing updatedin that state | 00:20 |
paladox | yeh, it's about making sure things are in sync | 00:21 |
clarkb | so spending 8 hours replicating it all is a massive waste | 00:21 |
paladox | 8 hours? | 00:21 |
paladox | wow | 00:21 |
paladox | dosen't take us long :P | 00:21 |
fungi | yeah, were we saw it cause an issue was we restarted (back when we were still performing full replication on start), some people merged new changes while there was still a lengthy replication backlog, then one of the merged changes altered the replication config, the remaining replication tasks were dropped from the queue, and anything which had merged between when replication started and when the | 00:21 |
fungi | config was altered wound up not appearing on mirrors | 00:21 |
clarkb | fungi: why havent we turnedit offthen? | 00:22 |
fungi | i thought we had | 00:22 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Fix gerrit execution path https://review.opendev.org/691451 | 00:24 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container (test only) https://review.opendev.org/630406 | 00:24 |
fungi | clarkb: but yeah, looks like autoReload = true is still set in the production config | 00:24 |
fungi | i'll see if i can find where that's set | 00:24 |
*** igordc has joined #openstack-infra | 00:25 | |
paladox | fungi i'm surprised your not affected by a majour auth bug in 2.13 | 00:26 |
clarkb | fungi: https://opendev.org/opendev/system-config/src/branch/master/modules/openstack_project/manifests/review.pp#L241 | 00:26 |
clarkb | paladox: the cache one? | 00:27 |
clarkb | we applied the patch you shared with us that fixes it | 00:27 |
paladox | clarkb nope, out of date index | 00:27 |
paladox | It hit us very badly :( | 00:27 |
clarkb | I havent noticed issues with ours | 00:27 |
paladox | to the point we had to apply luca fix (which was eventually abandoned) | 00:27 |
paladox | we had that applied till we went with 2.14/2.15 | 00:28 |
clarkb | apparently everyone hasproblems with ssh event streaming too | 00:28 |
clarkb | works great for us though | 00:28 |
paladox | all his fix did was made us use the source of truth (db) | 00:28 |
paladox | oh | 00:29 |
paladox | it works for us as far as i know | 00:29 |
fungi | clarkb: yeah, i've got a revert of the autoreload change queued up, just doing my due diligence of finding the event so i can cite it in the commit message | 00:29 |
*** igordc has quit IRC | 00:30 | |
paladox | clarkb and then the fix that upstream did for us, caused a security issue. | 00:30 |
paladox | it didn't affect ldap, it affected oauth i think? | 00:31 |
paladox | due to multiple auths | 00:31 |
mordred | paladox: in other news - thanks for making me look at that ansible template- because I had miscopied a different setting :) | 00:31 |
paladox | your welcome :) | 00:31 |
paladox | also in other news, i've managed to remove _legacyUndefinedCheck from everywhere in PolyGerrit! | 00:32 |
* paladox thinks he catched all the undefined cases | 00:33 | |
*** yamamoto has quit IRC | 00:33 | |
mordred | woot! | 00:34 |
openstackgerrit | Jeremy Stanley proposed opendev/system-config master: Revert "Enable gerrit replication.autoReload for review" https://review.opendev.org/691452 | 00:34 |
fungi | from the irc discussion it looks like we suspected it and had some strong coincidental evidence, but did not attempt to recreate the problem | 00:36 |
*** slaweq has quit IRC | 00:37 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run gerrit with podman instead https://review.opendev.org/689890 | 00:37 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Apply review container to review-dev https://review.opendev.org/690483 | 00:37 |
fungi | since we were also dealing with post opendev migration challenges at the same time | 00:37 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Use explicit image paths https://review.opendev.org/690511 | 00:37 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py https://review.opendev.org/690766 | 00:37 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit https://review.opendev.org/691171 | 00:37 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible https://review.opendev.org/691172 | 00:37 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible https://review.opendev.org/691173 | 00:37 |
mordred | fungi: I have updated the ansible replication patch ^^ to match | 00:37 |
fungi | i did not revert the application to review-dev nor the plumbing for that option | 00:38 |
mordred | I don't think it's necessary to | 00:38 |
fungi | as they were in a separate change | 00:38 |
*** igordc has joined #openstack-infra | 00:38 | |
fungi | yeah, with you replacing it anyway | 00:38 |
mordred | yeah | 00:38 |
fungi | it'll be dead code soon enough | 00:39 |
mordred | speaking of - holy crap it's a lot of puppet to translate :) | 00:39 |
fungi | i want to say, merging that revert will also cause the replication config to be autoreloaded, so i wonder if we should save it for a maintenance widow/restart | 00:39 |
fungi | s/widow/window/ | 00:40 |
mordred | we should have maintenance widows | 00:40 |
fungi | i had one for years | 00:40 |
clarkb | fwiw I strongly recall it requeuing things | 00:40 |
clarkb | just to the end of the queue | 00:40 |
clarkb | which is why I was confused and thought it had been fine for us | 00:41 |
fungi | it looked like we ended up starting another full replication, and did not attempt to reproduce the problem | 00:42 |
mordred | clarkb, fungi: while y'all are around - care to look at https://review.opendev.org/#/c/691451/ ? that fixes the issue in local testing | 00:42 |
mordred | also - turns out we werent' testing anyting in the first "run gerrit in a container" patch due to hostname mismatch. yay | 00:43 |
paladox | :D | 00:44 |
clarkb | fungi: the issue you refwr to was due to the gutea git lock files I think | 00:47 |
clarkb | basically full replication didntreplicate anyrhinf | 00:48 |
clarkb | and was due to gitea having stale locks | 00:48 |
fungi | clarkb: the one in the comment linked from the revert commit message? | 00:49 |
clarkb | I havent look atit yet just going of what you said here /me looks | 00:49 |
fungi | you may want to scroll up a ways from the timestamp i linked | 00:50 |
paladox | We learnt you need to do --delete whilst rsycing gerrit repos. Apparently if you doin't it causes all types of issues... | 00:50 |
mordred | corvus: if you're still around ... | 00:53 |
fungi | in unrelated news, if 691440 gets approved i can remove storyboard-dev from the emergency disable list | 00:53 |
mordred | corvus: https://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_49c/630406/58/check/system-config-run-review-container/49cbce3/ | 00:53 |
mordred | corvus: does not appear to have done speculative container images | 00:53 |
mordred | corvus: in that the error message given in the logs: https://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_49c/630406/58/check/system-config-run-review-container/49cbce3/review-container01.opendev.org/docker/gerrit-docker_gerrit_1.txt | 00:54 |
mordred | corvus: is the thing fixed by the patch ahead of it in the queue | 00:54 |
paladox | mordred j2 syntax now highlighted in gerrit :) | 00:54 |
* paladox added that to the mime type list | 00:55 | |
clarkb | ya so we cancel the events (not disputing that) but on further testing I thought they got requeued. | 00:55 |
*** weifan has joined #openstack-infra | 00:55 | |
clarkb | and we finally trackeddon stale refs to corrupt fson gitea06and the stale locks for gitea | 00:55 |
clarkb | but possible it doesnt requeue like I thought it did | 00:55 |
clarkb | wither way our replication config is pretty stable now so being cautious should be finw | 00:56 |
mordred | corvus: if you look at https://2516d8212df2e8c9afc4-5ac1a082e8ecb95d568fc64f7fecfb2c.ssl.cf1.rackcdn.com/691451/2/check/system-config-build-image-gerrit-2.13/4c9e180/job-output.txt timestamp "2019-10-26 00:34:05.670705" you'll see we push sha dd08322433354538cbd3b790ef6ad9de6fb8ec47e70e5ea467d10f4de9f74fdf | 00:56 |
mordred | corvus: but then in https://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_49c/630406/58/check/system-config-run-review-container/49cbce3/job-output.txt timestamp 2019-10-26 00:37:52.826342 when we pull opendevorg/gerrit:2.13 we get e85c49941bfefbbdb15f6770959c270b5e6191d8ea2ada15a3bdd72ec8a5422b | 00:57 |
*** weifan has quit IRC | 00:59 | |
*** weifan has joined #openstack-infra | 00:59 | |
*** weifan has quit IRC | 01:00 | |
*** weifan has joined #openstack-infra | 01:00 | |
*** weifan has quit IRC | 01:01 | |
*** weifan has joined #openstack-infra | 01:01 | |
mordred | corvus: aha! system-config-run-review-dev does not depend on opendev-buildset-registry | 01:01 |
*** weifan has quit IRC | 01:01 | |
*** weifan has joined #openstack-infra | 01:01 | |
*** weifan has quit IRC | 01:02 | |
*** weifan has joined #openstack-infra | 01:02 | |
*** weifan has quit IRC | 01:03 | |
*** weifan has joined #openstack-infra | 01:03 | |
*** weifan has quit IRC | 01:04 | |
*** weifan has joined #openstack-infra | 01:04 | |
*** weifan has quit IRC | 01:04 | |
*** weifan has joined #openstack-infra | 01:05 | |
mordred | corvus: this is a little trickier - I think for now I'll just land the image update and recheck - I'd like to talk with you about the right way to use the buildset registry from the system-config jobs | 01:05 |
*** weifan has quit IRC | 01:05 | |
*** weifan has joined #openstack-infra | 01:05 | |
*** weifan has quit IRC | 01:06 | |
*** weifan has joined #openstack-infra | 01:06 | |
*** weifan has quit IRC | 01:07 | |
*** weifan has joined #openstack-infra | 01:07 | |
*** weifan has quit IRC | 01:08 | |
*** weifan has joined #openstack-infra | 01:08 | |
*** weifan has quit IRC | 01:08 | |
*** weifan has joined #openstack-infra | 01:09 | |
*** weifan has quit IRC | 01:09 | |
*** weifan has joined #openstack-infra | 01:09 | |
*** weifan has quit IRC | 01:10 | |
*** weifan has joined #openstack-infra | 01:10 | |
*** weifan has quit IRC | 01:11 | |
*** slaweq has joined #openstack-infra | 01:11 | |
*** weifan has joined #openstack-infra | 01:11 | |
*** weifan has quit IRC | 01:12 | |
*** weifan has joined #openstack-infra | 01:12 | |
*** weifan has quit IRC | 01:12 | |
*** weifan has joined #openstack-infra | 01:13 | |
*** weifan has quit IRC | 01:13 | |
*** weifan has joined #openstack-infra | 01:13 | |
*** weifan has quit IRC | 01:14 | |
*** weifan has joined #openstack-infra | 01:14 | |
*** weifan has quit IRC | 01:15 | |
*** weifan has joined #openstack-infra | 01:15 | |
*** slaweq has quit IRC | 01:15 | |
*** weifan has quit IRC | 01:16 | |
*** weifan has joined #openstack-infra | 01:16 | |
*** weifan has quit IRC | 01:21 | |
*** weifan has joined #openstack-infra | 01:29 | |
*** weifan has quit IRC | 01:29 | |
*** weifan has joined #openstack-infra | 01:30 | |
*** jerryz has quit IRC | 01:33 | |
*** weifan has quit IRC | 01:38 | |
openstackgerrit | Merged opendev/system-config master: Fix gerrit execution path https://review.opendev.org/691451 | 01:39 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container (test only) https://review.opendev.org/630406 | 01:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run gerrit with podman instead https://review.opendev.org/689890 | 01:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Apply review container to review-dev https://review.opendev.org/690483 | 01:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Use explicit image paths https://review.opendev.org/690511 | 01:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py https://review.opendev.org/690766 | 01:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit https://review.opendev.org/691171 | 01:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible https://review.opendev.org/691172 | 01:43 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible https://review.opendev.org/691173 | 01:43 |
*** weifan has joined #openstack-infra | 01:52 | |
*** weifan has quit IRC | 01:54 | |
*** weifan has joined #openstack-infra | 01:57 | |
*** weifan has quit IRC | 01:57 | |
*** slaweq has joined #openstack-infra | 02:11 | |
*** slaweq has quit IRC | 02:15 | |
*** weifan has joined #openstack-infra | 02:19 | |
*** weifan has quit IRC | 02:19 | |
*** weifan has joined #openstack-infra | 02:20 | |
*** weifan has quit IRC | 02:21 | |
*** weifan has joined #openstack-infra | 02:21 | |
*** rfolco|rover has joined #openstack-infra | 02:21 | |
*** weifan has quit IRC | 02:22 | |
*** weifan has joined #openstack-infra | 02:22 | |
*** weifan has quit IRC | 02:23 | |
*** weifan has joined #openstack-infra | 02:26 | |
*** yamamoto has joined #openstack-infra | 02:27 | |
*** larainema has joined #openstack-infra | 02:29 | |
*** weifan has quit IRC | 02:37 | |
*** rfolco|rover has quit IRC | 02:38 | |
*** yamamoto has quit IRC | 02:40 | |
*** weifan has joined #openstack-infra | 02:42 | |
*** weifan has quit IRC | 02:45 | |
*** weifan has joined #openstack-infra | 02:45 | |
*** weifan has quit IRC | 02:46 | |
*** weifan has joined #openstack-infra | 02:46 | |
*** weifan has quit IRC | 02:50 | |
*** weifan has joined #openstack-infra | 02:51 | |
*** weifan has quit IRC | 02:55 | |
*** udesale has joined #openstack-infra | 03:02 | |
*** auristor has quit IRC | 03:04 | |
*** weifan has joined #openstack-infra | 03:05 | |
*** yamamoto has joined #openstack-infra | 03:09 | |
*** weifan has quit IRC | 03:10 | |
*** yamamoto has quit IRC | 03:12 | |
*** yamamoto has joined #openstack-infra | 03:14 | |
*** yamamoto_ has joined #openstack-infra | 03:15 | |
*** auristor has joined #openstack-infra | 03:18 | |
*** yamamoto has quit IRC | 03:19 | |
openstackgerrit | Xuan Yandong proposed openstack/pbr master: Switch to Ussuri jobs https://review.opendev.org/691456 | 03:22 |
*** igordc has quit IRC | 03:24 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 03:28 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Use explicit image paths https://review.opendev.org/690511 | 03:28 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py https://review.opendev.org/690766 | 03:28 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit https://review.opendev.org/691171 | 03:28 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible https://review.opendev.org/691172 | 03:28 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible https://review.opendev.org/691173 | 03:28 |
*** weifan has joined #openstack-infra | 03:33 | |
*** weifan has quit IRC | 03:35 | |
*** rh-jelabarre has joined #openstack-infra | 03:44 | |
*** igordc has joined #openstack-infra | 03:51 | |
*** weifan has joined #openstack-infra | 03:54 | |
*** weifan has quit IRC | 03:56 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 04:00 |
*** igordc has quit IRC | 04:00 | |
*** yamamoto_ has quit IRC | 04:01 | |
*** markvoelker has joined #openstack-infra | 04:04 | |
*** markvoelker has quit IRC | 04:09 | |
*** slaweq has joined #openstack-infra | 04:11 | |
*** yamamoto has joined #openstack-infra | 04:15 | |
*** slaweq has quit IRC | 04:16 | |
*** yamamoto has quit IRC | 04:17 | |
*** yamamoto has joined #openstack-infra | 04:18 | |
*** yamamoto has quit IRC | 04:20 | |
*** jamesmcarthur has joined #openstack-infra | 04:21 | |
*** yamamoto has joined #openstack-infra | 04:23 | |
*** udesale has quit IRC | 04:31 | |
*** yamamoto has quit IRC | 04:42 | |
*** ramishra has quit IRC | 04:47 | |
*** yamamoto has joined #openstack-infra | 05:01 | |
*** jamesmcarthur has quit IRC | 05:01 | |
*** jamesmcarthur has joined #openstack-infra | 05:02 | |
*** yamamoto has quit IRC | 05:03 | |
*** jamesmcarthur has quit IRC | 05:10 | |
*** yamamoto has joined #openstack-infra | 05:30 | |
*** weifan has joined #openstack-infra | 05:31 | |
*** yamamoto has quit IRC | 05:34 | |
*** weifan has quit IRC | 05:36 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 05:59 |
*** hwoarang has quit IRC | 06:08 | |
*** hwoarang has joined #openstack-infra | 06:08 | |
*** psachin has joined #openstack-infra | 06:09 | |
*** slaweq has joined #openstack-infra | 06:11 | |
*** slaweq has quit IRC | 06:15 | |
*** jtomasek has joined #openstack-infra | 06:31 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 06:51 |
*** ociuhandu has joined #openstack-infra | 07:00 | |
*** lxkong has joined #openstack-infra | 07:01 | |
*** dchen has quit IRC | 07:03 | |
*** ociuhandu has quit IRC | 07:05 | |
*** slaweq has joined #openstack-infra | 07:11 | |
*** slaweq has quit IRC | 07:15 | |
*** psachin has quit IRC | 07:17 | |
*** rcernin has joined #openstack-infra | 07:28 | |
openstackgerrit | zhurong proposed openstack/project-config master: Grant permissions for murano-dashboard to horizon-core https://review.opendev.org/691140 | 07:44 |
*** hwoarang has quit IRC | 07:56 | |
*** hwoarang has joined #openstack-infra | 08:02 | |
*** slaweq has joined #openstack-infra | 08:29 | |
*** markvoelker has joined #openstack-infra | 08:37 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 08:39 |
*** markvoelker has quit IRC | 08:42 | |
*** rcernin has quit IRC | 08:47 | |
openstackgerrit | Adam Coldrick proposed opendev/storyboard-webclient master: Fix truncated email addresses in user dropdowns https://review.opendev.org/691472 | 09:02 |
*** dciabrin has quit IRC | 09:08 | |
*** yamamoto has joined #openstack-infra | 09:18 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 09:25 |
*** yamamoto has quit IRC | 09:30 | |
*** yamamoto has joined #openstack-infra | 09:32 | |
*** Benibr has quit IRC | 09:36 | |
*** yamamoto has quit IRC | 09:43 | |
*** yamamoto has joined #openstack-infra | 09:46 | |
*** Benibr has joined #openstack-infra | 09:50 | |
*** slaweq has quit IRC | 09:58 | |
*** yamamoto has quit IRC | 10:12 | |
*** yamamoto has joined #openstack-infra | 10:24 | |
*** yamamoto has quit IRC | 10:28 | |
openstackgerrit | Adam Coldrick proposed opendev/storyboard-webclient master: Remove unused imagemin build step https://review.opendev.org/691050 | 10:41 |
openstackgerrit | Adam Coldrick proposed opendev/storyboard-webclient master: Update selenium-standalone and gifsicle https://review.opendev.org/691051 | 10:41 |
openstackgerrit | Adam Coldrick proposed opendev/storyboard-webclient master: Reinstate "Add transpiling as a step in the build process" https://review.opendev.org/691477 | 10:42 |
openstackgerrit | Adam Coldrick proposed opendev/storyboard-webclient master: Reinstate "Add transpiling as a step in the build process" https://review.opendev.org/691477 | 10:43 |
*** slaweq has joined #openstack-infra | 11:48 | |
*** slaweq has quit IRC | 11:59 | |
*** yamamoto has joined #openstack-infra | 11:59 | |
*** yamamoto has quit IRC | 12:03 | |
*** yamamoto has joined #openstack-infra | 12:07 | |
*** lxkong has quit IRC | 12:07 | |
*** yamamoto has quit IRC | 12:08 | |
*** tosky has joined #openstack-infra | 12:28 | |
*** ociuhandu has joined #openstack-infra | 12:29 | |
*** yamamoto has joined #openstack-infra | 12:32 | |
*** ociuhandu has quit IRC | 12:34 | |
*** markvoelker has joined #openstack-infra | 12:40 | |
*** markvoelker has quit IRC | 12:45 | |
*** yamamoto has quit IRC | 12:57 | |
*** yamamoto has joined #openstack-infra | 13:05 | |
*** eharney has quit IRC | 13:08 | |
*** dklyle has quit IRC | 13:26 | |
*** david-lyle has joined #openstack-infra | 13:27 | |
*** yamamoto has quit IRC | 13:32 | |
*** yamamoto has joined #openstack-infra | 14:08 | |
*** yamamoto has quit IRC | 14:10 | |
*** yamamoto has joined #openstack-infra | 14:11 | |
*** yamamoto has quit IRC | 14:12 | |
*** yamamoto has joined #openstack-infra | 14:12 | |
*** yamamoto has quit IRC | 14:13 | |
*** david-lyle is now known as dklyle | 14:13 | |
*** yamamoto has joined #openstack-infra | 14:16 | |
*** yamamoto has quit IRC | 14:18 | |
*** kambiz has quit IRC | 14:21 | |
*** kambiz has joined #openstack-infra | 14:25 | |
*** yamamoto has joined #openstack-infra | 14:39 | |
*** yamamoto has quit IRC | 15:08 | |
*** yamamoto has joined #openstack-infra | 15:13 | |
*** yamamoto has quit IRC | 15:15 | |
*** diablo_rojo has joined #openstack-infra | 15:21 | |
openstackgerrit | Adam Coldrick proposed opendev/storyboard-webclient master: WIP: Rework story search results https://review.opendev.org/691484 | 15:28 |
*** tosky has quit IRC | 15:41 | |
*** yamamoto has joined #openstack-infra | 15:45 | |
*** yamamoto has quit IRC | 15:52 | |
*** yamamoto has joined #openstack-infra | 16:04 | |
*** yamamoto has quit IRC | 16:07 | |
*** yamamoto has joined #openstack-infra | 16:07 | |
*** xenos76 has joined #openstack-infra | 16:17 | |
*** rascasoft has joined #openstack-infra | 16:19 | |
*** yamamoto has quit IRC | 16:22 | |
*** whoami-rajat has joined #openstack-infra | 16:23 | |
*** jamesmcarthur has joined #openstack-infra | 16:32 | |
*** jamesmcarthur has quit IRC | 16:37 | |
*** ykarel|away has joined #openstack-infra | 16:57 | |
*** yamamoto has joined #openstack-infra | 16:58 | |
*** yamamoto has quit IRC | 17:04 | |
*** ykarel|away has quit IRC | 17:05 | |
*** xek has joined #openstack-infra | 17:39 | |
*** prometheanfire has quit IRC | 17:41 | |
*** prometheanfire has joined #openstack-infra | 17:41 | |
*** xek has quit IRC | 17:44 | |
*** xenos76 has quit IRC | 18:19 | |
*** jamesmcarthur has joined #openstack-infra | 18:34 | |
*** dciabrin has joined #openstack-infra | 18:34 | |
*** ralonsoh has joined #openstack-infra | 18:36 | |
*** whoami-rajat has quit IRC | 18:43 | |
*** markvoelker has joined #openstack-infra | 18:43 | |
*** markvoelker has quit IRC | 18:48 | |
*** dciabrin has quit IRC | 19:04 | |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Add shared 'oslo', 'oslo-independent' ACL files https://review.opendev.org/670270 | 19:04 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Update ACLs for moved doc projects https://review.opendev.org/670269 | 19:04 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Update gerritbot channels for moved doc projects https://review.opendev.org/670483 | 19:04 |
*** vesper11 has joined #openstack-infra | 19:07 | |
*** vesper has quit IRC | 19:09 | |
*** jamesmcarthur has quit IRC | 19:12 | |
*** jamesmcarthur has joined #openstack-infra | 19:15 | |
*** jamesmcarthur has quit IRC | 19:18 | |
*** ralonsoh has quit IRC | 19:20 | |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Update ACLs for moved doc projects https://review.opendev.org/670269 | 19:27 |
openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Update gerritbot channels for moved doc projects https://review.opendev.org/670483 | 19:27 |
*** yamamoto has joined #openstack-infra | 19:31 | |
*** dave-mccowan has quit IRC | 19:35 | |
*** yamamoto has quit IRC | 19:37 | |
Zara | (hm, I've noticed that on the opendev homepage, there aren't any links to review.opendev.org. I'm out of the loop in general so not sure if this is wanted, if there's work going on etc, but wanted to mention it since it seemed a bit odd to me.) | 19:59 |
*** larainema has quit IRC | 20:04 | |
fungi | Zara: it's a great point. the content there was mostly a placeholder, it could use some serious help | 20:30 |
Zara | cool, I'll see if I can help with it sometime. :) (though this has made me realise I don't know how to update that page in general.) | 20:34 |
Zara | (this has reminded me that I also wanted to do something so that the repo list would show when the *code* in the repos was last updated, instead of their metadata, but thought that would be more involved) | 20:37 |
fungi | Zara: i'd have to refresh my recollection as to where it lives, but easiest way to fish is pick a unique-looking phrase and plug it into codesearch.openstack.org | 20:41 |
Zara | aha, I'll try it (had a similar idea but just used a web search engine; no dice ;_;) | 20:42 |
Zara | hm... https://opendev.org/opendev/system-config/src/branch/master/docker/gitea/custom/templates/home.tmpl | 20:43 |
fungi | winnner, winner, chicken dinner | 20:44 |
*** slaweq has joined #openstack-infra | 21:02 | |
openstackgerrit | Zara proposed opendev/system-config master: Add some details on where to find things to homepage https://review.opendev.org/691491 | 21:04 |
Zara | lazy commit title but hopefully a start | 21:05 |
*** slaweq has quit IRC | 21:07 | |
Zara | (tbh I just wanted to send a commit via gerrit again. been looking at a gh project where rebases are discouraged because they 'make it harder to see what changes from one patch version to the next', and am quietly crying.) | 21:10 |
*** xarses_ has joined #openstack-infra | 21:14 | |
*** xarses has quit IRC | 21:15 | |
*** xarses_ has quit IRC | 21:18 | |
*** xarses_ has joined #openstack-infra | 21:18 | |
*** markvoelker has joined #openstack-infra | 21:24 | |
*** jtomasek has quit IRC | 21:28 | |
*** markvoelker has quit IRC | 21:34 | |
*** markvoelker has joined #openstack-infra | 21:34 | |
*** markvoelker has quit IRC | 21:36 | |
*** markvoelker has joined #openstack-infra | 21:36 | |
*** armax has quit IRC | 21:38 | |
*** markvoelker has quit IRC | 21:47 | |
*** markvoelker has joined #openstack-infra | 21:47 | |
*** markvoelker has quit IRC | 21:52 | |
*** ociuhandu has joined #openstack-infra | 22:00 | |
*** ociuhandu has quit IRC | 22:05 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 22:13 |
fungi | Zara: i sob into my keyboard every time i need to make a pull request to github projects | 22:34 |
*** eernst_ has joined #openstack-infra | 22:34 | |
fungi | just started keeping a handkerchief next to my workstation | 22:35 |
Zara | ;_; it is a deeply distressing workflow. also I wound up on the gerrit wikipedia page, which has evidently been a battleground, as among other things I encountered a sentence that said it was impossible to cherry pick specific commits in gerrit | 22:36 |
Zara | there's literally a button... | 22:36 |
fungi | hwa?!? seriously, yes, there's a button (also a little drop-down you can cut-n-paste from if you want the git command to do the same to a local repo) | 22:37 |
Zara | I edited it, tried not to make it too partial but I probably failed https://en.wikipedia.org/w/index.php?title=Gerrit_(software)&diff=923183022&oldid=922889597 | 22:38 |
Zara | but eh, someone can always edit | 22:38 |
Zara | the history of that page is a gem | 22:38 |
fungi | maybe paladox also has a vested interest in the wikipedia article about gerrit | 22:38 |
paladox | the only edit i made was https://en.wikipedia.org/w/index.php?title=Gerrit_(software)&oldid=777848707 :P | 22:40 |
Zara | ah, there have been some exciting edits added and removed since. (eg: someone listed 'notable users', someone removed them) | 22:41 |
Zara | https://en.wikipedia.org/w/index.php?title=Gerrit_(software)&diff=884370613&oldid=884174089 | 22:42 |
*** jerryz has joined #openstack-infra | 22:49 | |
fungi | gotta love the ancient gerrit 2.2.1 screenshot that article uses | 22:49 |
fungi | also lists 2.14 as the current stable release | 22:50 |
Zara | the article now specifically mentions 'cherry pick' as something one *can* do | 22:50 |
Zara | am petty | 22:50 |
fungi | oh, wait, it was since updated to 3.0.0 as the stable release | 22:51 |
fungi | still has the 2.2.1 screenshot though ;) | 22:51 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01 https://review.opendev.org/630406 | 22:55 |
Zara | also lol I just noticed they tried to claim the workflow where people amend individual commits was 'gerrit-specific' | 22:59 |
*** diablo_rojo has quit IRC | 23:02 | |
*** dciabrin has joined #openstack-infra | 23:02 | |
clarkb | My petty thing was laying thorny branches over the spot the nrighborhood cat likes to use as a bathroom in my yard | 23:02 |
Zara | :P | 23:04 |
*** jerryz has quit IRC | 23:15 | |
*** diablo_rojo has joined #openstack-infra | 23:17 | |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Use explicit image paths https://review.opendev.org/690511 | 23:25 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py https://review.opendev.org/690766 | 23:25 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit https://review.opendev.org/691171 | 23:25 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible https://review.opendev.org/691172 | 23:25 |
openstackgerrit | Monty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible https://review.opendev.org/691173 | 23:25 |
*** dciabrin has quit IRC | 23:34 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!