Skip to content

Commit 2bc4510

Browse files
authored
chore: Update role to latest NGINX best practices (#878)
1 parent 0cb89b4 commit 2bc4510

15 files changed

+166
-108
lines changed

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 0 additions & 36 deletions
This file was deleted.

.github/ISSUE_TEMPLATE/bug_report.yml

Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,71 @@
1+
---
2+
name: 🐛 Bug report
3+
description: Create a report to help us improve
4+
labels: bug
5+
body:
6+
- type: markdown
7+
attributes:
8+
value: |
9+
Thanks for taking the time to fill out this bug report!
10+
11+
Before you continue filling out this report, please take a moment to check that your bug has not been [already reported on GitHub][issue search] 🙌
12+
13+
Remember to redact any sensitive information such as authentication credentials and/or license keys!
14+
15+
**Note:** If you are seeking community support or have a question, please consider starting a new thread via [GitHub discussions][discussions] or the [NGINX Community forum][forum].
16+
17+
[issue search]: https://github.com/nginxinc/ansible-role-nginx/issues
18+
[discussions]: https://github.com/nginxinc/ansible-role-nginx/discussions
19+
[forum]: https://community.nginx.org
20+
21+
- type: textarea
22+
id: overview
23+
attributes:
24+
label: Bug Overview
25+
description: A clear and concise overview of the bug.
26+
placeholder: When I do "X" with the NGINX Ansible role, "Y" happens instead of "Z".
27+
validations:
28+
required: true
29+
30+
- type: textarea
31+
id: behavior
32+
attributes:
33+
label: Expected Behavior
34+
description: A clear and concise description of what you expected to happen.
35+
placeholder: When I do "X" with the NGINX Ansible role, I expect "Z" to happen.
36+
validations:
37+
required: true
38+
39+
- type: textarea
40+
id: steps
41+
attributes:
42+
label: Steps to Reproduce the Bug
43+
description: Detail the series of steps required to reproduce the bug.
44+
value: |
45+
1. I have deployed/run the NGINX Ansible role using the following `playbook.yml`...
46+
2. I have seen the following error(s) on my terminal/logs...
47+
validations:
48+
required: true
49+
50+
- type: textarea
51+
id: environment
52+
attributes:
53+
label: Environment Details
54+
description: Please provide details about your environment.
55+
value: |
56+
- Target deployment platforms: [e.g. AWS/GCP/local cluster/etc...]
57+
- Target OSs: [e.g. RHEL 9/Ubuntu 24.04/etc...]
58+
- Host OS (where you are running Ansible from): [e.g. RHEL 9/Ubuntu 24.04/etc...]
59+
- Version of the NGINX Ansible role (or specific commit): [e.g. 0.25.0/commit hash]
60+
- Version of Ansible: [e.g. 2.16.5]
61+
- How is Ansible being managed: [e.g. CLI/pipeline/Automation Hub/etc...]
62+
- Version of Jinja2 (if you are using any templating capability): [e.g. 3.1.1]
63+
validations:
64+
required: true
65+
66+
- type: textarea
67+
id: context
68+
attributes:
69+
label: Additional Context
70+
description: Add any other context about the problem here.
71+
placeholder: Feel free to add any other context/information/screenshots/etc... that you think might be relevant to this issue in here.

.github/ISSUE_TEMPLATE/config.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
---
2+
blank_issues_enabled: false
3+
contact_links:
4+
- name: 💬 Talk to the NGINX community!
5+
url: https://community.nginx.org
6+
about: A community forum for NGINX users, developers, and contributors
7+
- name: 📝 Code of Conduct
8+
url: https://www.contributor-covenant.org/version/2/1/code_of_conduct
9+
about: NGINX follows the Contributor Covenant Code of Conduct to ensure a safe and inclusive community
10+
- name: 💼 For commercial & enterprise users
11+
url: https://www.f5.com/products/nginx
12+
about: F5 offers a wide range of NGINX products for commercial & enterprise users

.github/ISSUE_TEMPLATE/feature_request.md

Lines changed: 0 additions & 22 deletions
This file was deleted.
Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
---
2+
name: ✨ Feature request
3+
description: Suggest an idea for this project
4+
labels: enhancement
5+
body:
6+
- type: markdown
7+
attributes:
8+
value: |
9+
Thanks for taking the time to fill out this feature request!
10+
11+
Before you continue filling out this request, please take a moment to check that your feature has not been [already requested on GitHub][issue search] 🙌
12+
13+
**Note:** If you are seeking community support or have a question, please consider starting a new thread via [GitHub discussions][discussions] or the [NGINX Community forum][forum].
14+
15+
[issue search]: https://github.com/nginxinc/ansible-role-nginx/issues
16+
[discussions]: https://github.com/nginxinc/ansible-role-nginx/discussions
17+
[forum]: https://community.nginx.org
18+
19+
- type: textarea
20+
id: overview
21+
attributes:
22+
label: Feature Overview
23+
description: A clear and concise description of what the feature request is.
24+
placeholder: I would like the NGINX Ansible role to be able to do "X".
25+
validations:
26+
required: true
27+
28+
- type: textarea
29+
id: alternatives
30+
attributes:
31+
label: Alternatives Considered
32+
description: Detail any potential alternative solutions/workarounds you've used or considered.
33+
placeholder: I have done/might be able to do "X" in the NGINX Ansible role by doing "Y".
34+
35+
- type: textarea
36+
id: context
37+
attributes:
38+
label: Additional Context
39+
description: Add any other context about the problem here.
40+
placeholder: Feel free to add any other context/information/screenshots/etc... that you think might be relevant to this feature request here.

.github/pull_request_template.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ Describe the use case and detail of the change. If this PR addresses an issue on
77
Before creating a PR, run through this checklist and mark each as complete:
88

99
- [ ] I have read the [contributing guidelines](/CONTRIBUTING.md).
10-
- [ ] I have signed the [F5 Contributor License Agreement (CLA)](https://github.com/f5/.github/blob/main/CLA/cla-markdown.md).
10+
- [ ] I have signed the [F5 Contributor License Agreement (CLA)](https://github.com/f5/f5-cla/blob/main/docs/f5_cla.md).
1111
- [ ] If applicable, I have added Molecule tests that prove my fix is effective or that my feature works.
1212
- [ ] If applicable, I have checked that any relevant Molecule tests pass after adding my changes.
1313
- [ ] I have updated any relevant documentation ([`defaults/main/*.yml`](/defaults/main/), [`README.md`](/README.md) and [`CHANGELOG.md`](/CHANGELOG.md)).

.github/scorecard.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
---
2+
annotations:
3+
- checks:
4+
- fuzzing
5+
- packaging
6+
- pinned-dependencies
7+
- sast
8+
- signed-releases
9+
reasons:
10+
- reason: not-applicable

.github/workflows/f5-cla.yml renamed to .github/workflows/f5_cla.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,20 +19,21 @@ jobs:
1919
if: (github.event.comment.body == 'recheck' || github.event.comment.body == 'I have hereby read the F5 CLA and agree to its terms') || github.event_name == 'pull_request_target'
2020
uses: contributor-assistant/github-action@ca4a40a7d1004f18d9960b404b97e5f30a505a08 # v2.6.1
2121
with:
22-
# Any pull request targeting the following branch will trigger a CLA check.
23-
branch: main
2422
# Path to the CLA document.
25-
path-to-document: https://github.com/f5/.github/blob/main/CLA/cla-markdown.md
23+
path-to-document: https://github.com/f5/f5-cla/blob/main/docs/f5_cla.md
2624
# Custom CLA messages.
27-
custom-notsigned-prcomment: '🎉 Thank you for your contribution! It appears you have not yet signed the F5 Contributor License Agreement (CLA), which is required for your changes to be incorporated into an F5 Open Source Software (OSS) project. Please kindly read the [F5 CLA](https://github.com/f5/.github/blob/main/CLA/cla-markdown.md) and reply on a new comment with the following text to agree:'
25+
custom-notsigned-prcomment: '🎉 Thank you for your contribution! It appears you have not yet signed the [F5 Contributor License Agreement (CLA)](https://github.com/f5/f5-cla/blob/main/docs/f5_cla.md), which is required for your changes to be incorporated into an F5 Open Source Software (OSS) project. Please kindly read the [F5 CLA](https://github.com/f5/f5-cla/blob/main/docs/f5_cla.md) and reply on a new comment with the following text to agree:'
2826
custom-pr-sign-comment: 'I have hereby read the F5 CLA and agree to its terms'
2927
custom-allsigned-prcomment: '✅ All required contributors have signed the F5 CLA for this PR. Thank you!'
3028
# Remote repository storing CLA signatures.
3129
remote-organization-name: f5
3230
remote-repository-name: f5-cla-data
31+
# Branch where CLA signatures are stored.
32+
branch: main
3333
path-to-signatures: signatures/signatures.json
3434
# Comma separated list of usernames for maintainers or any other individuals who should not be prompted for a CLA.
35-
allowlist: alessfg, oxpa, bot*
35+
# NOTE: You will want to edit the usernames to suit your project needs.
36+
allowlist: bot*
3637
# Do not lock PRs after a merge.
3738
lock-pullrequest-aftermerge: false
3839
env:

.github/workflows/ossf-scorecard.yml renamed to .github/workflows/ossf_scorecard.yml

Lines changed: 4 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@ name: OSSF Scorecard
44
on:
55
# For Branch-Protection check. Only the default branch is supported. See https://github.com/ossf/scorecard/blob/main/docs/checks.md#branch-protection.
66
branch_protection_rule:
7-
push:
8-
branches: [main]
97
# To guarantee Maintained check is occasionally updated. See https://github.com/ossf/scorecard/blob/main/docs/checks.md#maintained.
108
schedule:
119
- cron: "0 0 * * 1"
10+
push:
11+
branches: [main]
1212
workflow_dispatch:
1313
# Declare default permissions as read only.
1414
permissions: read-all
@@ -17,16 +17,10 @@ jobs:
1717
name: Scorecard analysis
1818
runs-on: ubuntu-24.04
1919
permissions:
20-
# Needed if using Code Scanning alerts
20+
# Needed if using Code Scanning alerts.
2121
security-events: write
22-
# Needed for GitHub OIDC token if publish_results is true
22+
# Needed for GitHub OIDC token if publish_results is true.
2323
id-token: write
24-
# Uncomment the permissions below if installing on a private repository.
25-
# contents: read
26-
# actions: read
27-
# issues: read # To allow GraphQL ListCommits to work
28-
# pull-requests: read # To allow GraphQL ListCommits to work
29-
# checks: read # To detect SAST tools
3024
steps:
3125
- name: Check out the codebase
3226
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
@@ -38,14 +32,7 @@ jobs:
3832
with:
3933
results_file: results.sarif
4034
results_format: sarif
41-
# (Optional) fine-grained personal access token. Uncomment the `repo_token` line below if:
42-
# - You want to enable the Branch-Protection check on a *public* repository.
43-
# - You are installing the OSSF Scorecard on a *private* repository.
44-
# To create the PAT, follow the steps in https://github.com/ossf/scorecard-action#authentication-with-fine-grained-pat-optional.
45-
# repo_token: ${{ secrets.SCORECARD_TOKEN }}
46-
4735
# Publish the results for public repositories to enable scorecard badges. For more details, see https://github.com/ossf/scorecard-action#publishing-results.
48-
# For private repositories, `publish_results` will automatically be set to `false`, regardless of the value entered here.
4936
publish_results: true
5037

5138
# Upload the results as artifacts (optional). Commenting out will disable uploads of run results in SARIF format to the repository Actions tab.

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,10 @@ BUG FIXES:
1111
- Fix Ansible and Jinja versions validation tasks in ansible check mode.
1212
- Correctly use the `nginx_version` (if defined) for NGINX module versions.
1313

14+
DOCUMENTATION:
15+
16+
- Update community docs & required workflows per the latest [NGINX template repository](https://github.com/nginx/template-repository) guidelines.
17+
1418
## 0.25.0 (Nov 28, 2024)
1519

1620
BREAKING CHANGES:

CONTRIBUTING.md

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,21 +5,15 @@ The following is a set of guidelines for contributing to the NGINX Ansible role.
55
#### Table Of Contents
66

77
[Getting Started](#getting-started)
8-
98
[Contributing](#contributing)
10-
119
[Code Guidelines](#code-guidelines)
12-
13-
- [Git Guidelines](#git-guidelines)
14-
- [Ansible Guidelines](#ansible-guidelines)
15-
1610
[Code of Conduct](/CODE_OF_CONDUCT.md)
1711

1812
## Getting Started
1913

20-
Follow this project's [Installation Guide](/README.md#Installation) to install Ansible, Ansible Lint, and Molecule and get ready to develop and test the NGINX Ansible role.
14+
Follow the role's [installation guide](/README.md#Installation) to install Ansible, Ansible Lint, and Molecule and get ready to develop and test the NGINX Ansible role.
2115

22-
### Project Structure
16+
### Project Overview & Structure
2317

2418
- The NGINX Ansible role is written in [`yaml`](https://yaml.org) and supports NGINX Open Source, NGINX Plus, NGINX Agent and NGINX Amplify.
2519
- The project follows the standard [Ansible role directory structure](https://docs.ansible.com/ansible/latest/user_guide/playbooks_reuse_roles.html):
@@ -34,23 +28,23 @@ Follow this project's [Installation Guide](/README.md#Installation) to install A
3428

3529
### Report a Bug
3630

37-
To report a bug, open an issue on GitHub with the label `bug` using the available bug report issue template. Please ensure the bug has not already been reported. **If the bug is a potential security vulnerability, please report it using our [security policy](/SECURITY.md).**
31+
To report a bug, open an issue on GitHub with the label `bug` using the available [bug report issue form](/.github/ISSUE_TEMPLATE/bug_report.yml). Please ensure the bug has not already been reported. **If the bug is a potential security vulnerability, please report it using our [security policy](/SECURITY.md).**
3832

3933
### Suggest a Feature or Enhancement
4034

41-
To suggest a feature or enhancement, please create an issue on GitHub with the label `enhancement` using the available [feature request template](/.github/feature_request_template.md). Please ensure the feature or enhancement has not already been suggested.
35+
To suggest a feature or enhancement, please create an issue on GitHub with the label `enhancement` using the available [feature request issue form](/.github/ISSUE_TEMPLATE/feature_request.yml). Please ensure the feature or enhancement has not already been suggested.
4236

4337
### Open a Pull Request (PR)
4438

4539
- Fork the repo, create a branch, implement your changes, add any relevant tests, and submit a PR when your changes are **tested** (using Molecule) and ready for review.
4640
- Fill in the [PR template](/.github/pull_request_template.md).
4741

4842
> [!NOTE]
49-
> If you'd like to implement a new feature, please consider creating a [feature request issue](/.github/feature_request_template.md) first to start a discussion about the feature.
43+
> If you'd like to implement a new feature, please consider creating a [feature request issue](/.github/ISSUE_TEMPLATE/feature_request.yml) first to start a discussion about the feature.
5044
5145
#### F5 Contributor License Agreement (CLA)
5246

53-
F5 requires all external contributors to agree to the terms of the F5 CLA (available [here](https://github.com/f5/.github/blob/main/CLA/cla-markdown.md)) before any of their changes can be incorporated into an F5 Open Source repository.
47+
F5 requires all contributors to agree to the terms of the F5 CLA (available [here](https://github.com/f5/f5-cla/blob/main/docs/f5_cla.md)) before any of their changes can be incorporated into an F5 Open Source repository (even contributions to the F5 CLA itself!).
5448

5549
If you have not yet agreed to the F5 CLA terms and submit a PR to this repository, a bot will prompt you to view and agree to the F5 CLA. You will have to agree to the F5 CLA terms through a comment in the PR before any of your changes can be merged. Your agreement signature will be safely stored by F5 and no longer be required in future PRs.
5650

README.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
[![Ansible Galaxy](https://img.shields.io/badge/galaxy-nginxinc.nginx-5bbdbf.svg)](https://galaxy.ansible.com/nginxinc/nginx)
2-
[![Molecule CI/CD](https://github.com/nginxinc/ansible-role-nginx/workflows/Molecule%20CI/CD/badge.svg)](https://github.com/nginxinc/ansible-role-nginx/actions/workflows/molecule.yml)
2+
[![Molecule CI/CD](https://github.com/nginxinc/ansible-role-nginx/actions/workflows/molecule.yml/badge.svg)](https://github.com/nginxinc/ansible-role-nginx/actions/workflows/molecule.yml)
33
[![OpenSSF Scorecard](https://api.securityscorecards.dev/projects/github.com/nginxinc/ansible-role-nginx/badge)](https://securityscorecards.dev/viewer/?uri=github.com/nginxinc/ansible-role-nginx)
44
[![Project Status: Active – The project has reached a stable, usable state and is being actively developed.](https://www.repostatus.org/badges/latest/active.svg)](https://www.repostatus.org/#active)
55
[![Community Support](https://badgen.net/badge/support/community/cyan?icon=awesome)](/SUPPORT.md)
6-
[![Contributor Covenant](https://img.shields.io/badge/Contributor%20Covenant-2.1-4baaaa.svg)](/CODE_OF_CONDUCT.md)
6+
[![Community Forum](https://img.shields.io/badge/community-forum-009639?logo=discourse&link=https%3A%2F%2Fcommunity.nginx.org)](https://community.nginx.org)
77
[![License](https://img.shields.io/badge/License-Apache--2.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
8+
[![Contributor Covenant](https://img.shields.io/badge/Contributor%20Covenant-2.1-4baaaa.svg)](/CODE_OF_CONDUCT.md)
89

910
# 👾 *Help make the NGINX Ansible role better by participating in our [survey](https://forms.office.com/Pages/ResponsePage.aspx?id=L_093Ttq0UCb4L-DJ9gcUKLQ7uTJaE1PitM_37KR881UM0NCWkY5UlE5MUYyWU1aTUcxV0NRUllJSC4u)!* 👾
1011

@@ -361,4 +362,4 @@ You can find the Ansible NGINX App Protect role to install and configure NGINX A
361362

362363
[Tom Gamull](https://github.com/magicalyak)
363364

364-
© [F5, Inc.](https://www.f5.com/) 2018 - 2024
365+
© [F5, Inc.](https://www.f5.com/) 2018 - 2025

0 commit comments

Comments
 (0)