Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ZOOKEEPER-4784: Token based ASF JIRA authentication #2106

Merged
merged 1 commit into from Feb 9, 2024

Conversation

szucsvillo
Copy link
Contributor

@szucsvillo szucsvillo commented Jan 10, 2024

No description provided.

Change-Id: I71203b7c02d4e25ce47c11eea859c372ed0c18b6
Copy link
Contributor

@anmolnar anmolnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great stuff, thanks @szucsvillo !
I'll try it on a real patch soon.

Copy link
Contributor

@anmolnar anmolnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@szucsvillo

Got the following error message when merging #2100

Would you like to update an associated JIRA? (y/n): y
Traceback (most recent call last):
  File "zk-merge-pr.py", line 528, in <module>
    main()
  File "zk-merge-pr.py", line 514, in main
    resolve_jira_issues(commit_title, merged_refs, jira_comment)
  File "zk-merge-pr.py", line 321, in resolve_jira_issues
    resolve_jira_issue(merge_branches, comment, jira_id)
  File "zk-merge-pr.py", line 260, in resolve_jira_issue
    asf_jira = jira.client.JIRA(jira_server, token_auth=JIRA_ACCESS_TOKEN)
TypeError: __init__() got an unexpected keyword argument 'token_auth'

Please check.

Copy link
Contributor

@anmolnar anmolnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine now. I had to upgrade python-jira to 3.6.0. This should be noted on the wiki page later.
Thanks @szucsvillo !

@anmolnar anmolnar merged commit e571dd8 into apache:master Feb 9, 2024
1 check passed
AlphaCanisMajoris pushed a commit to AlphaCanisMajoris/zookeeper that referenced this pull request Mar 28, 2024
Reviewers: anmolnar, anmolnar
Author: szucsvillo
Closes apache#2106 from szucsvillo/ZOOKEEPER-4784
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants