Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(base-standalone)!: compose into 1 stack, and use parameter.ts #161

Merged

Conversation

konokenj
Copy link
Contributor

@konokenj konokenj commented Dec 29, 2022

Feature details are described at #214 .

Code changes

https://github.com/aws-samples/baseline-environment-on-aws/pull/161/files

git diff commands described below are expected to be in PR branch.

Common changes

  • Delete context from cdk.json and migrate to parameter.ts.
  • Delete getting context in bin/.ts and read parameter.ts. Generate Stack for each Environments.
    • Same to snapshot test

DELETE: lib/blea-chatbot-stack.ts

Manually configure Chatbot by AWS Management Console.

MERGE INTO: lib/construct/security-detection.ts

No logic changes.

  • lib/blea-config-ct-guardrail-stack.ts
  • lib/blea-config-rules-stack.ts
  • lib/blea-guardduty-stack.ts
  • lib/blea-security-hub-stack.ts
  • lib/blea-security-alarm-stack.ts
git diff main:usecases/base-standalone/lib/blea-config-ct-guardrail-stack.ts \
 @:usecases/blea-base-standalone/lib/construct/security-detection.ts

git diff main:usecases/base-standalone/lib/blea-config-rules-stack.ts \
 @:usecases/blea-base-standalone/lib/construct/security-detection.ts

git diff main:usecases/base-standalone/lib/blea-guardduty-stack.ts \
 @:usecases/blea-base-standalone/lib/construct/security-detection.ts

git diff main:usecases/base-standalone/lib/blea-security-hub-stack.ts \
 @:usecases/blea-base-standalone/lib/construct/security-detection.ts

RENAME INTO: lib/construct/security-iam.ts

No logic changes.

  • lib/blea-iam-stack.ts
git diff main:usecases/base-standalone/lib/blea-iam-stack.ts \
 @:usecases/blea-base-standalone/lib/construct/security-iam.ts

MERGE INTO: lib/construct/security-logging.ts

No logic changes. Some methods moved.

  • lib/blea-trail-stack.ts
  • lib/blea-config-stack.ts
git diff main:usecases/base-standalone/lib/blea-trail-stack.ts \
 @:usecases/blea-base-standalone/lib/construct/security-logging.ts

git diff main:usecases/base-standalone/lib/blea-config-stack.ts \
 @:usecases/blea-base-standalone/lib/construct/security-logging.ts

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT No Attribution (MIT-0).

@konokenj konokenj marked this pull request as ready for review March 13, 2023 06:04
@konokenj konokenj force-pushed the feature/v3-base-standalone-1stack branch from ceb9a9d to 8494a7e Compare March 15, 2023 09:06
@ohmurayu
Copy link
Contributor

Needs to add some comments and change variable name for easy to understand.
However code is OK, so I'll go forward merge. I'll create issue for easy to understand.

@ohmurayu ohmurayu merged commit 762ed58 into aws-samples:dev.v3 Mar 16, 2023
@konokenj konokenj deleted the feature/v3-base-standalone-1stack branch April 17, 2023 11:43
konokenj added a commit that referenced this pull request Apr 20, 2023
* feat(base-ct-audit)!: delete base-ct-audit (#221)

Delete Audit account baseline. This is discussed in #237.

* feat(base-standalone)!: compose into 1 stack, and use parameter.ts (#161)

* refactor(base-standalone): compose into 1 stack and use parameter.ts

* feat(base-standalone): refactor file and construct names

* build: fix eslint fails on jest.config.js

* fix(base-standalone): remove chatbot params

* test(base-standalone): fix snapshot test

* Revert "build: fix eslint fails on jest.config.js"

This reverts commit 18f4275.

* build(base-standalone): remove unused settings from package.json

* update package-lock.json

* feat(base-ct-guest)!: compose into 1 stack, and use parameter.ts (#220)

* feat(base-ct-guest)!: compose into 1 stack and use parameter.ts

* feat(base-ct-guest)!: rename usecase

* build(base-ct-guest): remove lint, format npm tasks

* test(base-ct-guest): fix snapshot test

* feat(base-ct-guest): deploy baseline via Service Catalog / AFC

* remove product-stack-snapshots

* add CDK Pipelines into blea-base-ct-guest

---------

Co-authored-by: Yukitaka Ohmura <[email protected]>

* refactor(guest-apiapp-sample): merge into single stack and change method to access parameter (#240)

* Merge multiple stacks to one stack.
* Merge two language (Python, Node.js) sample applications into single application.
* Change method to access parameters to use parameter.ts

* refactor(guest-webapp-sample)!: split into 2 usecases, and decrease number of stacks (#239)

* refactor(guest-webapp-sample)!: split into 2 usecases, and decrease number of stacks

* delete .gitignore from ec2app-sample

* Update package-lock.json

---------

Co-authored-by: Yukitaka Ohmura <[email protected]>

* refactor(blea-base-ct-guest): Separate deploy parameter from pipeline

* doc: Update Japanese docs to align v3 procedure

* refactor: renew deprecated method and class on CloudWatch

* fix: delete unnecessary information (#245)

* refactor: rename Construct IDs and variable names

* refactor: set different names to ct/standalone

* refactor: rename packages

* test: add missing snapshot tests for gov-base-ct and fix formats

* refactor: remove "security" prefix

* build: refactor npm packages and build scripts

* ci: remove product-stack-snapshots generated by test

* ci: fix escape

* refactor: fix ecs-app class names

* fix: pass env parameter to stacks

* test: update snapshots

* fix: specify env

* build: remove parameter.js when clean, and update snapshots

* feat: re-add Chatbot to gov-base when deploy via CLI

* test: fix tests

* feat(ecs-app): deploy without custom domain

* refactor: rename Construct IDs and variable names for consistent use in this project (#286)

* refactor: rename Construct IDs and variable names

* refactor: set different names to ct/standalone

* refactor: rename packages

* test: add missing snapshot tests for gov-base-ct and fix formats

* refactor: remove "security" prefix

* build: refactor npm packages and build scripts

* ci: remove product-stack-snapshots generated by test

* ci: fix escape

* refactor: fix ecs-app class names

* fix: pass env parameter to stacks

* test: update snapshots

* fix: specify env

* build: remove parameter.js when clean, and update snapshots

---------

Co-authored-by: Yukitaka Ohmura <[email protected]>

* docs: restore codes to use custom domain on CloudFront as a comment

* fix(gov-base-ct, gov-base-standalone): adopt s3's new bucket default security policy (#296)

* fix(gov-base-ct, gov-base-standalone): adopt s3's new bucket default security policy

* doc: Update English docs to align v3 (#297)

* docs: add versioning policy and some fixes

* build: remove cdk

---------

Co-authored-by: Kenji Kono <[email protected]>
Co-authored-by: Kenji Kono <[email protected]>
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