Skip to content

Conversation

@RedbackThomson
Copy link
Contributor

Description of changes:
Updates the ACK runtime version in go.mod to v0.25.0

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

@ack-prow ack-prow bot requested review from jljaco and vijtrip2 March 22, 2023 18:55
@ack-prow ack-prow bot added the approved label Mar 22, 2023
@a-hilaly
Copy link
Member

/retest


type fakeDescriptor struct{}

func (fd *fakeDescriptor) GroupKind() *metav1.GroupKind {
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this change belong in this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I suppose this is a miss from my previous PR. Want me to split this out and merge separately?

@ack-prow
Copy link

ack-prow bot commented Mar 22, 2023

@RedbackThomson: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ec2-controller-test 6f95b4c link true /test ec2-controller-test

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jljaco jljaco self-requested a review March 22, 2023 21:09
Copy link
Contributor

@jljaco jljaco left a comment

Choose a reason for hiding this comment

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

looks good. happy to /lgtm once these tests pass (pending a fix in a different PR)

@RedbackThomson
Copy link
Contributor Author

looks good. happy to /lgtm once these tests pass (pending a fix in a different PR)

That PR has already been merged and this PR rebased on top of it

Copy link
Member

@a-hilaly a-hilaly left a comment

Choose a reason for hiding this comment

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

Ty ty!

@ack-prow
Copy link

ack-prow bot commented Mar 22, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: A-Hilaly, jljaco, RedbackThomson

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 [A-Hilaly,RedbackThomson,jljaco]

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

@a-hilaly a-hilaly merged commit fa24753 into aws-controllers-k8s:main Mar 22, 2023
@RedbackThomson RedbackThomson deleted the release-0.25.0 branch March 22, 2023 21:30
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.

3 participants