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

Fix automatic release on Fedora by importing python-jira in makebumpver late #5400

Conversation

jkonecny12
Copy link
Member

With the current approach we are adding unnecessary package requirement for Fedora releases where JIRA is not used at all. Let's move python-jira usage to place where it's used to avoid complexity for Fedora release process and automation.

With the current approach we are adding unnecessary package requirement
for Fedora releases where JIRA is not used at all. Let's move
python-jira usage to place where it's used to avoid complexity for
Fedora release process and automation.
@jkonecny12 jkonecny12 added master Please, use the `f39` label instead. infrastructure Changes affecting mostly infrastructure labels Jan 12, 2024
@github-actions github-actions bot added f40 and removed infrastructure Changes affecting mostly infrastructure labels Jan 12, 2024
Copy link

Infrastructure check failed on these files. Please do a double check of these files before merge!

scripts/makebumpver

@jkonecny12
Copy link
Member Author

/kickstart-test --waive infra only

@@ -328,6 +328,9 @@ class MakeBumpVer:
continue

if self.rhel:
# First place where we need to use JIRA server, don't require it until now
self._jira.connect()
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't it s/self._jira/self.jira?

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh ignore I see now.

Copy link
Member Author

Choose a reason for hiding this comment

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

Probably should be but let's not complicate things here. :)

@jkonecny12 jkonecny12 merged commit b330905 into rhinstaller:master Jan 12, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f40 master Please, use the `f39` label instead.
Development

Successfully merging this pull request may close these issues.

2 participants