Skip to content

Commit 378dd0f

Browse files
authored
add GH PR/issue templates (#74)
1 parent 5ffb072 commit 378dd0f

2 files changed

Lines changed: 72 additions & 0 deletions

File tree

‎.github/ISSUE_TEMPLATE.md‎

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
## Summary
2+
3+
Give a brief description of the problem.
4+
5+
### Details
6+
7+
Explain with further technical details, if applies.
8+
9+
### Impact
10+
11+
Why is it important?
12+
13+
## How to reproduce?
14+
15+
### Environment
16+
- Helm chart version:
17+
- Graylog version:
18+
- Kubernetes version:
19+
- Cloud provider / platform:
20+
21+
### Pre-flight checks
22+
- [ ] `helm lint ./graylog` passes
23+
- [ ] Checked existing issues for duplicates
24+
25+
### Steps
26+
1. ...
27+
2. ...
28+
29+
## Notes for maintainers
30+
_Any additional context, logs, or screenshots_

‎.github/PULL_REQUEST_TEMPLATE.md‎

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
## Summary
2+
3+
Short description of the change.
4+
5+
## What changed
6+
- List of meaningful changes
7+
8+
## Checklist
9+
- [ ] Tests added/updated
10+
- [ ] Documentation updated
11+
- [ ] This PR includes a new feature
12+
- [ ] This PR includes a bugfix
13+
- [ ] This PR includes a refactor
14+
15+
## Testing Checklist
16+
17+
### Static Validation
18+
- [ ] `helm lint ./graylog` passes
19+
- [ ] `helm template graylog ./graylog --validate` passes
20+
21+
### Installation
22+
- [ ] Fresh installation completes successfully
23+
- [ ] All pods reach Running state
24+
- [ ] `helm test graylog -n graylog` passes
25+
26+
### Functional (if applicable)
27+
- [ ] Web UI accessible and login works
28+
- [ ] DataNodes visible in System > Data Nodes
29+
- [ ] Inputs can be created and receive data
30+
31+
### Upgrade (if applicable)
32+
- [ ] Upgrade from previous release succeeds
33+
- [ ] Scaling up/down works correctly
34+
- [ ] Configuration changes apply correctly
35+
36+
### Specific to this PR
37+
- [ ] _describe what was specifically tested_
38+
39+
## Notes for reviewers
40+
- [ ] Verify all tests pass
41+
- [ ] Sync up with the author before merging
42+
- [ ] The commit history must be preserved - please use the rebase-merge or standard merge options

0 commit comments

Comments
 (0)