| *** andrebeltrami has quit IRC | 00:01 | |
| *** yamamoto has joined #openstack-meeting-alt | 00:10 | |
| *** tetsuro has joined #openstack-meeting-alt | 00:10 | |
| *** gyee has quit IRC | 00:52 | |
| *** enriquetaso has quit IRC | 01:11 | |
| *** knomura has joined #openstack-meeting-alt | 01:13 | |
| *** knomura has quit IRC | 01:17 | |
| *** knomura has joined #openstack-meeting-alt | 01:21 | |
| *** knomura has quit IRC | 01:25 | |
| *** knomura has joined #openstack-meeting-alt | 01:27 | |
| *** knomura has quit IRC | 01:32 | |
| *** knomura has joined #openstack-meeting-alt | 01:39 | |
| *** knomura has quit IRC | 01:44 | |
| *** ayoung has quit IRC | 01:56 | |
| *** knomura has joined #openstack-meeting-alt | 02:09 | |
| *** knomura has quit IRC | 02:13 | |
| *** knomura has joined #openstack-meeting-alt | 02:14 | |
| *** apetrich has quit IRC | 02:14 | |
| *** knomura has quit IRC | 02:18 | |
| *** knomura has joined #openstack-meeting-alt | 02:24 | |
| *** knomura has quit IRC | 02:28 | |
| *** knomura has joined #openstack-meeting-alt | 02:38 | |
| *** knomura has quit IRC | 02:43 | |
| *** rfolco has quit IRC | 02:59 | |
| *** ricolin_ has joined #openstack-meeting-alt | 03:00 | |
| *** tetsuro has quit IRC | 03:04 | |
| *** knomura has joined #openstack-meeting-alt | 03:49 | |
| *** m5z has quit IRC | 03:54 | |
| *** lhinds has quit IRC | 03:54 | |
| *** knomura has quit IRC | 03:54 | |
| *** knomura has joined #openstack-meeting-alt | 03:54 | |
| *** lhinds|out has joined #openstack-meeting-alt | 03:54 | |
| *** knomura has quit IRC | 03:58 | |
| *** tetsuro has joined #openstack-meeting-alt | 04:22 | |
| *** tetsuro_ has joined #openstack-meeting-alt | 04:26 | |
| *** tetsuro has quit IRC | 04:27 | |
| *** tetsuro_ has quit IRC | 04:41 | |
| *** tetsuro has joined #openstack-meeting-alt | 04:46 | |
| *** tetsuro has quit IRC | 04:58 | |
| *** vishalmanchanda has joined #openstack-meeting-alt | 05:41 | |
| *** links has joined #openstack-meeting-alt | 05:50 | |
| *** knomura has joined #openstack-meeting-alt | 06:07 | |
| *** knomura has quit IRC | 06:09 | |
| *** knomura_ has joined #openstack-meeting-alt | 06:09 | |
| *** knomura_ has quit IRC | 06:13 | |
| *** ccamacho has joined #openstack-meeting-alt | 06:18 | |
| *** diurnalist has quit IRC | 06:22 | |
| *** knomura has joined #openstack-meeting-alt | 06:23 | |
| *** knomura has quit IRC | 06:28 | |
| *** diurnalist has joined #openstack-meeting-alt | 06:29 | |
| *** knomura has joined #openstack-meeting-alt | 06:31 | |
| *** knomura has quit IRC | 06:36 | |
| *** SotK has quit IRC | 06:54 | |
| *** SotK has joined #openstack-meeting-alt | 06:55 | |
| *** rdopiera has joined #openstack-meeting-alt | 07:05 | |
| *** rcernin has quit IRC | 07:05 | |
| *** ralonsoh has joined #openstack-meeting-alt | 07:31 | |
| *** ccamacho has quit IRC | 07:32 | |
| *** yamamoto has quit IRC | 07:38 | |
| *** yamamoto has joined #openstack-meeting-alt | 07:40 | |
| *** m5z has joined #openstack-meeting-alt | 07:41 | |
| *** diurnalist has quit IRC | 07:44 | |
| *** e0ne has joined #openstack-meeting-alt | 08:01 | |
| *** rcernin has joined #openstack-meeting-alt | 08:16 | |
| *** rcernin has quit IRC | 08:42 | |
| *** derekh has joined #openstack-meeting-alt | 08:48 | |
| *** ccamacho has joined #openstack-meeting-alt | 09:00 | |
| *** apetrich has joined #openstack-meeting-alt | 09:00 | |
| *** diurnalist has joined #openstack-meeting-alt | 09:43 | |
| *** frickler is now known as frickler_pto | 09:44 | |
| *** frickler_pto is now known as frickler | 09:47 | |
| *** yamamoto has quit IRC | 10:16 | |
| *** yamamoto has joined #openstack-meeting-alt | 10:22 | |
| *** gouthamr has quit IRC | 10:22 | |
| *** gouthamr has joined #openstack-meeting-alt | 10:23 | |
| *** yamamoto has quit IRC | 10:26 | |
| *** yamamoto has joined #openstack-meeting-alt | 11:02 | |
| *** yamamoto has quit IRC | 11:08 | |
| *** rcernin has joined #openstack-meeting-alt | 11:14 | |
| *** yamamoto has joined #openstack-meeting-alt | 11:40 | |
| *** raildo has joined #openstack-meeting-alt | 12:02 | |
| *** rfolco has joined #openstack-meeting-alt | 12:03 | |
| *** yamamoto has quit IRC | 12:08 | |
| *** yamamoto has joined #openstack-meeting-alt | 12:13 | |
| *** derekh has quit IRC | 12:23 | |
| *** yamamoto has quit IRC | 12:27 | |
| *** vishakha has joined #openstack-meeting-alt | 12:28 | |
| *** yamamoto has joined #openstack-meeting-alt | 12:41 | |
| *** rcernin has quit IRC | 12:48 | |
| *** baojg has joined #openstack-meeting-alt | 12:54 | |
| *** derekh has joined #openstack-meeting-alt | 13:03 | |
| *** yamamoto has quit IRC | 13:10 | |
| *** eharney has joined #openstack-meeting-alt | 13:11 | |
| *** baojg has quit IRC | 13:15 | |
| *** baojg has joined #openstack-meeting-alt | 13:16 | |
| *** enriquetaso has joined #openstack-meeting-alt | 13:22 | |
| *** benj_ has quit IRC | 13:39 | |
| *** benj_ has joined #openstack-meeting-alt | 13:41 | |
| *** ralonsoh has quit IRC | 13:49 | |
| *** frickler is now known as frickler_pto | 13:50 | |
| *** ralonsoh has joined #openstack-meeting-alt | 14:00 | |
| *** john153 has joined #openstack-meeting-alt | 14:10 | |
| *** knikolla has joined #openstack-meeting-alt | 14:21 | |
| *** yamamoto has joined #openstack-meeting-alt | 14:28 | |
| *** dave-mccowan has joined #openstack-meeting-alt | 14:38 | |
| *** johnthetubaguy has quit IRC | 14:50 | |
| *** johnthetubaguy has joined #openstack-meeting-alt | 14:52 | |
| gagehugo | #startmeeting openstack-helm | 15:00 |
|---|---|---|
| *** andrii_ostapenko has joined #openstack-meeting-alt | 15:00 | |
| openstack | Meeting started Tue Jul 14 15:00:16 2020 UTC and is due to finish in 60 minutes. The chair is gagehugo. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: openstack-helm)" | 15:00 | |
| openstack | The meeting name has been set to 'openstack_helm' | 15:00 |
| gagehugo | #link https://etherpad.opendev.org/p/openstack-helm-weekly-meeting agenda | 15:00 |
| stevthedev | Hello | 15:00 |
| gagehugo | o/ | 15:00 |
| *** lamt has joined #openstack-meeting-alt | 15:00 | |
| andrii_ostapenko | o/ | 15:01 |
| lamt | \o | 15:01 |
| gagehugo | we will start in a couple mins | 15:02 |
| *** yamamoto has quit IRC | 15:03 | |
| *** sangeet has joined #openstack-meeting-alt | 15:04 | |
| gagehugo | ok | 15:04 |
| gagehugo | #topic congrats andrii | 15:05 |
| *** openstack changes topic to "congrats andrii (Meeting topic: openstack-helm)" | 15:05 | |
| stevthedev | \o/ | 15:05 |
| andrii_ostapenko | thank you :) | 15:05 |
| gagehugo | the OSH core team would like to welcome andrii as a core reviewer, thanks andrii for all your hard work! | 15:05 |
| gagehugo | and I believe andrii has more things to discuss today too :) | 15:07 |
| lamt | \o/ | 15:07 |
| gagehugo | #topic Improvements in jobs execution time | 15:07 |
| *** openstack changes topic to "Improvements in jobs execution time (Meeting topic: openstack-helm)" | 15:07 | |
| gagehugo | andrii_ostapenko: the floor is yours | 15:07 |
| andrii_ostapenko | yes. Thank you, gagehugo. I started with small think for now and sorry for commit message. I'm introducing a mechanism to parallelize gate scripts execution that at this point saves 10-20% of execution time. https://review.opendev.org/739659 https://review.opendev.org/739891 | 15:08 |
| andrii_ostapenko | if we're going in this direction i have several more optimizations in my mind | 15:09 |
| andrii_ostapenko | like script refactoring and running all tests in parallel at the end | 15:10 |
| gagehugo | The gates could use cleanup like this, so I'm in favor of something like this | 15:10 |
| andrii_ostapenko | also looking to switch back to kubeadm but move kubeadm-aio image build to osh images and reuse it in osh during the build | 15:10 |
| andrii_ostapenko | this should be evaluated if we can save couple mins there, so rnd for now | 15:11 |
| andrii_ostapenko | also pre-pulling some images in parallel | 15:11 |
| andrii_ostapenko | for now only https://review.opendev.org/739659 https://review.opendev.org/739891 - i'll prepare commits for review later | 15:11 |
| andrii_ostapenko | so if you have some other ideas - please share | 15:12 |
| gagehugo | I think there was discussion previously about hosting some images in the infra mirror | 15:12 |
| gagehugo | idk what happened there though | 15:12 |
| andrii_ostapenko | my goal is to have minimum time for compute-kit ~30 mins | 15:12 |
| gagehugo | That would be amazing | 15:13 |
| *** tosky has joined #openstack-meeting-alt | 15:13 | |
| lamt | ++ | 15:13 |
| andrii_ostapenko | i don't feel we can have an average 30 mins, but at least we should move in this direction | 15:14 |
| gagehugo | I'm for it | 15:15 |
| andrii_ostapenko | also i noticed it's a bit worse on 8GB nodes. maybe we have memory consumption spikes due to parallel execution. still it's faster than consequent one | 15:15 |
| gagehugo | There may be things we can reduce there as well | 15:15 |
| gagehugo | replica counts | 15:15 |
| andrii_ostapenko | i'll prepare some comparison based on statistics | 15:15 |
| gagehugo | min memory values | 15:15 |
| andrii_ostapenko | yes. looking at individual charts is the next step | 15:16 |
| gagehugo | ok cool | 15:16 |
| andrii_ostapenko | minute here, minute there - every minute matters | 15:17 |
| stevthedev | ++ | 15:17 |
| gagehugo | yup, might just be a bunch of small tweaks that add up to larger gains | 15:17 |
| gagehugo | #topic osh-infra-images testing | 15:17 |
| *** openstack changes topic to "osh-infra-images testing (Meeting topic: openstack-helm)" | 15:17 | |
| gagehugo | andrii_ostapenko: this is you as well I believe | 15:18 |
| andrii_ostapenko | https://review.opendev.org/#/q/topic:test-osh-images | 15:18 |
| andrii_ostapenko | it's a mechanism to reuse osh jobs in osh-images on post-review | 15:18 |
| andrii_ostapenko | so what is suggest is to build and upload images during the gates on osh-images and test it there as well | 15:19 |
| andrii_ostapenko | also in future I want periodic builds for loci images, testing and promotion for loci images. we often have a situation when the last latest image is more then 1 month old | 15:20 |
| andrii_ostapenko | because we build them only on commits to corresponding code in osh-images | 15:20 |
| gagehugo | yeah, we've already hit issues where some images are old and out-of-date with upstream changes | 15:20 |
| gagehugo | I'm in favor of this as well | 15:21 |
| gagehugo | it's been some pain points | 15:21 |
| andrii_ostapenko | this topic is ready for review. it provides mechanism and implements it for loci stein and train on gates | 15:21 |
| stevthedev | If we build on each commit, why do we need to build periodically? | 15:21 |
| andrii_ostapenko | we build on each commit to osh-images to corresponding to particular image code | 15:22 |
| andrii_ostapenko | but don't build for new openstack component merges | 15:22 |
| stevthedev | Ah okay | 15:22 |
| stevthedev | I don't think I run into this problem much with the LMA images | 15:23 |
| andrii_ostapenko | yes. we need periodic one only for loci | 15:23 |
| gagehugo | It's been a bit annoying with things like the requirements image and stable images for openstack | 15:23 |
| stevthedev | Thanks for clarifying for me | 15:23 |
| gagehugo | branches get updated with bug fixes or reqs change, but the image is months behind | 15:23 |
| stevthedev | Makes sense | 15:23 |
| gagehugo | and suddenly images fail because the reqs match up game breaks | 15:24 |
| andrii_ostapenko | periodic ones will be available if i manage to merge https://review.opendev.org/#/c/740560/ | 15:24 |
| andrii_ostapenko | but now we can at least run test deployment on gates and topic is ready for review | 15:24 |
| andrii_ostapenko | i'm going to implement this logic for other images, not only loci | 15:24 |
| andrii_ostapenko | if we have a corresponding test job in osh or osh-infra | 15:25 |
| stevthedev | I think thats a good idea. I remember Steve W wanting more robust testing between osh-infra and osh-images too | 15:25 |
| andrii_ostapenko | e.g. for some changes in kibana image we can run osh-infra-logging job | 15:25 |
| stevthedev | ++ | 15:26 |
| andrii_ostapenko | also i want to get rid of some duplication across the repos https://review.opendev.org/#/c/740530/ | 15:26 |
| andrii_ostapenko | and inherit from infra job and reuse osh-infra-gate-runner instead of having it in both repos | 15:27 |
| stevthedev | I will go through the chain of patches and take a look | 15:27 |
| andrii_ostapenko | thank you | 15:27 |
| gagehugo | sounds good! | 15:28 |
| gagehugo | thanks andrii! | 15:28 |
| gagehugo | #topic Open Discussion/Reviews | 15:28 |
| *** openstack changes topic to "Open Discussion/Reviews (Meeting topic: openstack-helm)" | 15:28 | |
| gagehugo | #link https://review.opendev.org/#/c/740825/ node-down detector | 15:28 |
| gagehugo | #undo | 15:28 |
| openstack | Removing item from minutes: #link https://review.opendev.org/#/c/740825/ | 15:28 |
| stevthedev | I want to pass on a Q from my team | 15:28 |
| stevthedev | Radhika has been working on a dockerfile for the node-problem-detector. The dockerfile merged, and she is following that up with a zuul job for the image (https://review.opendev.org/#/c/740825/). The job log shows the image was built successfully, but we don't see it on dockerhub (https://hub.docker.com/u/openstackhelm/) | 15:28 |
| gagehugo | #link https://review.opendev.org/#/c/740825/ node problem detector | 15:28 |
| gagehugo | yeah it may need to get added | 15:29 |
| stevthedev | Is there a missing component, does the repo need to be created manually, will this get fixed when PS merges? | 15:29 |
| andrii_ostapenko | it will be uploaded after merge only | 15:29 |
| gagehugo | but also there's credentials I believe in the repo | 15:29 |
| andrii_ostapenko | actually it's being uploaded on post-review | 15:29 |
| andrii_ostapenko | with temp tag | 15:30 |
| andrii_ostapenko | and promoted to defined tag after the merge | 15:30 |
| andrii_ostapenko | image is not uploaded to dockerhub in untrusted pre-review pipeline in case if zuul-jobs *-docker-image jobs are used | 15:31 |
| stevthedev | Upon merging https://review.opendev.org/#/c/740825/ do we have to make a change to the dockerfile to trigger the build? Or will it run once the zuul addition is merged? | 15:32 |
| stevthedev | to rephrase that - is it merge to the image directory that triggers the build, or any change to osh-images | 15:33 |
| andrii_ostapenko | - node-problem-detector/.* | 15:34 |
| andrii_ostapenko | - zuul.d/node-problem-detector.yaml | 15:34 |
| andrii_ostapenko | so it's defined in zuul.d/node-problem-detector.yaml | 15:34 |
| andrii_ostapenko | when to run it | 15:34 |
| stevthedev | Ah I see now. ty | 15:34 |
| *** baojg has quit IRC | 15:37 | |
| gagehugo | Anyone else have anything for this week? | 15:38 |
| andrii_ostapenko | that actually have a relation to testing an image | 15:38 |
| andrii_ostapenko | we'll be able to do it on post-review for this image as well | 15:38 |
| *** baojg has joined #openstack-meeting-alt | 15:38 | |
| andrii_ostapenko | if there's any job in osh or osh-infra that deploys this chart | 15:39 |
| stevthedev | Yep :) | 15:39 |
| andrii_ostapenko | but not in pre-review | 15:39 |
| stevthedev | I'm not sure if it's been included in an osh-infra job yet. But I'll have to double check | 15:39 |
| stevthedev | might be in monitoring? | 15:39 |
| andrii_ostapenko | https://review.opendev.org/#/c/668754/ | 15:40 |
| andrii_ostapenko | aio-monitoring | 15:41 |
| gagehugo | Thanks everyone have a good rest of the week | 15:44 |
| gagehugo | #endmeeting | 15:44 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 15:44 | |
| openstack | Meeting ended Tue Jul 14 15:44:24 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:44 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_helm/2020/openstack_helm.2020-07-14-15.00.html | 15:44 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_helm/2020/openstack_helm.2020-07-14-15.00.txt | 15:44 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_helm/2020/openstack_helm.2020-07-14-15.00.log.html | 15:44 |
| *** gyee has joined #openstack-meeting-alt | 15:50 | |
| *** andrii_ostapenko has quit IRC | 15:51 | |
| *** links has quit IRC | 15:59 | |
| *** knomura has joined #openstack-meeting-alt | 16:38 | |
| *** e0ne has quit IRC | 16:38 | |
| *** derekh has quit IRC | 16:58 | |
| knikolla | #startmeeting keystone | 16:59 |
| openstack | Meeting started Tue Jul 14 16:59:12 2020 UTC and is due to finish in 60 minutes. The chair is knikolla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:59 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:59 |
| *** openstack changes topic to " (Meeting topic: keystone)" | 16:59 | |
| openstack | The meeting name has been set to 'keystone' | 16:59 |
| *** knomura has quit IRC | 16:59 | |
| knikolla | o/ | 16:59 |
| cmurphy | o/ | 16:59 |
| vishakha | o/ | 17:00 |
| knikolla | how's everyone? | 17:00 |
| *** yamamoto has joined #openstack-meeting-alt | 17:01 | |
| vishakha | Good. How are you? | 17:02 |
| cmurphy | not too bad | 17:03 |
| knikolla | doing alright. | 17:04 |
| knikolla | going to maine the weekend next week. | 17:04 |
| knikolla | I guess that's all of us for this meeting. | 17:05 |
| knikolla | #topic Review Requests | 17:05 |
| *** openstack changes topic to "Review Requests (Meeting topic: keystone)" | 17:05 | |
| vishakha | I have few review requests that closes some bugs | 17:06 |
| vishakha | #link https://review.opendev.org/#/c/731087/ | 17:07 |
| vishakha | #link https://review.opendev.org/#/c/740473/ | 17:07 |
| vishakha | #link https://review.opendev.org/#/c/737225/ | 17:07 |
| vishakha | #link https://review.opendev.org/#/c/739784/ | 17:08 |
| vishakha | #link https://review.opendev.org/#/c/738190/ | 17:08 |
| knikolla | Thanks vishakha | 17:09 |
| knikolla | about https://review.opendev.org/#/c/731087/ i'm not entirely sure about the best time to prune stale assignment, i' | 17:10 |
| knikolla | i'm not sure delete role is the best time, but i'd like to hear other opinions | 17:10 |
| cmurphy | we should just keep it consistent with how we do it with project and domain assignments right? | 17:11 |
| knikolla | do we prune those? | 17:13 |
| vishakha | I did not saw anywhere in keystone pruning assignments | 17:14 |
| cmurphy | oh maybe i misunderstood, by stale you mean the ones that exist because of this bug? | 17:14 |
| cmurphy | sorry i'm not sure i'm being clear, i haven't looked too deeply at this bug yet | 17:15 |
| knikolla | yes, the ones prior to this bug being fixed. | 17:16 |
| vishakha | This bug is specific to system assignments, where while deleting system assignments , the assignments are not deleted from the system assignment table | 17:17 |
| cmurphy | i don't really think we should be going and retroactively fixing people's deployments and messing with their databases, and leaving in code that does a one-time cleanup but then forever exists in the code base | 17:20 |
| cmurphy | i would rather the release note included mysql instruction to help them clean it up one time | 17:20 |
| bnemec | I could use another opinion on https://review.opendev.org/#/c/733881/ It's what we agreed to at the PTG, but there was some disagreement on the review. I left more thoughts related to it on https://review.opendev.org/#/c/726929 | 17:22 |
| bnemec | (for oslo.limit and keystoneauth) | 17:22 |
| knikolla | cmurphy: hmmm... unsure if a good idea, but maybe a sql migration? | 17:23 |
| cmurphy | knikolla: i'd be more in favor of that than code that runs in the driver | 17:24 |
| cmurphy | but that wouldn't fix older deployments | 17:25 |
| knikolla | that is true. | 17:25 |
| vishakha | Still not sure should I add a note in the document informing the deployers? | 17:29 |
| knikolla | yes, that while the bug is fixed moving forward, the previously stale system role assignments will need to be cleaned manually. | 17:30 |
| knikolla | we should provide some guidance on how to do that. | 17:31 |
| cmurphy | i'm saying i disagree with lance's suggestion to add remove_stale_role_assignments and i would prefer documenting manual steps in the release note | 17:31 |
| cmurphy | i'll comment on the review and see what lbragstad thinks | 17:32 |
| vishakha | Thanks cmurphy knikolla | 17:32 |
| * lbragstad walks in late | 17:32 | |
| knikolla | another possible solution is to catch the exception when a stale assignment is encountered, and clean it then. | 17:32 |
| lbragstad | sorry - i got wrapped up reading some docs | 17:32 |
| tosky | and also https://review.opendev.org/#/c/733801/ which removes the last legacy job, which already received a +2 from last week | 17:37 |
| knikolla | lbragstad, cmurphy: thoughts on cleaning up on exception encountered? | 17:39 |
| lbragstad | knikolla cmurphy yep - i just responded on the review, but i can repeat here, too | 17:40 |
| lbragstad | i can empathize with not running that code every time we delete a role | 17:40 |
| lbragstad | not wanting to run that code* | 17:40 |
| lbragstad | but a better alternative wasn't coming to me | 17:40 |
| lbragstad | to make sure i understand correctly, cmurphy is your recommendation to provide documentation that tells operators how to fix this with SQL statements? | 17:42 |
| cmurphy | lbragstad: yes that is my proposal, it should be a pretty simple sql query, and also a pretty easy test to see if you are affected by it (see if you can run role assignment list --names) | 17:43 |
| lbragstad | what about an upgrade check? | 17:44 |
| knikolla | That is a better option than running it each time during delete_role, but I would love to avoid adding more work to operators, even if it is a one time thing. | 17:44 |
| lbragstad | we typically put things like this in keystone-manage | 17:45 |
| lbragstad | instead of asking operators to tinker with the database directly | 17:45 |
| lbragstad | (keystone-manage token_flush was an example) | 17:46 |
| knikolla | What about adding a try/catch in listing roles with names and doing the cleanup if the exception is caught? That will add more code with limited usefulness, but we would be doing the same by adding documentation. | 17:46 |
| cmurphy | token_flush was a task that needed to happen every so often | 17:46 |
| cmurphy | this is a one time thing | 17:46 |
| knikolla | And the code won't run if the exception is never encountered. | 17:46 |
| cmurphy | we have a precedent for asking operators to run db queries to fix a broken state left by a bug | 17:49 |
| cmurphy | https://bugs.launchpad.net/keystone/+bug/1848342 | 17:49 |
| openstack | Launchpad bug 1848342 in OpenStack Identity (keystone) "Duplicated entries in users API" [Medium,Fix released] - Assigned to Pedro Henrique Pereira Martins (pedrohpmartins) | 17:49 |
| cmurphy | i don't really see why it's that big of a deal | 17:49 |
| cmurphy | it feels really inelegant to leave code in keystone that will last forever that will only ever run once on a deployment in its lifetime | 17:50 |
| knikolla | hmm, i wasn't aware of the precedent. interesting. | 17:51 |
| lbragstad | i don't recall seeing this bug | 17:51 |
| *** yamamoto has quit IRC | 17:53 | |
| knikolla | i'm okay with a release not for this. given that very few deployments will be using system role assignments at this point in time, it feels like very few people will be running into this. | 17:54 |
| knikolla | note* | 17:54 |
| lbragstad | if others are ok with the approach, i won't get in the way | 17:57 |
| *** ralonsoh has quit IRC | 17:57 | |
| lbragstad | i was unaware of cases where we did this in the past' | 17:57 |
| knikolla | i was unaware too | 17:57 |
| knikolla | vishakha: you can revise your spec with the new approach, and we can continue talking on the review, or in next weeks meeting | 17:58 |
| cmurphy | this may be the only case, if we've come up with another solution for a similar problem in the past we could look at that but i'm unaware of any | 17:58 |
| *** diurnalist has quit IRC | 17:59 | |
| cmurphy | it looks like it was mostly just me involved in the review of that last bugfix so i may be biased | 17:59 |
| lbragstad | the only other case that's jogging my memory is asking operators to adjust character encoding in the database... which in my opinion is a bit different than this because it's a database setting out of keystone's control | 18:00 |
| cmurphy | yeah | 18:01 |
| knikolla | either way, we're out of time | 18:01 |
| knikolla | thanks all | 18:01 |
| vishakha | Sure, i will wait until we agree on something. Thanks knikolla lbragstad cmurphy | 18:01 |
| knikolla | #endmeeting | 18:01 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 18:01 | |
| openstack | Meeting ended Tue Jul 14 18:01:25 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/keystone/2020/keystone.2020-07-14-16.59.html | 18:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/keystone/2020/keystone.2020-07-14-16.59.txt | 18:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/keystone/2020/keystone.2020-07-14-16.59.log.html | 18:01 |
| *** tosky has left #openstack-meeting-alt | 18:03 | |
| *** diurnalist has joined #openstack-meeting-alt | 18:26 | |
| *** ricolin_ has quit IRC | 18:41 | |
| *** slaweq has joined #openstack-meeting-alt | 18:54 | |
| *** enriquetaso has quit IRC | 18:55 | |
| *** slaweq has quit IRC | 18:56 | |
| *** sangeet has quit IRC | 19:10 | |
| *** knomura has joined #openstack-meeting-alt | 19:11 | |
| *** knomura has quit IRC | 19:15 | |
| *** sangeet has joined #openstack-meeting-alt | 19:41 | |
| *** sangeet has left #openstack-meeting-alt | 19:41 | |
| *** yamamoto has joined #openstack-meeting-alt | 19:51 | |
| *** vishakha has quit IRC | 20:09 | |
| *** yamamoto has quit IRC | 20:51 | |
| *** e0ne has joined #openstack-meeting-alt | 20:57 | |
| *** enriquetaso has joined #openstack-meeting-alt | 21:05 | |
| *** markvoelker has joined #openstack-meeting-alt | 21:23 | |
| *** markvoelker has quit IRC | 21:27 | |
| *** yamamoto has joined #openstack-meeting-alt | 21:35 | |
| *** e0ne has quit IRC | 21:37 | |
| *** e0ne has joined #openstack-meeting-alt | 21:38 | |
| *** rdopiera has quit IRC | 21:59 | |
| *** knomura has joined #openstack-meeting-alt | 22:11 | |
| *** knomura has quit IRC | 22:16 | |
| *** e0ne has quit IRC | 22:21 | |
| *** jamesmcarthur has joined #openstack-meeting-alt | 22:32 | |
| *** vishalmanchanda has quit IRC | 22:39 | |
| *** rcernin has joined #openstack-meeting-alt | 22:45 | |
| *** rcernin has quit IRC | 22:51 | |
| *** rcernin has joined #openstack-meeting-alt | 22:51 | |
| *** yamamoto has quit IRC | 23:02 | |
| *** e0ne has joined #openstack-meeting-alt | 23:11 | |
| *** e0ne has quit IRC | 23:13 | |
| *** yamamoto has joined #openstack-meeting-alt | 23:33 | |
| *** jamesmcarthur has quit IRC | 23:36 | |
| *** jamesmcarthur has joined #openstack-meeting-alt | 23:48 | |
| *** enriquetaso has quit IRC | 23:51 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!