Skip to content

feat: init v1beta1 work controller when enabling v1beta1 api#515

Merged
zhiying-lin merged 2 commits intoAzure:mainfrom
zhiying-lin:work-controller-caller
Sep 8, 2023
Merged

feat: init v1beta1 work controller when enabling v1beta1 api#515
zhiying-lin merged 2 commits intoAzure:mainfrom
zhiying-lin:work-controller-caller

Conversation

@zhiying-lin
Copy link
Contributor

@zhiying-lin zhiying-lin commented Sep 4, 2023

Description of your changes

  • init v1beta1 work controller when enabling v1beta1 api
  • update v1beta1 work controller to use the v1beta1 annotation & finalizer
  • remove manager.go file as the CreateControllers are only used by the integration tests.

Fixes #

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

@zhiying-lin zhiying-lin changed the title feat: init v1beta1 work controller when enabling v1beta1 api feat: init work controller when enabling v1beta1 api Sep 4, 2023
@zhiying-lin zhiying-lin changed the title feat: init work controller when enabling v1beta1 api feat: create work controller when enabling v1beta1 api Sep 4, 2023
@zhiying-lin zhiying-lin force-pushed the work-controller-caller branch from ae96aa5 to 865ee1b Compare September 4, 2023 06:59
@zhiying-lin zhiying-lin changed the title feat: create work controller when enabling v1beta1 api feat: init v1beta1 work controller when enabling v1beta1 api Sep 4, 2023
michaelawyu
michaelawyu previously approved these changes Sep 7, 2023
Copy link
Contributor

@michaelawyu michaelawyu left a comment

Choose a reason for hiding this comment

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

LGTM, except one comment.

@zhiying-lin zhiying-lin merged commit 90dd8a1 into Azure:main Sep 8, 2023
@zhiying-lin zhiying-lin deleted the work-controller-caller branch September 8, 2023 02:12
weng271190436 added a commit to weng271190436/fleet that referenced this pull request Mar 18, 2026
@weng271190436 weng271190436 mentioned this pull request Mar 18, 2026
1 task
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