| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-06-30 | |||
| 14:22:00 | chandankumar | will take a look at those this week. | |
| 14:22:05 | sean-k-mooney | i approved the irrelevent files change this morning https://review.opendev.org/c/openstack/cyborg/+/994816/1 | |
| 14:22:17 | melwitt | ah thanks | |
| 14:23:12 | jgilaber | thanks, I'll add them to my review list as well | |
| 14:23:22 | jgilaber | let's move on | |
| 14:23:28 | jgilaber | #topic Bugs | |
| 14:23:36 | jgilaber | I saw we have two untriaged | |
| 14:23:44 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2158058 | |
| 14:24:41 | chandankumar | we discussed about this one in last meeting | |
| 14:24:52 | chandankumar | [cyborg-tempest-plugin][fake driver]Improve the scenario test coverage for instance operation in fake driver | |
| 14:25:26 | chandankumar | sean-k-mooney proposed https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580 pci driver based instance lifecycle test | |
| 14:25:33 | chandankumar | we will reuse the same format | |
| 14:25:46 | chandankumar | for this one, can we mark them as a wishlist? | |
| 14:25:57 | jgilaber | does that patch completely address the bug report or will we need more? | |
| 14:26:45 | chandankumar | nono, we need to duplicate the same tests for fake driver based on above patch | |
| 14:26:59 | jgilaber | ah ok, I see | |
| 14:27:15 | jgilaber | wishlist/medium seems good to me | |
| 14:27:17 | sean-k-mooney | so it kind of superceed it | |
| 14:27:39 | sean-k-mooney | i actully think there is very littel point in useing the fake driver for the majoryit of the lifecycle operations | |
| 14:27:53 | sean-k-mooney | because the only thing that tests is the arq creation | |
| 14:28:02 | sean-k-mooney | on the nova side we entirly drop the arq info | |
| 14:28:29 | melwitt | yeah I kind of wondered about that ... how helpful is the fake driver for lifecycle testing | |
| 14:28:32 | sean-k-mooney | so i woudl prefer to only test program and create with the fake driver | |
| 14:28:46 | sean-k-mooney | and test the rest with pci/mdev/nvme drivers | |
| 14:29:04 | chandankumar | for program I have awip patch https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580 | |
| 14:29:21 | sean-k-mooney | chandankumar: so we have a exsitng boot test with the fake driver | |
| 14:29:28 | sean-k-mooney | once we merge your program tests | |
| 14:29:34 | sean-k-mooney | i woudl close https://bugs.launchpad.net/openstack-cyborg/+bug/2158058 | |
| 14:29:57 | jgilaber | close after the patch merges or close it now? | |
| 14:30:27 | sean-k-mooney | well i guess it depend on if we add Close-Bug to oen of them :) | |
| 14:30:46 | sean-k-mooney | or related bug | |
| 14:30:46 | chandankumar | I can add closes -bug , then it will automatically closes | |
| 14:30:57 | sean-k-mooney | cool those that work for others? | |
| 14:31:09 | melwitt | probably want to add a comment to it about not doing lifecycle tests as part of it. or edit the description | |
| 14:31:16 | jgilaber | ack, so in the meantime we set it as wishlist and low/medium? | |
| 14:32:05 | chandankumar | I will add a comment to the bug | |
| 14:32:13 | chandankumar | low/medium +1 | |
| 14:32:42 | chandankumar | low | |
| 14:33:02 | jgilaber | ack set then | |
| 14:33:39 | jgilaber | next one | |
| 14:33:43 | opendevreview | Merged openstack/cyborg master: Add api-ref/ to irrelevant-files in Zuul config https://review.opendev.org/c/openstack/cyborg/+/994816 | |
| 14:33:43 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2158147 | |
| 14:34:11 | melwitt | oh, that's the one in flight patch approved from earlier | |
| 14:34:43 | melwitt | so it will get closed when the patch merges | |
| 14:34:45 | jgilaber | ok, so that seems valid and is in progress, only missing the importnace | |
| 14:34:53 | jgilaber | medium? | |
| 14:35:11 | sean-k-mooney | ya it has no end user impact directly | |
| 14:35:20 | melwitt | oh wait sorry. I misread that it's after the api-ref thing | |
| 14:35:22 | sean-k-mooney | but melwitt has already fixed some incorrect examples | |
| 14:35:24 | melwitt | anyway. yeah not high importance | |
| 14:35:39 | jgilaber | ack thanks melwitt | |
| 14:35:58 | jgilaber | that was the last one | |
| 14:36:07 | jgilaber | #topic Open discussion | |
| 14:36:16 | jgilaber | any last minute topic to raise? | |
| 14:36:42 | sean-k-mooney | maybe we coudl chat abouy https://github.com/adamchainz/flake8-logging/pull/187 quickly | |
| 14:36:50 | jgilaber | looking | |
| 14:37:08 | sean-k-mooney | melwitt: so we have moved away form using flake8 in general to using ruff for linting | |
| 14:37:22 | sean-k-mooney | we coudl run that check via the hacking env | |
| 14:37:30 | sean-k-mooney | since it uses flake internally | |
| 14:37:34 | melwitt | ok that's good to know | |
| 14:37:44 | sean-k-mooney | but woudl it be better to just have a local hacking check instead | |
| 14:38:24 | melwitt | I don't know a lot about it so I wasn't aware we could make local checks. if that's an option then seems like a nice idea? | |
| 14:39:03 | sean-k-mooney | so there are 2 ways we coudl do this | |
| 14:39:13 | melwitt | I'm skeptical my PR will go anywhere but I tried it anyway | |
| 14:39:14 | sean-k-mooney | a pre-commit hook or a local hackign check | |
| 14:39:19 | sean-k-mooney | the later is proably simpler | |
| 14:39:37 | sean-k-mooney | this might be genericlly usefully enough to put in hacking propper as well | |
| 14:40:54 | melwitt | ok, I will look into that then, thanks | |
| 14:41:47 | sean-k-mooney | ah right | |
| 14:41:49 | sean-k-mooney | https://github.com/openstack/cyborg/commit/8eb3a5a4b2a8fa0fc969290ceace9f574715d027 | |
| 14:42:16 | sean-k-mooney | i delete our hacking plugin | |
| 14:42:20 | sean-k-mooney | but we can add one back | |
| 14:42:25 | melwitt | :) | |
| 14:42:42 | sean-k-mooney | i deleted it because i didnt like the preior rules it used in general | |
| 14:43:05 | sean-k-mooney | well or the were duplicats of other checks | |
| 14:43:10 | melwitt | oh ok | |
| 14:43:51 | sean-k-mooney | nova has a bunch of hackign plugin examples too for what its worth | |
| 14:44:14 | sean-k-mooney | anyway i think we can chat about that outside the meeting | |
| 14:44:23 | sean-k-mooney | any other topics or questions on this topic? | |
| 14:44:28 | melwitt | sure. I will put something together. thanks for the info | |
| 14:45:10 | sean-k-mooney | ... https://github.com/openstack/cyborg/blob/master/cyborg/__init__.py sorry got distracted | |
| 14:45:19 | sean-k-mooney | we need to fix ^ | |
| 14:45:35 | sean-k-mooney | i nocied that lookign at the old commit | |
| 14:45:52 | sean-k-mooney | but we have been asked to remove any runtim usage of pbr by the pbr maintianers | |
| 14:46:04 | melwitt | ok good to know | |
| 14:46:07 | sean-k-mooney | ill file a bug for that later | |
| 14:46:23 | sean-k-mooney | we are emnt to use import lib metadta or soemthign instead | |
| 14:46:35 | sean-k-mooney | or just stop providing __version__ | |
| 14:46:50 | sean-k-mooney | there was a mail thread about it a month ago or there abouts | |
| 14:47:11 | jgilaber | ack thanks sean-k-mooney we can look at the bug in the next meeting | |
| 14:47:23 | jgilaber | I think I missed this email | |
| 14:47:26 | sean-k-mooney | https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/V7JFTBRA3CHHIWCZM5CM2LE27ESDJHO7/#V7JFTBRA3CHHIWCZM5CM2LE27ESDJHO7 | |
| 14:48:06 | sean-k-mooney | https://review.opendev.org/c/openstack/python-openstackclient/+/987782/2/openstackclient/__init__.py | |
| 14:48:14 | sean-k-mooney | so that the exampel fo what we shoudl do instead | |
| 14:48:45 | jgilaber | seems simple enough | |
| 14:48:51 | sean-k-mooney | ok ill file a bug for that and create a patch later | |
| 14:49:02 | sean-k-mooney | ya pretty much | |
| 14:49:27 | sean-k-mooney | they just would prefer to keep pbr as a build time only tool going forward to limit the maintance | |
| 14:51:17 | jgilaber | thanks for bringing it up sean-k-mooney, anything else on this topic or any other? | |
| 14:53:00 | jgilaber | last thing for today | |