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

ci: optimize build rpm workflow with apisix-build-tools v2.2.0 #2133

Merged
merged 2 commits into from
Sep 12, 2021
Merged

ci: optimize build rpm workflow with apisix-build-tools v2.2.0 #2133

merged 2 commits into from
Sep 12, 2021

Conversation

qsliu2017
Copy link
Contributor

@qsliu2017 qsliu2017 commented Sep 8, 2021

Please answer these questions before submitting a pull request, or your PR will get closed.

Why submit this pull request?

  • Bugfix
  • New feature provided
  • Improve performance
  • Backport patches

What changes will this PR take into?

As discussed in #2072, we delay using buildx in auto-build-rpm workflow since it relates to the build tool. Now we have updated the build tool(api7/apisix-build-tools#74), it's time to make this change!

Related issues

fix #2128

Checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@imjoey
Copy link
Member

imjoey commented Sep 8, 2021

Hi @qsliu2017, happy to see you committed this PR. I've closed the #2132, which did the same thing. Let's work on this. Thanks.

@imjoey imjoey changed the title ci: optimize build rpm workflow ci: optimize build rpm workflow with apisix-build-tools v2.2.0 Sep 8, 2021
@qsliu2017
Copy link
Contributor Author

Hi @qsliu2017, happy to see you committed this PR. I've closed the #2132, which did the same thing. Let's work on this. Thanks.

Thank you so much, I've updated build tool version to v2.2.0

Copy link
Member

@imjoey imjoey left a comment

Choose a reason for hiding this comment

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

/lgtm

@liuxiran liuxiran merged commit 65e2528 into apache:master Sep 12, 2021
@qsliu2017 qsliu2017 deleted the optimize-build-rpm-ci branch September 12, 2021 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Bump apisix-build-tools version to v2.2.0
6 participants