Skip to content

Conversation

@a-hilaly
Copy link
Member

Issue: aws-controllers-k8s/community#582

Description of changes:

  • Added a new field named Policy to the Repository CRD (from the
    SetRepositoryPolicy API Call.
  • Add e2e tests for CRUD operations on the Policy field
  • Refactor setAddtionalFields hook function
  • Fix some e2e rare error appearing when trying to patch an outdated CR

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

Issue: aws-controllers-k8s/community#582

Description of changes:
- Added a new field named `Policy `to the `Repository` CRD (from the
`SetRepositoryPolicy` API Call.
- Add e2e tests for CRUD operations on `Policy` field
- Refactor `setAddtionalFields` hook function
- Fix some e2e rare error appearing when trying to patch a outdated CR
Copy link
Contributor

@jaypipes jaypipes left a comment

Choose a reason for hiding this comment

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

I can't find a single thing wrong with this. Excellent job, @a-hilaly. :)

@jaypipes
Copy link
Contributor

/lgtm

@ack-bot ack-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 21, 2022
@ack-bot
Copy link
Collaborator

ack-bot commented Apr 21, 2022

[APPROVALNOTIFIER] This PR is APPROVED

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

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:

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

@ack-bot ack-bot merged commit 1a1bf07 into aws-controllers-k8s:main Apr 21, 2022
@jackivanov
Copy link

@jaypipes @a-hilaly there seem to be a bug or some kind of race condition with this feature - aws-controllers-k8s/community#1873

@a-hilaly
Copy link
Member Author

@jackivanov Fixed in v1.0.9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants