| *** ysandeep|out is now known as ysandeep | 03:58 | |
| opendevreview | Ian Wienand proposed opendev/system-config master: Run a base test against "old" bridge https://review.opendev.org/c/opendev/system-config/+/860802 | 04:04 |
|---|---|---|
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: remove testinfra version install workaround https://review.opendev.org/c/opendev/system-config/+/852475 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: testinfra: install with ansible extras https://review.opendev.org/c/opendev/system-config/+/852476 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: remove stevedore workaround https://review.opendev.org/c/opendev/system-config/+/852477 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: install-ansible: remove stub install for ARA https://review.opendev.org/c/opendev/system-config/+/852478 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: bootstrap-bridge: drop pip3 role, add venv https://review.opendev.org/c/opendev/system-config/+/856593 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: run-selenium: Use latest tag on firefox image https://review.opendev.org/c/opendev/system-config/+/857803 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: afs-release: better info when can not get lockfile https://review.opendev.org/c/opendev/system-config/+/858009 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: Run jobs with a jammy bridge.openstack.org https://review.opendev.org/c/opendev/system-config/+/857799 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: testinfra: Update selenium calls https://review.opendev.org/c/opendev/system-config/+/858003 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: Abstract name of bastion host for testing path https://review.opendev.org/c/opendev/system-config/+/858476 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: Run a base test against "old" bridge https://review.opendev.org/c/opendev/system-config/+/860802 | 05:31 |
| opendevreview | Ian Wienand proposed opendev/system-config master: Convert production playbooks to bastion host group https://review.opendev.org/c/opendev/system-config/+/858486 | 05:31 |
| *** ysandeep is now known as ysandeep|afk | 06:23 | |
| *** jpena|off is now known as jpena | 07:07 | |
| gnuoy | Hi, I'd like to share some ansible roles that define test setup and execution across the charm projects. Where would be the right place to store these roles ? | 07:14 |
| *** ysandeep|afk is now known as ysandeep | 07:39 | |
| opendevreview | Artem Goncharov proposed openstack/project-config master: Add post-review pipeline https://review.opendev.org/c/openstack/project-config/+/859977 | 07:44 |
| opendevreview | Artem Goncharov proposed openstack/project-config master: Add Allow-Post-Review flag to OpenStackSDK project https://review.opendev.org/c/openstack/project-config/+/859976 | 07:46 |
| opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Added cloud-init growpart element https://review.opendev.org/c/openstack/diskimage-builder/+/855856 | 08:03 |
| *** ramishra_ is now known as ramishra | 08:56 | |
| *** ysandeep is now known as ysandeep|lunch | 09:52 | |
| *** ysandeep|lunch is now known as ysandeep | 11:15 | |
| frickler | so ... didn't in earlier times we still collect gate results when a dependency failed? to me it doesn't make much sense to abort already running jobs instead of collecting their results, see e.g. https://review.opendev.org/c/openstack/devstack/+/684183/4 | 11:15 |
| frickler | or do I misremember things? | 11:16 |
| *** dviroel|out is now known as dviroel | 11:24 | |
| fungi | gnuoy: you could put them somewhere central, for example in the openstack/charms.openstack repo or create a new repository specifically for the shared configuration. probably best way to look at it is who is best placed to review and approve changes to those roles, and then put them somewhere that group of people can do that | 11:36 |
| gnuoy | fungi, ah ok, thanks, I can do that. How do I then make sure that the ansible definitions in the new repo are available when zuul reads the charms zuul.yaml ? Is that a project-config change? | 11:39 |
| fungi | gnuoy: you add a "roles" list to the job definition to tell it you want to include roles from that repository | 11:44 |
| fungi | so it depends on where the job which will use those roles is defined as to where you'd set it | 11:45 |
| fungi | frickler: i don't recall that zuul ever included dependent pipeline build results in a "this change depends on a change that failed to merge" result, but we do still retain the results in the db. in that example, it was this buildset: https://zuul.opendev.org/t/openstack/buildset/f9b1a390037b401fbbdb017603483199 | 11:46 |
| gnuoy | fungi, the jobs are running in opendev zuul. At the moment I have https://opendev.org/openstack/charm-keystone-k8s/src/branch/main/.zuul.yaml where 'build-charm' and 'func-tests' are defined in the playbooks/ subdirectory of the charm. If I move those definitions to another repo can I reference that repo in the .zuul.yaml in the charm ? | 11:52 |
| frickler | fungi: I was wondering about results for the dependent patch, which via some searching I found at https://zuul.opendev.org/t/openstack/buildset/d1cf3c8d868c4dda8e6ab642f215ea90 . so a) can we (and do we want to) have zuul add a reference to this partially cancelled buildset in the V-2 message? and b) would it make sense to at least have an option to not cancel jobs in this scenario? similar to like we | 11:57 |
| frickler | don't cancel the remainder of the buildset on the first failing gate job? | 11:57 |
| fungi | gnuoy: yes, you would list the repositories containing roles used by the job like this: https://opendev.org/openstack/heat-dashboard/src/branch/master/.zuul.yaml#L9-L11 | 12:00 |
| gnuoy | fungi, great, thanks for all the help | 12:01 |
| fungi | frickler: oh, you're right i searched on the change it said it depended on. looks like https://zuul.opendev.org/t/openstack/builds?job_name=684183 returns no search results | 12:02 |
| fungi | i wonder if freeing up build resources for all changes depending on the one which is failing is an intentional optimization, but if so i definitely don't recall when it would have been originally implemented | 12:04 |
| frickler | fungi: I found the buildset by looking at the history of builds for the "devstack" jobs and then selecting the one matching the change I'm interested in | 12:05 |
| fungi | gah, nevermind, i shouldn't be doing this before i've finished pouring my coffee for the morning | 12:06 |
| fungi | https://zuul.opendev.org/t/openstack/builds?job_name=684183 was the search i meant to try, and it did return results | 12:07 |
| fungi | https://zuul.opendev.org/t/openstack/builds?change=684183 | 12:07 |
| fungi | pasted from the wrong buffer | 12:07 |
| fungi | filtering down to the gate pipeline it gets me to https://zuul.opendev.org/t/openstack/buildset/d1cf3c8d868c4dda8e6ab642f215ea90 which does indeed show some of the builds were cancelled | 12:09 |
| *** dviroel is now known as dviroel|biab | 12:29 | |
| *** dasm|off is now known as dasm | 12:32 | |
| *** pojadhav is now known as pojadhav|sick | 13:13 | |
| *** dviroel|biab is now known as dviroel | 13:31 | |
| TheJulia | Question, will you folks be taking part in the PTG? I was kind of hoping so to have a few high bandwidth discussions I have been putting off until then. | 13:50 |
| fungi | i will be, certainly | 13:58 |
| fungi | we didn't book any slots specific to the opendev collaboratory, but many of us still participate in sessions for other projects/teams where relevant | 13:59 |
| frickler | but I don't about about a specific infra/opendev session, is there one planned? I for sure would prefer IRC discussions | 13:59 |
| TheJulia | Even a office hours session would be good. Truthfully, I'd prefer higher bandwidth because I think tone, emotion, and body language are important when discussing topics that may be contentious or where highly opinionated individuals take place. Granted, I too prefer text at times, but sometimes not everything will translate through | 14:02 |
| TheJulia | I think also, some may find a time to "break the ice" might be helpful where as jumping into irc might be a little intimidating for just trying to build mutual context | 14:03 |
| fungi | worth looping our service coordinator (clarkb) into this when he's awake/around, but it's certainly possible to add a session in an available slot on the schedule if people have topics they want to cover | 14:09 |
| Clark[m] | We stopped doing office hours because no one showed up so it was just effort on my part that felt wasted. I'm happy to join in on ptg sessions where I can be helpful though | 14:09 |
| *** dviroel_ is now known as dviroel | 14:10 | |
| fungi | yeah, even for the in-person ptgs, it was mostly us sitting around waiting and nobody coming into the room with questions | 14:10 |
| TheJulia | I know you guys got visited at least once :) But yeah, I can see that as frustrating | 14:13 |
| Clark[m] | I think we went 3 ptgs in a row where no one took advantage of the time I scheduled before we stopped | 14:15 |
| Clark[m] | I think the best approach is to in where we know there is interest rather than the other way around. I'm happy to add sessions to my calendar for that | 14:15 |
| *** ysandeep is now known as ysandeep|out | 14:35 | |
| TheJulia | ack, okay | 14:47 |
| TheJulia | let me see if I can get some others to join in since I'd hate for it to just be me | 14:47 |
| fungi | but if there's a topic specific to a particular project or subproject and you want to cover some things that are relevant to the collaboratory services, i'm happy to join that so long as i don't have other schedule conflicts | 14:54 |
| *** dviroel is now known as dviroel|lunch | 15:03 | |
| clarkb | and we don't necessarily need to organize this as part of the ptg either. | 15:13 |
| *** marios is now known as marios|out | 15:38 | |
| clarkb | fungi: when you get a chance today can you review https://review.opendev.org/c/opendev/system-config/+/855472 that is one piece of not getting /tmp files for zuul console on bridge and static | 15:55 |
| clarkb | (we need a second change in a different repo to complete the cleanup | 15:55 |
| clarkb | I think we can also hold off on approving that until ianw is around if we like | 15:55 |
| *** jpena is now known as jpena|off | 16:10 | |
| *** dviroel|lunch is now known as dviroel| | 16:21 | |
| *** dviroel| is now known as dviroel | 16:21 | |
| *** dasm is now known as dasm|off | 17:49 | |
| *** dviroel_ is now known as dviroel | 18:28 | |
| dmendiza[m] | diablo_rojo 👋 | 19:40 |
| dmendiza[m] | diablo_rojo: I can't seem to find the link to the ethercalc for PTG slot reservation. Would you happen to have that handy? | 19:41 |
| Clark[m] | dmendiza: there isn't one. Instead you can use the ptg bot to reserve slots on the schedule directly | 19:48 |
| dmendiza[m] | Clark: ack, that would explain why I can't find the ethercalc | 19:50 |
| dmendiza[m] | Clark: thanks! | 19:50 |
| fungi | dmendiza[m]: hop into #openinfra-events and use the book command to book the times you want: https://ptg.opendev.org/ptg.html | 19:50 |
| fungi | see the help near the bottom of that page | 19:50 |
| *** dviroel is now known as dviroel|biab | 19:51 | |
| diablo_rojo_phone | dmendiza[m]: if you need help with the bot, the docs are really good. | 20:24 |
| clarkb | I'm updating the meeting agenda for tomorrow. Please add any content I've missed soon | 20:34 |
| clarkb | or let me know what I forgot and I'll do my best to add it | 20:34 |
| fungi | i'm all good, thanks | 20:40 |
| ianw | clarkb: https://review.opendev.org/c/opendev/system-config/+/858476 and https://review.opendev.org/c/opendev/system-config/+/858486 are ready for review in the venv stack | 21:05 |
| ianw | 476 is the biggest one -- that abstracts us from places where we use "bridge.openstack.org" to using a "bastion" group -- so we can swap the name of the bastion host | 21:06 |
| ianw | 476 changes the testing path, 486 is a follow on that changes the production run playbooks (which really just consists of adding groups["bastion"][0] in add_host, instead of "bridge.openstack.org" hard-coded) | 21:07 |
| ianw | both intended to be no-ops in current production :) | 21:08 |
| clarkb | ianw: 486 will also need a similar update in base-jobs | 21:29 |
| clarkb | in the same bit of code that needs updating for console log disabling | 21:29 |
| clarkb | er I guess that is true for both of them 476 as well | 21:31 |
| ianw | hrm, good point. have to think about where that can get the variable name | 21:32 |
| ianw | shouldn't break, as long as bridge.openstack.org is around | 21:33 |
| ianw | i'll put something together today, and work on 855472 | 21:35 |
| clarkb | ianw: I left some other comments inline on 476 | 21:38 |
| *** dasm|off is now known as dasm | 21:52 | |
| fungi | i just noticed that meetings.opendev.org content has browser-side script callouts to https://ajax.googleapis.com/ajax/libs/jquery/1.12.4/jquery.min.js and https://maxcdn.bootstrapcdn.com/bootstrap/3.3.5/js/bootstrap.min.js | 22:33 |
| fungi | we should ideally be serving that stuff directly to clients, right? otherwise it's a privacy issue? | 22:33 |
| fungi | (since browsers are notifying google and bootstrap that they're visiting the site) | 22:34 |
| JayF | There have been some EU companies getting in trouble for that; some court over there ruled it was the site sharing the users' IP with the remote servers hosting the js (IIRC) | 22:41 |
| fungi | well, i mean, they're right, it totally is | 22:43 |
| JayF | :) | 22:43 |
| fungi | we've already stripped out the built-in gravatar "integration" from some of the services we run for precisely that reason | 22:43 |
| fungi | gravatar is a bit more egregious though, since they have a tos that says you basically can't proxy content to your users so that they can preserve the revenue stream of knowing who's going to what websites | 22:44 |
| JayF | I'm just going to say, as an aside; I <3 that we care enough about this to have a conversation about it. After having worked for places that were data vacuums it is appreciated that it's table-stakes-assumed that we're protecting our contributors from tracking :) | 22:46 |
| fungi | oh, i could tell you some stories. the first start-up i worked for, my job was securing packet sniffers for collecting browser traffic for "market analysis" from partnering isps who got paid a monthly stipend for letting us snoop on their customers | 22:48 |
| fungi | back in the innocent times where we thought that doing double-blind anonymization and superquantization to zip codes was sufficient to avoid the creepy factor | 22:49 |
| fungi | also long before widespread adoption of https, and when most people still used dial-up services over phone lines to reach the web | 22:51 |
| JayF | I didn't realize for a long time how many folks thought that kind of data collection and advertising was *actually good for people* instead of a neccessary evil. | 22:51 |
| fungi | we basically intercepted their ras credentials, did a lookup against a database of home addresses given to us by the isp, and then created a one-way hash which we used to associate all their http requests, creating timelines of every url they visited | 22:53 |
| JayF | Closest thing for me is a company that I worked for that did POS API integration that also made money by selling aggreggated reciept data | 22:54 |
| ianw | fwiw i think that the google api stuff says they don't track things on an individual level. but hosting them sounds like a better idea | 22:54 |
| fungi | "google says" is enough of a reason for me to assume they're probably lying | 22:55 |
| fungi | i'll try to remember to push up a change to copy those bits to the server deployment | 22:56 |
| opendevreview | Ian Wienand proposed opendev/base-jobs master: Add zuul_console_disable flag to added hosts https://review.opendev.org/c/opendev/base-jobs/+/860883 | 23:07 |
| ianw | i've approved 855472 and cleared out the old console files to make sure they're not coming back -- i will keep an eye | 23:13 |
| fungi | thanks! | 23:26 |
| *** rcastillo_ is now known as rcastillo | 23:30 | |
| opendevreview | Merged opendev/system-config master: Add zuul_console_disable flag to added hosts https://review.opendev.org/c/opendev/system-config/+/855472 | 23:36 |
| opendevreview | Merged opendev/system-config master: install-ansible: remove testinfra version install workaround https://review.opendev.org/c/opendev/system-config/+/852475 | 23:36 |
| opendevreview | Merged opendev/system-config master: testinfra: install with ansible extras https://review.opendev.org/c/opendev/system-config/+/852476 | 23:44 |
| opendevreview | Merged opendev/base-jobs master: Add zuul_console_disable flag to added hosts https://review.opendev.org/c/opendev/base-jobs/+/860883 | 23:44 |
| opendevreview | Merged opendev/system-config master: install-ansible: remove stevedore workaround https://review.opendev.org/c/opendev/system-config/+/852477 | 23:46 |
| opendevreview | Merged opendev/system-config master: install-ansible: remove stub install for ARA https://review.opendev.org/c/opendev/system-config/+/852478 | 23:46 |
| opendevreview | Ian Wienand proposed opendev/system-config master: run-selenium: Use latest tag on firefox image https://review.opendev.org/c/opendev/system-config/+/857803 | 23:53 |
| opendevreview | Ian Wienand proposed opendev/system-config master: afs-release: better info when can not get lockfile https://review.opendev.org/c/opendev/system-config/+/858009 | 23:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!