| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-12-09 | |||
| 04:36:50 | iokiwi | neither is_bootable=True nor bootable=True results in a bootable volume. | |
| 04:37:20 | iokiwi | Creating bootable volume via openstack client works fine though | |
| 05:03:47 | eandersson | iokiwi what is the output if you use bootable=True instead, identical? | |
| 05:05:09 | eandersson | but I see the bug | |
| 05:05:30 | eandersson | https://github.com/openstack/openstacksdk/blob/140eaf7547cf9f557fdd119bf278c41f7b49e1e2/openstack/cloud/_block_storage.py#L242 | |
| 05:05:59 | eandersson | When bootable is set, the volume is just return, but not normalized. | |
| 05:06:12 | eandersson | mordred ^ | |
| 05:07:07 | eandersson | So don't think is_bootable could ever be true? | |
| 09:56:47 | iokiwi | eandersson yes I tried bootable=True | |
| 10:00:48 | iokiwi | Yes, the volume will go to status 'CREATING' first so that condition will probably never be met. But I dont see a way via the sdk to set the volume bootable after the volume has been created | |
| 10:13:38 | iokiwi | Oh nevermind, I am inside the wait block. I am not sure what normalizing the volume means? | |
| 15:04:30 | mordred | eandersson: oh - yeah. that's unfortunate there | |
| 15:06:34 | mordred | eandersson: but iokiwi is using the resource layer | |
| 15:06:51 | mordred | iokiwi: can you try con.create_volume instead of con.block_storage.create_volume? | |
| 15:10:06 | mordred | or - alternately, if you want to use the resource layer - you need to get the Volume object back from that create call, then you need to update it with the bootable flag - like v.is_bootable = True ; v.commit(con.block_storage) | |
| 17:02:39 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: Add support for setting and removing tags from server instance https://review.opendev.org/697674 | |
| 17:18:07 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: WIP Make an OpenStackModule base class https://review.opendev.org/698044 | |
| 18:16:11 | sshnaidm | is it a duplicate? or intentional https://github.com/openstack/openstacksdk/blob/master/.zuul.yaml#L298-L314 | |
| 18:17:49 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Add collections metadata info https://review.opendev.org/684787 | |
| 18:17:49 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Import modules from ansible/ansible https://review.opendev.org/684788 | |
| 18:17:50 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: WIP Make an OpenStackModule base class https://review.opendev.org/698044 | |
| 18:17:50 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Add tox.ini file for running pep8 https://review.opendev.org/698056 | |
| 18:18:16 | mordred | sshnaidm: duplicate :) | |
| 18:18:45 | mordred | sshnaidm, dtantsur|afk, Shrews, mnaser: ^^ added a pep8 job and rebased the other stuff on it | |
| 18:21:09 | sshnaidm | mordred, commented | |
| 18:22:58 | openstackgerrit | Sagi Shnaidman proposed openstack/openstacksdk master: Remove duplicate job definition https://review.opendev.org/698057 | |
| 18:29:27 | eandersson | oh yea my bad mordred | |
| 18:29:32 | eandersson | but still that should probably be fixed :p | |
| 18:47:27 | mordred | eandersson: ++ | |
| 18:52:30 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: WIP Make an OpenStackModule base class https://review.opendev.org/698044 | |
| 18:52:30 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Fix F841 and remove exclusion https://review.opendev.org/698063 | |
| 18:52:31 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Fix E128 and remove exclusion https://review.opendev.org/698064 | |
| 18:52:31 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Fix F401 and remove exclusion https://review.opendev.org/698065 | |
| 18:52:32 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Fix H236 and remove exclusion https://review.opendev.org/698066 | |
| 18:52:32 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Remove from __future__ lines https://review.opendev.org/698067 | |
| 18:52:33 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Fix H301 and remove exclusion https://review.opendev.org/698068 | |
| 18:52:40 | mordred | sshnaidm, mnaser: enjoy | |
| 18:55:25 | sshnaidm | mordred, ok, anyway we assigned all work to you iirc :D | |
| 18:59:03 | mordred | :) | |
| 20:54:48 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM: test job https://review.opendev.org/698085 | |
| 21:15:41 | iokiwi | Cheers mordred and eanderson I will try use con.create_volume instead | |
| 21:25:47 | mordred | iokiwi: sorry for the confusion there - it's sadly a two-step process which isn't modelled well down at the resource layer | |
| 21:29:26 | iokiwi | No worries - I can deal with that. Thanks for the help | |
| #openstack-sdks - 2019-12-10 | |||
| 06:32:45 | openstackgerrit | Shogo Saito proposed openstack/openstacksdk master: Adding basic implementation for Accelerator(Cyborg) https://review.opendev.org/679914 | |
| 08:26:51 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: Add support for setting and removing tags from server instance https://review.opendev.org/697674 | |
| 10:18:51 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM: test job https://review.opendev.org/698085 | |
| 19:06:30 | cmurphy | dtroyer: next time you have a few minutes, could i ask you to add https://review.opendev.org/677857 to your review queue? there's an open question on what the command name should be | |
| #openstack-sdks - 2019-12-11 | |||
| 14:30:05 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: Add dns_publish_fixed_ip attribute to subnets https://review.opendev.org/679833 | |
| 16:19:41 | openstackgerrit | Michael Johnson proposed openstack/python-openstackclient master: Switch to using osc_lib.utils.tags https://review.opendev.org/662864 | |
| #openstack-sdks - 2019-12-12 | |||
| 15:59:09 | sshnaidm | hello everyone | |
| 15:59:23 | dtantsur | we're about to hijack this slot again \o/ | |
| 16:00:21 | sshnaidm | etherpad is here: https://etherpad.openstack.org/p/openstack-ansible-modules | |
| 16:00:31 | sshnaidm | please add topic for discussions there | |
| 16:00:40 | sshnaidm | dtantsur, can you run the bot please? | |
| 16:00:48 | dtantsur | #startmeeting api-sig | |
| 16:00:49 | openstack | Meeting started Thu Dec 12 16:00:48 2019 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:50 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:53 | openstack | The meeting name has been set to 'api_sig' | |
| 16:00:59 | dtantsur | #chair sshnaidm elmiko | |
| 16:01:00 | openstack | Current chairs: dtantsur elmiko sshnaidm | |
| 16:01:05 | sshnaidm | dtantsur, thanks! | |
| 16:01:15 | sshnaidm | o/ | |
| 16:01:26 | dtantsur | #link https://etherpad.openstack.org/p/openstack-ansible-modules agenda | |
| 16:01:38 | dtantsur | anyone else here? :) | |
| 16:01:42 | gtema | me | |
| 16:02:53 | dtantsur | a nice private party :) | |
| 16:02:57 | gtema | sure | |
| 16:03:00 | gtema | as usual | |
| 16:03:06 | sshnaidm | mnaser, mordred, odyssey4me | |
| 16:03:11 | gtema | last week was weird - too much people :D | |
| 16:03:18 | sshnaidm | just calling people from etherpad :) | |
| 16:03:21 | odyssey4me | o/ | |
| 16:03:27 | sshnaidm | holidays I assume | |
| 16:03:51 | gtema | it's tooooooo early for holidays, who will do the work | |
| 16:04:03 | sshnaidm | so I've sent a mail to legal-discussion: http://lists.openstack.org/pipermail/legal-discuss/2019-December/thread.html | |
| 16:04:15 | sshnaidm | it's the only thread there | |
| 16:04:48 | sshnaidm | got responses that we'd better to keep developing in ansible sig repo under GPL | |
| 16:05:41 | sshnaidm | any other opinions about that? ^ | |
| 16:05:45 | odyssey4me | hmm, is that an OK thing for openstack repositories? | |
| 16:05:50 | mordred | ohai! | |
| 16:06:05 | gtema | well, reading inside I would say there are no answers (mordred doesn't count) | |
| 16:06:10 | sshnaidm | odyssey4me, well, it's not really openstack repository | |
| 16:06:38 | sshnaidm | gtema, yeah, responses were from people of this meeting actually :) | |
| 16:07:10 | mordred | yeah - although while I don't count - the takeaway I've gotten from past encounters is that as along as it's not part of "the software" called openstack, it's ok | |
| 16:07:10 | dtantsur | it seems that we can host GPL code, just not make it official openstack delivarables | |
| 16:07:18 | mordred | yah | |
| 16:07:26 | odyssey4me | is that a problem? | |
| 16:07:39 | sshnaidm | I'd like to know what it blocks for us | |
| 16:07:42 | mordred | nope. should be fine. this is an effort of a sig | |
| 16:08:00 | mordred | I don't thnk it's blocks anything. ansible is already GPL | |
| 16:08:00 | dtantsur | nothing blocking. just a bit unfortunate that we cannot proudly call these modules official | |
| 16:08:03 | sshnaidm | what is consequences of not having this as deliverable? | |
| 16:08:13 | odyssey4me | we have imports both ways, right - the module imports ansible (GPL) stuff, and also imports openstack (Apache) stuff | |
| 16:08:15 | mordred | so anything that would use these modules must by definition be ok consuming gpl software | |
| 16:08:22 | dtantsur | sshnaidm: no promotion on openstack.org? | |
| 16:08:38 | mordred | I don't think it's a matter of not being able to call these modules official | |
| 16:08:49 | dtantsur | one things that I think is important re licensing: | |
| 16:08:50 | sshnaidm | mordred, this won't change anyway, we don't relicense them | |
| 16:08:51 | mordred | it's that they can't be part of an OpenStack release | |
| 16:09:01 | mordred | which they wouldn't be anyway, since that makes no sense | |