Skip to content

Conversation

@nmvk
Copy link
Contributor

@nmvk nmvk commented Jul 21, 2021

  • Secret issues to be terminal conditions.
  • Updated runtime
  • New code with updated code generator
  • Removed AutoMinorVersionUpgrade and NumCacheClusters
  • Updated an error and error message to be terminal condition

Secret not found

 Conditions:
    Message:  kubernetes secret not found
    Status:   True
    Type:     ACK.Terminal

Negative node group

  Conditions:
    Message:    At least one node group should be present.
    Status:     True
    Type:       ACK.Terminal
    Status:     True

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@nmvk
Copy link
Contributor Author

nmvk commented Jul 21, 2021

/retest

- Secret issues to be terminal conditions.
- Updated runtime
- New code with updated code generator
- Removed AutoMinorVersionUpgrade and NumCacheClusters
- Updated an error and error message to be terminal condition
@ack-bot
Copy link
Collaborator

ack-bot commented Jul 22, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: echen-98, kumargauravsharma, nmvk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [echen-98,kumargauravsharma,nmvk]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kumargauravsharma kumargauravsharma merged commit 15ef81c into aws-controllers-k8s:main Jul 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants