Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-05-12
15:20:20 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
15:20:21 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Remove incorrect os_ prefix in documentation https://review.opendev.org/727226
15:20:55 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
15:25:37 mordred noonedeadpunk: I haven't - but I try to stay as far away from python-*client as I can, so I'm not much help
15:32:07 noonedeadpunk mordred: ok, I see:) I didn't found anything there as well actualy:( so maybe it's in horizon...
15:35:15 mordred noonedeadpunk: we should add to our already-way-too-large-todo-list updating horizon to use sdk :)
15:36:03 noonedeadpunk that would be great... but tough:)
15:49:42 mordred noonedeadpunk: yeah. tell me about it
16:03:34 openstackgerrit Merged openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952
18:13:55 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
18:16:45 openstackgerrit Merged openstack/ansible-collections-openstack master: Remove incorrect os_ prefix in documentation https://review.opendev.org/727226
18:16:46 openstackgerrit Merged openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
18:24:34 openstackgerrit Merged openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538
18:39:46 openstackgerrit Merged openstack/openstacksdk master: Fix issues found by latest flake8 https://review.opendev.org/727204
19:22:20 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
19:34:42 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Revert "Run linters in a temp dir instead of an install" https://review.opendev.org/727400
19:40:00 tremble mordred, baremetal_node_action (Ansible module) - We talked about fixing some of the 'bool's that are currently strings when I did the big batch of cleanup, technically right now we allow several values that aren't 'bool's (and aren't in the description as valid) would you rather a formal deprecation cycle, or just clean it up for v1 of the collection?
19:40:48 mordred tremble: I say just clean it up for v1 of the collection ... sshnaidm what do you think?
19:41:47 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
19:42:12 sshnaidm mordred, yeah, i think it's ok
19:43:11 tremble ok
19:52:13 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Improve ansible-test sanity for collection https://review.opendev.org/727411
19:54:52 AJaeger mordred: https://review.opendev.org/#/c/727352/ removes pbrx, hope that the jobs now passes. could you review later, please?
19:56:24 mordred AJaeger: oh thanks! yes - that looks good
19:58:33 AJaeger mordred: doesn't work, it's failing ;(
19:58:57 AJaeger do we need to remove the repos from the constraints list first?
20:01:02 tremble mordred, Do you happen to know the history of https://github.com/openstack/ansible-collections-openstack/blob/master/plugins/modules/baremetal_node_action.py#L212 I think that's actually unreachable because of the preceding if statement
20:01:47 AJaeger mordred: yeah, this basically needs rewrite of the siblings role ;( should we keep pbrx or move it over?
20:02:11 mordred tremble: I don't - I think we'd want to ask dtantsur|afk or TheJulia
20:02:24 AJaeger mordred: we could also do some ansible magic...
20:02:50 mordred AJaeger: hrm
20:03:25 AJaeger cp constraitns-file and then iterate with lineinfile to remove the lines
20:04:41 tremble mordred, Ok, I'll try dtantsur|afk in our morning
20:08:44 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
20:09:07 AJaeger mordred: let's try again - if you have a better idea or a fix, feel free to take over. I'm signing off for today
20:09:22 mordred AJaeger: :) thank you!
20:10:26 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Improve ansible-test sanity for collection https://review.opendev.org/727411
20:20:59 TheJulia tremble: I have this fear I may have been on a third whiskey at a hotel bar in downtown seattle when I wrote that
20:21:24 tremble Think I just nuke the lines?
20:21:33 TheJulia I remember feeling inspired and working out such caseI feel like your likely safe doing so.
20:35:20 TheJulia tremble: may not be a bad idea. I can alloc brain cells tomorrow because the module was basically overloaded
20:43:33 mordred TheJulia: we might also be in a position now where we have some of this logic covered down in sdk but we never got around to nuking it from the modules
21:43:21 TheJulia mordred: yeah, I think a lot of it was to cover "you should try to do this and that at the same time" kind of stuff
21:53:08 openstackgerrit Merged openstack/openstacksdk master: Set BaseImageProxy.create_image validate_checksum default to False https://review.opendev.org/726465
#openstack-sdks - 2020-05-13
00:25:44 openstackgerrit Ghanshyam Mann proposed openstack/keystoneauth master: Fix hacking min version to 3.0.1 https://review.opendev.org/727498
01:37:42 openstackgerrit Steve Baker proposed openstack/keystoneauth master: Implement HTTP Basic client support in keystoneauth1 https://review.opendev.org/727562
04:10:06 iokiwi known issues with openstack clients on ubuntu 20.04? Running into a problem with openstack image save freezing after authenticating
04:11:48 iokiwi I can see it does keystone auth and gets a token and then tries to start downloading from glance, but it just gets stuck on the first GET. I can dive deeper I just wondered if anyone else had experienced something similar / could confirm the behavior
04:23:58 openstackgerrit Merged openstack/openstacksdk master: Remove some unneeded things from test-requirements https://review.opendev.org/715707
04:23:59 openstackgerrit Merged openstack/openstacksdk master: Remove uses of from six.moves https://review.opendev.org/715712
04:25:35 openstackgerrit Merged openstack/openstacksdk master: Remove use of six https://review.opendev.org/715713
05:22:51 openstackgerrit Pete Zaitcev proposed openstack/python-openstackclient master: Cleanup: remove a useless reference to "object" https://review.opendev.org/727632
05:54:05 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
05:54:42 openstackgerrit Merged openstack/openstacksdk master: Fix image owner field https://review.opendev.org/726627
05:54:43 openstackgerrit Merged openstack/openstacksdk stable/train: Don't error if clouds.yaml is not readable https://review.opendev.org/726963
06:12:53 iokiwi disregard my previous query I think I found my problem
06:14:01 iokiwi mem leak with openssl on client with whatever cipher / tls version negotiated with the API
06:28:42 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
06:30:51 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: DNM: Test openstackclient-check-plugins https://review.opendev.org/727637
06:44:47 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
06:47:37 AJaeger mordred: the above should be fine now - the test is failing on cloudkittyclient.v1.collector_cli (see https://zuul.opendev.org/t/openstack/build/ac3fbea5df684718956ebf2c3a79b7d1 )
06:54:23 AJaeger mordred: looking at 727637: That one passes. So, my test is not accurate since it tests too many libs, cloudkitty is not part of the requirements that we should test.
07:10:05 openstackgerrit Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
07:23:09 AJaeger mordred: yeah, passing now ^
10:28:51 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Improve ansible-test sanity for collection https://review.opendev.org/727411
10:31:40 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Improve ansible-test sanity for collection https://review.opendev.org/727411
11:05:48 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add template for generation of artibtrary module https://review.opendev.org/727216
11:35:51 openstackgerrit Merged openstack/openstacksdk master: Stop subclassing object https://review.opendev.org/715714
12:13:00 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Redesign OpenstackModule class https://review.opendev.org/722283
12:49:39 openstackgerrit Merged openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352
13:02:27 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add Contributing file https://review.opendev.org/727752
13:07:04 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Add contributing file https://review.opendev.org/727753
14:02:15 mordred AJaeger: thanks for following up on that! ^^ pbrx removal has been merged
14:02:33 AJaeger great, mordred ! That was the only blocker for pbrx retirement.
14:02:37 AJaeger Fun ansible hacking ;)
14:03:24 AJaeger mordred: I'm glad that my solution is soo much shorter than the pbrx siblings implementation ;)
14:05:36 mordred AJaeger: me too!
14:11:43 openstackgerrit Merged openstack/ansible-collections-openstack master: Add contributing file https://review.opendev.org/727753
14:18:48 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Link to Contributing file in README https://review.opendev.org/727776
15:19:01 openstackgerrit Merged openstack/ansible-collections-openstack master: Improve ansible-test sanity for collection https://review.opendev.org/727411
19:03:47 openstackgerrit Merged openstack/ansible-collections-openstack master: Link to Contributing file in README https://review.opendev.org/727776
#openstack-sdks - 2020-05-14
05:40:00 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Update README with additional information https://review.opendev.org/727927
06:53:53 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097
07:36:06 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add template for generation of artibtrary module https://review.opendev.org/727216
09:31:26 sshnaidm where can I see all available filters for list_server for example? https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/cloud/_compute.py#L292
11:04:53 sshnaidm dtantsur, gtema do you know maybe? ^
11:05:12 gtema in the server resource itself
11:05:42 gtema https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/compute/v2/server.py#L32
13:21:17 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Fix AttributeError exception during authorization https://review.opendev.org/728077
14:02:15 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Remove ANSIBLE_METADATA from all modules https://review.opendev.org/728093
14:06:18 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add template for generation of artibtrary module https://review.opendev.org/727216
15:23:25 openstackgerrit Merged openstack/ansible-collections-openstack master: Update README with additional information https://review.opendev.org/727927
15:25:43 openstackgerrit Tim Burke proposed openstack/python-openstackclient master: Fix pep8 job https://review.opendev.org/728116
15:43:04 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add OpenstackModule to os_server_action https://review.opendev.org/721746
16:00:17 elmiko API SIG office hour now open \o/
16:07:47 openstackgerrit Merged openstack/ansible-collections-openstack master: Remove ANSIBLE_METADATA from all modules https://review.opendev.org/728093
17:05:31 elmiko have a good weekend all, stay safe out there! o/
17:05:53 gtema_ thks elmiko
18:55:07 openstackgerrit Pete Zaitcev proposed openstack/python-openstackclient master: Resolve PEP8 https://review.opendev.org/728258
#openstack-sdks - 2020-05-15
01:25:31 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: Use elif for processing is_domain https://review.opendev.org/728320

Earlier   Later