Earlier  
Posted Nick Remark
#openstack-nova - 2021-05-12
10:13:48 gibi as I always forget that openstack resource provider trait list needs 1.6 at least
10:13:48 gibi as I always forget that openstack resource provider trait list needs 1.6 at least
10:14:03 sean-k-mooney gibi: i would be fine with auto negociation but negociating the latest i think is not correct. also yes a flag for enabling it i think would be fine
10:14:03 sean-k-mooney gibi: i would be fine with auto negociation but negociating the latest i think is not correct. also yes a flag for enabling it i think would be fine
10:14:27 gibi I understand your point
10:14:27 gibi I understand your point
10:14:31 stephenfin sean-k-mooney: osc is slowly switching to using the latest as part of the migration to sdk
10:14:31 stephenfin sean-k-mooney: osc is slowly switching to using the latest as part of the migration to sdk
10:14:33 sean-k-mooney if we want to do it by default we coudl but that in my view would need a major version bump
10:14:33 sean-k-mooney if we want to do it by default we coudl but that in my view would need a major version bump
10:14:40 gibi I migth disagree that we need to support people scripting against the CLI
10:14:40 gibi I migth disagree that we need to support people scripting against the CLI
10:15:04 sean-k-mooney it was one of the desigin reason why it had its current behavior
10:15:04 sean-k-mooney it was one of the desigin reason why it had its current behavior
10:15:06 stephenfin scripting users should use the sdk now. They can keep using older versions of osc also
10:15:06 stephenfin scripting users should use the sdk now. They can keep using older versions of osc also
10:15:13 gibi * gibi wonders why the IRC log is not updated in the web :/
10:15:15 sean-k-mooney stephenfin: that assume they are using python
10:15:15 sean-k-mooney stephenfin: that assume they are using python
10:15:38 stephenfin yes
10:15:40 sean-k-mooney gibi: it lags there is a chorn script i think that runs evey few minutes
10:15:40 sean-k-mooney gibi: it lags there is a chorn script i think that runs evey few minutes
10:15:49 sean-k-mooney stephenfin: that is not a valid assumtion
10:15:49 sean-k-mooney stephenfin: that is not a valid assumtion
10:16:00 sean-k-mooney osc was create to be a stabel cli that is scriptable
10:16:00 sean-k-mooney osc was create to be a stabel cli that is scriptable
10:16:13 stephenfin maybe not, but we've got limited maintainers and the ones that are there want auto-negotiation
10:16:13 stephenfin maybe not, but we've got limited maintainers and the ones that are there want auto-negotiation
10:16:37 stephenfin osc was also created when we had many more maintainers
10:16:37 stephenfin osc was also created when we had many more maintainers
10:16:44 sean-k-mooney which is why we can channge that if we want to but it need to be a major version bump to singnale we are breakign backward compatiabiltiy
10:16:44 sean-k-mooney which is why we can channge that if we want to but it need to be a major version bump to singnale we are breakign backward compatiabiltiy
10:17:41 stephenfin yes, the last release was a major version bump iirc
10:17:41 stephenfin yes, the last release was a major version bump iirc
10:18:03 sean-k-mooney the release contiang the default change patch would have to be a major version bump
10:18:03 sean-k-mooney the release contiang the default change patch would have to be a major version bump
10:18:23 sean-k-mooney having the ablity to auto negociate is not a breaking change turning it on by default is
10:18:23 sean-k-mooney having the ablity to auto negociate is not a breaking change turning it on by default is
10:21:33 sean-k-mooney gibi: stephenfin if ye want to not revert and you make sure the next version of osc is a major version then i could proably live with that but i still think min required would be better that said i would like thinks like flavor validation to be on by default
10:21:33 sean-k-mooney gibi: stephenfin if ye want to not revert and you make sure the next version of osc is a major version then i could proably live with that but i still think min required would be better that said i would like thinks like flavor validation to be on by default
10:22:05 stephenfin this only affects osc-placement, not osc btw
10:22:05 stephenfin this only affects osc-placement, not osc btw
10:22:19 gibi * gibi lunches 
10:22:27 sean-k-mooney oh it does?
10:22:27 sean-k-mooney oh it does?
10:22:44 sean-k-mooney i tought it was all of osc
10:22:44 sean-k-mooney i tought it was all of osc
10:23:06 stephenfin no. That's happening but separately. The change above was just for osc-placement
10:23:06 stephenfin no. That's happening but separately. The change above was just for osc-placement
10:23:46 sean-k-mooney i still woudl prefer auto negociating the min reqruied not latest but maybe i have to just live with it being unstable
10:23:46 sean-k-mooney i still woudl prefer auto negociating the min reqruied not latest but maybe i have to just live with it being unstable
10:24:08 sean-k-mooney i rarely have need to script against osc so maybe it not worth avocating for those that do
10:24:08 sean-k-mooney i rarely have need to script against osc so maybe it not worth avocating for those that do
10:24:33 stephenfin tbf, those users could simply request an explicit base version manually
10:24:33 stephenfin tbf, those users could simply request an explicit base version manually
10:24:48 stephenfin '--os-compute-api-version 2.1' or whatever
10:24:48 stephenfin '--os-compute-api-version 2.1' or whatever
10:24:50 sean-k-mooney i just dont want verizon complaining that they cant create flavor when the insitent on setting hw:numa_mem_policy that does not exist
10:24:50 sean-k-mooney i just dont want verizon complaining that they cant create flavor when the insitent on setting hw:numa_mem_policy that does not exist
10:26:23 sean-k-mooney stephenfin: ya i know but many dont know what microverions are or care they just want the command they always used to work
10:26:23 sean-k-mooney stephenfin: ya i know but many dont know what microverions are or care they just want the command they always used to work
10:27:27 sean-k-mooney stephenfin: gibi if the desisin to move all of osc to latest has aleready been made then dont revert the placment patch
10:27:27 sean-k-mooney stephenfin: gibi if the desisin to move all of osc to latest has aleready been made then dont revert the placment patch
10:29:39 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Enable 'vmcoreinfo' feature by default https://review.opendev.org/c/openstack/nova/+/752912
10:29:40 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Make vmcoreinfo dependent on feature flag https://review.opendev.org/c/openstack/nova/+/779342
10:29:40 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove 'create_fake_libvirt_mock' https://review.opendev.org/c/openstack/nova/+/790945
10:29:41 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: libvirt: Make vmcoreinfo checks guest arch specific https://review.opendev.org/c/openstack/nova/+/790946
10:33:57 gibi sean-k-mooney: OK, then I won't revert.
10:33:57 gibi sean-k-mooney: OK, then I won't revert.
10:34:10 gibi I honestly don't know what will be the overall osc policy
10:34:10 gibi I honestly don't know what will be the overall osc policy
10:35:16 sean-k-mooney gibi: right now i really dont like that its different per command
10:35:16 sean-k-mooney gibi: right now i really dont like that its different per command
10:35:27 sean-k-mooney as it make it much harder to use
10:35:27 sean-k-mooney as it make it much harder to use
10:36:09 sean-k-mooney that said i know ironic broke the consitency years ago so its not a new thing
10:36:09 sean-k-mooney that said i know ironic broke the consitency years ago so its not a new thing
10:36:10 gibi I think it is the same for each call . max(server,client)
10:36:10 gibi I think it is the same for each call . max(server,client)
10:36:35 sean-k-mooney gibi: not i mean nova and cinder command wont auto negociate
10:36:35 sean-k-mooney gibi: not i mean nova and cinder command wont auto negociate
10:36:41 sean-k-mooney but ironic and plamcent will
10:36:41 sean-k-mooney but ironic and plamcent will
10:36:47 gibi I see
10:36:47 gibi I see
10:37:04 gibi but if the overall osc change is coming the it will be in sync again
10:37:04 gibi but if the overall osc change is coming the it will be in sync again
10:37:26 sean-k-mooney yes if but there was never a cross project agreement to do that as far as i knew
10:37:26 sean-k-mooney yes if but there was never a cross project agreement to do that as far as i knew
10:37:52 sean-k-mooney but if its happeing then i just hope it gets done this cycle
10:37:52 sean-k-mooney but if its happeing then i just hope it gets done this cycle
10:38:07 sean-k-mooney draging it out over multiple cycle will not be a good ux
10:38:07 sean-k-mooney draging it out over multiple cycle will not be a good ux
10:38:39 sean-k-mooney which is why i was expecting this type of change to be a cross project goal
10:38:39 sean-k-mooney which is why i was expecting this type of change to be a cross project goal
10:38:40 gibi OK, then please propose a revert on the osc patch and lets bring it up on cross project levels
10:38:40 gibi OK, then please propose a revert on the osc patch and lets bring it up on cross project levels
10:39:04 stephenfin I don't think this is a serious issue
10:39:04 stephenfin I don't think this is a serious issue
10:39:16 stephenfin If people don't like it, then they'll let us know

Earlier   Later