Saturday, 2019-10-26

*** markvoelker has joined #openstack-infra00:02
paladoxmordred you may want to switch off autoReload at https://review.opendev.org/#/c/691173/3/playbooks/roles/gerrit/templates/replication.config.j200:03
paladoxthere's a bug that when the config is touched whilst gerrit is running, it'll just stop all replications until gerrit's restarted00:03
paladoxthat kept hitting us until i switched it off :P00:06
fungii think what we observed was it cancelled any queued replications00:06
fungibut i thought we did turn it back off00:06
fungithough maybe we left it active on review-dev?00:06
*** weifan has joined #openstack-infra00:06
*** markvoelker has quit IRC00:06
paladoxoh00:07
*** weifan has quit IRC00:07
*** weifan has joined #openstack-infra00:07
paladoxAll 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
paladoxSince replicating through ssh failed too (e.g forcing the replication using the ssh command)00:08
*** weifan has quit IRC00:08
*** weifan has joined #openstack-infra00:08
*** weifan has quit IRC00:09
*** weifan has joined #openstack-infra00:09
*** weifan has quit IRC00:10
*** weifan has joined #openstack-infra00:10
paladoxi think luca was fixing it. Not sure if there was a patch already.00:10
*** weifan has quit IRC00:10
*** weifan has joined #openstack-infra00:10
*** weifan has quit IRC00:11
*** weifan has joined #openstack-infra00:11
*** weifan has quit IRC00:12
*** weifan has joined #openstack-infra00:12
*** STEWIE925 has quit IRC00:12
*** weifan has quit IRC00:13
*** weifan has joined #openstack-infra00:13
*** weifan has quit IRC00:13
*** weifan has joined #openstack-infra00:14
*** xek has quit IRC00:14
*** jamesmcarthur has quit IRC00:14
*** weifan has quit IRC00:14
*** weifan has joined #openstack-infra00:14
*** weifan has quit IRC00:15
*** weifan has joined #openstack-infra00:15
clarkbya its been working fine on vurrent gerrit00:16
*** weifan has quit IRC00:16
*** weifan has joined #openstack-infra00:16
fungiuh, not "fine" as far as i'm aware00:16
clarkboh?00:16
fungiit ended up causing missed commits if a replication event was still queued when the config changed00:16
*** weifan has quit IRC00:17
*** weifan has joined #openstack-infra00:17
fungiobserved if we altered the config on disk while a lengthy full replication was underway (for example following a restart)00:17
*** weifan has quit IRC00:17
*** weifan has joined #openstack-infra00:18
fungiany of the pending replication tasks in the queue were simply dropped00:18
clarkbThe only missed commits I remember were due to broken filesystem on the remote00:18
paladoxfungi does replication still replicate new changes/00:18
paladox*?00:18
*** weifan has quit IRC00:18
clarkbpaladox: yes00:18
paladoxoh00:18
paladoxthen this bug is in 2.1500:18
paladoxi dunno about 2.1400:18
fungipaladox: yeah, i don't think we observed it failing to replicate new commits00:19
paladoxsince as soon as the replication config is reloaded, all replications is stopped00:19
fungiwell, regardless, it sounds fairly unsafe00:19
clarkbnote the replication on start is its own anti feature but enabaled by d efault which we have disabled00:20
paladoxheh we enabled that00:20
paladoxto prevent things getting out of sync00:20
paladoxmore importantly our replica00:20
clarkbthere is ttpically nothing newto replicate if gerrit is off though00:20
clarkbbecausegit repos arentbeing updatedin that state00:20
paladoxyeh, it's about making sure things are in sync00:21
clarkbso spending 8 hours replicating it all is a massive waste00:21
paladox8 hours?00:21
paladoxwow00:21
paladoxdosen't take us long :P00:21
fungiyeah, 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 the00:21
fungiconfig was altered wound up not appearing on mirrors00:21
clarkbfungi: why havent we turnedit offthen?00:22
fungii thought we had00:22
openstackgerritMonty Taylor proposed opendev/system-config master: Fix gerrit execution path  https://review.opendev.org/69145100:24
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container (test only)  https://review.opendev.org/63040600:24
fungiclarkb: but yeah, looks like autoReload = true is still set in the production config00:24
fungii'll see if i can find where that's set00:24
*** igordc has joined #openstack-infra00:25
paladoxfungi i'm surprised your not affected by a majour auth bug in 2.1300:26
clarkbfungi: https://opendev.org/opendev/system-config/src/branch/master/modules/openstack_project/manifests/review.pp#L24100:26
clarkbpaladox: the cache one?00:27
clarkbwe applied the patch you shared with us that fixes it00:27
paladoxclarkb nope, out of date index00:27
paladoxIt hit us very badly :(00:27
clarkbI havent noticed issues with ours00:27
paladoxto the point we had to apply luca fix (which was eventually abandoned)00:27
paladoxwe had that applied till we went with 2.14/2.1500:28
clarkbapparently everyone hasproblems with ssh event streaming too00:28
clarkbworks great for us though00:28
paladoxall his fix did was made us use the source of truth (db)00:28
paladoxoh00:29
paladoxit works for us as far as i know00:29
fungiclarkb: 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 message00:29
*** igordc has quit IRC00:30
paladoxclarkb and then the fix that upstream did for us, caused a security issue.00:30
paladoxit didn't affect ldap, it affected oauth i think?00:31
paladoxdue to multiple auths00:31
mordredpaladox: in other news - thanks for making me look at that ansible template- because I had miscopied a different setting :)00:31
paladoxyour welcome :)00:31
paladoxalso in other news, i've managed to remove _legacyUndefinedCheck from everywhere in PolyGerrit!00:32
* paladox thinks he catched all the undefined cases00:33
*** yamamoto has quit IRC00:33
mordredwoot!00:34
openstackgerritJeremy Stanley proposed opendev/system-config master: Revert "Enable gerrit replication.autoReload for review"  https://review.opendev.org/69145200:34
fungifrom the irc discussion it looks like we suspected it and had some strong coincidental evidence, but did not attempt to recreate the problem00:36
*** slaweq has quit IRC00:37
openstackgerritMonty Taylor proposed opendev/system-config master: Run gerrit with podman instead  https://review.opendev.org/68989000:37
openstackgerritMonty Taylor proposed opendev/system-config master: Apply review container to review-dev  https://review.opendev.org/69048300:37
fungisince we were also dealing with post opendev migration challenges at the same time00:37
openstackgerritMonty Taylor proposed opendev/system-config master: Use explicit image paths  https://review.opendev.org/69051100:37
openstackgerritMonty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py  https://review.opendev.org/69076600:37
openstackgerritMonty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit  https://review.opendev.org/69117100:37
openstackgerritMonty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible  https://review.opendev.org/69117200:37
openstackgerritMonty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible  https://review.opendev.org/69117300:37
mordredfungi: I have updated the ansible replication patch ^^ to match00:37
fungii did not revert the application to review-dev nor the plumbing for that option00:38
mordredI don't think it's necessary to00:38
fungias they were in a separate change00:38
*** igordc has joined #openstack-infra00:38
fungiyeah, with you replacing it anyway00:38
mordredyeah00:38
fungiit'll be dead code soon enough00:39
mordredspeaking of - holy crap it's a lot of puppet to translate :)00:39
fungii 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/restart00:39
fungis/widow/window/00:40
mordredwe should have maintenance widows00:40
fungii had one for years00:40
clarkbfwiw I strongly recall it requeuing things00:40
clarkbjust to the end of the queue00:40
clarkbwhich is why I was confused and thought it had been fine for us00:41
fungiit looked like we ended up starting another full replication, and did not attempt to reproduce the problem00:42
mordredclarkb, fungi: while y'all are around - care to look at https://review.opendev.org/#/c/691451/ ? that fixes the issue in local testing00:42
mordredalso - turns out we werent' testing anyting in the first "run gerrit in a container" patch due to hostname mismatch. yay00:43
paladox:D00:44
clarkbfungi: the issue you refwr to was due to the gutea git lock files I think00:47
clarkbbasically full replication didntreplicate anyrhinf00:48
clarkband was due to gitea having stale locks00:48
fungiclarkb: the one in the comment linked from the revert commit message?00:49
clarkbI havent look atit yet just going of what you said here /me looks00:49
fungiyou may want to scroll up a ways from the timestamp i linked00:50
paladoxWe learnt you need to do --delete whilst rsycing gerrit repos. Apparently if you doin't it causes all types of issues...00:50
mordredcorvus: if you're still around ...00:53
fungiin unrelated news, if 691440 gets approved i can remove storyboard-dev from the emergency disable list00:53
mordredcorvus: https://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_49c/630406/58/check/system-config-run-review-container/49cbce3/00:53
mordredcorvus: does not appear to have done speculative container images00:53
mordredcorvus: 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.txt00:54
mordredcorvus: is the thing fixed by the patch ahead of it in the queue00:54
paladoxmordred j2 syntax now highlighted in gerrit :)00:54
* paladox added that to the mime type list00:55
clarkbya so we cancel the events (not disputing that) but on further testing I thought they got requeued.00:55
*** weifan has joined #openstack-infra00:55
clarkband we finally trackeddon stale refs to corrupt fson gitea06and the stale locks for gitea00:55
clarkbbut possible it doesnt requeue like I thought it did00:55
clarkbwither way our replication config is pretty stable now so being cautious should be finw00:56
mordredcorvus: 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 dd08322433354538cbd3b790ef6ad9de6fb8ec47e70e5ea467d10f4de9f74fdf00:56
mordredcorvus: 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 e85c49941bfefbbdb15f6770959c270b5e6191d8ea2ada15a3bdd72ec8a5422b00:57
*** weifan has quit IRC00:59
*** weifan has joined #openstack-infra00:59
*** weifan has quit IRC01:00
*** weifan has joined #openstack-infra01:00
*** weifan has quit IRC01:01
*** weifan has joined #openstack-infra01:01
mordredcorvus: aha! system-config-run-review-dev does not depend on opendev-buildset-registry01:01
*** weifan has quit IRC01:01
*** weifan has joined #openstack-infra01:01
*** weifan has quit IRC01:02
*** weifan has joined #openstack-infra01:02
*** weifan has quit IRC01:03
*** weifan has joined #openstack-infra01:03
*** weifan has quit IRC01:04
*** weifan has joined #openstack-infra01:04
*** weifan has quit IRC01:04
*** weifan has joined #openstack-infra01:05
mordredcorvus: 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 jobs01:05
*** weifan has quit IRC01:05
*** weifan has joined #openstack-infra01:05
*** weifan has quit IRC01:06
*** weifan has joined #openstack-infra01:06
*** weifan has quit IRC01:07
*** weifan has joined #openstack-infra01:07
*** weifan has quit IRC01:08
*** weifan has joined #openstack-infra01:08
*** weifan has quit IRC01:08
*** weifan has joined #openstack-infra01:09
*** weifan has quit IRC01:09
*** weifan has joined #openstack-infra01:09
*** weifan has quit IRC01:10
*** weifan has joined #openstack-infra01:10
*** weifan has quit IRC01:11
*** slaweq has joined #openstack-infra01:11
*** weifan has joined #openstack-infra01:11
*** weifan has quit IRC01:12
*** weifan has joined #openstack-infra01:12
*** weifan has quit IRC01:12
*** weifan has joined #openstack-infra01:13
*** weifan has quit IRC01:13
*** weifan has joined #openstack-infra01:13
*** weifan has quit IRC01:14
*** weifan has joined #openstack-infra01:14
*** weifan has quit IRC01:15
*** weifan has joined #openstack-infra01:15
*** slaweq has quit IRC01:15
*** weifan has quit IRC01:16
*** weifan has joined #openstack-infra01:16
*** weifan has quit IRC01:21
*** weifan has joined #openstack-infra01:29
*** weifan has quit IRC01:29
*** weifan has joined #openstack-infra01:30
*** jerryz has quit IRC01:33
*** weifan has quit IRC01:38
openstackgerritMerged opendev/system-config master: Fix gerrit execution path  https://review.opendev.org/69145101:39
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container (test only)  https://review.opendev.org/63040601:43
openstackgerritMonty Taylor proposed opendev/system-config master: Run gerrit with podman instead  https://review.opendev.org/68989001:43
openstackgerritMonty Taylor proposed opendev/system-config master: Apply review container to review-dev  https://review.opendev.org/69048301:43
openstackgerritMonty Taylor proposed opendev/system-config master: Use explicit image paths  https://review.opendev.org/69051101:43
openstackgerritMonty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py  https://review.opendev.org/69076601:43
openstackgerritMonty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit  https://review.opendev.org/69117101:43
openstackgerritMonty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible  https://review.opendev.org/69117201:43
openstackgerritMonty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible  https://review.opendev.org/69117301:43
*** weifan has joined #openstack-infra01:52
*** weifan has quit IRC01:54
*** weifan has joined #openstack-infra01:57
*** weifan has quit IRC01:57
*** slaweq has joined #openstack-infra02:11
*** slaweq has quit IRC02:15
*** weifan has joined #openstack-infra02:19
*** weifan has quit IRC02:19
*** weifan has joined #openstack-infra02:20
*** weifan has quit IRC02:21
*** weifan has joined #openstack-infra02:21
*** rfolco|rover has joined #openstack-infra02:21
*** weifan has quit IRC02:22
*** weifan has joined #openstack-infra02:22
*** weifan has quit IRC02:23
*** weifan has joined #openstack-infra02:26
*** yamamoto has joined #openstack-infra02:27
*** larainema has joined #openstack-infra02:29
*** weifan has quit IRC02:37
*** rfolco|rover has quit IRC02:38
*** yamamoto has quit IRC02:40
*** weifan has joined #openstack-infra02:42
*** weifan has quit IRC02:45
*** weifan has joined #openstack-infra02:45
*** weifan has quit IRC02:46
*** weifan has joined #openstack-infra02:46
*** weifan has quit IRC02:50
*** weifan has joined #openstack-infra02:51
*** weifan has quit IRC02:55
*** udesale has joined #openstack-infra03:02
*** auristor has quit IRC03:04
*** weifan has joined #openstack-infra03:05
*** yamamoto has joined #openstack-infra03:09
*** weifan has quit IRC03:10
*** yamamoto has quit IRC03:12
*** yamamoto has joined #openstack-infra03:14
*** yamamoto_ has joined #openstack-infra03:15
*** auristor has joined #openstack-infra03:18
*** yamamoto has quit IRC03:19
openstackgerritXuan Yandong proposed openstack/pbr master: Switch to Ussuri jobs  https://review.opendev.org/69145603:22
*** igordc has quit IRC03:24
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040603:28
openstackgerritMonty Taylor proposed opendev/system-config master: Use explicit image paths  https://review.opendev.org/69051103:28
openstackgerritMonty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py  https://review.opendev.org/69076603:28
openstackgerritMonty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit  https://review.opendev.org/69117103:28
openstackgerritMonty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible  https://review.opendev.org/69117203:28
openstackgerritMonty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible  https://review.opendev.org/69117303:28
*** weifan has joined #openstack-infra03:33
*** weifan has quit IRC03:35
*** rh-jelabarre has joined #openstack-infra03:44
*** igordc has joined #openstack-infra03:51
*** weifan has joined #openstack-infra03:54
*** weifan has quit IRC03:56
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040604:00
*** igordc has quit IRC04:00
*** yamamoto_ has quit IRC04:01
*** markvoelker has joined #openstack-infra04:04
*** markvoelker has quit IRC04:09
*** slaweq has joined #openstack-infra04:11
*** yamamoto has joined #openstack-infra04:15
*** slaweq has quit IRC04:16
*** yamamoto has quit IRC04:17
*** yamamoto has joined #openstack-infra04:18
*** yamamoto has quit IRC04:20
*** jamesmcarthur has joined #openstack-infra04:21
*** yamamoto has joined #openstack-infra04:23
*** udesale has quit IRC04:31
*** yamamoto has quit IRC04:42
*** ramishra has quit IRC04:47
*** yamamoto has joined #openstack-infra05:01
*** jamesmcarthur has quit IRC05:01
*** jamesmcarthur has joined #openstack-infra05:02
*** yamamoto has quit IRC05:03
*** jamesmcarthur has quit IRC05:10
*** yamamoto has joined #openstack-infra05:30
*** weifan has joined #openstack-infra05:31
*** yamamoto has quit IRC05:34
*** weifan has quit IRC05:36
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040605:59
*** hwoarang has quit IRC06:08
*** hwoarang has joined #openstack-infra06:08
*** psachin has joined #openstack-infra06:09
*** slaweq has joined #openstack-infra06:11
*** slaweq has quit IRC06:15
*** jtomasek has joined #openstack-infra06:31
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040606:51
*** ociuhandu has joined #openstack-infra07:00
*** lxkong has joined #openstack-infra07:01
*** dchen has quit IRC07:03
*** ociuhandu has quit IRC07:05
*** slaweq has joined #openstack-infra07:11
*** slaweq has quit IRC07:15
*** psachin has quit IRC07:17
*** rcernin has joined #openstack-infra07:28
openstackgerritzhurong proposed openstack/project-config master: Grant permissions for murano-dashboard to horizon-core  https://review.opendev.org/69114007:44
*** hwoarang has quit IRC07:56
*** hwoarang has joined #openstack-infra08:02
*** slaweq has joined #openstack-infra08:29
*** markvoelker has joined #openstack-infra08:37
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040608:39
*** markvoelker has quit IRC08:42
*** rcernin has quit IRC08:47
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Fix truncated email addresses in user dropdowns  https://review.opendev.org/69147209:02
*** dciabrin has quit IRC09:08
*** yamamoto has joined #openstack-infra09:18
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040609:25
*** yamamoto has quit IRC09:30
*** yamamoto has joined #openstack-infra09:32
*** Benibr has quit IRC09:36
*** yamamoto has quit IRC09:43
*** yamamoto has joined #openstack-infra09:46
*** Benibr has joined #openstack-infra09:50
*** slaweq has quit IRC09:58
*** yamamoto has quit IRC10:12
*** yamamoto has joined #openstack-infra10:24
*** yamamoto has quit IRC10:28
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Remove unused imagemin build step  https://review.opendev.org/69105010:41
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Update selenium-standalone and gifsicle  https://review.opendev.org/69105110:41
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Reinstate "Add transpiling as a step in the build process"  https://review.opendev.org/69147710:42
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Reinstate "Add transpiling as a step in the build process"  https://review.opendev.org/69147710:43
*** slaweq has joined #openstack-infra11:48
*** slaweq has quit IRC11:59
*** yamamoto has joined #openstack-infra11:59
*** yamamoto has quit IRC12:03
*** yamamoto has joined #openstack-infra12:07
*** lxkong has quit IRC12:07
*** yamamoto has quit IRC12:08
*** tosky has joined #openstack-infra12:28
*** ociuhandu has joined #openstack-infra12:29
*** yamamoto has joined #openstack-infra12:32
*** ociuhandu has quit IRC12:34
*** markvoelker has joined #openstack-infra12:40
*** markvoelker has quit IRC12:45
*** yamamoto has quit IRC12:57
*** yamamoto has joined #openstack-infra13:05
*** eharney has quit IRC13:08
*** dklyle has quit IRC13:26
*** david-lyle has joined #openstack-infra13:27
*** yamamoto has quit IRC13:32
*** yamamoto has joined #openstack-infra14:08
*** yamamoto has quit IRC14:10
*** yamamoto has joined #openstack-infra14:11
*** yamamoto has quit IRC14:12
*** yamamoto has joined #openstack-infra14:12
*** yamamoto has quit IRC14:13
*** david-lyle is now known as dklyle14:13
*** yamamoto has joined #openstack-infra14:16
*** yamamoto has quit IRC14:18
*** kambiz has quit IRC14:21
*** kambiz has joined #openstack-infra14:25
*** yamamoto has joined #openstack-infra14:39
*** yamamoto has quit IRC15:08
*** yamamoto has joined #openstack-infra15:13
*** yamamoto has quit IRC15:15
*** diablo_rojo has joined #openstack-infra15:21
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: WIP: Rework story search results  https://review.opendev.org/69148415:28
*** tosky has quit IRC15:41
*** yamamoto has joined #openstack-infra15:45
*** yamamoto has quit IRC15:52
*** yamamoto has joined #openstack-infra16:04
*** yamamoto has quit IRC16:07
*** yamamoto has joined #openstack-infra16:07
*** xenos76 has joined #openstack-infra16:17
*** rascasoft has joined #openstack-infra16:19
*** yamamoto has quit IRC16:22
*** whoami-rajat has joined #openstack-infra16:23
*** jamesmcarthur has joined #openstack-infra16:32
*** jamesmcarthur has quit IRC16:37
*** ykarel|away has joined #openstack-infra16:57
*** yamamoto has joined #openstack-infra16:58
*** yamamoto has quit IRC17:04
*** ykarel|away has quit IRC17:05
*** xek has joined #openstack-infra17:39
*** prometheanfire has quit IRC17:41
*** prometheanfire has joined #openstack-infra17:41
*** xek has quit IRC17:44
*** xenos76 has quit IRC18:19
*** jamesmcarthur has joined #openstack-infra18:34
*** dciabrin has joined #openstack-infra18:34
*** ralonsoh has joined #openstack-infra18:36
*** whoami-rajat has quit IRC18:43
*** markvoelker has joined #openstack-infra18:43
*** markvoelker has quit IRC18:48
*** dciabrin has quit IRC19:04
openstackgerritAndreas Jaeger proposed openstack/project-config master: Add shared 'oslo', 'oslo-independent' ACL files  https://review.opendev.org/67027019:04
openstackgerritAndreas Jaeger proposed openstack/project-config master: Update ACLs for moved doc projects  https://review.opendev.org/67026919:04
openstackgerritAndreas Jaeger proposed openstack/project-config master: Update gerritbot channels for moved doc projects  https://review.opendev.org/67048319:04
*** vesper11 has joined #openstack-infra19:07
*** vesper has quit IRC19:09
*** jamesmcarthur has quit IRC19:12
*** jamesmcarthur has joined #openstack-infra19:15
*** jamesmcarthur has quit IRC19:18
*** ralonsoh has quit IRC19:20
openstackgerritAndreas Jaeger proposed openstack/project-config master: Update ACLs for moved doc projects  https://review.opendev.org/67026919:27
openstackgerritAndreas Jaeger proposed openstack/project-config master: Update gerritbot channels for moved doc projects  https://review.opendev.org/67048319:27
*** yamamoto has joined #openstack-infra19:31
*** dave-mccowan has quit IRC19:35
*** yamamoto has quit IRC19: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 IRC20:04
fungiZara: it's a great point. the content there was mostly a placeholder, it could use some serious help20:30
Zaracool, 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
fungiZara: 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.org20:41
Zaraaha, I'll try it (had a similar idea but just used a web search engine; no dice ;_;)20:42
Zarahm... https://opendev.org/opendev/system-config/src/branch/master/docker/gitea/custom/templates/home.tmpl20:43
fungiwinnner, winner, chicken dinner20:44
*** slaweq has joined #openstack-infra21:02
openstackgerritZara proposed opendev/system-config master: Add some details on where to find things to homepage  https://review.opendev.org/69149121:04
Zaralazy commit title but hopefully a start21:05
*** slaweq has quit IRC21: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-infra21:14
*** xarses has quit IRC21:15
*** xarses_ has quit IRC21:18
*** xarses_ has joined #openstack-infra21:18
*** markvoelker has joined #openstack-infra21:24
*** jtomasek has quit IRC21:28
*** markvoelker has quit IRC21:34
*** markvoelker has joined #openstack-infra21:34
*** markvoelker has quit IRC21:36
*** markvoelker has joined #openstack-infra21:36
*** armax has quit IRC21:38
*** markvoelker has quit IRC21:47
*** markvoelker has joined #openstack-infra21:47
*** markvoelker has quit IRC21:52
*** ociuhandu has joined #openstack-infra22:00
*** ociuhandu has quit IRC22:05
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040622:13
fungiZara: i sob into my keyboard every time i need to make a pull request to github projects22:34
*** eernst_ has joined #openstack-infra22:34
fungijust started keeping a handkerchief next to my workstation22: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 gerrit22:36
Zarathere's literally a button...22:36
fungihwa?!? 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
ZaraI 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=92288959722:38
Zarabut eh, someone can always edit22:38
Zarathe history of that page is a gem22:38
fungimaybe paladox also has a vested interest in the wikipedia article about gerrit22:38
paladoxthe only edit i made was https://en.wikipedia.org/w/index.php?title=Gerrit_(software)&oldid=777848707 :P22:40
Zaraah, there have been some exciting edits added and removed since. (eg: someone listed 'notable users', someone removed them)22:41
Zarahttps://en.wikipedia.org/w/index.php?title=Gerrit_(software)&diff=884370613&oldid=88417408922:42
*** jerryz has joined #openstack-infra22:49
fungigotta love the ancient gerrit 2.2.1 screenshot that article uses22:49
fungialso lists 2.14 as the current stable release22:50
Zarathe article now specifically mentions 'cherry pick' as something one *can* do22:50
Zaraam petty22:50
fungioh, wait, it was since updated to 3.0.0 as the stable release22:51
fungistill has the 2.2.1 screenshot though ;)22:51
openstackgerritMonty Taylor proposed opendev/system-config master: Run a gerrit container on review-dev01  https://review.opendev.org/63040622:55
Zaraalso lol I just noticed they tried to claim the workflow where people amend individual commits was 'gerrit-specific'22:59
*** diablo_rojo has quit IRC23:02
*** dciabrin has joined #openstack-infra23:02
clarkbMy petty thing was laying thorny branches over the spot the nrighborhood cat likes to use as a bathroom in my yard23:02
Zara:P23:04
*** jerryz has quit IRC23:15
*** diablo_rojo has joined #openstack-infra23:17
openstackgerritMonty Taylor proposed opendev/system-config master: Use explicit image paths  https://review.opendev.org/69051123:25
openstackgerritMonty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py  https://review.opendev.org/69076623:25
openstackgerritMonty Taylor proposed opendev/system-config master: Add service playbook and test run for prod gerrit  https://review.opendev.org/69117123:25
openstackgerritMonty Taylor proposed opendev/system-config master: Add launchpadlib credentials to gerrit ansible  https://review.opendev.org/69117223:25
openstackgerritMonty Taylor proposed opendev/system-config master: Add replication config to gerrit ansible  https://review.opendev.org/69117323:25
*** dciabrin has quit IRC23:34

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!