| *** pojadhav is now known as pojadhav|ruck | 01:53 | |
| *** lajoskatona_ is now known as lajoskatona | 03:17 | |
| *** ysandeep- is now known as ysandeep | 03:26 | |
| opendevreview | chandan kumar proposed zuul/zuul-jobs master: Setup RDO gpg keys and repo for CentOS distros only https://review.opendev.org/c/zuul/zuul-jobs/+/837850 | 03:58 |
|---|---|---|
| *** pojadhav is now known as pojadhav|ruck | 04:18 | |
| *** ysandeep is now known as ysandeep|afk | 05:36 | |
| *** ysandeep|afk is now known as ysandeep | 06:08 | |
| *** marios_ is now known as marios | 07:04 | |
| *** pojadhav|ruck is now known as pojadhav|lunch | 07:33 | |
| *** pojadhav|lunch is now known as pojadhav|ruck | 08:30 | |
| *** chandankumar is now known as chkumar|ruck | 08:33 | |
| *** ysandeep is now known as ysandeep|lunch | 08:37 | |
| *** ysandeep|lunch is now known as ysandeep | 08:54 | |
| *** ysandeep is now known as ysandeep|coffee | 10:54 | |
| *** dviroel__ is now known as dviroej | 11:26 | |
| *** ysandeep|coffee is now known as ysandeep | 11:33 | |
| *** dviroej is now known as dviroel | 11:39 | |
| *** ysandeep is now known as ysandeep|afk | 12:04 | |
| *** rcastillo is now known as rcastillo|ruck | 13:01 | |
| *** ysandeep|afk is now known as ysandeep | 13:02 | |
| *** chkumar|ruck is now known as chandankumar | 14:32 | |
| *** dviroel is now known as dviroel|lunch | 15:20 | |
| *** marios is now known as marios|out | 15:34 | |
| *** ysandeep is now known as ysandeep|out | 15:36 | |
| clarkb | its looking like tomorrow might not have a very large crowd for our meeting. I'm tempted to cancel it and we can use that time for less formal discussions if anyone is around and has something to bring up | 15:38 |
| clarkb | fungi: any thoughts on bindep and git review releases today/tomorrow/soon? I'm happy to push them up and keep an eye out for trouble if others are at least around to double check the tag inputs. | 15:40 |
| clarkb | bindep: 5d6a6c58b0eda84615d8d57fd2fb3c58768cc61b tag as 2.11.0 due to new functionaltiy supporting alma linux | 15:40 |
| clarkb | looks like for bindep we're still waiting on https://review.opendev.org/c/opendev/git-review/+/823318 | 15:41 |
| clarkb | er for git-review | 15:42 |
| fungi | yeah, i'm happy to push tags and send release announcements today, last week was just a bit crazy with the git excitement and i didn't want to push a release on friday of a holiday weekend | 15:42 |
| fungi | we could remove frickler's workflow -1 from 823318 and assume ianw's -1 is a +2 with the followup change already approved | 15:43 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove health and subunit works from inventory https://review.opendev.org/c/opendev/system-config/+/838320 | 15:45 |
| clarkb | gmann: fungi ^ fyi noticed that when I started looking at cleaning up the subunit workers. I think landing that soon would be good but not necessary to proceed on the other health cleanups | 15:46 |
| clarkb | fungi: yes I think that ianw's followup is an implicit method of addressing that concern | 15:47 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove config management for subunit2sql workers https://review.opendev.org/c/opendev/system-config/+/838322 | 15:54 |
| clarkb | fungi: gmann ^ and that is the subunit2sql worker removal from config management | 15:55 |
| opendevreview | Merged opendev/system-config master: Retire opendev/puppet-openstack_health: remove from system-config https://review.opendev.org/c/opendev/system-config/+/836879 | 15:55 |
| chandankumar | Hello infra, can we get this https://review.opendev.org/c/zuul/zuul-jobs/+/837850 merged? please have a look, when free, thanks! | 16:04 |
| clarkb | chandankumar: one note: zuul-jobs has its own core team and they hang out in the zuul room on matrix. That said I did have a question about that, what about rocky/alma/openeuler which are not RHEL but would still trip the old condition. Are they not expected to work with the old code so the new code isn't a regression for them? | 16:06 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove configuration management for ELK stack https://review.opendev.org/c/opendev/system-config/+/838324 | 16:06 |
| clarkb | fungi: gmann ^ and that can land once we stop zuul sneding gearman events to the geard | 16:06 |
| clarkb | I set the depends on for that so we should be good to land those as dependencies land | 16:07 |
| chandankumar | clarkb: RHEL ships openvswitch via different repos, earlier we were mixing RHEL and CentOS content, It fixes the issue and installs it on CentOS so it might work for rocky/alma/openeuler if CentOS repos get enabled or others adds conditions for enabling these repos on those distros | 16:09 |
| clarkb | chandankumar: right thats my concern. You are disabling it for all of those distros too | 16:10 |
| clarkb | it might be better to exclude rhel instaed | 16:10 |
| clarkb | and then those distros can update if they are broken | 16:10 |
| opendevreview | Merged opendev/system-config master: Remove health and subunit works from inventory https://review.opendev.org/c/opendev/system-config/+/838320 | 16:11 |
| chandankumar | ok let me update this condition | 16:13 |
| opendevreview | chandan kumar proposed zuul/zuul-jobs master: Setup RDO gpg keys and repo for CentOS distros only https://review.opendev.org/c/zuul/zuul-jobs/+/837850 | 16:14 |
| chandankumar | clarkb: ^^ | 16:15 |
| clarkb | chandankumar: I think the condition in https://review.opendev.org/c/zuul/zuul-jobs/+/837850/3/roles/multi-node-bridge/templates/zuul-multi-node-bridge-ovs.repo.j2 needs to update too? | 16:16 |
| opendevreview | chandan kumar proposed zuul/zuul-jobs master: Donot Setup RDO gpg keys and repo for RHEL and Fedora https://review.opendev.org/c/zuul/zuul-jobs/+/837850 | 16:21 |
| chandankumar | clarkb: ^^ removed centos conditionals from repo | 16:21 |
| gmann | clarkb: so for ELK, we are stopping the old server now ? or I think we said to keep for 1-2 month and give people time to move to new dashbaord and feedback if any ? | 16:34 |
| clarkb | gmann: we said after yoga we would remove it | 16:39 |
| clarkb | and it broke during the release and slowed it down | 16:39 |
| clarkb | the problem ultimately is that no one is around trying to make it work anymore and it is causing problems for things like the release | 16:39 |
| clarkb | I think if it hadn't impacted the release then I would be fine trying to keep it around longer but it did impact the reelase and if no one is workign to fix it we should remove it? | 16:40 |
| clarkb | it was always a "if it doesn't cause problems we'll keep it up" statement. It has been causing problems | 16:40 |
| gmann | because we have not send on ML about the new dashbaord and documentation patch of url and its credentials to use is not merged. | 16:41 |
| gmann | may be we can wait for an month and finish ^^ work. I am not sure if dpawlik is online/back to work so that we can merge those things soon and send on ML. | 16:43 |
| clarkb | ok I don't think the existing system ahs been working is the problem | 16:44 |
| clarkb | it was definitely not working during the release and fungi had to restart some stuff | 16:44 |
| clarkb | whether or not it is happy now I don't know | 16:44 |
| gmann | clarkb: ok, it be good if we can wait, at least until dpawlik patches merged and he announce the new dashboard on ML. | 16:49 |
| clarkb | ok even if the dat isn't populating and jobs like releases are impacted? | 16:50 |
| clarkb | thats my concern. We can say this is up but if it isn't actually functional and things that we expect to be functional like release jobs are impacted then maybe we need to prioritize | 16:51 |
| fungi | note it's not just release jobs, it's basically slowing down all jobs because of the gearman submission taking minutes | 16:51 |
| fungi | so consuming a non-negligible proportion of our test resource quota by making all jobs incur that additional time penalty | 16:52 |
| clarkb | basically what we aleays said was we'd try to make it last until yoga but it was dependent on it not causing problems | 16:52 |
| clarkb | it is causing problems now | 16:52 |
| clarkb | it is after yoga | 16:52 |
| clarkb | I think we should turn it off | 16:52 |
| clarkb | we've asked for help to manage this system in opendev and not got it. We've done a fair bit of assistance to help with the migration to opensearch. I don't think we should pretend the existing service is happy and functional if it is not even if the openasrch stuff isn't quite ready | 16:53 |
| clarkb | if the release was not impacted then I wouldn't mind leaving it up a bit | 16:54 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove config management for subunit2sql workers https://review.opendev.org/c/opendev/system-config/+/838322 | 17:04 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove configuration management for ELK stack https://review.opendev.org/c/opendev/system-config/+/838324 | 17:04 |
| clarkb | testing caught that logstash puppetry depends on subunit2sql puppet module so can't remove that puppet module from stuff until we remove logstash | 17:04 |
| clarkb | hrm related ti seems liek we've got another backlog of tag jobs | 17:12 |
| clarkb | on the whole not very busy though and our node usage is fairly low | 17:12 |
| *** dviroel|lunch is now known as dviroel | 17:23 | |
| clarkb | https://review.opendev.org/c/opendev/system-config/+/838322 is hapyp now which removes the subunit workers | 17:29 |
| opendevreview | Merged openstack/project-config master: Retire openstack-health projects: remove project from infra https://review.opendev.org/c/openstack/project-config/+/836709 | 18:11 |
| opendevreview | Merged zuul/zuul-jobs master: Donot Setup RDO gpg keys and repo for RHEL and Fedora https://review.opendev.org/c/zuul/zuul-jobs/+/837850 | 18:29 |
| opendevreview | Merged opendev/git-review master: Fix submitting signed patches https://review.opendev.org/c/opendev/git-review/+/823318 | 18:36 |
| fungi | i guess we can tag git-review today with those in | 18:39 |
| fungi | i'll get to work on both that and bindep releases shortly | 18:39 |
| clarkb | fungi: the second git-review change hasn't landed yet though | 18:40 |
| clarkb | but yes once that lands I think we can ship them with tags | 18:40 |
| fungi | yeah, still waiting on it | 18:40 |
| opendevreview | Merged opendev/git-review master: Enforce minimum git version https://review.opendev.org/c/opendev/git-review/+/837451 | 18:45 |
| clarkb | there it is | 18:45 |
| clarkb | I'm going to single core approve https://review.opendev.org/c/opendev/system-config/+/838322 to remove the subunit2sql workers since that seems to be not controversial and I don't know that I'll get another set of eyeballs for a few days | 18:46 |
| fungi | thanks | 18:46 |
| clarkb | I'll leave the ELK removal change up for a bit longer | 18:46 |
| clarkb | and others can chime in if they want to figure out what is going on with that to help gmann out. | 18:46 |
| clarkb | fungi: any objection to cnacelling tomorrow's meeting? | 18:47 |
| fungi | none, since i don't think i'll be able to make it myself | 18:47 |
| fungi | would be sort of silly for me to insist there be a meeting i'm unable to attend ;) | 18:47 |
| fungi | bindep commit 5d6a6c58b0eda84615d8d57fd2fb3c58768cc61b (HEAD, tag: 2.11.0, origin/master, origin/HEAD, gerrit/master, refs/changes/18/837118/5) | 18:49 |
| fungi | git-review commit 82a2c8df2e6deada5770d1ea48e5d0ce0eaeab36 (HEAD, tag: 2.3.0, origin/master, origin/HEAD, gerrit/master) | 18:49 |
| fungi | those look right? | 18:49 |
| clarkb | looking | 18:50 |
| clarkb | 5d6a6c58b0eda84615d8d57fd2fb3c58768cc61b and 2.11.0 look right to me for bindep | 18:51 |
| clarkb | git review's log is a bit more extensive let me read it over really quickly | 18:52 |
| clarkb | yes 82a2c8df2e6deada5770d1ea48e5d0ce0eaeab36 as 2.3.0 looks good on git-review | 18:53 |
| fungi | cool, pushing both now in that case | 18:59 |
| fungi | once those are done, i'll see if i need to do anything to refresh the release notes pages for them | 19:00 |
| opendevreview | Merged opendev/system-config master: Remove config management for subunit2sql workers https://review.opendev.org/c/opendev/system-config/+/838322 | 19:00 |
| fungi | release jobs completed, though the release notes predictably still list the unreleased state as docs were not rebuilt with the tags present | 19:14 |
| fungi | looks like we're relying on opendev-promote-docs in the promote pipeline, and reenqueuing the most recent changes into promote is probably not advisable? | 19:15 |
| fungi | yeah, it would just republish the same docs already built in the gate pipeline for those changes | 19:18 |
| fungi | not build new docs | 19:18 |
| Clark[m] | We can land some noopy changes just to build new docs | 19:20 |
| fungi | i've got one better | 19:20 |
| opendevreview | Jeremy Stanley proposed opendev/bindep master: Run opendev-publish-tox-docs on release https://review.opendev.org/c/opendev/bindep/+/838340 | 19:25 |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Run opendev-publish-tox-docs on release https://review.opendev.org/c/opendev/git-review/+/838341 | 19:25 |
| fungi | mmm, though i wonder if that's going to publish under something other than latest | 19:26 |
| fungi | target_dict: "{{ afs.targets.tag }}" | 19:28 |
| fungi | yeah, we probably want one which relies on zuul's ability to guess the most appropriate branch from the tag? | 19:28 |
| fungi | i suspect that's going to need a new job definition | 19:29 |
| clarkb | ya I think the zuul and nodepool doc build suffer from this too | 19:33 |
| clarkb | they push to the tag but then you need something on master to lupdate the latest/ path | 19:33 |
| fungi | right. the docs job they use has the same parent as opendev-publish-tox-docs | 19:33 |
| fungi | (opendev-publish-tox-docs-base) | 19:33 |
| fungi | though how does the branch-guessing-from-tag work again? or did that not actually end up implemented? looking at a tag-triggered build, i don't see any zuul var which reflects the likely branch for that tag | 19:35 |
| fungi | i think openstack stable releases rely on this for their releasenotes jobs, so maybe i can track it down there | 19:36 |
| fungi | ahh, no they don't | 19:40 |
| fungi | every python3-releasenotes build generates branch-specific release notes for all available branches, and republishes that entire tree | 19:41 |
| fungi | which i should have remembered, since that's what causes them to sometimes race one another when two tags are created on different branches at roughly the same moment | 19:41 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Switch Refstack image over to python3.9 https://review.opendev.org/c/opendev/system-config/+/838344 | 19:43 |
| clarkb | I've been thinking it would be good for us to get to python3.9 then we can add python3.10 images | 19:43 |
| clarkb | I'm hopeful that we can also drop the 3.7 and maybe even 3.8 base images onces we get everything on 3.9 | 19:43 |
| clarkb | that is a first change to work towards that. I think zuul and nodepool may also by on 3.8 so some work to be done elsewhere | 19:44 |
| clarkb | There is some other general zuul config cleanup related to that I'll work on next | 19:44 |
| clarkb | mnaser: are you using our base python images? I'm curious if you are still using the buster images or if you ahve moved everything to bullseye. Also what is your oldest python version that you are using. I'm hopeful I can delete all of the buster images and the python3.7 images | 19:47 |
| opendevreview | Jeremy Stanley proposed opendev/bindep master: Run opendev-publish-tox-docs on release https://review.opendev.org/c/opendev/bindep/+/838340 | 19:48 |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Run opendev-publish-tox-docs on release https://review.opendev.org/c/opendev/git-review/+/838341 | 19:48 |
| fungi | oh, it looks like we an override the "tag" jobvar for the parent. clarkb ^ is that how you do it? | 19:48 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Fixup some image dependencies in our zuul configs https://review.opendev.org/c/opendev/system-config/+/838345 | 19:49 |
| *** dviroel is now known as dviroel|biab | 19:49 | |
| clarkb | fungi: I think you may need to indent vars: an extra level? | 19:50 |
| fungi | obviously not a solution for multi-branch projects, but for single-branch projects like bindep and git-review it should suffice, i think | 19:50 |
| fungi | oh, yep | 19:50 |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Run opendev-publish-tox-docs on release https://review.opendev.org/c/opendev/git-review/+/838341 | 19:50 |
| opendevreview | Jeremy Stanley proposed opendev/bindep master: Run opendev-publish-tox-docs on release https://review.opendev.org/c/opendev/bindep/+/838340 | 19:50 |
| fungi | but you can override job vars like that in a variant, right? even a variant of a job inheriting those vars from an abstract parent? | 19:51 |
| clarkb | I think so | 19:51 |
| clarkb | it should merge the vars together with the parents with children overriding iirc | 19:52 |
| fungi | i guess we can find out the next time we tag one of those | 19:52 |
| clarkb | fungi: https://review.opendev.org/c/openstack/project-config/+/837621 is an easy one that I think we can land even while the logstash geard remains up for the next little bit | 19:55 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove our buster python images https://review.opendev.org/c/opendev/system-config/+/838347 | 20:00 |
| clarkb | I'm hoping ^ will shake out things that are still using those images if they exist | 20:00 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove python3.7-bullseye docker images https://review.opendev.org/c/opendev/system-config/+/838348 | 20:03 |
| clarkb | fungi: https://review.opendev.org/c/opendev/system-config/+/831441 another straightforward chagne if you have time (this cleans up our clouds.yaml to remove the airship cloud stuff) | 20:11 |
| opendevreview | Merged openstack/project-config master: Remove geard graphing from zuul-status dashboard https://review.opendev.org/c/openstack/project-config/+/837621 | 20:11 |
| clarkb | oh heh a number of those chagnes have piled up behind the one that updated the inventory. I guess that isn't a major problem. Just slow | 20:21 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Switch Refstack image over to python3.9 https://review.opendev.org/c/opendev/system-config/+/838344 | 20:23 |
| fungi | infra-prod-remote-puppet-else failed deploy for 838320 | 20:26 |
| fungi | i can probably take a look after dinner to see why | 20:27 |
| clarkb | fungi: it looks like ti failed on logstash01.o.o trying to do stuff in hte kibana repo | 20:28 |
| clarkb | more reason to turn things off ? | 20:28 |
| clarkb | error: some local refs could not be updated; try running 'git remote prune origin' to remove any old, conflicting branches | 20:28 |
| clarkb | but double check me on that to make sure nothing else is unhappy | 20:28 |
| clarkb | my kids get out of school early tomorrow and that conflicts with the meeting time. More reason to cancel. I'll wait on sending an email about that until ianw's theoretical morning would start if he ends up workign today to chime in though | 20:30 |
| clarkb | well https://review.opendev.org/c/opendev/system-config/+/838347 didn't explode. Maybe this is safe afterall. /me checks codesearch | 20:32 |
| clarkb | hrm codesearch immediately finds that gear and storyboard rely on that maybe | 20:33 |
| opendevreview | Clark Boylan proposed opendev/gear master: Update the docker image to python3.9 and buster https://review.opendev.org/c/opendev/gear/+/838402 | 20:36 |
| opendevreview | Clark Boylan proposed opendev/storyboard master: Update container image to bullseye and python3.9 https://review.opendev.org/c/opendev/storyboard/+/838403 | 20:39 |
| clarkb | I'm not finding any other uses which means we should be good once those two changes land. Let me set a depends on | 20:40 |
| *** dviroel|biab is now known as dviroel | 20:42 | |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove our buster python images https://review.opendev.org/c/opendev/system-config/+/838347 | 20:42 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove python3.7-bullseye docker images https://review.opendev.org/c/opendev/system-config/+/838348 | 20:42 |
| clarkb | looks like storyboard is broken due to a jinja2 thing. I'll see if I have time after school run to look at it more closely | 20:55 |
| opendevreview | Merged opendev/system-config master: Remove airship-kna1 from bridge clouds.yaml https://review.opendev.org/c/opendev/system-config/+/831441 | 21:06 |
| *** dviroel is now known as dviroel|afk | 21:37 | |
| fungi | i think i looked into the storyboard problem already... checking my swiss cheese memory | 21:40 |
| fungi | doesn't look like i pushed any fix if so | 21:40 |
| fungi | https://review.opendev.org/715229 seems related to container base images though | 21:41 |
| fungi | clarkb: 838340 and 838341 are both passing if you're up for approving. their merging should update docs as a side effect, probably? then i can send release announcements without worrying about people thinking there haven't been releases | 21:50 |
| clarkb | fungi: in this case the storyboard issue are all outside teh containers | 21:59 |
| clarkb | fungi: ya let me take a look | 21:59 |
| clarkb | ianw: wondering if you think we should have a meeting tomorrow. I think it may just be the two of us and I've got kids that need picking up from school early tomorrow during the meeting time | 22:00 |
| *** prometheanfire is now known as Guest2246 | 22:00 | |
| ianw | sorry just reading backwards. skipping is fine if nobody has anything. i'm assuming there was some fallout of git things while I was Easter-ing to catch up on :) | 22:01 |
| clarkb | ianw: there was some and its a bit all over as to what appraoch peopel took. But I think the issue is understood and people know how to address it | 22:01 |
| ianw | thanks for the fix to source-repositories | 22:05 |
| clarkb | fungi: left a comment on https://review.opendev.org/c/opendev/bindep/+/838340 I didn't realize that was part of an afs secrets dict and thought i t was a normal var. I'm not longer certain this will do anything | 22:11 |
| clarkb | I think the comment applies to both changes | 22:11 |
| fungi | oh. hrm | 22:13 |
| clarkb | fungi: ya and if you look at opendev/base-jobs/playbooks/tox-docs/publish.yaml it does target_dir: "{{ target_dict.path.format(zuul=zuul) }}" | 22:15 |
| fungi | yeah, i missed the indentation in the description, i guess it's something like afs[targets][tag]? | 22:15 |
| clarkb | so its taking the value out of zuul.stuff | 22:15 |
| clarkb | and when you set vars on a job I think they are just normal inventory vars not part of the zuul dict either | 22:15 |
| clarkb | fungi: ya and sorry its literally tag: path: /some/path/here | 22:16 |
| fungi | oh, right: "This is expected to be a dict with a key value pair: `path: the full docs publication path to use if the job is run on a tag.`" | 22:17 |
| clarkb | looking at storyboard more closely it apepars there are linter issues, sqlalchemy issues, and also the jinja2 docs thing :/ | 22:22 |
| clarkb | I'll start on a likely incimplete change but would be good to see some things happier | 22:22 |
| fungi | i'd also be fine forcing the next storyboard change to work with newer container base images | 22:22 |
| clarkb | the problem isn't the container images its storyboard itself | 22:26 |
| fungi | so anyway, to do docs republish on tag... we need a different secret i guess (reencoding the same kerberos ticket)? | 22:26 |
| clarkb | the jobs related to container buidls seem fine | 22:26 |
| fungi | clarkb: right, i mean i'd be fine merging a change elsewhere which leaves storyboard jobs broken until we update the container builds to suit | 22:27 |
| clarkb | oh I suee | 22:28 |
| fungi | i guess i could work through tag-triggered docs builds by pushing tags to opendev/sandbox | 22:30 |
| clarkb | fungi: ya I think we need a different secret that sets a different path value that just hardcodes to /latest rather than /{zuul.tag} or whatever it does now | 22:32 |
| fungi | okay cool, i'll try to figure that out | 22:32 |
| fungi | i guess i can just duplicate the encrypted token since it'll be for the same project, no need to reencode | 22:33 |
| opendevreview | Clark Boylan proposed opendev/storyboard master: Omnibus fixups for storyboard testing and CI https://review.opendev.org/c/opendev/storyboard/+/838409 | 22:41 |
| clarkb | ok I'm going to send email skipping tomorrows meeting | 22:41 |
| clarkb | infra-root https://review.opendev.org/c/opendev/system-config/+/838345 and https://review.opendev.org/c/opendev/system-config/+/838344 should be easy reviews. I'm going to WIP https://review.opendev.org/c/opendev/base-jobs/+/837619 since there is some contention over whether or not we need to keep running that now ... | 22:46 |
| ianw | hrm, i'm getting a 502 from paste trying to publish some of the long query logs we've seen | 22:54 |
| clarkb | there is a size limit. IN the past it would just truncate and silently fail returning success but maybe now it doesn't do that? | 22:55 |
| ianw | pymysql.err.DataError: (1406, "Data too long for column 'code' at row 1") | 22:55 |
| ianw | i wonder if this is a result of the switch to our own hosted db | 22:55 |
| ianw | | code | text | YES | | NULL | | | 22:57 |
| ianw | so that would be 64k i guess | 22:58 |
| fungi | i'd be fine increasing the size limit for that column if it's easy | 23:00 |
| opendevreview | Clark Boylan proposed opendev/storyboard master: Omnibus fixups for storyboard testing and CI https://review.opendev.org/c/opendev/storyboard/+/838409 | 23:02 |
| ianw | i think it's probably a decent size, it would just be better if it didn't 502 over the limit | 23:02 |
| ianw | next size up is 16mb which seems too big | 23:02 |
| clarkb | I have to pop out for kids swim calss now but will check oin that storyboard change after. I think it is actually pretty close now | 23:09 |
| *** dviroel|afk is now known as dviroel|out | 23:51 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!