senlin

parameters for keystoneauth1 is wrong

Bug #2033750 reported by Xing Zhang on 2023-09-01

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	senlin	Invalid	Undecided	Xing Zhang

Bug Description

zaqar section are update from string to cfg.OptGroup in [1][2], but the second parameter for
ks_loading.register_session_conf_options and register_auth_conf_options is string[3][4]

also, the type in sample config docs[5] are unknown type.

[1] https://github.com/openstack/senlin/commit/01eff94b7625c64b3342830ea2b574be15dbef8d#diff-0cb20a6e04ea9bc84a80ac5ba046cd68a04eb465a05736a54bd6944cf2e811c2L286-L287
[2] https://github.com/openstack/senlin/commit/01eff94b7625c64b3342830ea2b574be15dbef8d#diff-32adfd6ff262eef4848530d4229436bc6087a556db3a51d5bec43f40fb8663ceR25-R26
[3] https://github.com/openstack/keystoneauth/blob/e6f3999c6f2f846e3dda505343166ab8c8346c2a/keystoneauth1/loading/__init__.py#L29
https://github.com/openstack/keystoneauth/blob/e6f3999c6f2f846e3dda505343166ab8c8346c2a/keystoneauth1/loading/conf.py#L82
[4] https://github.com/openstack/keystoneauth/blob/e6f3999c6f2f846e3dda505343166ab8c8346c2a/keystoneauth1/loading/__init__.py#L34
https://github.com/openstack/keystoneauth/blob/e6f3999c6f2f846e3dda505343166ab8c8346c2a/keystoneauth1/loading/session.py#L216
[5] https://docs.openstack.org/senlin/latest/configuration/config.html#zaqar

Xing Zhang (xingzhang) on 2023-09-01

Changed in senlin:
assignee:	nobody → Xing Zhang (xingzhang)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2023-09-01: Fix proposed to senlin (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/senlin/+/893480

Changed in senlin:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2023-09-06: Change abandoned on senlin (master)

Change abandoned by "Xing Zhang <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/senlin/+/893480
Reason: Not a bug, won't fix. Only affect docs description: https://docs.openstack.org/senlin/latest/configuration/config.html#zaqar.auth_type

Revision history for this message

Xing Zhang (xingzhang) wrote on 2023-09-06:

These are used in:

auth = ks_loading.load_auth_from_conf_options(CONF, 'zaqar')
session = ks_loading.load_session_from_conf_options(CONF, 'zaqar')

https://github.com/openstack/senlin/blob/cef004c05bbc209239e749270ad090063971b4d7/senlin/engine/receivers/message.py#L80-L81

Not a bug, won't fix. Only affect docs description: https://docs.openstack.org/senlin/latest/configuration/config.html#zaqar.auth_type

Changed in senlin:
status:	In Progress → Invalid

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.