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

Prod - DTSSE: Remove dtsse from prod (not needed) #34511

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

danielwilsonkainos
Copy link
Contributor

@danielwilsonkainos danielwilsonkainos commented Sep 20, 2024

Change description

  • dtsse ardoq-adapter seems to have non-working prod image
  • this is not needed on prod env
  • causing blocker for AKS upgrade process due to pod disruption budget 'allowed disruptions' calculating to 0

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

🤖AEP PR SUMMARY🤖

I'm a bot that generates AI summaries of pull requests, see AEP for more details

  • Deleted the following files:
    • apps/dtsse/prod/00/kustomization.yaml
    • apps/dtsse/prod/01/kustomization.yaml
    • apps/dtsse/prod/base/kustomization.yaml
  • Updated clusters/prod/base/kustomization.yaml:
    • Removed the reference to apps/dtsse/base/kustomize.yaml

@danielwilsonkainos danielwilsonkainos changed the title DTSSE -Prod: Remove dtsse from prod (not needed) Prod - DTSSE: Remove dtsse from prod (not needed) Sep 20, 2024
@danielwilsonkainos danielwilsonkainos marked this pull request as ready for review September 20, 2024 14:15
@danielwilsonkainos danielwilsonkainos requested a review from a team as a code owner September 20, 2024 14:15
@danielwilsonkainos danielwilsonkainos merged commit 05735cd into master Sep 20, 2024
18 checks passed
@danielwilsonkainos danielwilsonkainos deleted the remove-dtsse-prod branch September 20, 2024 14:28
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.

3 participants