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

Created a CDK Construct to deploy Komiser helm chart on EKS cluster. #19

Merged
merged 4 commits into from
Apr 2, 2024

Conversation

Azanul
Copy link
Collaborator

@Azanul Azanul commented Mar 25, 2024

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ”Œ Chore (Release)
  • ⏩ Revert

Description

Created AWS CDK app that contains komiser construct which helps to deploy the helm chart to AWS EKS cluster.

Related Issues

Refer the issue tailwarden/komiser#897 for more details.

Mobile & Desktop Screenshots/Recordings

Need help, while running cdk deploy it always gives the below error. Although I am using a EKS supported KubernetesVersion by considering the official documentation.

Received response status [FAILED] from custom resource. Message returned: unsupported Kubernetes version 

Screenshot 2023-10-02 at 4 50 16 PM

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.komiser.io
  • πŸ“• storybook
  • πŸ—’ Code comment intelliSense
  • πŸ™… no documentation needed

Complete the checklist before considering this PR as "Ready to review"

  • Pull the Code - Ensure the functionality works locally.
  • Compare to the Design - Check if everything matches the design specifications. Validate the consistency of colors, fonts, paddings, etc.
  • Responsive Testing - Resize the browser window to ensure nothing breaks on mobile
  • Code Quality - Ensure best coding practices are used *
  • Ensure no commented-out code or debug logs are left behind.
  • Check for code modularity and reusability.
  • Check for descriptive variable names: e.g., "totalPrice" not "tp".
  • Ensure there are no linting errors npm run lint
  • Ensure the code passes the build npm run build
  • Ensure there are no inline error disabling/ignores
  • Ensure all newly added dependencies are actually necessary (package.json)
  • Check that the PR has a title and description

@Azanul Azanul merged commit b6f37fc into tailwarden:master Apr 2, 2024
1 check failed
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