Skip to content

Run full CI test workflow on release tag pushes#9

Merged
matthias-lay merged 2 commits into
mainfrom
copilot/update-github-actions-for-releases
May 12, 2026
Merged

Run full CI test workflow on release tag pushes#9
matthias-lay merged 2 commits into
mainfrom
copilot/update-github-actions-for-releases

Conversation

Copilot AI commented May 12, 2026

Copy link
Copy Markdown
Contributor

This updates GitHub Actions so the existing full test matrix also runs when release tags are created, while keeping current CI behavior for main pushes and pull requests unchanged.

  • Workflow trigger update

    • Extended .github/workflows/test-nginx-1.30.yml on.push trigger to include release-style tags.
    • Added tag patterns to cover common conventions used in this repo/ecosystem:
      • v* (e.g. v2.0.0)
      • [0-9]* (e.g. 2.0.0)
  • Behavior preserved

    • No job/step/test logic changed.
    • Existing push to main and pull_request triggers remain intact.
on:
  push:
    branches:
      - main
    tags:
      - v*
      - '[0-9]*'
  pull_request:

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • cpanmetadb.plackperl.org
    • Triggering command: /usr/bin/wget /usr/bin/wget REDACTED --user-agent cpanminus/1.7047 --retry-connrefused -q -O - -edge.gpg (dns block)
    • Triggering command: /usr/bin/wget /usr/bin/wget REDACTED --user-agent cpanminus/1.7047 --retry-connrefused -q -O - am_ubuntu_ppa.gptar (dns block)
    • Triggering command: /usr/bin/wget /usr/bin/wget REDACTED --user-agent cpanminus/1.7047 --retry-connrefused -q -O - (dns block)
  • dl.google.com
    • Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https (dns block)
    • Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https --print-foreign-architectures v2 -modulev2 (dns block)
  • fastapi.metacpan.org
    • Triggering command: /usr/bin/wget /usr/bin/wget REDACTED --user-agent cpanminus/1.7047 --retry-connrefused -q -O - credential.userndpkg-deb (dns block)
    • Triggering command: /usr/bin/wget /usr/bin/wget REDACTED --user-agent cpanminus/1.7047 --retry-connrefused -q -O - am_ubuntu_ppa.gpdpkg-deb (dns block)
    • Triggering command: /usr/bin/wget /usr/bin/wget REDACTED --user-agent cpanminus/1.7047 --retry-connrefused -q -O - yring-2012-cdimarm (dns block)
  • http://www.cpan.org:80/modules/02packages.details.txt.gz
    • Triggering command: /usr/bin/wget /usr/bin/wget REDACTED --user-agent cpanminus/1.7047 --retry-connrefused -q -O /home/REDACTED/.cpanm/sources/http%www.cpan.org/02packages.details.txt.gz am_ubuntu_ppa.gptar (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Create a pull request in Securepoint/nginx-ntlm-modulev2 that updates GitHub Actions so all tests run when a release tag is created.

Context:

  • The repository default branch is main.
  • There is an existing workflow at .github/workflows/test-nginx-1.30.yml.
  • That workflow currently runs on pushes to main and on pull requests.
  • The user wants a workflow to run all tests on release tagging.

Required changes:

  • Update the existing test workflow, or add a new workflow if that is cleaner, so that the full test suite runs when a release tag is pushed.
  • Use GitHub Actions trigger configuration appropriate for release tags. Prefer a tag pattern that matches the repository's release tagging convention if it can be inferred from the repo; otherwise use a sensible general release-tag trigger and document it in the PR.
  • Ensure the existing test behavior for main pushes and pull requests is preserved unless there is a strong reason to change it.
  • Keep the workflow valid YAML and consistent with existing style.
  • If needed, add a brief note to repository documentation or workflow naming so it is clear that tests run on release tags.

Acceptance criteria:

  • A GitHub Actions workflow is present that runs the same test coverage on release tag creation.
  • Existing CI for branch pushes/PRs continues to work.
  • The PR clearly explains what tag events now trigger the workflow.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: create a workflow to run all tests on release tagging

Assistant: I’m waiting for your response to the confirmation dialog so I can continue creating the PR for the release-tag test workflow.

This pull request was created from Copilot chat.

Agent-Logs-Url: https://github.com/Securepoint/nginx-ntlm-modulev2/sessions/92a35f7a-97d2-410a-b05c-a1f87594c82e

Co-authored-by: matthias-lay <163420385+matthias-lay@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub Actions to run all tests on release tagging Run full CI test workflow on release tag pushes May 12, 2026
Copilot AI requested a review from matthias-lay May 12, 2026 09:22
@matthias-lay matthias-lay marked this pull request as ready for review May 12, 2026 10:01
@matthias-lay matthias-lay merged commit d006c98 into main May 12, 2026
2 checks passed
@matthias-lay matthias-lay deleted the copilot/update-github-actions-for-releases branch May 12, 2026 11: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.

2 participants