)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7c87bb22ea11bc05e123b1771351f8adcd2febe5","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Takashi Kajinami \u003ctkajinam@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-05-07 00:36:40 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deprecate [designate] auth_* parameters"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The auth_* parameters are implementing the same functionality as"},{"line_number":10,"context_line":"keystoneauth parameters alghouth these doesn\u0027t fully implement required"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bfd2d3d8_faad86ef","line":7,"range":{"start_line":7,"start_character":22,"end_line":7,"end_character":26},"updated":"2021-05-06 16:02:52.000000000","message":"admin?","commit_id":"d276cb5ec32242ea74d4eb7abe5ac3da04b23166"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a9d6c3286f63ca86fb5cb65e515aca8e9d0ec042","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Takashi Kajinami \u003ctkajinam@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-05-07 00:36:40 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deprecate [designate] auth_* parameters"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The auth_* parameters are implementing the same functionality as"},{"line_number":10,"context_line":"keystoneauth parameters alghouth these doesn\u0027t fully implement required"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9969b644_eab044dd","line":7,"range":{"start_line":7,"start_character":22,"end_line":7,"end_character":26},"in_reply_to":"bfd2d3d8_faad86ef","updated":"2021-05-06 16:12:46.000000000","message":"Done","commit_id":"d276cb5ec32242ea74d4eb7abe5ac3da04b23166"}],"neutron/conf/services/extdns_designate_driver.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"6779bb834b7d084d709473f31d8a961c12efacdf","unresolved":true,"context_lines":[{"line_number":49,"context_line":"    cfg.StrOpt(\u0027admin_auth_url\u0027,"},{"line_number":50,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":51,"context_line":"               deprecated_reason\u003d(\"This option will be completely replaced by \""},{"line_number":52,"context_line":"                                  \"keystoneauth parameters.\"),"},{"line_number":53,"context_line":"               help\u003d_(\u0027Authorization URL for connecting to designate in admin \u0027"},{"line_number":54,"context_line":"                      \u0027context\u0027)),"},{"line_number":55,"context_line":"    cfg.BoolOpt(\u0027allow_reverse_dns_lookup\u0027, default\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9e745f4e_e8ceafa7","line":52,"updated":"2021-05-10 21:18:16.000000000","message":"nit: can You add \"deprecated_since\" parameter to all of those? It will be easier in the future to find out when it was deprecated and if it\u0027s time to remove it already or not yet :)","commit_id":"5366217fadf2b15c86f2ad015a7f9a929e10d53e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a13b562d52684e05bfbb527c4a1a7ec1bc85efa3","unresolved":true,"context_lines":[{"line_number":49,"context_line":"    cfg.StrOpt(\u0027admin_auth_url\u0027,"},{"line_number":50,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":51,"context_line":"               deprecated_reason\u003d(\"This option will be completely replaced by \""},{"line_number":52,"context_line":"                                  \"keystoneauth parameters.\"),"},{"line_number":53,"context_line":"               help\u003d_(\u0027Authorization URL for connecting to designate in admin \u0027"},{"line_number":54,"context_line":"                      \u0027context\u0027)),"},{"line_number":55,"context_line":"    cfg.BoolOpt(\u0027allow_reverse_dns_lookup\u0027, default\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"bec4d1c1_a14ef758","line":52,"in_reply_to":"9e745f4e_e8ceafa7","updated":"2021-05-10 23:30:26.000000000","message":"That makes sense. I updated the patch according to your suggestion !","commit_id":"5366217fadf2b15c86f2ad015a7f9a929e10d53e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"3a71b8e26d40571f79f51be98c00913d2a4c6138","unresolved":true,"context_lines":[{"line_number":24,"context_line":"               help\u003d_(\u0027URL for connecting to designate\u0027)),"},{"line_number":25,"context_line":"    cfg.StrOpt(\u0027admin_username\u0027,"},{"line_number":26,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":27,"context_line":"               deprecated_since\u003dversionutils.deprecated.XENA,"},{"line_number":28,"context_line":"               deprecated_reason\u003d(\"This option will be completely replaced by \""},{"line_number":29,"context_line":"                                  \"keystoneauth parameters.\"),"},{"line_number":30,"context_line":"               help\u003d_(\u0027Username for connecting to designate in admin \u0027"}],"source_content_type":"text/x-python","patch_set":7,"id":"b1eb24df_c830c927","line":27,"range":{"start_line":27,"start_character":32,"end_line":27,"end_character":60},"updated":"2021-05-11 03:09:31.000000000","message":"It seems this is not available in oslo.log which is installed according to upper constraints now.","commit_id":"c06817284a6a5d470a219b7675a993e07576d3d1"}],"neutron/services/externaldns/drivers/designate/driver.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2b21e993aa08c9ef78fa104d8d2f8c33e63e2a9c","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        admin_auth \u003d loading.load_auth_from_conf_options("},{"line_number":52,"context_line":"            CONF, \u0027designate\u0027)"},{"line_number":53,"context_line":"    else:"},{"line_number":54,"context_line":"        LOG.warning(\u0027[designate] auth_type and keystoneauth parameters will \u0027"},{"line_number":55,"context_line":"                    \u0027be required instead of [designate] admin_* parameters \u0027"},{"line_number":56,"context_line":"                    \u0027in a future release.\u0027)"},{"line_number":57,"context_line":"        admin_auth \u003d password.Password("}],"source_content_type":"text/x-python","patch_set":5,"id":"368c3a99_2a209929","line":54,"range":{"start_line":54,"start_character":8,"end_line":54,"end_character":11},"updated":"2021-05-06 16:58:07.000000000","message":"I don\u0027t know if this is good idea: with \"warning\" the message will be shown only once; with LOG.warning, the message will be printed always.\n\nWe have documented the deprecation of those config vars and we have the original \"warning\". IMO, this is enough.","commit_id":"70d1546c89b6ab0840319692fce9bcbdd5615162"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"98e8c09e1ca3d059d4bfa0531c7a08dc1ccf1986","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        admin_auth \u003d loading.load_auth_from_conf_options("},{"line_number":52,"context_line":"            CONF, \u0027designate\u0027)"},{"line_number":53,"context_line":"    else:"},{"line_number":54,"context_line":"        LOG.warning(\u0027[designate] auth_type and keystoneauth parameters will \u0027"},{"line_number":55,"context_line":"                    \u0027be required instead of [designate] admin_* parameters \u0027"},{"line_number":56,"context_line":"                    \u0027in a future release.\u0027)"},{"line_number":57,"context_line":"        admin_auth \u003d password.Password("}],"source_content_type":"text/x-python","patch_set":5,"id":"e53a56b0_e13a5f60","line":54,"range":{"start_line":54,"start_character":8,"end_line":54,"end_character":11},"in_reply_to":"368c3a99_2a209929","updated":"2021-05-07 06:14:44.000000000","message":"I agree with you and removed this warning.\n\nI left TODO comment to record my next plan for removal but hope this looks good.","commit_id":"70d1546c89b6ab0840319692fce9bcbdd5615162"}],"releasenotes/notes/deprecate-designate-auth-opts-c2b1050a0360981c.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7c87bb22ea11bc05e123b1771351f8adcd2febe5","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"deprecations:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The following parameters in the ``designate`` section has been deprecated"},{"line_number":5,"context_line":"    and will be removed in a future release."},{"line_number":6,"context_line":"    The ``[designate] auth_type`` parameter and required keystoneauth"},{"line_number":7,"context_line":"    parameters should be used instead."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"83312283_d59b76e3","line":4,"range":{"start_line":4,"start_character":58,"end_line":4,"end_character":61},"updated":"2021-05-06 16:02:52.000000000","message":"have","commit_id":"d276cb5ec32242ea74d4eb7abe5ac3da04b23166"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a9d6c3286f63ca86fb5cb65e515aca8e9d0ec042","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"deprecations:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The following parameters in the ``designate`` section has been deprecated"},{"line_number":5,"context_line":"    and will be removed in a future release."},{"line_number":6,"context_line":"    The ``[designate] auth_type`` parameter and required keystoneauth"},{"line_number":7,"context_line":"    parameters should be used instead."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"058d83e6_83b6c5dd","line":4,"range":{"start_line":4,"start_character":58,"end_line":4,"end_character":61},"in_reply_to":"83312283_d59b76e3","updated":"2021-05-06 16:12:46.000000000","message":"Done","commit_id":"d276cb5ec32242ea74d4eb7abe5ac3da04b23166"}]}
