Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-06-09
15:26:27 gtema_ and this is really "not critical" - will not be there in the real run, who cares.
15:26:47 gtema_ Perhaps message in the cleanup should be "at least those resources will be deleted"
15:30:02 openstackgerrit Merged openstack/osc-lib master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694510
16:36:12 openstackgerrit Merged openstack/cliff master: Import command group support from osc-lib https://review.opendev.org/733964
18:11:12 openstackgerrit Merged openstack/openstacksdk master: Fix AttributeError exception during authorization https://review.opendev.org/728077
18:43:10 openstackgerrit Gabriel Ramirez proposed openstack/python-openstackclient master: Allow openstack flavor set to update flavor description using name https://review.opendev.org/733688
19:20:04 openstackgerrit Monty Taylor proposed openstack/cliff master: Fix compatibility with new cmd2 https://review.opendev.org/712591
19:34:22 mordred TheJulia: heya - iurygregory was just asing in #openstack-ansible-sig about an issue that turns out to be about running bifrost from a virtualenv and then not finding libs ... are there any bifrost docs/recommendations about that usage pattern?
19:34:52 iurygregory mordred, im using the bifrost docs hehe
19:35:33 iurygregory https://docs.openstack.org/bifrost/latest/install/index.html and https://docs.openstack.org/bifrost/latest/user/howto.html
19:35:38 mordred heh. well - ansible+virtualenv is always special - so I figured maybe it was a chance to learn or improve the docs or something
19:37:38 iurygregory well so far what I can say is that for me on Fedora30 i need to set variables inside the venv so things will work
22:34:09 TheJulia iurygregory: Interesting :(
#openstack-sdks - 2020-06-10
06:33:15 openstackgerrit Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
07:03:28 openstackgerrit Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
07:09:10 openstackgerrit Vishakha Agarwal proposed openstack/openstacksdk master: NIT: Fix application credential https://review.opendev.org/734756
08:49:01 openstackgerrit Merged openstack/cliff master: Fix compatibility with new cmd2 https://review.opendev.org/712591
11:12:58 nightmare_unreal where should I discuss cliff related stuff ?
11:13:13 nightmare_unreal is this channel right for it
11:16:24 dtantsur nightmare_unreal: probably yes
11:17:43 nightmare_unreal okay so I have submitted this patch : https://review.opendev.org/#/c/733573/ . It's regarding the json output format in osc cli. I found out that it's cliff that actually manages/handles the output (tables, json, bash etc)
11:18:07 nightmare_unreal so should I abandon this patch and create one for cliff ?
11:18:59 nightmare_unreal in the above patch I made the changes to correct the json but because of it other formats are messed up. Like mentioned by frickler in the comment that I need to do it in a way where the format is processed.
11:19:15 nightmare_unreal I hope I am able to express myself clearly .
11:24:57 openstackgerrit Hervé Beraud proposed openstack/keystoneauth master: drop mock from lower-constraints https://review.opendev.org/734359
11:24:58 openstackgerrit Hervé Beraud proposed openstack/keystoneauth master: fix reno lower-constraints for py35 https://review.opendev.org/734799
11:43:00 openstackgerrit Sean McGinnis proposed openstack/keystoneauth master: Update lower-constraints versions https://review.opendev.org/734803
12:20:34 mordred nightmare_unreal: yes - I think that is a thing that needs to be fixed in cliff
12:21:55 nightmare_unreal thanks mordred . I will create new patch and submit it to cliff :)
12:22:39 mordred nightmare_unreal: cool! I'm not 100% sure what the answer will be - the nested data types have always been a little wonky
12:23:52 nightmare_unreal i am trying it. but I am not sure if it will be right way. I am currently looking at the keys that are formatted wrongly and just fix them up
12:24:09 nightmare_unreal but I think cliff is more generic so adding a specific code will be bad
12:24:15 nightmare_unreal not sure I will submit though
12:24:59 mordred yeah - it's possible this can't be solved in cliff and we still might need to fix it somehow in osc
12:25:12 mordred I'll try to look in a little bit and see if I can make better suggestions
12:40:31 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Fix idempotency with server security groups https://review.opendev.org/734810
13:45:58 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Fix idempotency with server security groups https://review.opendev.org/734810
13:46:34 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test jobs https://review.opendev.org/732100
13:49:39 dtroyer mordred, nightmare_unreal: solving the general problem of nested structured output in cliff is the right place, however you may find that you still need to address individual situations in OSC too that don't fit the general solution.
13:51:06 nightmare_unreal yeah , makes sense. I will update . Thanks
14:04:13 dtroyer Dang, you guys have been busy! I can't wait to see what the command structure change does for startup times :)
14:04:36 gtema will be lightning fast :D
14:09:38 openstackgerrit Andreas Jaeger proposed openstack/service-types-authority master: Switch to newer openstackdocstheme version https://review.opendev.org/734852
14:11:15 mordred dtroyer: me either! :)
15:17:49 openstackgerrit Merged openstack/ansible-collections-openstack master: Add notes to README about deps and versions https://review.opendev.org/734557
19:20:38 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient stable/ussuri: Client should parse string to boolean for value 'is_domain' https://review.opendev.org/732397
19:23:56 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient stable/ussuri: Client should parse string to boolean for value 'is_domain' https://review.opendev.org/732397
19:27:48 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient stable/train: Client should parse string to boolean for value 'is_domain' https://review.opendev.org/732398
19:30:26 lbragstad mordred ^ that's a pretty straight forward backport and we have the tests working against the train/ussuri branches
21:07:59 openstackgerrit Merged openstack/openstacksdk master: Do not clean keypairs in the project cleanup https://review.opendev.org/734019
22:51:09 openstackgerrit Gabriel Ramirez proposed openstack/python-openstackclient master: Allow openstack flavor set to update flavor description using name https://review.opendev.org/733688
22:51:28 mordred lbragstad: lgtm
#openstack-sdks - 2020-06-11
00:10:31 openstackgerrit Monty Taylor proposed openstack/osc-lib master: Remove commandmanager subclass https://review.opendev.org/734000
02:37:03 openstackgerrit Merged openstack/python-openstackclient stable/ussuri: Client should parse string to boolean for value 'is_domain' https://review.opendev.org/732397
08:51:11 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add note about py3 in readme https://review.opendev.org/735081
09:45:52 openstackgerrit Dirk Mueller proposed openstack/osc-lib stable/ussuri: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/735090
09:48:06 openstackgerrit Dirk Mueller proposed openstack/python-openstackclient stable/ussuri: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/735091
09:55:57 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Fix ansible-devel sanity tests for deprecations https://review.opendev.org/735095
12:18:20 mordred dtroyer, mnaser, dtantsur|afk, frickler: with the move of the group code to cliff, the osc-lib patch is now green: https://review.opendev.org/#/c/734000/
12:34:54 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add note about py3 in readme https://review.opendev.org/735081
12:48:12 sshnaidm hmm.. I've got a weird error while trying to use conn.compute.wait_for_server() - AttributeError: 'Munch' object has no attribute 'fetch'
12:48:25 sshnaidm anybody knows what is the issue? ^
12:49:06 openstackgerrit jayaditya gupta proposed openstack/cliff master: Fix json output for security_groups property in "openstack server show -f json" command https://review.opendev.org/735128
12:50:39 nightmare_unreal frickler mordred , This is how I am thinking to fix the json properties issue. This i probably not the right way so please comment on it when anyone gets time :) Thanks
12:50:39 nightmare_unreal Patch : https://review.opendev.org/735128
12:57:16 mordred sshnaidm: sounds like you're using wait_for_server on something that's returning just the munch and not the Server resource - we haven't finished reworking the shade api to completely use the underlying sdk resource layer yet
12:57:35 mordred sshnaidm: so there are places still where the shade layer is still making direct rest calls
12:58:17 jawad_axd Hi all, Is it to possible to allocate cidr(subnet) automatically when creating network? I came acorss with get-me-network, but I dont really need router creation, just network with automatic cidr. Any pointers?
13:00:41 sshnaidm mordred, hmm.. maybe I don't understand something, but where is "resource" reinitialization in the loop? https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/resource.py#L2035
13:01:11 sshnaidm does it just repeat "resource = resource.fetch(session)" ?
13:01:39 mordred yes. iterate_timeout is a loop-until-timeout function
13:01:58 mordred sshnaidm: where are you calling that?
13:02:38 mordred sshnaidm: (I can almost certaily tell you how to do things in your code differently so it works, but what would be great is to take this as an opportunity to fix whatever gave you a server that doesn't work there)
13:03:16 sshnaidm mordred, I'm calling from ansible module: self.conn.compute.wait_for_server(os_server,status=_action_map[self.params['action']],wait=self.params['timeout'])
13:03:39 mordred sshnaidm: nod - and what did you call to get teh os_server object?
13:04:21 sshnaidm mordred, self.conn.get_server(self.params['server'])
13:04:34 mordred sshnaidm: wow
13:05:16 mordred oh - wait
13:05:19 mordred no, that's expected
13:05:24 mordred and we should fix it - but for now
13:05:34 mordred use self.conn.wait_for_server if you use self.conn.get_server
13:06:08 mordred (although that's not going to let you pass in a status)
13:06:37 sshnaidm yeah, server is <class 'munch.Munch'>
13:06:39 mordred sshnaidm: actually - for this case ...
13:06:57 mordred sshnaidm: try using self.conn.compute.get_server(self.params['server'])
13:06:58 frickler jawad_axd: with subnetpools creating a subnet is pretty simple, but still a dedicated action, I don't think there's a way around that
13:08:35 mordred sshnaidm: we need to finish https://review.opendev.org/#/c/630912 to fix the other thing, and that's important, but probably not a thing you want to wait on
13:08:40 jawad_axd @frickler Thanks.
13:09:26 sshnaidm mordred, hmm, it didn't find server with self.conn.compute.get_server(self.params['server'])
13:11:09 sshnaidm and w/o compute: TypeError: wait_for_server() got an unexpected keyword argument 'status'
13:11:31 sshnaidm I think I'll wait a little to introduce this function..
13:11:49 mordred sshnaidm: yeah - there's no status parameter for self.conn.wait_for_server
13:12:37 mordred sshnaidm: one more thign to try (sorry, still on first coffee)
13:12:50 mordred sshnaidm: self.conn.compute.find_server(self.params['server'])
13:15:43 sshnaidm mordred, that seems working
13:21:34 sshnaidm mordred, <class 'openstack.compute.v2.server.Server'> - it seems to have all attributes as "None"
13:25:32 sshnaidm I've got only ID and name
13:35:09 openstackgerrit Merged openstack/ansible-collections-openstack master: Fix ansible-devel sanity tests for deprecations https://review.opendev.org/735095
13:41:43 mordred sshnaidm: that resource doesn't seem to have been fetched
13:42:31 sshnaidm mordred, well, it got id and name

Earlier   Later