| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-01 | |||
| 03:10:44 | melwitt | gmann: ok. thanks. sorry, was just wondering whether that was something to also mention in the nova-status upgrade mention or not, where it said blank file. I wasn't sure if blank file would also do the same as "no file" for option 3 in the message | |
| 07:18:22 | bauzas | gibi: bank holiday today, will be back on Monday | |
| 09:06:51 | stephenfin | lyarwood: you stable-core on os-vif? https://review.opendev.org/#/c/724784/ | |
| 09:11:58 | openstackgerrit | Merged openstack/os-vif master: [Community goal] Update contributor documentation https://review.opendev.org/722392 | |
| 09:12:46 | lyarwood | stephenfin: yes | |
| 09:13:40 | stephenfin | Cool. Would you mind? There are also a few stable branch creation patches there. They're -1 but that's because we need the patch I linked to merge first | |
| 09:14:47 | lyarwood | stephenfin: ack, +W'd | |
| 09:14:58 | stephenfin | thanks | |
| 09:15:20 | openstackgerrit | Lee Yarwood proposed openstack/os-vif stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/724712 | |
| 09:15:29 | openstackgerrit | Lee Yarwood proposed openstack/os-vif stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/724713 | |
| 09:16:09 | lyarwood | stephenfin: https://review.opendev.org/#/c/723853/ - trivial one if you have time, any idea why that link doesn't work? | |
| 09:17:02 | stephenfin | looking | |
| 09:20:58 | stephenfin | lyarwood: it's the os-api-ref extension | |
| 09:21:32 | stephenfin | this is what the URL looks like if you navigate to the "Execute Server" section first: ?expanded=evacuate-server-evacuate-action-detail#forced-down | |
| 09:21:51 | stephenfin | that doesn't work since the block the anchor relates to is not visible | |
| 09:22:33 | stephenfin | if you navigate to the 'Update Forced Down' section first, followed by 'Evacuate Server', you get this: ?expanded=update-forced-down-detail,evacuate-server-evacuate-action-detail#forced-down | |
| 09:22:36 | stephenfin | which does work | |
| 09:24:00 | openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for --force flag for nova-manage placement heal_allocations command use this flag to forcefully call heal allocation for a specific instance https://review.opendev.org/715395 | |
| 09:24:23 | lyarwood | stephenfin: ah, so is the best we can do? | |
| 09:24:53 | lyarwood | stephenfin: I just copied the approach from another section FWIW | |
| 09:25:51 | stephenfin | lyarwood: try removing the anchor you added and using ':ref:`update-forced-down` instead | |
| 09:26:03 | lyarwood | ack | |
| 09:26:36 | stephenfin | because that already exists https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_348/723853/1/check/build-openstack-api-ref/348c486/docs/#update-forced-down | |
| 09:31:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add evacuation pre-conditions around the src host https://review.opendev.org/723853 | |
| 09:32:54 | lyarwood | huh that only took a while as my upgrade to f32 somehow removed precommit | |
| 09:33:05 | lyarwood | weird | |
| 10:26:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add evacuation pre-conditions around the src host https://review.opendev.org/723853 | |
| 11:05:56 | openstackgerrit | Merged openstack/os-vif stable/ussuri: Fix doc build job for wanring turn into error https://review.opendev.org/724784 | |
| 11:09:58 | sean-k-mooney | lyarwood: thanks for merging ^ | |
| 11:11:37 | lyarwood | np | |
| 11:11:45 | sean-k-mooney | i still haven't gotten around to creating a sepeart stable team for os-vif specifcally but since i have generally been able to fine people to merge teh patches its been a low priority but those are the kinds of patches i want to be able to merge on stable | |
| 11:12:38 | lyarwood | sean-k-mooney: yeah, I've just ack'd the two above it btw enabling stable/ussuri stuff | |
| 11:12:53 | lyarwood | sean-k-mooney: after that I'll leave others to that branch as I've not worked on os-vif | |
| 11:13:41 | sean-k-mooney | lyarwood: well nova-stable-maint are the only people that can merge stuff other then the gloabl stable mainaters | |
| 11:13:51 | sean-k-mooney | there is not os-vif stable team | |
| 11:14:23 | sean-k-mooney | so im happy you are taking a look as it means i dont have to try and find someone to review | |
| 11:14:35 | lyarwood | ah okay | |
| 11:15:30 | sean-k-mooney | that is why i have considerd createding an os-vif-release team a few times as i dont really want o join nova-stable-maint or simialr for the low volume of os-vif stable patches | |
| 11:19:21 | sean-k-mooney | speaking of finding someone tonyb: would you be able to take a look at two procedual patches .gitreview update https://review.opendev.org/#/c/724712/2 and tox constraits update https://review.opendev.org/#/c/724713/2 for os-vif | |
| 11:19:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/714676 | |
| 11:19:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove future imports https://review.opendev.org/714675 | |
| 11:19:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: force explicit import of python's mock https://review.opendev.org/708768 | |
| 11:21:32 | sean-k-mooney | oh im not on irc downstream my vpn is still down | |
| 11:34:11 | openstackgerrit | Merged openstack/nova stable/rocky: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/723050 | |
| 12:29:29 | openstackgerrit | Merged openstack/os-vif master: trivial: Remove some rules from flake8 ignore list https://review.opendev.org/716223 | |
| 13:10:42 | lyarwood | stephenfin: another trivial docs question for you, is there a way to reference the values of a given nova.objects.field in the api-ref? | |
| 13:11:21 | lyarwood | stephenfin: adding a validation of block_device_mapping_v2.disk_bus in the compute API and wanted to update the docs with the possible values without listing them out as a static list | |
| 13:11:34 | artom | Wait, we have the nova objects fields documented somewhere? | |
| 13:11:41 | stephenfin | Not easily. You'd have to write a custom extension to do it. That might be worth it if you plan to do it multiple times, but otherwise I'd just add a NOTE reminding people to update the docs if they change the o.vo | |
| 13:12:53 | lyarwood | artom: no this is in the api-ref for the actual request, I'm just using the fields to validate the input | |
| 13:13:21 | lyarwood | stephenfin: ack understood, I couldn't find one so just wanted to check that I wasn't missing anything | |
| 13:21:05 | openstackgerrit | tianmaofu proposed openstack/nova master: Delete queued image after deleting instance when creating image from instance https://review.opendev.org/724839 | |
| 13:40:46 | sean-k-mooney | lyarwood: out side of the notification object there should be non ovos at teh api level currently | |
| 13:41:02 | sean-k-mooney | we dont use ovo over any of the rest apis | |
| 13:41:23 | dansmith | aren't you guys on holiday today? | |
| 13:41:23 | sean-k-mooney | so i would not expect ovo to be mentioned in the api-ref at all | |
| 13:41:29 | sean-k-mooney | monday | |
| 13:41:34 | sean-k-mooney | in ireland | |
| 13:41:34 | dansmith | oh I thought both days | |
| 13:42:05 | sean-k-mooney | no ireland just moves the bank holiday to the first monday in may | |
| 13:42:24 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/724842 | |
| 13:42:25 | dansmith | okay but is UK of today? | |
| 13:42:30 | sean-k-mooney | the uk normally doesd too but the have friday next week off instead as due to the 75 aniversry of ve day | |
| 13:42:58 | dansmith | okay I thought it was today and monday | |
| 13:43:04 | lyarwood | sean-k-mooney: ^ I'm not using the o.vo objects, just the associated field to validate the values in the compute API | |
| 13:43:07 | sean-k-mooney | dansmith: i think france is today | |
| 13:43:11 | dansmith | ahh | |
| 13:43:20 | dansmith | of course france | |
| 13:44:23 | sean-k-mooney | i assume the use is monday? | |
| 13:44:27 | sean-k-mooney | *us | |
| 13:44:42 | dansmith | nope, our next one is end of may | |
| 13:44:58 | sean-k-mooney | oh i tought ye had memorial day or something this weekend | |
| 13:45:15 | sean-k-mooney | oh that the end of may | |
| 13:45:18 | dansmith | that's end yeah | |
| 13:46:41 | sean-k-mooney | dansmith: you on nova-stable-maint right mind hitting these too os-vif stabel branch creation changes https://review.opendev.org/#/c/724712/2 and https://review.opendev.org/#/c/724713/2 | |
| 13:49:38 | artom | dansmith, hey, can you hit https://review.opendev.org/#/c/687404/ when you get a chance? If you're cool with it as-is, I can do all of the NUMA LM outstanding stuff in a single FUP | |
| 13:50:30 | dansmith | oh did I miss this? sorry | |
| 13:50:51 | sean-k-mooney | dansmith: thanks :) | |
| 13:50:56 | artom | dansmith, no worries, I think I linked you the bottom patch last time I pinged you. And thanks :) | |
| 13:51:10 | dansmith | I was really planing for all you people to be gone and me not have to work today | |
| 13:51:22 | dansmith | hmm, that sounds like the manifesto of a madma | |
| 13:51:24 | dansmith | *madman | |
| 13:51:26 | artom | Hey now, I'm only French in language | |
| 13:51:50 | artom | dansmith, or a serial killer congratulating himself on a job well done | |
| 13:51:52 | sean-k-mooney | dansmith: look forward to monday i guess | |
| 13:51:58 | dansmith | heh | |
| 13:52:31 | lyarwood | dansmith turning into our very own Milton | |
| 13:52:49 | dansmith | hey now, that's going a little far | |
| 13:53:41 | sean-k-mooney | dansmith: at least he is not confusting you with a boomer like stephenfin did | |
| 13:53:42 | lyarwood | apologies | |
| 13:54:29 | dansmith | stephenfin confused me with a boomer? | |
| 13:54:39 | stephenfin | yeah, what? | |
| 13:55:14 | sean-k-mooney | it was at the ptg in vancover and you quickly tried to gloss over it | |
| 13:55:17 | sean-k-mooney | maybe it worked | |
| 13:56:08 | stephenfin | say wha | |
| 13:56:36 | lyarwood | stephenfin: officespace, burns the office down. | |
| 13:56:42 | stephenfin | ahhhh | |
| 13:58:13 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/724842 | |
| 13:59:00 | sean-k-mooney | stephenfin: by the way is https://review.opendev.org/#/c/724188/4 and https://review.opendev.org/#/c/724725/1 what you wanted | |
| 13:59:18 | sean-k-mooney | we already have the min version set to 10. something on master | |