Skip to content

Custom Domain for Microservice #365

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

Merged
merged 9 commits into from
Jun 24, 2024

Conversation

williamputraintan
Copy link
Member

@williamputraintan williamputraintan commented Jun 24, 2024

  • Add custom domain name for microservice api-gateway
    • metadata-manager -> metadata
    • file-manager -> file
    • sequence-run-manager -> sequence-run sequence
    • workflow-manager -> workflow
  • api-gw CW logs limited for 2 years in prod, and 14 days in dev/stg

Resolve #361

@williamputraintan williamputraintan added the metadata_manager anything related to metadata manager label Jun 24, 2024
@williamputraintan williamputraintan self-assigned this Jun 24, 2024
@williamputraintan williamputraintan added the feature New feature label Jun 24, 2024
Copy link
Member

@victorskl victorskl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Just minor request.

@@ -114,6 +115,7 @@ export class SequenceRunManagerStack extends Stack {
const srmApi = new ApiGatewayConstruct(this, 'ApiGateway', {
region: this.region,
apiName: 'SequenceRunManager',
customDomainNamePrefix: 'sequence-run',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this case, could you change it to just sequence pls.

@@ -117,6 +118,7 @@ export class ProjectNameStack extends Stack { // FIXME change construct name
const srmApi = new ApiGatewayConstruct(this, 'ApiGateway', {
region: this.region,
apiName: 'SequenceRunManager',
customDomainNamePrefix:'sequence-run',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup. Just sequence as noun pattern.

@williamputraintan
Copy link
Member Author

Updated! Will merge once test passed!

@williamputraintan williamputraintan merged commit 46244d7 into main Jun 24, 2024
5 checks passed
@williamputraintan williamputraintan deleted the feature(metadata-manager)/custom-domain-name branch June 24, 2024 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature metadata_manager anything related to metadata manager
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom Domain Name for OrcaBus (MetadataManager)
2 participants