Skip to content

ci: run prometheus regtest as final step for ASAN builds#24

Draft
Copilot wants to merge 2 commits intomasterfrom
copilot/add-prometheus-query-regtests
Draft

ci: run prometheus regtest as final step for ASAN builds#24
Copilot wants to merge 2 commits intomasterfrom
copilot/add-prometheus-query-regtests

Conversation

Copy link

Copilot AI commented Feb 20, 2026

ASAN builds include USE_PROMEX=1 but had no dedicated final step to exercise the prometheus exporter under the sanitizer. This adds an explicit prometheus query at the end of ASAN CI runs to catch memory errors that only surface during prometheus scrapes.

Changes

  • .github/workflows/vtest.yml: Add a new step "Run Prometheus query (ASAN only)" conditioned on contains(matrix.name, 'ASAN'), placed after the main reg-tests step:
- name: Run Prometheus query (ASAN only)
  if: ${{ contains(matrix.name, 'ASAN') }}
  run: |
    make reg-tests VTEST_PROGRAM=../vtest/vtest REG_TEST_FILES=reg-tests/contrib/prometheus.vtc

Uses REG_TEST_FILES to run only reg-tests/contrib/prometheus.vtc — no full suite re-run. The contains(matrix.name, 'ASAN') condition is already used elsewhere in the same workflow (line 73) for ASAN detection.

Original prompt

This section details on the original issue you should resolve

<issue_title>invoke prometheus in regtests</issue_title>
<issue_description>if haproxy is built using address sanitizer we want to include additional step to regtests which performs a query to prometheus as a final step of test</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: chipitsine <2217296+chipitsine@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Prometheus query step in regtests for AddressSanitizer builds ci: run prometheus regtest as final step for ASAN builds Feb 20, 2026
Copilot AI requested a review from chipitsine February 20, 2026 19:41
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.

invoke prometheus in regtests

2 participants