| *** changcheng has joined #starlingx | 03:04 | |
| *** yaawang_ has quit IRC | 05:18 | |
| *** yaawang_ has joined #starlingx | 05:20 | |
| *** hyunsikyang has quit IRC | 05:33 | |
| *** changcheng has quit IRC | 05:37 | |
| *** hyunsikyang has joined #starlingx | 06:00 | |
| *** hyunsikyang__ has joined #starlingx | 07:00 | |
| *** hyunsikyang has quit IRC | 07:02 | |
| *** bengates has joined #starlingx | 07:11 | |
| *** bengates has quit IRC | 07:21 | |
| *** bengates has joined #starlingx | 07:21 | |
| *** sgw has quit IRC | 09:09 | |
| *** hyunsikyang__ has quit IRC | 09:59 | |
| *** ijolliffe has joined #starlingx | 12:23 | |
| *** irclogbot_1 has quit IRC | 13:06 | |
| *** irclogbot_0 has joined #starlingx | 13:08 | |
| *** sgw has joined #starlingx | 13:24 | |
| sgw | Morning all | 13:33 |
|---|---|---|
| *** hoonetorg has quit IRC | 13:40 | |
| bwensley | morning | 13:40 |
| sgw | dpenney__: Morning updated that cve_support patch | 13:45 |
| dpenney__ | sge: ok, I'll look for it | 13:46 |
| sgw | dpenney__: jsomervi1, slittle1: It seems that the .gitreview was forgotten for the kernel repo, I posted a review for that. | 13:49 |
| dpenney__ | Davlet had that here: https://review.opendev.org/#/c/720004/ | 13:49 |
| dpenney__ | looks like it didn't merge? | 13:50 |
| sgw | Ah interesting, right it did not merge | 13:50 |
| sgw | that's why I posted a review, I guess I should have looked first | 13:51 |
| dpenney__ | there was an email about zuul setup missing... maybe the .zuul.yaml is the missing piece? | 13:51 |
| sgw | I was just checking for that also. | 13:51 |
| sgw | Will davlet handle that or do you want me to look into it? | 13:53 |
| dpenney__ | I'll ping him about it... I assume if there's no .zuul.yaml, that would explain why it hasn't been picked up? | 13:56 |
| sgw | Not sure I looked an another older repo and the .gitreview goes in correctly and then a .zuul.yaml file commit was added followed by tox.ini | 13:57 |
| sgw | I will check in the #opendev channel | 13:58 |
| *** hoonetorg has joined #starlingx | 13:59 | |
| -openstackstatus- NOTICE: Zuul is temporarily offline; service should be restored in about 15 minutes. | 14:00 | |
| sgw | looks like we do need to get the .zuul.yaml in first | 14:01 |
| sgw | dpenney__: should just be a copy of the integ .zuul.yaml file | 14:03 |
| dpenney__ | ok | 14:03 |
| sgw | Actually both could go together, Davlet could just amend his .gitreview commit | 14:03 |
| dpenney__ | probably don't need all the same jobs, since it's just kernel code | 14:05 |
| sgw | Right, I don't think we need all the devstack stuff | 14:06 |
| sgw | I had not looking in that .zuul.yaml file in a while | 14:06 |
| dpenney__ | I think just the openstack-tox-linters for the check/gate jobs, since there'd be no python... but presumably need the same templates and post... and cleanup unneeded stuff like devstack | 14:07 |
| sgw | Seems right | 14:11 |
| *** dpanech has joined #starlingx | 14:29 | |
| sgw | dpanech: morning! | 14:30 |
| sgw | dpanech: why did you -W your commit? | 15:18 |
| dpanech | Open issue re SSH keys, see my last comment | 15:19 |
| sgw | Did you check the other repos, I think they share the same keys, I am still trying to work out with the Infra team why the commit is not getting verified | 15:34 |
| dpanech | I checked a few, they seem to be different | 15:37 |
| sgw | Hmm, then you need to check with the person that set up those repo | 15:39 |
| *** hoonetorg has quit IRC | 15:46 | |
| dpenney__ | dpanech: I added a comment to your review... I think the "parent" is wrong for the mirror job, which may be why Zuul isn't able to verify it | 15:47 |
| dpenney__ | all other mirror jobs in the other repos set parent to upload-git-mirror, presumably a centrally defined job | 15:48 |
| -openstackstatus- NOTICE: Gerrit will be restarted to correct a misconfiguration which caused some git mirrors to have outdated references. | 15:48 | |
| dpenney__ | yeah, it's a reference to this job: https://opendev.org/zuul/zuul-jobs/src/branch/master/playbooks/upload-git-mirror/run.yaml | 15:49 |
| *** hoonetorg has joined #starlingx | 15:59 | |
| *** bengates has quit IRC | 16:02 | |
| *** bengates has joined #starlingx | 16:02 | |
| dpenney__ | odd... still not being picked up | 16:03 |
| *** bengates has quit IRC | 16:07 | |
| dpanech | FYI: I did a -W on that review because of the SSH keys thing | 16:09 |
| sgw | Hmm, then you need to check with the person that set up those repo with on github, I don't remember who that was | 16:23 |
| sgw | dpenney__: the kernel got caught up in the zuul configuration side changes they were making, it should get checked soon ish, I am tracking it on the #opendev channel | 16:26 |
| dpanech | ok | 16:26 |
| sgw | dpenney__: does Al ever show up here? | 16:30 |
| *** yaawang has joined #starlingx | 16:33 | |
| *** yaawang_ has quit IRC | 16:33 | |
| dpenney__ | right now, Al is working with an odd WFH setup in a rural area with low bandwidth... likely IRC is only setup on his office computer | 16:38 |
| dpenney__ | sgw: zuul picked up the kernel review on your recheck! woohoo! | 16:48 |
| sgw | dpenney__: Ok, I will follow-up on emalil | 16:48 |
| sgw | yuppers | 16:49 |
| sgw | dpanech: did you fix up the secrets? | 16:49 |
| sgw | dpenney__: not sure if anyone else is aware of the bandit work that was done in sysinv? | 16:50 |
| dpanech | Still talking to people | 16:51 |
| *** rcw has joined #starlingx | 16:51 | |
| dpenney__ | sgw: not sure about the bandit stuff... it may just be Al | 16:52 |
| sgw | thanks | 16:52 |
| dpanech | sgw: SSH keys now sorted out in https://review.opendev.org/#/c/720004/ | 17:25 |
| dpanech | sgw: wait they are saying I have to create target repo on github as well ... | 17:28 |
| *** portdirect has quit IRC | 17:30 | |
| *** portdirect has joined #starlingx | 17:31 | |
| sgw | Makes sense! | 17:31 |
| dpanech | sgw: ok it's done now | 17:56 |
| dpanech | sgw: now Dariush is checking write permissions in Github for the new repo. I have no direct access to that... | 17:58 |
| dpanech | sgw: ok, I think it's all done now | 18:03 |
| *** slittle1 has quit IRC | 18:17 | |
| *** slittle1 has joined #starlingx | 18:21 | |
| *** slittle1 has quit IRC | 18:33 | |
| *** slittle1 has joined #starlingx | 18:41 | |
| sgw | Hi dpenney__: Wayback machine needed! You added python-daemon to logmgmt a long time ago! When you did you use /dev/null for stdin/stdout/stderr, well python3 used different semancitc in Open and is failing. The Python3 guy are proposing adding a patch to ptyhon3-daemon (more tech debt and non-standard) to deal with, do you know if there is alternative? | 19:33 |
| sgw | dpanech: looks like there was some Zuul issues, I recently saw the same POST_FAILURE with zuul-jobs and am working on an answer there. For the tox one you need to add a tox.ini file also to this commit, I should have seen that sooner, sorry. | 19:35 |
| dpenney__ | sgw: logmgmt is just a basic daemon... I wouldn't think we should need to modify any central pkg to support it... if it's doing things in an old and weird way, I would assume it would be easier to just update it to behave in a python3 manner, rather than vice-versa | 19:41 |
| dpenney__ | it was probably the second thing I ever wrote in python | 19:42 |
| sgw | dpenney__: see this review: https://review.opendev.org/#/c/717211/, the root cause of needing to patch the python3-daemon is the usage of /dev/null and the way open changed for unbuffered files we need a less invasive solution. | 19:46 |
| sgw | I agree with you we should not be patching a central package which is why I ahve been pushingback on the review so hard! | 19:47 |
| sgw | There is also this one: https://review.opendev.org/#/c/714813/ which I am not wild about, but have not figured out what to suggest as an alterniatve yet! | 19:49 |
| *** ijolliffe has quit IRC | 19:53 | |
| *** sgw has quit IRC | 21:09 | |
| *** sgw has joined #starlingx | 21:18 | |
| sgw | Hi Team, has something changed with the build-openstack-docs-pti template? We are seeing a POST_FAILURE in the starlingx/zuul-jobs repo with this change: https://review.opendev.org/721294 | 21:19 |
| sgw | This repo does not generate any docs | 21:19 |
| sgw | Oops wrong channel, will move this to Infra! | 21:20 |
| *** rcw has quit IRC | 21:30 | |
| *** hyunsikyang has joined #starlingx | 21:40 | |
| sgw | dpenney__: slittle1: Please see the zuul-jobs change I just proposed | 22:58 |
| sgw | dpanech: you will need to remove the build-openstack-docs-pti from the kernel .zuul.yaml for now until we add release note generation, that will get you based the POST_FAILURES | 22:59 |
| dpanech | its already removed, its failing due to missing tox.ini, I'll work on that as soon as I can | 23:08 |
| sgw | Right, I mentioned that earlier | 23:14 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!