Skip to content

Conversation

@base698
Copy link

@base698 base698 commented Oct 9, 2025

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

This PR improves container build flexibility and unifies the JIRA epic collector to use the v3 /search/jql endpoint exclusively.

Key changes:

  • Makefiles
    • Introduces CONTAINER_CMD (defaults to docker) and PLATFORM (defaults to linux/amd64) for flexible builds across environments.
    • Updates all docker build commands to respect $(CONTAINER_CMD) and --platform=$(PLATFORM).
  • Backend Dockerfile
    • Adds -Wno-incompatible-pointer-types and USE_BUNDLED_ZLIB=ON to CMake flags for stable cross-compile of libgit2.
    • Simplifies make install -j invocation.
  • JIRA Epic Collector
    • Removes api/2/search (Server/DC) collector path.
    • Always uses the api/3/search/jql endpoint.
    • Inlines next-page handling via nextPageToken and removes redundant v2/v3 helper functions.

Does this close any open issues?


Screenshots

N/A


Other Information

Motivation

  • Jira Collection broken against atlassian.net

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Oct 9, 2025
Copy link
Contributor

@klesh klesh left a comment

Choose a reason for hiding this comment

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

Hi, we need to support Jira Server 7, which doesn't support api v3, removing api v2 is not acceptable.

@base698
Copy link
Author

base698 commented Oct 14, 2025

Closing in lieu of: #8612

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants