| rosmaita | dansmith: https://review.opendev.org/c/openstack/devstack/+/842390 | 01:20 |
|---|---|---|
| opendevreview | Pranali Deore proposed openstack/glance master: Correction in zed milestone 1 relesenotes https://review.opendev.org/c/openstack/glance/+/842457 | 05:03 |
| opendevreview | Pranali Deore proposed openstack/glance master: Correction in zed milestone 1 relesenotes https://review.opendev.org/c/openstack/glance/+/842457 | 06:00 |
| abhishekk | sentinel | 06:16 |
| *** akekane_ is now known as abhishekk | 06:46 | |
| opendevreview | Abhishek Kekane proposed openstack/glance master: [WIP] Immediate caching of an image https://review.opendev.org/c/openstack/glance/+/842070 | 07:29 |
| opendevreview | Rajat Dhasmana proposed openstack/glance master: Revert "Override GLANCE_USE_IMPORT_WORKFLOW in cinder jobs" https://review.opendev.org/c/openstack/glance/+/842400 | 08:26 |
| opendevreview | Rajat Dhasmana proposed openstack/glance-specs master: Add new location APIs https://review.opendev.org/c/openstack/glance-specs/+/840882 | 11:16 |
| opendevreview | Rajat Dhasmana proposed openstack/glance-specs master: Add new location APIs https://review.opendev.org/c/openstack/glance-specs/+/840882 | 11:18 |
| opendevreview | Rajat Dhasmana proposed openstack/glance-specs master: Add new location APIs https://review.opendev.org/c/openstack/glance-specs/+/840882 | 11:18 |
| abhishekk | rosmaita, around? | 13:19 |
| rosmaita | hello | 13:19 |
| abhishekk | sorry to bother you | 13:19 |
| rosmaita | am i missing the meeting? | 13:19 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/842400 | 13:19 |
| abhishekk | it is after 45 minutes from now | 13:19 |
| rosmaita | ok | 13:20 |
| abhishekk | there is requirement related error | 13:20 |
| rosmaita | i will look real quick, clarkb told me how to troubleshoot these a few weeks ago | 13:20 |
| rosmaita | and if i remember, i will tell you too | 13:21 |
| abhishekk | ack | 13:21 |
| abhishekk | cool | 13:21 |
| opendevreview | Pranali Deore proposed openstack/glance master: Correction in zed milestone 1 releasenotes https://review.opendev.org/c/openstack/glance/+/842457 | 13:26 |
| rosmaita | abhishekk: this is going to take a while | 13:34 |
| abhishekk | rosmaita, ack | 13:34 |
| abhishekk | we are stuck due to this issue, I will also try to find alternative | 13:35 |
| rosmaita | well, if i change upper constraints to 'glance-store===3.0.0', 'tox -e py36 --notest' installs fine | 13:39 |
| abhishekk | ok | 13:40 |
| rosmaita | did we change some requirements/test-requirements recently? | 13:40 |
| abhishekk | no | 13:40 |
| abhishekk | let me confirm though | 13:40 |
| rosmaita | could be an indirect dependency, since 3.6 is officially EPL, some projects are dropping support for it | 13:40 |
| rosmaita | (i mean, EOL) | 13:41 |
| abhishekk | yes, glance-store also dropped support for it | 13:41 |
| abhishekk | We just added upper constraint to install command | 13:42 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/839786 | 13:42 |
| abhishekk | other than this there is no change in requirements/test-requirements | 13:42 |
| rosmaita | why are we testing with released glance-store instead of glance-store from master? | 13:44 |
| rosmaita | (not related to this problem, i'm just wondering) | 13:44 |
| abhishekk | this is from beginning | 13:45 |
| rosmaita | yeah, forget that, i'm just grasping for straws | 13:46 |
| pdeore | abhishekk, rosmaita, jokke_, dansmith, croelandt, mrjoshi glance weekly meeting in 10 minutes at #openstack-meeting | 13:50 |
| abhishekk | ty! | 13:50 |
| rosmaita | abhishekk: as an emergency, we can change glance/requirements.txt to have the line 'glance-store>=2.3.0,<4.0.0 # Apache-2.0' | 13:51 |
| rosmaita | i can get the tox py36 environment to build with that change and current upper constraints | 13:51 |
| abhishekk | ack | 13:52 |
| rosmaita | actually, let me check that | 13:52 |
| rosmaita | not true, i forgot to -r my tox build | 13:52 |
| abhishekk | The conflict is caused by: | 13:53 |
| abhishekk | The user requested glance-store<4.0.0 and >=2.3.0 | 13:53 |
| abhishekk | The user requested (constraint) glance-store===4.0.0 | 13:53 |
| rosmaita | yeah | 13:54 |
| abhishekk | can we check with infra or release team? | 13:54 |
| rosmaita | i think let's ask in #opendev | 13:55 |
| rosmaita | which is the infra team | 13:55 |
| abhishekk | ok | 13:55 |
| abhishekk | rosmaita, in this case can we mark py36 jobs non voting? | 14:09 |
| rosmaita | i am so freaking confused at this point i don't know what is going on | 14:11 |
| abhishekk | same here | 14:11 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: DNM: Remove py36 jobs support https://review.opendev.org/c/openstack/glance/+/842549 | 14:14 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: DNM: Remove py36 jobs support https://review.opendev.org/c/openstack/glance/+/842549 | 14:16 |
| rosmaita | abhishekk: we are still running yoga jobs | 14:16 |
| rosmaita | https://opendev.org/openstack/glance/src/branch/master/.zuul.yaml#L324 | 14:16 |
| rosmaita | https://review.opendev.org/c/openstack/glance/+/841695 | 14:17 |
| abhishekk | hmm, https://review.opendev.org/c/openstack/glance/+/841350/3/.zuul.yaml | 14:18 |
| rosmaita | i don't understand why https://review.opendev.org/c/openstack/glance/+/841350 has a depends-on? | 14:22 |
| abhishekk | because py38 fips definition is not present in openstack-zuul-jobs | 14:24 |
| abhishekk | same goes for openstack-tox-py38-with-oslo-master | 14:24 |
| abhishekk | DNM Remove py36 jobs support https://review.opendev.org/c/openstack/glance/+/842549 | 14:25 |
| abhishekk | this I am removing all py36 jobs from glance including periodic but it is giving error as I m making fips job non voting | 14:25 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: DNM: Remove py36 jobs support https://review.opendev.org/c/openstack/glance/+/842549 | 14:31 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: DNM: Remove py36 jobs support https://review.opendev.org/c/openstack/glance/+/842549 | 14:39 |
| abhishekk | So this is how we can get rid of this requirement and constraint conflict | 14:43 |
| abhishekk | 1 remove py36 support and related jobs | 14:43 |
| abhishekk | 2 move to use zed-testing env | 14:43 |
| abhishekk | 3 make fips job non voting atm as we don't know whether it has support for py38 (it does not have a master job for py38 in openstack-zuul-jobs) | 14:44 |
| abhishekk | This all is covered in above DNM job | 14:44 |
| abhishekk | once this patch merged and we have new definitions for py38 in openstack-zuul-jobs (https://review.opendev.org/#/q/I3be69c3b42e9920e0de0299ae7e97026cdfd12e3) then we can add periodic jobs back for py38 | 14:46 |
| rosmaita | ok, that sounds like a way forward | 14:47 |
| abhishekk | I will change the commit message once everything passes | 14:47 |
| rosmaita | abhishekk: what patch(s) should i be watching? | 15:02 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/842549 | 15:02 |
| jokke_ | abhishekk: rosmaita: the pypi glance_store 4.0.0 does not have the Py3.6 tag anymore ... could it be that what is putting pip off? | 15:02 |
| abhishekk | jokke_, because we dropped support for py36 and py37 | 15:03 |
| jokke_ | abhishekk: correct | 15:03 |
| *** pdeore is now known as pdeore|afk | 15:03 | |
| rosmaita | yes, and even worse it has python_requires = >=3.8 | 15:04 |
| abhishekk | Lesson learned, we need to be very very careful when dropping any python support | 15:04 |
| jokke_ | and it's the py3.6 jobs (unit and functional) that are breaking | 15:04 |
| abhishekk | fips as well | 15:05 |
| jokke_ | abhishekk: and I think that fips job is still on centos 8 stream which is <3.8, right? | 15:06 |
| abhishekk | yeah | 15:06 |
| jokke_ | yup, so we need to drop those tests | 15:06 |
| abhishekk | atm I made it non-voting | 15:06 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/841034 | 15:07 |
| abhishekk | I guess this one patch is ok to keep cinder-fips job | 15:07 |
| jokke_ | or we can put conditionals into the upper constrains for >py3.8< | 15:07 |
| abhishekk | but functional-py36-fips needs to be dropped until we have functional-py38-fips in zuul templates | 15:08 |
| jokke_ | yeah | 15:08 |
| abhishekk | I will drop it with a note in the commit message rather than keeping it non voting | 15:08 |
| jokke_ | I'd say just drop them both | 15:10 |
| jokke_ | pointless to eat up infra resources for something we know will always fail from now to eternity | 15:10 |
| abhishekk | 2nd one is periodic job and which is now using centos 9 | 15:12 |
| jokke_ | abhishekk: but if it's specifically configured to do py36 the job will still always fail | 15:14 |
| abhishekk | Lets do it as followup patch? | 15:15 |
| jokke_ | like makes no sense to keep any py36 jobs around voting or not as they will not be able to run due to the g_s dependency | 15:15 |
| jokke_ | Don't you need to repsin the patch anyways as you marked it DNM? | 15:16 |
| abhishekk | I was going to update commit message via browser :P | 15:16 |
| abhishekk | Ok,let me do it manually then | 15:16 |
| jokke_ | I think it still retriggers zuul, no? | 15:16 |
| jokke_ | if not then followup is fine | 15:16 |
| abhishekk | Not now a days | 15:17 |
| jokke_ | oh gr8 | 15:17 |
| abhishekk | but I will respin it | 15:17 |
| jokke_ | cool ... yeah just put something like "glance_store 4.0.0 requires >=py3.8" into the commit message to make it clear why this is necessary | 15:18 |
| abhishekk | jokke_, I think periodic fips job is definitely using centos 9 stream | 15:19 |
| abhishekk | so it will be running on py38 | 15:19 |
| jokke_ | oh sorry, I was referring to the functional 36 fips job | 15:20 |
| abhishekk | sounds good then | 15:20 |
| jokke_ | I thought you put that on non-voting | 15:20 |
| abhishekk | I put functional 36 fips job non voting in DNM and now removing it in new PS | 15:22 |
| jokke_ | ++ ... will keep eye on the patch until we get vote from zuul | 15:24 |
| abhishekk | cool,thanks | 15:24 |
| jokke_ | well on the positive note, I love almost as much removing obsolete redundant test jobs burdening infra as I do removing code :D | 15:28 |
| abhishekk | :d | 15:28 |
| abhishekk | DNM verified, I will push a new PS | 15:30 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: Update python testing as per zed cycle testing runtime https://review.opendev.org/c/openstack/glance/+/842549 | 15:34 |
| abhishekk | rosmaita, jokke_, dansmith ^^ | 15:35 |
| abhishekk | dansmith, is there a different core team for devstack stable | 15:58 |
| rosmaita | zuul estimates 50 more minutes for the check on https://review.opendev.org/c/openstack/glance/+/842549 | 16:13 |
| dansmith | abhishekk: I don't know | 16:14 |
| rosmaita | dansmith: see if you have +2 powers on https://review.opendev.org/c/openstack/devstack/+/842390 | 16:14 |
| abhishekk | yeah, I asked for this only | 16:14 |
| dansmith | just did | 16:14 |
| dansmith | I do | 16:14 |
| rosmaita | \o/ | 16:14 |
| abhishekk | great | 16:14 |
| dansmith | still don't know the answer to the separate-team question, but.. if so, I'm on it :) | 16:14 |
| abhishekk | no there is not | 16:15 |
| abhishekk | there is failure, so need to add recheck on https://review.opendev.org/c/openstack/glance/+/842549 :/ | 16:35 |
| abhishekk | jokke_, rosmaita can either of you keep watching if I will not be around? | 16:36 |
| jokke_ | yup | 16:38 |
| abhishekk | next PTG is in person \o/ | 16:54 |
| abhishekk | adding recheck | 17:13 |
| rosmaita | oh noes ... volume related failure! | 17:16 |
| gmann | abhishekk: not sure what the issue in https://review.opendev.org/c/openstack/glance/+/841350 you abandoned ? | 17:28 |
| *** timburke__ is now known as timburke | 17:28 | |
| gmann | abhishekk: new patch remove the coverage of py38 which i was adding in 841350 | 17:28 |
| gmann | i can remove the depends-on which was not i meant to add there | 17:28 |
| abhishekk | gmann, basically it has a dependency for some of the periodic job templates | 17:28 |
| abhishekk | also, there is no template for fips functional py38 job | 17:29 |
| gmann | abhishekk: ok but i got confused with your comment "This patch is not required any more as we decided to drop py36 jobs for now " which what 841350 doing | 17:29 |
| abhishekk | so your patch will never pass the zuul | 17:29 |
| gmann | abhishekk: let me fix that. i did not intent to touch fips jobs but it was updated | 17:30 |
| abhishekk | so you will remove the fips functional py36 job as well? | 17:30 |
| gmann | we should move them to py38 not remove, that is what you are removing the test coverage then moving them to new python version | 17:31 |
| abhishekk | we are stuck at this moment and that is why decided to go out with this way | 17:31 |
| abhishekk | I will be happy to abandon my patch if you quickly fix yours | 17:32 |
| gmann | issue is centos8 does not have 38 I think and that is why fips jobs are stuck | 17:32 |
| gmann | let me see if we can move them to py39 | 17:32 |
| abhishekk | yes | 17:32 |
| abhishekk | and if you move it to py39 you need to create a new template in openstack-zuul-jobs | 17:33 |
| abhishekk | once that is merged then we can use it in glance | 17:33 |
| * abhishekk will be back after dinner | 17:42 | |
| rosmaita | btw, that volume related failure was a timeout trying to ssh into an instance ... just sayin' | 17:57 |
| opendevreview | Ghanshyam proposed openstack/glance master: Update python testing as per zed cycle teting runtime https://review.opendev.org/c/openstack/glance/+/841350 | 17:57 |
| gmann | rosmaita: which test that is, resize fix for SSH-able in gate https://review.opendev.org/c/openstack/tempest/+/842140 | 17:58 |
| rosmaita | gmann: tempest.api.compute.volumes.test_attach_volume_negative.AttachVolumeNegativeTest.test_attach_attached_volume_to_different_server on https://zuul.opendev.org/t/openstack/build/55de00f999fc45cbbedddb8662d551b9 | 17:59 |
| rosmaita | tempest-integrated-storage-import-standalone on patch https://review.opendev.org/842549 | 18:00 |
| gmann | ok, yeah that is already SSH-able and ssh timeout. I think we can recheck and see | 18:02 |
| rosmaita | ok, i think it is in recheck already | 18:02 |
| rosmaita | i just didn't want people thinking it was a cinder problem :P | 18:02 |
| abhishekk | :D | 18:05 |
| abhishekk | rosmaita, jokke_ so we need to watch original patch from gmann https://review.opendev.org/c/openstack/glance/+/841350 now | 18:06 |
| abhishekk | Just ignore my patch, I will abandon it later | 18:06 |
| rosmaita | ok, i'm on it | 18:07 |
| abhishekk | thank you | 18:07 |
| abhishekk | I will sign off as it is close to midnight here | 18:08 |
| abhishekk | you or jokke_ can ninja approve it to unblock the gate | 18:08 |
| rosmaita | ok, will do | 18:09 |
| opendevreview | Ghanshyam proposed openstack/glance master: Add openstack-tox-functional-py39-fips job https://review.opendev.org/c/openstack/glance/+/842595 | 18:09 |
| gmann | abhishekk: ^^ does it work for you. 841350 will migrate py36 testing to py38 and 842595 fips 36 to fips 39 | 18:09 |
| rosmaita | abhishekk: have a good evening, and enjoy your recharge day | 18:10 |
| abhishekk | gmann, that will work | 18:10 |
| gmann | abhishekk: cool, thanks I will get zuul-jobs change merged once it pass | 18:10 |
| abhishekk | just we will be keeping periodic jobs running with failures until those are migrated to py38 | 18:10 |
| abhishekk | cool, thank you | 18:10 |
| abhishekk | then no need to worry at all | 18:10 |
| abhishekk | rosmaita, thanks and wish you the same | 18:11 |
| rosmaita | :D | 18:11 |
| abhishekk | I will abandon mine to take it immediately out of the zuul | 18:11 |
| gmann | oh it all redhat guys off tomorrow? | 18:12 |
| rosmaita | gmann: yes | 18:12 |
| gmann | it/is | 18:12 |
| rosmaita | openstack will grind to a halt | 18:12 |
| rosmaita | :D | 18:12 |
| gmann | :) true. I can plan some backlogs then :P | 18:12 |
| abhishekk | :D | 18:12 |
| abhishekk | gmann, if possible could you please also look out for, https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/841368 | 18:15 |
| gmann | abhishekk: yeah, I checked that and seems c8s does not have py38. I think we can test fips on py39 only. let's see how that goes. I will ping ade to confirm about py38 thing | 18:16 |
| gmann | abhishekk: i think all periodic jobs will also pass but let's wait for the result by tomorrow | 18:16 |
| abhishekk | gmann, ack, I will not be around but we should get 841350 merge today | 18:17 |
| gmann | abhishekk: sure, will ping rosmaita dansmith jokke_ once it pass gate | 18:18 |
| abhishekk | ack, thank you again | 18:18 |
| rosmaita | yeah, thanks for helping on this, gmann | 18:19 |
| gmann | sorry for updating it late, did not know glance was blocked on this | 18:19 |
| rosmaita | we didn't either! | 18:20 |
| rosmaita | it was approved a while back, i don't think anyone noticed that it hadn't merged | 18:20 |
| abhishekk | no problem, we also not hoping to get block this way :D | 18:20 |
| abhishekk | I noticed it and added comment also about dependency but at that time we were stuck solving import issue for devstack :D and tempest-plugin issue of rbac | 18:21 |
| * abhishekk will shutup and go to bed now | 18:22 | |
| abhishekk | o/~ | 18:22 |
| opendevreview | Ghanshyam proposed openstack/glance master: DNM: testing py38 fips on c8s(if py38 works on c8s) https://review.opendev.org/c/openstack/glance/+/842608 | 18:34 |
| opendevreview | Ghanshyam proposed openstack/glance master: DNM: testing py38 fips on c8s(if py38 works on c8s) https://review.opendev.org/c/openstack/glance/+/842608 | 18:35 |
| opendevreview | Ghanshyam proposed openstack/glance master: DNM: testing py38 fips on c8s(if py38 works on c8s) https://review.opendev.org/c/openstack/glance/+/842608 | 18:48 |
| opendevreview | Ghanshyam proposed openstack/glance master: DNM: testing py38 fips on c8s(if py38 works on c8s) https://review.opendev.org/c/openstack/glance/+/842608 | 19:09 |
| rosmaita | zuul estimating 1 hour 15 min before https://review.opendev.org/c/openstack/glance/+/841350 merges | 19:48 |
| jokke_ | thanks rosmaita | 19:50 |
| rosmaita | np, let's hope nothing else breaks before this thing merges! | 20:02 |
| rosmaita | now showing 1 hour 8 minutes ... must be using microsoft time | 20:42 |
| dansmith | rosmaita: are you going to send that email to the list? | 21:43 |
| dansmith | was hoping to ack it before I disappear fro the long weekend | 21:43 |
| dansmith | and that's becoming..soon | 21:43 |
| rosmaita | just finishing it now | 21:43 |
| dansmith | okay | 21:43 |
| rosmaita | sent | 21:44 |
| rosmaita | http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028608.html | 21:44 |
| rosmaita | dammit, forgot to include a link to the backport patch | 21:45 |
| opendevreview | Merged openstack/glance master: Update python testing as per zed cycle teting runtime https://review.opendev.org/c/openstack/glance/+/841350 | 21:47 |
| rosmaita | \o/ | 21:47 |
| jokke_ | yeii! | 21:48 |
| jokke_ | Have a great Recharge weekend rosmaita | 21:48 |
| rosmaita | you too! | 21:48 |
| rosmaita | dansmith: http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028608.html , and have a good recharge day | 21:52 |
| dansmith | waiting for it to show up here | 21:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!