Skip to content

CI: add PR workflow to check IPFIX element consistency with libfds #295

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

Merged
merged 1 commit into from
Jul 30, 2025

Conversation

SiskaPavel
Copy link
Collaborator

@SiskaPavel SiskaPavel commented Jul 22, 2025

Summary

Add GitHub Actions workflow to verify consistency of IPFIX elements defined in copr repository @CESNET/NEMEA in cesnet-ipfix-elements package.

Details

Introduced a new reusable workflow .github/workflows/checkIpfixElemets.yml:

  • Runs in Oracle Linux 9 container.
  • Installs necessary dependencies, including cesnet-ipfix-elements.
  • Compares locally defined IPFIX elements (via ipfix-elements.hpp) against elements from libfds.
  • Fails the job if any elements are missing.

✅ Checks

  • Relevant documentation was updated (if needed)
  • CI pipeline passes (build, tests, lint, static analysis)
  • New code follows clang-tidy rules
  • The change is clearly related to a specific issue / requirement
    - [ ] Documentation (e.g. Doxygen) is updated if needed
  • The PR is rebased on the latest master
  • No unrelated changes are included

@SiskaPavel SiskaPavel self-assigned this Jul 22, 2025
@SiskaPavel SiskaPavel marked this pull request as draft July 22, 2025 09:01
@SiskaPavel SiskaPavel force-pushed the ci-check-ipfix-elements branch 6 times, most recently from d06c0e4 to 506ca8c Compare July 22, 2025 09:34
@SiskaPavel SiskaPavel force-pushed the ci-check-ipfix-elements branch from 506ca8c to bb1e399 Compare July 29, 2025 18:19
@SiskaPavel SiskaPavel force-pushed the ci-check-ipfix-elements branch from bb1e399 to 8d1a29c Compare July 30, 2025 12:55
@SiskaPavel SiskaPavel changed the base branch from master to rtsp-ipfix-elements July 30, 2025 13:15
@SiskaPavel SiskaPavel requested a review from hynekkar July 30, 2025 13:19
@SiskaPavel SiskaPavel marked this pull request as ready for review July 30, 2025 13:19
Base automatically changed from rtsp-ipfix-elements to master July 30, 2025 13:40
@SiskaPavel SiskaPavel merged commit 00a630e into master Jul 30, 2025
8 checks passed
@SiskaPavel SiskaPavel deleted the ci-check-ipfix-elements branch July 30, 2025 13:51
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.

2 participants