Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2022-03-21
15:13:23 sean-k-mooney should we add that blueprint to the meeting adgenda or wait untill after the release
15:22:17 sean-k-mooney elodilles: could you take a look at https://review.opendev.org/c/openstack/nova/+/828413/2 the follow up has +2+w and is waiting on the job being enabled
15:29:31 elodilles sean-k-mooney: hmmm, jobs aren't backported usually, and as I see it's non-voting
15:31:14 sean-k-mooney well we dont currently have coverage for hybrid plug in xena
15:31:25 sean-k-mooney it was the default before xena
15:31:44 sean-k-mooney but we did not readd it when we changed the devstack default
15:32:22 sean-k-mooney so this is really just add back coverage we had for wallaby to xena
15:32:40 sean-k-mooney which we now have on yoga again
15:33:15 elodilles (at least the new job nicely shows that the bug fix works)
15:33:20 sean-k-mooney it is non voting but it actully should get fliped to voting at some point
15:33:31 sean-k-mooney i think that is in the follow up patch
15:33:32 elodilles yes, i see now
15:34:02 sean-k-mooney yep so we were using that like a regression test to demonstrate the issue
15:54:41 elodilles sean-k-mooney: +2+W'd, thanks for the details about the fix
15:56:59 sean-k-mooney :)
16:00:19 bauzas sean-k-mooney: I prepared the zed cycle
16:00:33 bauzas sean-k-mooney: but actually I forgot to update the launchpad main branch
16:15:00 artom bauzas, sean-k-mooney, https://review.opendev.org/c/openstack/nova/+/806049/
16:17:54 sean-k-mooney thanks :)
16:52:56 bauzas fwiw, I'm about to give a go to https://review.opendev.org/c/openstack/nova/+/833440 as it won't harm our future tick-tock cadence
16:53:20 bauzas this is about removing Xena computes support for Zed
16:53:34 bauzas but we definitely need to discuss this at the PTG
17:30:55 gibi_pto bauzas: correct, we have to change that limitation in the future to support N-2 -> N upgrades
17:31:29 bauzas I'll leave this change open until tomorrow and we'll discuss at the nova meeting
17:31:34 sean-k-mooney gibi_pto: well that is the topic to discuss at the PTG apprently we dont actully have too but likely should
17:32:12 sean-k-mooney gibi_pto: just repeating what i have heard but appretly live rolling upgrade supprot for n-2 is left to the projects to opt in too
17:33:08 sean-k-mooney so if we elect to supprot n-2 rolling upgrades upstream then we shoudl alter that check and add a job to validate it if we dont then we could keep it at n-1
17:33:27 gibi_pto ahh, OK, lets talk that through on the PTHG
17:33:29 gibi_pto PTG
17:33:57 gibi_pto bauzas: that change fails in grenade on the service version check
17:34:11 gibi_pto https://zuul.opendev.org/t/openstack/build/a24b974ae9fd487694724da795bfcd2a/log/controller/logs/screen-n-cond.txt#1140
17:34:12 dansmith bauzas: I think we should have something like this for nova: https://docs.openstack.org/cinder/latest/contributor/gerrit.html#ci-job-rechecks
17:34:36 dansmith some discussion of late in the tc about how most people just do blind rechecks all the time
17:35:09 dansmith probably don't need that much detail for nova given that we don't have all the drivers, but still, I see a lot of "recheck" with no explanation, which I think often means no inspection
17:35:40 dansmith I'm hoping we'll have some general direction out of the tc about this, but would be nice to jumpstart in the nova community with reminders and constructive peer pressure :)
17:36:50 gibi_pto dansmith: I suppor that idea!
17:37:11 dansmith gibi_pto: I support your support :)
17:37:21 sean-k-mooney we used to have a section that said you should file a bug and add an elastic recheck query i think
17:37:23 dansmith however, I do not support you spending pto hanging out in irc :P
17:37:24 gmann like neutron, we can monitor how many recheck for this week we had
17:38:05 dansmith sean-k-mooney: yeah, but we need more constructive peer pressure regardless.. clear documentation to point to would be helpful when doing that
17:38:16 gmann s/gibi_pto/gibi_pto_but_work_on_irc :)
17:38:24 dansmith sean-k-mooney: if we've got it, then cool, but a concerted team-wide effort is needed I think
17:38:40 dansmith gmann: paid time on work is just work :P
17:38:40 gibi_pto nah, I just randomly hanging around :)
17:38:41 sean-k-mooney dansmith: i think we removed it at som epoint im checking to see if we still do
17:38:53 sean-k-mooney i tought it was in https://github.com/openstack/nova/blob/master/doc/source/contributor/code-review.rst but not seeign it
17:40:21 dansmith I can certainly write up some stuff
17:40:23 sean-k-mooney its also not in our testing doc https://github.com/openstack/nova/blob/master/doc/source/contributor/testing.rst so ya i think we remved it
17:47:49 sean-k-mooney dansmith: this is proably where it need to be updated https://opendev.org/openstack/nova/src/branch/master/doc/source/contributor/process.rst#how-can-i-get-my-code-merged-faster
17:48:23 sean-k-mooney we currently say """If you're fixing something that only occasionally failed before, maybe recheck a few times to prove the tests stay passing."""
17:48:41 sean-k-mooney but we dont give guidnce on when its appropate to recheck in general
17:48:51 gmann dansmith: or may be to writeup in project-team-guide and point projects doc to that ?
17:49:45 sean-k-mooney gmann: we do link to this https://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing
17:50:12 sean-k-mooney we used to have a version fo that in nova
17:50:48 gmann I mean to tell "no blind recheck" and 'best way to check logs and find issue' explicitly
17:50:51 sean-k-mooney step 2 """Examine the console log or other relevant log files to determine the cause of the error. If it is related to your change, you should fix the problem and upload a new patchset. Do not use “recheck”."""
17:52:45 sean-k-mooney we do link to that form our testing.rst doc
17:53:13 gmann yeah, I think due length of this doc it got hidden and people ignore to read the complete doc. we can add link to this doc for details but have a explicit section about recheck and how to use it in best way
17:53:21 gmann dur to
17:53:52 sean-k-mooney if we had a centralised one we coudl have zuul link top it when it fails
17:54:00 sean-k-mooney by updating the failure message
17:55:04 sean-k-mooney in any case it took me 5 mins to find that and i more or less know what im looking for so a new contibutor is likely not going to find it
17:56:02 gmann true
18:44:13 stephenfin sean-k-mooney: https://review.opendev.org/c/openstack/os-traits/+/833826 easy patch
18:47:01 sean-k-mooney stephenfin: ah has the discussion on that been resovled
18:47:25 sean-k-mooney i saw the thread you started but did not keep track of it while i was off
18:49:14 sean-k-mooney oh its just the reference syntax in zuul. that keeps duplicaiton down at least.
18:49:29 sean-k-mooney the zuul change that is https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/833286/2/zuul.d/project-templates.yaml
18:54:34 sean-k-mooney stephenfin: ok done.#
18:54:43 stephenfin spot on, thanks!
18:57:42 sean-k-mooney did you also do os-resource classes
18:59:01 sean-k-mooney stephenfin: dont see it here https://review.opendev.org/q/topic:add-python3-unversioned-jobs but i think its independent too https://github.com/openstack/releases/blob/master/deliverables/_independent/os-resource-classes.yaml
18:59:21 stephenfin Oh, my hacky script probably just missed that
19:00:16 sean-k-mooney i need to add a PTG topic about this by the way
19:00:32 sean-k-mooney we might want to merge those back into placment
19:00:53 sean-k-mooney we have a cirular dep issue with testing and releases
19:01:29 sean-k-mooney we can either relax the test that assert the amount of traits/resouce classes or we can put them all in the same repo
19:02:06 sean-k-mooney stephenfin: im going to go eat something. if you push a patch add me and ill look at it tomorrow
19:02:54 opendevreview Stephen Finucane proposed openstack/os-resource-classes master: Update CI to use unversioned jobs template https://review.opendev.org/c/openstack/os-resource-classes/+/834537
19:03:01 stephenfin sean-k-mooney: Cool. It's there btw ^
19:04:08 opendevreview Merged openstack/os-traits master: Update CI to use unversioned jobs template https://review.opendev.org/c/openstack/os-traits/+/833826
19:13:18 gmann stephenfin I missed it for os-traits but please update setup.cfg too to reflect the tested versions https://review.opendev.org/c/openstack/os-resource-classes/+/834537
19:14:30 stephenfin gmann: I'd rather not mix those two things though I'll admit they're related. Do we want to bump 'python-requires' also?
19:14:38 gmann also, like i mentioned in ML reply, I have added to remove the release specific templates itself which will fix all these issue and less work during releases https://etherpad.opendev.org/p/tc-zed-ptg#L108
19:15:37 gmann stephenfin: yes we can bump python-requires also. They are related as if we do not test py3.6/7 then we should update it in classifier too.
19:16:01 gmann in zed release template we will do the same for branched repo also
19:16:24 stephenfin Oh, that's a good idea
19:16:55 gmann in Zed: python-requires >= py3.8 and remove py3.6 and py3.7 from list
19:17:26 stephenfin _Personally_, I'd question the value the classifiers bring. They aren't used by any tooling that I'm aware of, unlike python-requires. I wonder if we should get rid of them?
19:19:14 gmann I think someone might be checking it from pypi doc but yes I do not have anyone asked about it explicitly
19:25:45 dansmith gmann: yeah, maybe some centralized docs would be good
19:25:57 dansmith gmann: I definitely think we need PTLs to get the word out though
19:26:56 gmann dansmith: +1 on PTL one. I like to neutron way to track it in their weekly meeting. that is something we can suggest PTL to do.
19:27:24 gmann I have added it in TC+PTL PTG etherpad
19:30:58 dansmith yeah
#openstack-nova - 2022-03-22
10:17:57 opendevreview Stephen Finucane proposed openstack/os-resource-classes master: setup: Update Python testing classifiers https://review.opendev.org/c/openstack/os-resource-classes/+/834643
10:17:57 opendevreview Stephen Finucane proposed openstack/os-resource-classes master: setup: Replace dashes with underscores, add links https://review.opendev.org/c/openstack/os-resource-classes/+/834644
10:51:30 zigo Is there a way to evacuate a host that has 3 VMs that have affinity? Can I somehow tell nova "migrate them together" ?
12:41:50 opendevreview anguoming proposed openstack/nova master: fix the bug of the log line has no request_id info at source host when live migration https://review.opendev.org/c/openstack/nova/+/834677

Earlier   Later