Tuesday, 2020-12-08

*** tosky has quit IRC00:54
*** openstackgerrit has quit IRC00:58
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-sdks05:33
*** ricolin has quit IRC06:24
*** ricolin has joined #openstack-sdks06:30
*** ricolin has quit IRC06:36
*** ricolin has joined #openstack-sdks06:53
*** lxkong has quit IRC07:09
*** ricolin has quit IRC07:23
*** nikparasyr has joined #openstack-sdks07:26
*** ricolin has joined #openstack-sdks07:39
*** gtema has joined #openstack-sdks07:48
*** openstackgerrit has joined #openstack-sdks07:48
openstackgerritPolina Gubina proposed openstack/ansible-collections-openstack master: Enabling update for recordset  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76591307:48
*** slaweq has joined #openstack-sdks08:08
*** rpittau|afk is now known as rpittau08:10
*** lxkong has joined #openstack-sdks09:00
*** jpich has joined #openstack-sdks09:01
*** Hybrid512 has joined #openstack-sdks09:02
openstackgerritTino Schreiber proposed openstack/ansible-collections-openstack master: Add security_group_rule_info module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76558009:55
*** jpich has quit IRC10:01
*** jpich has joined #openstack-sdks10:02
openstackgerritShnaidman Sagi proposed openstack/ansible-collections-openstack master: Fix docs-args mismatch in modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76596010:06
*** jpich has quit IRC10:52
*** jpich has joined #openstack-sdks10:53
*** dtantsur|afk is now known as dtantsur11:00
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Fix lower-constraints job  https://review.opendev.org/c/openstack/python-openstackclient/+/76596811:23
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Update lower-constraints  https://review.opendev.org/c/openstack/python-openstackclient/+/76596911:23
stephenfingtema: dtantsur: That should fix the OSC gate ^11:23
gtemais it again broken?11:24
stephenfinI don't know if you've a patch proposed to fix it but master is broken locally for me, yes11:24
gtemaI haven't recognized so far it was broken11:24
stephenfinit's the new pip dependency resolver highlighting bugs in our lower-cosntraints11:25
stephenfin*constraints11:25
stephenfinopenstacksdk is okay11:25
gtemaokay. Let's wait what zuul tells11:31
*** jpich has quit IRC11:41
*** jpich has joined #openstack-sdks11:42
dtantsurpip releases are always fun11:45
openstackgerritTino Schreiber proposed openstack/openstacksdk master: Add id query parameter to sg rules  https://review.opendev.org/c/openstack/openstacksdk/+/76598011:49
*** jpich has quit IRC11:58
*** jpich has joined #openstack-sdks11:59
*** dtantsur has quit IRC12:50
*** Luzi has joined #openstack-sdks13:00
*** dtantsur has joined #openstack-sdks13:06
*** jpich has quit IRC13:14
*** jpich has joined #openstack-sdks13:15
openstackgerritMerged openstack/ansible-collections-openstack master: Fix docs-args mismatch in modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76596013:26
*** tosky has joined #openstack-sdks13:44
openstackgerritPolina Gubina proposed openstack/ansible-collections-openstack master: Enabling update for recordset  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76591313:58
*** slaweq has quit IRC14:03
*** slaweq has joined #openstack-sdks14:04
*** tkajinam has quit IRC14:47
*** tkajinam has joined #openstack-sdks14:47
openstackgerritShnaidman Sagi proposed openstack/ansible-collections-openstack master: Add security_group_rule_info module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76558014:52
openstackgerritShnaidman Sagi proposed openstack/ansible-collections-openstack master: Migrating network from AnsibleModule to OpenStackModule  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76441114:56
openstackgerritShnaidman Sagi proposed openstack/ansible-collections-openstack master: WIP add network tests for versioned args  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76601315:05
openstackgerritGhanshyam proposed openstack/openstackclient master: Remove retired Qinling & Searchlight usage  https://review.opendev.org/c/openstack/openstackclient/+/76455615:05
*** tkajinam has quit IRC15:18
gmannstephenfin: amotoki gtema squashed the Qinling and Searchlight retirement patches as they conflict with each other: https://review.opendev.org/c/openstack/openstackclient/+/764556/215:27
gtemathanks15:27
*** Luzi has quit IRC15:31
*** tosky has quit IRC15:45
openstackgerritMerged openstack/openstackclient master: Remove retired Qinling & Searchlight usage  https://review.opendev.org/c/openstack/openstackclient/+/76455615:46
*** nikparasyr has left #openstack-sdks15:53
sshnaidmgtema, do you know maybe since which version openstack client has arg '--long'? Is it old enough?16:00
gtemayes, enough age16:00
gtemaat least since queens, but sure it was even earlier16:01
gtemait all depends on concrete command16:01
openstackgerritMerged openstack/ansible-collections-openstack master: Migrating volume from AnsibleModule to OpenStackModule  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76554216:05
openstackgerritMerged openstack/python-openstackclient master: Fix lower-constraints job  https://review.opendev.org/c/openstack/python-openstackclient/+/76596816:13
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Show flavor in 'server list' with API >= 2.47  https://review.opendev.org/c/openstack/python-openstackclient/+/76177216:32
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add missing options for 'server rebuild'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536016:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: trivial: Cleanup docs for 'server list'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536116:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add missing options for 'server list'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536216:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add missing options for 'server set'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536316:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add missing options for 'keypair list'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536416:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add missing options for 'server image create'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536516:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add 'server volume update' command  https://review.opendev.org/c/openstack/python-openstackclient/+/76536616:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add 'server volume list' command  https://review.opendev.org/c/openstack/python-openstackclient/+/76536716:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add missing options for 'hypervisor list'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536816:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add '--force' option to 'server delete'  https://review.opendev.org/c/openstack/python-openstackclient/+/76536916:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Improve 'server migration list' options  https://review.opendev.org/c/openstack/python-openstackclient/+/76537016:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: tests: Remove unused fake method  https://review.opendev.org/c/openstack/python-openstackclient/+/76604016:38
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Make use of comparable 'FormattableColumn' subclasses  https://review.opendev.org/c/openstack/python-openstackclient/+/76144716:39
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server * -f yaml' output  https://review.opendev.org/c/openstack/python-openstackclient/+/76120516:39
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'usage * -f yaml' output  https://review.opendev.org/c/openstack/python-openstackclient/+/76159516:39
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server group * -f yaml' output  https://review.opendev.org/c/openstack/python-openstackclient/+/76159616:39
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'hypervisor show -f yaml' output  https://review.opendev.org/c/openstack/python-openstackclient/+/76300416:39
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add 'server group create --rule' option  https://review.opendev.org/c/openstack/python-openstackclient/+/76159716:39
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Add 'server show --topology' option  https://review.opendev.org/c/openstack/python-openstackclient/+/68092816:39
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: trivial: Use plural for appended parameters  https://review.opendev.org/c/openstack/python-openstackclient/+/76159816:39
stephenfinalright, I'll go review stuff inside of writing code now :)16:40
stephenfinThough i would appreciate reviews. It's all very trivial and everything has tests fwict16:40
*** dulek has quit IRC16:44
*** dulek has joined #openstack-sdks16:53
openstackgerritWill Tome proposed openstack/ansible-collections-openstack master: add option to exclude legacy groups  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76604716:54
*** efried has quit IRC17:00
*** jpich has quit IRC17:10
*** efried has joined #openstack-sdks17:15
*** tosky has joined #openstack-sdks17:26
openstackgerritMerged openstack/python-openstackclient master: Update lower-constraints  https://review.opendev.org/c/openstack/python-openstackclient/+/76596917:28
*** gtema has quit IRC17:41
*** rpittau is now known as rpittau|afk17:43
*** Hybrid512 has quit IRC17:48
*** dtantsur is now known as dtantsur|afk18:05
openstackgerritAnanya proposed openstack/ansible-collections-openstack master: WIP add network tests for versioned args  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/76601318:05
openstackgerritGhanshyam proposed openstack/python-openstackclient master: Remove retired Searchlight support  https://review.opendev.org/c/openstack/python-openstackclient/+/76456919:50
gmannstephenfin: ^^ resolved merge conflict for lower-constraints, please check again19:51
rm_workSo, looks like the SDK supports a cert/key for mTLS with services -- does this for sure work with EVERY service? do they all generate their clients in a unified way? or is it possible some SDK service doesn't support this and silently ignores it?21:11
*** artom has quit IRC21:39
*** artom has joined #openstack-sdks21:40
*** ebbex has quit IRC21:55
*** ebbex has joined #openstack-sdks21:55
*** tkajinam has joined #openstack-sdks23:01
mordredrm_work: it's actually keystoneauth that's the one that supports that - it's all passthrough/exposure support from an openstacksdk perspective23:21
rm_workok cool23:21
mordredrm_work: that said - unless you say "insecure: true" - any cert/ca settings that are invalid will cause keystoneuath to vomit - so I do not think silent ignoring will be a thing23:23
mordredworth testing though23:23
rm_worki was more concerned with like, one of the clients not actually using the Session thing23:23
rm_workand therefore just not using the settings23:23
mordredrm_work: openstacksdk does not use any of the clients23:25
mordredit only taks to services using keystoneauth23:25
mordredthe thing you mention is one of many reasons this is true23:25
rm_workhmm k23:26
rm_worki guess i meant like, the individual service-client implementations INSIDE sdk23:27
rm_workbut maybe the requirement for them to be in SDK is they must use that auth framework23:27
mordredyes. they actually arent' individual implementations23:27
mordredthey are all subclasses of openstack.proxy.Proxy which is a subclass of keystoneauth1.adapter.Adapter :)23:27
mordredthe amount of work to make a service proxy in sdk that did not use keystoneauth properly would be GIANT23:28
mordredI'm honestly not sure I could even write that code23:28

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!