| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Implement zookeeper-auth https://review.openstack.org/619155 | 05:04 |
|---|---|---|
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Implement zookeeper-auth https://review.openstack.org/619156 | 05:04 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: scheduler: add job's tags to the rpc job_list method https://review.openstack.org/633653 | 05:12 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add tags to jobs list https://review.openstack.org/633654 | 05:14 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Unmount internal mounts on finalise errors https://review.openstack.org/642953 | 06:17 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: DNM: Test zuul test case sharding https://review.openstack.org/642956 | 06:37 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/nodepool master: k8s: Fix delete_namespace to use kwargs properly https://review.openstack.org/642964 | 08:12 |
| openstackgerrit | Javier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support https://review.openstack.org/529376 | 09:32 |
| openstackgerrit | Ankita Bansal proposed openstack-infra/storyboard-webclient master: Displayed tags should link to search view for the tag https://review.openstack.org/642262 | 11:22 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck master: Add query for neutron API did not start bug 1819897 https://review.openstack.org/643019 | 11:42 |
| openstack | bug 1819897 in neutron "Intermittent "Neutron did not start" failures in the gate" [Medium,Confirmed] https://launchpad.net/bugs/1819897 | 11:42 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a linter for our .scss files https://review.openstack.org/643025 | 11:52 |
| openstackgerrit | Merged openstack-infra/elastic-recheck master: Add query for neutron API did not start bug 1819897 https://review.openstack.org/643019 | 12:07 |
| openstack | bug 1819897 in neutron "Intermittent "Neutron did not start" failures in the gate" [Medium,Confirmed] https://launchpad.net/bugs/1819897 | 12:07 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a linter for HTML templates https://review.openstack.org/643032 | 12:17 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a linter for HTML templates https://review.openstack.org/643032 | 13:26 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Use the latest active LTS version (10) of Node.js for testing https://review.openstack.org/643046 | 13:26 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Update to a newer version (8) of Node.js for testing https://review.openstack.org/643046 | 14:10 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a linter for HTML templates https://review.openstack.org/643032 | 14:10 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Update to a newer version (8) of Node.js for testing https://review.openstack.org/643046 | 14:30 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a linter for HTML templates https://review.openstack.org/643032 | 14:30 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Update to a newer version (8) of Node.js for testing https://review.openstack.org/643046 | 14:34 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a linter for HTML templates https://review.openstack.org/643032 | 14:34 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck master: Add query for DestinationHypervisorTooOld bug 1819910 https://review.openstack.org/643064 | 14:52 |
| openstack | bug 1819910 in tempest "test_server_connectivity_live_migration intermittently fails with NoValidHost due to DestinationHypervisorTooOld" [High,In progress] https://launchpad.net/bugs/1819910 - Assigned to Matt Riedemann (mriedem) | 14:52 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Add missing doc for 'execute_time' in MQTT schema https://review.openstack.org/643069 | 15:03 |
| openstackgerrit | Merged openstack-infra/elastic-recheck master: Add query for DestinationHypervisorTooOld bug 1819910 https://review.openstack.org/643064 | 15:17 |
| openstack | bug 1819910 in tempest "test_server_connectivity_live_migration intermittently fails with NoValidHost due to DestinationHypervisorTooOld" [High,In progress] https://launchpad.net/bugs/1819910 - Assigned to Matt Riedemann (mriedem) | 15:17 |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Add cgit-alias for zuul-preview https://review.openstack.org/643083 | 15:49 |
| openstackgerrit | Frank Kloeker proposed openstack-infra/openstack-zuul-jobs master: Disable releasenote sync to Zanata temporary https://review.openstack.org/643085 | 15:54 |
| openstackgerrit | Riju Khatri proposed openstack-infra/storyboard-webclient master: Enhance adding tags on story details page https://review.openstack.org/642643 | 16:00 |
| openstackgerrit | Merged openstack-infra/project-config master: Add cgit-alias for zuul-preview https://review.openstack.org/643083 | 16:14 |
| openstackgerrit | Riju Khatri proposed openstack-infra/storyboard-webclient master: Make task ordering in Story view selectable https://review.openstack.org/641930 | 16:50 |
| openstackgerrit | James E. Blair proposed opendev/base-jobs master: Document the OpenDev docker image jobs https://review.openstack.org/643108 | 17:32 |
| openstackgerrit | James E. Blair proposed opendev/base-jobs master: Document the OpenDev docker image jobs https://review.openstack.org/643108 | 17:47 |
| openstackgerrit | Merged openstack-dev/pbr master: Change openstack-dev to openstack-discuss https://review.openstack.org/625390 | 17:51 |
| openstackgerrit | James E. Blair proposed opendev/base-jobs master: Document the OpenDev docker image jobs https://review.openstack.org/643108 | 18:31 |
| openstackgerrit | Ankita Bansal proposed openstack-infra/storyboard-webclient master: Pagination error in active/merged/invalid stories in projects resolved. https://review.openstack.org/642224 | 19:38 |
| openstackgerrit | Clark Boylan proposed openstack-infra/project-config master: Disable wheel mirror updates https://review.openstack.org/643146 | 19:42 |
| openstackgerrit | Clark Boylan proposed openstack-infra/project-config master: Revert "Disable wheel mirror updates" https://review.openstack.org/643147 | 19:42 |
| openstackgerrit | Merged openstack-infra/project-config master: Disable wheel mirror updates https://review.openstack.org/643146 | 20:02 |
| openstackgerrit | Ankita Bansal proposed openstack-infra/storyboard-webclient master: Stars should be next to the name of the entity https://review.openstack.org/643153 | 20:17 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Migrating legacy base jobs nodeset from Xenial to Bionic https://review.openstack.org/639096 | 21:41 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Move zuul-legacy-jobs.yaml jobs to Bionic https://review.openstack.org/641886 | 21:43 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Document how to build a buildset registry https://review.openstack.org/643167 | 21:50 |
| openstackgerrit | Merged opendev/base-jobs master: Switch default nodeset to ubuntu-bionic https://review.openstack.org/641897 | 21:53 |
| openstackgerrit | Merged openstack-infra/project-config master: Revert "Disable wheel mirror updates" https://review.openstack.org/643147 | 22:02 |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Remove trusty testing flag from nodes that are xenial https://review.openstack.org/643168 | 22:03 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/openstack-zuul-jobs master: Run tox-py35 on Xenial for no-constraints template https://review.openstack.org/643180 | 23:03 |
| corvus | mordred, clarkb, fungi: i'm going to say some stuff about gitea and service workers here: | 23:04 |
| corvus | here's the relevant file in gitea: https://github.com/go-gitea/gitea/blob/master/templates/pwa/serviceworker_js.tmpl | 23:04 |
| fungi | neat! | 23:05 |
| corvus | that confirms the service worker is caching "gitea-sm.png", which is the thing used in the navbar | 23:05 |
| fungi | i wonder why it would be cached in the service worker instead of just relying on the browser cache | 23:05 |
| mordred | corvus: did we update the logo by replacing the file - or by updating template files and css entries? | 23:06 |
| corvus | you can also see here: https://github.com/go-gitea/gitea/blob/master/templates/pwa/serviceworker_js.tmpl#L64-L76 that they implement the "cache falling back to the network" approach described in https://developers.google.com/web/ilt/pwa/caching-files-with-service-worker | 23:06 |
| corvus | mordred: we replaced the file | 23:06 |
| mordred | nod | 23:06 |
| corvus | mordred: which, given that they implemented cache->network rather than network->cache is not going to work | 23:07 |
| corvus | so that explains our observations | 23:07 |
| mordred | yeah | 23:07 |
| corvus | i can think of 3 things: | 23:07 |
| corvus | 1) expire the cache when we install a new "version" of the service worker (see "removing outdated caches" in the google link) -- that should allow us to change icons with each new build of gitea | 23:08 |
| corvus | 2) replace the templates rather than the file. however, that particular icon appears in several template files -- by the time we're done we may have local copies of a significant amount of gitea templates | 23:09 |
| corvus | 3) i'm not sure i had a third thing. um. let's call it "some other idea" :) | 23:09 |
| mordred | yeah | 23:09 |
| mordred | oh - well | 23:09 |
| clarkb | 1 seems perfectly fine | 23:09 |
| mordred | actually - it's only in one template we care about | 23:09 |
| corvus | mordred: heh, that's a good point | 23:10 |
| mordred | the other templates its in are repo settings and new project creation screens | 23:10 |
| mordred | and I thnik we already have a fork of head_navbar | 23:10 |
| corvus | https://github.com/go-gitea/gitea/blob/master/templates/base/head_navbar.tmpl#L4 is the spot in question | 23:10 |
| corvus | mordred: oh do we already? | 23:10 |
| corvus | then #1 may be the way to go... | 23:10 |
| mordred | yup | 23:10 |
| corvus | next question though -- is https://github.com/go-gitea/gitea/blob/master/templates/pwa/manifest_json.tmpl going to be a problem? | 23:11 |
| mordred | yeah - we do - so if we just add it as a new image and update that template we should be fine | 23:11 |
| clarkb | if we do it as a new image we won't cache it though? | 23:11 |
| corvus | mordred: i guess ^ that means that if someone adds opendev.org to their homescreen, they're going to get the gitea icons? | 23:11 |
| clarkb | (it seems that it might be desireable to cache these often used artifacts) | 23:12 |
| mordred | hrm. yeah | 23:12 |
| corvus | clarkb: right, we'll rely on the browser cache, which as fungi suggests, seems perfectly reasonable :) | 23:12 |
| clarkb | ah | 23:12 |
| mordred | so yeah - we'd need to update that manifest_json.tmpl too | 23:12 |
| fungi | i've usually found browser caching of images reasonable | 23:12 |
| mordred | but actually - we shoudl do that anyway | 23:12 |
| fungi | i just wonder why they thought the service worker was a sensible place to do that instead | 23:13 |
| corvus | mordred: ok -- and that's not cached, so that should be fine | 23:13 |
| mordred | so that the homescreen entry created is "OpenDev: Free Software Needs Free Tools" instead of "gitea - Git with a cup of tea" | 23:13 |
| corvus | so basically, next steps are: revert the gitea-sm.png change, update the navbar to use a new image, then update the manifest (and all of its images should be new) along with the usual favicon stuff | 23:14 |
| mordred | yeah. that's what I'd vote for at least | 23:14 |
| mordred | sine I think we should update that manifest template | 23:14 |
| corvus | cool, i'll give that a shot tomorrow | 23:15 |
| corvus | thanks! | 23:15 |
| mordred | \o/ | 23:15 |
| clarkb | probably want an explanation in the commit message so 3 years from now when we are wondering wtf its weird we know :) | 23:15 |
| mordred | corvus: this makes me really appreciate the work webpack does for us | 23:15 |
| clarkb | though I fnd the borwser to be a black box so that may just be me wondering wtf my browser is doing | 23:15 |
| mordred | clarkb: it's using all your RAM, that's what it's doing | 23:16 |
| clarkb | mordred: exactly | 23:16 |
| clarkb | firefox alone is like 8GB | 23:17 |
| clarkb | I wish I was kidding | 23:17 |
| clarkb | also if you run the memory measuremtn on about:memory in firefox they don't show you an aggregate usage | 23:19 |
| clarkb | I think beacuse you'd probably spit your drink out | 23:19 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Run nodejs4- jobs on ubuntu-xenial https://review.openstack.org/643182 | 23:20 |
| mordred | clarkb: bwahaha | 23:21 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Run tox-py35 on Xenial for no-constraints template https://review.openstack.org/643180 | 23:27 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Run nodejs4- jobs on ubuntu-xenial https://review.openstack.org/643182 | 23:37 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!