Skip to content

Commit

Permalink
Merge pull request #2 from anyscale/release-0.3.0
Browse files Browse the repository at this point in the history
[Release] v0.3.0 - Common Name for Resources
  • Loading branch information
brent-anyscale authored Feb 24, 2023
2 parents bfcec6a + a732148 commit 333f79a
Show file tree
Hide file tree
Showing 36 changed files with 973 additions and 157 deletions.
8 changes: 8 additions & 0 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@

# Use this file to define individuals or teams that are responsible for code in a repository.
# Read more: <https://help.github.com/articles/about-codeowners/>

# Order is important: the last matching pattern has the highest precedence

# These owners will be the default owners for everything
* @anyscale/solutionarchitects
60 changes: 33 additions & 27 deletions .github/ISSUE_TEMPLATE/bug-report.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
name: Bug Report
description: Issue/Bug with the Terraform Modules. Please include repro steps!
title: "[Bug]: "
title: "[bug]: "
labels: ["bug", "P1", "terraform-modules"]
assignees: ["brent-anyscale"]

Expand All @@ -11,31 +11,46 @@ body:
Thank you for taking the time to file a bug report.
Please make sure to search for existing issues before filing a new one.
- type: textarea
id: what-happened
attributes:
label: What happened?
description: What did you expect to happen? (provide screenshots if possible)
placeholder: "A bug happened!"
label: Current Behavior
description: A clear description of what the bug is and how it manifests.
validations:
required: true
- type: input
id: which-module
- type: textarea
attributes:
label: Expected Behavior
description: A clear description of what you expected to happen.
validations:
required: true
- type: textarea
attributes:
label: Which module?
description: "Which module(s) are you having problems with?"
placeholder: "ex. Root, S3, EFS, IAM, VPC, S3-Policy, SecurityGroup"
label: Steps to Reproduce
description: Please explain the steps required to duplicate this issue.
validations:
required: true
- type: checkboxes
attributes:
label: Which module(s)?
description: "Please select which module(s) this issue impacts."
options:
- label: Root Anyscale Module
- label: aws-anyscale-efs
- label: aws-anyscale-iam
- label: aws-anyscale-s3
- label: aws-anyscale-s3-policy
- label: aws-anyscale-securitygroups
- label: aws-anyscale-vpc
- type: dropdown
id: version
attributes:
label: Module version
label: Anyscale Terraform Module version
description: What version of the modules are you using?
multiple: false
options:
- 0.1.0 (Default)
- v0.2.x (Default)
- v0.1.x
validations:
required: true
- type: dropdown
id: terraform_version
attributes:
label: Terraform version
description: What version of Terraform are you using?
Expand All @@ -49,7 +64,6 @@ body:
validations:
required: true
- type: dropdown
id: blocking_work
attributes:
label: Is it blocking work?
description: Please choose one?
Expand All @@ -59,15 +73,7 @@ body:
- "yes"
validations:
required: true

# - [ ] I searched the [issues](https://github.com/anyscale/terraform-cloudfoundation-modules/issues) and found no similar issues.

### Terraform Module (EFS, IAM, S3, S3-Policy, SecurityGroups, VPC)

### What happened + What you expected to happen (provide screenshots if possible)

### Versions / Dependencies (Terraform, AWS Provider, Modules, etc)

### Reproduction script

### Anything else
- type: textarea
attributes:
label: Additional Information
description: List any other information that is relevant to your issue.
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -1 +1 @@
blank_issues_enabled: false
blank_issues_enabled: false
31 changes: 11 additions & 20 deletions .github/ISSUE_TEMPLATE/feature-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,21 +11,25 @@ body:
Thank you for taking the time to request a new feature.
Please make sure to search for existing issues before filing a new one.
- type: textarea
id: new-feature
attributes:
label: What are you looking for?
description: Please provide a detailed explanation of the new feature request you are looking for.
placeholder: "Feature request!!"
validations:
required: true
- type: input
id: which-module
- type: checkboxes
attributes:
label: Which module?
description: "Which module(s) are you having problems with?"
placeholder: "ex. Root, S3, EFS, IAM, VPC, S3-Policy, SecurityGroup"
label: Which module(s)?
description: "Please select which module(s) this issue impacts."
options:
- label: Root Anyscale Module
- label: aws-anyscale-efs
- label: aws-anyscale-iam
- label: aws-anyscale-s3
- label: aws-anyscale-s3-policy
- label: aws-anyscale-securitygroups
- label: aws-anyscale-vpc
- type: dropdown
id: blocking_work
attributes:
label: Is it blocking work?
description: Please choose one?
Expand All @@ -36,7 +40,6 @@ body:
validations:
required: true
- type: dropdown
id: terraform_version
attributes:
label: Terraform version
description: What version of Terraform are you using?
Expand All @@ -49,15 +52,3 @@ body:
- Other
validations:
required: false

# - [ ] I searched the [issues](https://github.com/anyscale/terraform-cloudfoundation-modules/issues) and found no similar issues.

### Terraform Module (EFS, IAM, S3, S3-Policy, SecurityGroups, VPC)

### What happened + What you expected to happen (provide screenshots if possible)

### Versions / Dependencies (Terraform, AWS Provider, Modules, etc)

### Reproduction script

### Anything else
23 changes: 11 additions & 12 deletions .github/ISSUE_TEMPLATE/security-issue.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,23 +10,22 @@ body:
value: |
Thank you for taking the time to report a security issue.
If this is a critical security issue that needs to be addressed immediately, email [email protected]
- type: input
id: contact
attributes:
label: Contact Details
description: An email address that we can reach out to you if we need more information.
placeholder: ex. [email protected]
- type: textarea
id: security_issue
attributes:
label: What is the security issue?
description: Please provide a detailed explanation of the security issue.
value: "<<SECURITY ISSUE>>"
validations:
required: true
- type: input
id: which_module
- type: checkboxes
attributes:
label: Which module?
description: "Which module(s) are you having problems with?"
value: "ex. Root, S3, EFS, IAM, VPC, S3-Policy, SecurityGroup"
label: Which module(s)?
description: "Please select which module(s) this issue impacts."
options:
- label: Root Anyscale Module
- label: aws-anyscale-efs
- label: aws-anyscale-iam
- label: aws-anyscale-s3
- label: aws-anyscale-s3-policy
- label: aws-anyscale-securitygroups
- label: aws-anyscale-vpc
34 changes: 34 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
## Pull request checklist

Please check if your PR fulfills the following requirements:
- [ ] pre-commit has been run
- [ ] Tests for the changes have been added (for bug fixes / features)
- [ ] All tests passing
- [ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)

## Pull Request Type

- [ ] Bugfix
- [ ] New feature
- [ ] Refactoring (no functional changes)
- [ ] Documentation change
- [ ] Other (please describe):


## Description
<!-- Please describe the current behavior that you are modifying. -->
<!-- Please describe the behavior or changes that are being added by this PR. -->
-
-
-

## Does this introduce a breaking change?
- [ ] Yes
- [ ] No

<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->


## Other information

<!-- Any other information that is important to this PR such as screenshots. -->
54 changes: 54 additions & 0 deletions .github/create-release.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
name-template: 'v$RESOLVED_VERSION'
tag-template: 'v$RESOLVED_VERSION'
version-template: '$MAJOR.$MINOR.$PATCH'
version-resolver:
major:
labels:
- 'major'
minor:
labels:
- 'minor'
- 'enhancement'
patch:
labels:
- 'auto-update'
- 'patch'
- 'fix'
- 'bugfix'
- 'bug'
- 'hotfix'
- 'documentation'
- 'examples'
default: 'minor'

categories:
- title: '🚀 Enhancements'
labels:
- 'enhancement'
- 'patch'
- title: '🐛 Bug Fixes'
labels:
- 'fix'
- 'bugfix'
- 'bug'
- 'hotfix'
- title: '📚 Documentation/Examples'
labels:
- 'documentation'
- 'examples'
- title: 'Github Chores'
labels:
- 'github'
# - title: 'Automatic Updates'
# labels:
# - 'auto-update'

change-template: |
<details>
<summary>$TITLE @$AUTHOR (#$NUMBER)</summary>
$BODY
</details>
template: |
$CHANGES
19 changes: 17 additions & 2 deletions .github/labeler.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,20 @@
# See https://github.com/actions/labeler for details
# "dependencies" label for any changes in dependency files
# Add 'repo' label to any root file changes
repo:
- '*'
documentation:
- examples/**/*
- README.md

examples:
- examples/**/*

terraform:
- "*.tf"
- modules/**/*

unittests:
- test/**/*
- modules/**/test/*

github:
- .github/**/*
44 changes: 44 additions & 0 deletions .github/workflows/create-release.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
name: create-release

on:
push:
# branches to consider in the event; optional, defaults to all
branches:
- main
# pull_request event is required only for autolabeler
pull_request:
# Only following types are handled by the action, but one can default to all as well
types: [closed]
# pull_request_target event is required for autolabeler to support PRs from forks
# pull_request_target:
# types: [opened, reopened, synchronize]

permissions:
contents: read

jobs:
create_release:
if: github.event.pull_request.merged == true
permissions:
# write permission is required to create a github release
contents: write
# write permission is required for autolabeler
# otherwise, read permission is required at least
pull-requests: write
runs-on: ubuntu-latest
steps:
# Get PR from merged commit to main
- uses: actions-ecosystem/action-get-merged-pull-request@v1
id: get-merged-pull-request
with:
github_token: "${{ secrets.GITHUB_TOKEN }}"

# Drafts your next Release notes as Pull Requests are merged into "master"
- uses: release-drafter/release-drafter@v5
# (Optional) specify config name to use, relative to .github/. Default: release-drafter.yml
with:
publish: ${{ !contains(steps.get-merged-pull-request.outputs.labels, 'no-release') }}
# prerelease: false
config-name: create-release.yml
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
4 changes: 3 additions & 1 deletion .github/workflows/labeler.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
name: "Pull Request Labeler"
on:
- pull_request
- pull_request_target

permissions: read-all

jobs:
add-label:
Expand Down
9 changes: 0 additions & 9 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,6 @@ permissions: read-all
# A workflow run is made up of one or more jobs that can run sequentially or in parallel
jobs:
# This workflow contains a single job called "build"
pre-commit:
runs-on: ubuntu-latest
name: Static pre-commit analysis
steps:
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
- name: Checkout
uses: actions/checkout@v3
- name: Run a one-line script
run: echo "pre-commit (not active)"
unit-tests:
runs-on: ubuntu-latest
name: Terraform Unit Tests
Expand Down
Loading

0 comments on commit 333f79a

Please sign in to comment.