| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-03-09 | |||
| 12:42:06 | sean-k-mooney | we are currently minimising the patches we merge to master until after the final RC deadline | |
| 12:42:13 | sean-k-mooney | which is march 20th | |
| 12:43:06 | sean-k-mooney | we technically could merge this now but we likely will wait until after that | |
| 12:43:20 | ratailor_ | sean-k-mooney, ack. sure. Thanks! | |
| 12:43:32 | sean-k-mooney | ill see if i can find tiem to review but we will likely hold +w for another week or so | |
| 12:43:56 | ratailor_ | sean-k-mooney, ack. np. | |
| 12:50:15 | bauzas | we can merge some bugfixes if we want, that's not a problem | |
| 12:51:47 | dvo-plv | Hello, Could you please help me with zuul regression. https://review.opendev.org/c/openstack/nova/+/876075/ I have two components, nova depends on os-traits. I use Depends-on variable to link them, but it does not work for me. | |
| 12:52:26 | dvo-plv | I found that it install os-trait from pip | |
| 12:52:27 | dvo-plv | 2023-03-09 11:15:37.630 | Collecting os-traits===2.10.0 2023-03-09 11:15:37.634 | Downloading https://mirror-int.dfw.rax.opendev.org/pypifiles/packages/77/ce/66b9fc86ab69e4ad60d07901e9324cb44d72c75b9b0a989bfef0fc4ad4e3/os_traits-2.10.0-py3-none-any.whl (41 kB) | |
| 12:52:38 | dvo-plv | this log from here | |
| 12:52:39 | dvo-plv | https://6e5de493453a596bf54b-e12a251c5f6363a4d35eb8aac39c4442.ssl.cf2.rackcdn.com/876075/3/check/tempest-integrated-compute-ubuntu-focal/270fa66/controller/logs/devstacklog.txt | |
| 12:55:09 | sean-k-mooney | if its not listed as a required project depends on does not work | |
| 12:55:18 | sean-k-mooney | i would guess that that is the isseu there with that job | |
| 12:57:08 | sean-k-mooney | looking at the parent of the job that is indeed the case | |
| 12:58:25 | sean-k-mooney | you could add it as a required project here https://opendev.org/openstack/tempest/src/branch/master/zuul.d/integrated-gate.yaml#L146-L166 | |
| 12:59:14 | sean-k-mooney | you woudl add | |
| 12:59:18 | sean-k-mooney | required-projects: | |
| 12:59:28 | sean-k-mooney | - openstack/os-traits | |
| 13:04:29 | opendevreview | Jorge San Emeterio proposed openstack/nova master: Re-enable "test_tagged_attachment" test. https://review.opendev.org/c/openstack/nova/+/876699 | |
| 13:06:45 | opendevreview | Merged openstack/nova master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/nova/+/876553 | |
| 13:41:25 | dvo-plv | Sean, do I need to fix all gates or only for integrated-compute. Because it looks like depends-on veriable does not work for all tempest jobs | |
| 14:26:01 | sean-k-mooney | it will work only for required proejct im not sure we want to supprot it in general for os-vif | |
| 14:26:06 | sean-k-mooney | *os-traits | |
| 14:26:14 | sean-k-mooney | tis somethign we shoud discuss | |
| 14:26:41 | sean-k-mooney | if we were to fix it for all tempet jobs we woudl do so in the devstack base job | |
| 14:27:03 | sean-k-mooney | however os-traits is a libary and we normally want ot only use that from pypi release | |
| 14:27:29 | sean-k-mooney | dvo-plv: it is expected that your code will fail until an os-traits release is doen | |
| 14:28:01 | sean-k-mooney | i woudl focuse on the reapoval of the spec and we can do an os-traits release with teh traits after march 20th | |
| 14:28:28 | sean-k-mooney | that will fix all the jobs with out always installing os-traits form git | |
| 14:29:19 | sean-k-mooney | if we add os-traits as a required proejct it will always beinstalled form git instead of the most recent release even when we dont have a depens-on | |
| 14:29:27 | sean-k-mooney | which is not what we want in general | |
| 14:32:10 | bauzas | sean-k-mooney: well, we could ask devstack to run our librairies from git | |
| 14:32:23 | bauzas | but I'm not super OK if this is about ci jobs | |
| 14:32:36 | bauzas | it's simplier to release a library if needed | |
| 14:38:14 | sean-k-mooney | bauzas: we alredy can do this | |
| 14:38:19 | sean-k-mooney | btu we intentionally dont | |
| 14:38:34 | bauzas | I know and I agree | |
| 14:38:40 | bauzas | hence the "could" | |
| 14:38:45 | sean-k-mooney | ack | |
| 14:39:20 | bauzas | dvo-plv: if you get stuck by a lagging os-traits, come to us | |
| 14:39:54 | sean-k-mooney | dvo-plv: the other thing you can do is for now just use a custome trait in the nova code and you can update it to the real one later but it woudl be simpler ot just approvhe the spec agains and merge the trait and do a release in 2 weeks | |
| 14:39:56 | bauzas | but I think we recently released the last traits update | |
| 14:40:16 | sean-k-mooney | bauzas: the new triat is not merged yet becuase the spec need to be aprpoved first | |
| 14:40:24 | bauzas | sean-k-mooney: maybe we could have an experimental job that would pull os-traits from git | |
| 14:40:39 | bauzas | if that helps | |
| 14:40:43 | sean-k-mooney | that wont help they wanted to get a clen ci run | |
| 14:40:48 | bauzas | sean-k-mooney: ah I see, that's for a new feature | |
| 14:41:00 | sean-k-mooney | yep https://review.opendev.org/c/openstack/nova/+/876075/ | |
| 14:41:36 | dvo-plv | I will re-execute all tests locally to be sure that all works properly and will wait for spec approve and os-traits merge. Thnak you | |
| 14:41:38 | sean-k-mooney | dvo-plv: for now you could just use a custom trait and or comment it out | |
| 14:42:01 | bauzas | is it for helping to implement ? | |
| 14:42:07 | bauzas | or is it for us reviewing ? | |
| 14:42:31 | bauzas | if the former, nothing prevents you to test your patch against a running devstack which pulls ostraits from git | |
| 14:42:35 | sean-k-mooney | i.e more the request filter and traits reproting to a seocnd patch | |
| 14:43:13 | bauzas | if the latter, add a bit fat comment in the commit msg saying the CI is broken because of the missing release | |
| 14:43:26 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/876075/3/nova/scheduler/request_filter.py is just brekign things currently | |
| 14:43:35 | sean-k-mooney | since os_traits.COMPUTE_NET_VIRTIO_PACKED does not exist yet | |
| 14:43:54 | dvo-plv | yes, exactly | |
| 14:43:54 | bauzas | yeah got it | |
| 14:44:41 | bauzas | but again, nothing prevents you to test this filter with a devstack having a LIBS_FROM_GIT | |
| 14:45:16 | bauzas | once the spec is approved, we could move forward and release ostraits soon | |
| 14:45:22 | dvo-plv | I just wanted to be sure that all works properly with zull environment. But I will wait for approve and do best to fix all problems, what I will find locally | |
| 14:46:07 | bauzas | dvo-plv: if you really want to make yourself confident of your patch, then use a custom trait indeed | |
| 14:46:55 | bauzas | but this will have to be cleaned up before review time | |
| 14:48:21 | bauzas | gibi: a quick swing would be appreciated on https://review.opendev.org/c/openstack/nova/+/874515 | |
| 14:49:32 | gibi | bauzas: looks good | |
| 14:49:39 | bauzas | thanks | |
| 14:49:52 | bauzas | gibi: sean-k-mooney: I have a few other nitty changes in my gerrit that are overdue | |
| 14:50:00 | bauzas | stay around :) | |
| 14:53:20 | dvo-plv | btw, we already update and move packed virtqueue support spec file to the 2023/2 https://review.opendev.org/c/openstack/nova-specs/+/868377 | |
| 14:54:17 | sean-k-mooney | i need to go to the chemeyist and pick up my perscirption but ill be back in a while and can take a look then | |
| 14:55:04 | bauzas | dvo-plv: this week, I'm overbusy with PTL scrubbing tasks, but I'll give a round on reviews next week | |
| 14:55:10 | bauzas | (specs reviews I mean) | |
| 14:56:41 | dvo-plv | Thank you | |
| 18:41:21 | spatel | sean-k-mooney Quick question related nova-schedule not able to find sriov nodes. I do have this in nova - https://paste.opendev.org/show/bASaMtV2z9K8N24FZAk4/ | |
| 18:41:37 | spatel | I have created AZ group and added single compute node to that AZ | |
| 18:41:44 | opendevreview | Merged openstack/nova master: fup for power management series https://review.opendev.org/c/openstack/nova/+/874515 | |
| 18:41:55 | spatel | In flavor i didn't specify any properties | |
| 18:45:54 | spatel | Does placement has any entry of PCI_DEVICE? | |
| #openstack-nova - 2023-03-10 | |||
| 02:14:36 | opendevreview | melanie witt proposed openstack/nova master: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/877055 | |
| 02:14:37 | opendevreview | melanie witt proposed openstack/nova master: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/877056 | |
| 09:15:35 | zigo | My first Antelope VM pings ! :) | |
| 09:20:38 | gibi | zigo: awesome \o/ | |
| 09:23:22 | bauzas | pong | |
| 10:05:41 | bauzas | gibi: sean-k-mooney: I would appreciate your comments on a PTL doc amendment https://review.opendev.org/c/openstack/nova/+/875730 | |
| 10:06:24 | bauzas | also, | |
| 10:06:43 | bauzas | gibi: sean-k-mooney: some operators would like to know which specs were implemented :) https://review.opendev.org/c/openstack/nova-specs/+/876887 | |
| 10:07:27 | bauzas | and eventually, a easy peasy for count_blueprints.py https://review.opendev.org/c/openstack/nova-specs/+/876888 | |
| 10:14:32 | bauzas | https://docs.openstack.org/infra/storyboard/gui/theory.html#the-rest-api looks like there is a REST API and a CLI, lovely | |
| 10:14:49 | bauzas | way better to play with than the web ui from what I've seen in the last 30 mins | |
| 10:35:28 | gibi | bauzas: I have only one small commment on the PTL guid patch | |
| 10:35:57 | bauzas | cool | |
| 10:36:20 | bauzas | btw. this isn't a simple task to leave Storyboard AFAIK | |
| 10:36:48 | bauzas | I'll need to discuss this with the infra folks and the storyboard team (if it still exists :) ) | |
| 10:47:43 | bauzas | gibi: that's fun but fwiw, I just noticed last week of takashi's diligence of creating those schedule wikipages every cyvcle | |
| 10:48:19 | gibi | :) | |
| 10:48:23 | bauzas | IMHO if we really want to add more nova-specific deadlines, we would need to stuff it into https://releases.openstack.org/bobcat/schedule.html | |
| 10:48:55 | bauzas | not sure a lot of us still checkouts the wiki besides the meeting agenda :) | |
| 10:49:14 | bauzas | (but don't get me wrong, I like the wiki for its convenience) | |