Posted	Nick	Remark
#openstack-sdks - 2019-07-26
10:58:56	openstackgerrit	Akihiro Motoki proposed openstack/openstackclient master: check-plugins job: switch to python3 https://review.opendev.org/672949
11:37:51	efried	dtantsur: a couple of notes, but +1. Nice and clean.
11:37:59	dtantsur	thx!
11:50:15	mordred	dtantsur: I agree with efried - but I think all of those are things that could be done in a followup
11:50:55	dtantsur	cool, I'll see what other folks think
14:44:55	amotoki	dtroyer: I just started to check python3 status in OSC. Is anyone checking it?
14:45:17	amotoki	dtroyer: I started check-plugins job in the openstack/openstackclient repo https://review.opendev.org/#/q/project:openstack/openstackclient+topic:switch-to-py3+(status:open+OR+status:merged)
14:45:49	amotoki	dtroyer: if someone is working on py3 stuff, I would like to sync with them.
15:14:32	dtroyer	amotoki: we've been py3 clean for a long time, I use it locally only in py3. I saw those reviews, thanks for catching that, it was just missed
15:21:35	tosky	dtroyer: talking about check-plugins... do you rememer our old discussion about openstackclient-check-plugins failing on older stable branches? Do you think that those changes may fix them?
15:29:29	dtroyer	tosky: I vaguely recall that, but am fairly certain it will not make a difference unless amotoki did that in the stable branches too
15:30:11	tosky	dtroyer: the problem is exactly that openstackclient.git has no stable branches
15:30:45	dtroyer	oh, duh, I alwasy do that, right different repo…
15:32:11	tosky	dtroyer: this is the last discussion about this: http://eavesdrop.openstack.org/irclogs/%23openstack-sdks/%23openstack-sdks.2019-03-04.log.html#t2019-03-04T21:23:09
15:32:28	tosky	with the links to the previous discussions :)
15:44:57	amotoki	dtroyer: thanks. I am using py3 by default locally these days. perhaps what we need to check is just about CI jobs (though most have been done perhaps)
15:46:20	amotoki	dtroyer: tosky: regarding openstackclient-check-plugins, the main purpose of the job is to check command duplicates. Taking into account our stable policy, it looks like that the job needs to be run only in the master branch.
15:47:12	tosky	amotoki: oh
15:47:41	tosky	well, that would be a solution as well; what I see now is a set of red jobs and they should be either fixed or removed
15:48:49	amotoki	tosky: this is what the current version of openstackclient-check-plugins checks, so it sounds reasonable to drop it from the stable branches up to stein.
15:49:14	amotoki	tosky: our stable policy does not allow to add new commands :)
15:55:05	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841
16:28:48	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841
21:43:05	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Make configuration show not require auth https://review.opendev.org/673110
#openstack-sdks - 2019-07-27
18:10:03	openstackgerrit	Yuval Shalev proposed openstack/openstacksdk master: Added all-projects headers option to dns proxy. https://review.opendev.org/669547
20:42:56	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Make configuration show not require auth https://review.opendev.org/673110
#openstack-sdks - 2019-07-28
10:31:31	openstackgerrit	LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.opendev.org/650062
#openstack-sdks - 2019-07-29
09:48:54	openstackgerrit	Merged x/promstat master: Remove publishing to docs.openstack.org https://review.opendev.org/672058
11:07:51	openstackgerrit	Dmitry Tantsur proposed openstack/keystoneauth master: Allow requesting fixed retry delay instead of exponential https://review.opendev.org/672930
16:13:27	mriedem	dtroyer: mordred: following up (way later) on the osc compute api gaps forum discussion in denver, i've created a story for making boot-from-volume better in osc https://storyboard.openstack.org/#!/story/2006302,
16:13:33	mriedem	the tasks are in order,
16:13:44	mriedem	and i have questions on the implementation of the 2nd one that adds a --boot-from-volume option
16:14:15	mriedem	tl;dr i want to just do: openstack server create --flavor $flavor --image $image --boot-from-volume $server-name
16:14:25	mriedem	the problem is you have to pass a size for the volume that nova will create from the image,
16:14:39	mriedem	i think the easiest solution is just make the --boot-from-volume option take a size value
16:14:48	mriedem	so: openstack server create --flavor $flavor --image $image --boot-from-volume 1 $server-name
16:14:56	mriedem	that would create a 1GB volume from the given image
16:16:02	mriedem	the image has some properties which we could maybe use, like min_disk, but they are all optional so i don't have much confidence on relying on the image *size properties.
16:16:22	mordred	mriedem: fwiw, the API in sdk is boot_from_volume=True, volume_size=1 - it's that way because it's also possible to do boot_from_volume=True, boot_volume=some_volume
16:16:41	mordred	I would not rely on min_disk from the image
16:16:43	mriedem	osc already has a --volume option which is in a mutex with --image
16:16:53	mordred	nod
16:16:53	mriedem	so i think the cli is already covered there
16:17:03	mriedem	in that case nova doesn't create a volume since the user is providing it
16:17:16	mordred	then sure - I think a number argument to boot_from_volume seems like a fine idea
16:17:25	mriedem	ack, i'll start with that anyway
16:21:43	dtroyer	mriedem: I think I am ok with that, but I don't have that all in my head at the moment… it seems to map to the API fine
16:24:24	mriedem	oh also, i'm going to first add type=image as an option for --block-device-mapping; the question there is if --image is specified along with --block-device-mapping vda:$image_id:image:1:true - do we validate the --image id against the block device image id? fail if they are different? make --image optional if an image bdm is specified? make the bdm a non-root volume if --image is specified and the ids are different?
16:24:48	mriedem	i can take this to the ML if you want wider input and/or time to mull the options
17:03:11	openstackgerrit	Matt Riedemann proposed openstack/python-openstackclient master: WIP: Support type=image with --block-device-mapping option https://review.opendev.org/673340
17:03:13	mriedem	^ for chewing on that question
17:03:35	mriedem	i likely need to play with those scenarios in devstack to figure out what's best
20:48:25	openstackgerrit	Merged openstack/python-openstackclient master: Fix module paths for volumev3 volume backup commands https://review.opendev.org/672914
21:08:40	openstackgerrit	Michael Johnson proposed openstack/python-openstackclient master: Switch to using osc_lib.utils.tags https://review.opendev.org/662864
#openstack-sdks - 2019-07-30
03:09:12	openstackgerrit	yanpuqing proposed openstack/osc-lib master: Add error message when occurrence Forbidden error https://review.opendev.org/673389
05:06:52	openstackgerrit	Andreas Jaeger proposed openstack/openstacksdk master: Update links https://review.opendev.org/673401
05:16:49	openstackgerrit	Andreas Jaeger proposed openstack/openstacksdk master: Update links https://review.opendev.org/673401
08:38:02	dtantsur	mordred: morning, could you take another look? https://review.opendev.org/#/c/672930/
10:21:24	openstackgerrit	Vishakha Agarwal proposed openstack/openstacksdk master: Add application credential CRUD support https://review.opendev.org/669331
11:55:47	mordred	dtantsur: +A
12:22:54	dtantsur	thx!
13:33:04	openstackgerrit	Merged openstack/keystoneauth master: Allow requesting fixed retry delay instead of exponential https://review.opendev.org/672930
15:08:53	Shrews	mordred: now that https://review.opendev.org/672573 is merged, when do you think another release can go out?
15:10:16	dtantsur	mordred, Shrews, can we merge https://review.opendev.org/#/c/672233/ first please?
15:11:24	gtema	dtantsur: +2ed
15:14:35	dtantsur	thx! any reason not to +W?
15:15:03	gtema	done ;-) wanted to give others possibility to look as well
16:53:45	openstackgerrit	Merged openstack/openstacksdk master: Add node traits support to baremetal https://review.opendev.org/672233
16:59:21	openstackgerrit	Michael Johnson proposed openstack/python-openstackclient master: Switch to using osc_lib.utils.tags https://review.opendev.org/662864
17:01:21	openstackgerrit	Michael Johnson proposed openstack/python-openstackclient master: Switch to using osc_lib.utils.tags https://review.opendev.org/662864
17:58:21	openstackgerrit	Andreas Jaeger proposed openstack/os-client-config master: Fix broken link to openstacksdk https://review.opendev.org/673603
18:50:07	openstackgerrit	Tim Burke proposed openstack/python-openstackclient master: Fix typo: "to and endpoint" https://review.opendev.org/673614
23:31:06	openstackgerrit	Michael Johnson proposed openstack/python-openstackclient master: Switch to using osc_lib.utils.tags https://review.opendev.org/662864
#openstack-sdks - 2019-07-31
04:28:57	openstackgerrit	Brin Zhang proposed openstack/python-openstackclient master: Microversion 2.76: Add delete_on_termination to volume-attach API https://review.opendev.org/673725
07:36:04	openstackgerrit	Akihiro Motoki proposed openstack/openstackclient master: check-plugins job: convert the job to tox https://review.opendev.org/672942
07:36:05	openstackgerrit	Akihiro Motoki proposed openstack/openstackclient master: check-plugins job: switch to python3 https://review.opendev.org/672949
09:11:06	openstackgerrit	Brin Zhang proposed openstack/python-openstackclient master: Microversion 2.76: Add delete_on_termination to volume-attach API https://review.opendev.org/673725
09:23:49	openstackgerrit	Brin Zhang proposed openstack/python-openstackclient master: Microversion 2.74: Support Specifying AZ to unshelve https://review.opendev.org/665336
09:45:38	openstackgerrit	Brin Zhang proposed openstack/python-openstackclient master: Microversion 2.75: Support Specifying AZ to unshelve https://review.opendev.org/665336
10:14:02	openstackgerrit	Brin Zhang proposed openstack/python-openstackclient master: Microversion 2.75: Support Specifying AZ to unshelve https://review.opendev.org/665336
14:42:24	openstackgerrit	Vishakha Agarwal proposed openstack/openstacksdk master: Add application credential CRUD support https://review.opendev.org/669331
19:37:33	openstackgerrit	Matt Riedemann proposed openstack/python-openstackclient master: Support type=image with --block-device-mapping option https://review.opendev.org/673340
22:19:53	efried	mordred: nova is using sdk for placement as of this morning \o/
22:24:16	mordred	efried: woot! welp - I guess that's a thing now
22:24:33	mordred	Shrews: it seems we probably shouldn't screw things up too much
22:26:33	efried	betcher @$$. We're watching you 0.0
22:29:55	Shrews	Or nova could, ya know, help improve our testing ;)
22:32:47	efried	Shrews: That's effectively what we're doing. Not so much with the placement thing, which just using simple passthrough to Adapter primitives, but with ironic we're actually working on replacing python-ironicclient entirely https://review.opendev.org/#/q/topic:bp/openstacksdk-in-nova+(status:open+OR+status:merged)
22:33:09	efried	reviews there would be most helpful
22:33:54	efried	o/
#openstack-sdks - 2019-08-01
13:57:34	Shrews	mordred: why don't i see a tag: 0.27.0 in the openstacksdk repo? I see 0.26.0 and then a jump to 0.28.0
13:57:57	mordred	Shrews: I see 0.26.0
13:57:59	mordred	0.27.0
13:58:01	mordred	0.28.0
13:58:03	mordred	0.29.0
13:58:17	mordred	Shrews: maybe try git fetch origin --tags ?
13:59:01	Shrews	mordred: nope. what commit was the 0.27.0 tag?
13:59:36	mordred	41740827c4702e8fcaebdf6adf7a2d49a557ec14
13:59:46	mordred	weird - I checked opendev.org/openstack/openstacksdk and it's there too
14:00:36	Shrews	super weird that I do not have that commit id

Earlier   Later