Skip to content

feat(azurefrontdoor): [PATCH 04/08] SDK client bundle + export-mode a…#383

Open
rchincha wants to merge 1 commit into
Azure:mainfrom
rchincha:rchinchani/afd-04-sdk-bundle
Open

feat(azurefrontdoor): [PATCH 04/08] SDK client bundle + export-mode a…#383
rchincha wants to merge 1 commit into
Azure:mainfrom
rchincha:rchinchani/afd-04-sdk-bundle

Conversation

@rchincha

Copy link
Copy Markdown

…nnotations

Part 04 of an 8-patch split of #373. Adds the client-side plumbing that subsequent reconciler patches depend on. No reconciler logic yet.

  • pkg/common/azurefrontdoor/client.go — armcdn client bundle (Profiles, Endpoints, OriginGroups, Origins, WAFPolicies, SecurityPolicies, AFDCustomDomains) with a single constructor and unit tests exercising the wiring.
  • pkg/common/objectmeta/objectmeta.go — add the export-mode annotation key + PLS annotation keys used by the member serviceexport controller (PATCH 08) and helper ExtractExportModeFromServiceExport.
  • cmd/net-crd-installer/utils/util_test.go — expect AFD CRDs in the hub-mode CRD list (they are added by PATCH 03).

Copilot-Session: a8fabc68-e113-49ef-b82d-d99a8502da11

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Requirements:

How has this code been tested

Special notes for your reviewer

…nnotations

Part 04 of an 8-patch split of Azure#373. Adds the client-side plumbing that
subsequent reconciler patches depend on. No reconciler logic yet.

- pkg/common/azurefrontdoor/client.go — armcdn client bundle
  (Profiles, Endpoints, OriginGroups, Origins, WAFPolicies,
  SecurityPolicies, AFDCustomDomains) with a single constructor and
  unit tests exercising the wiring.
- pkg/common/objectmeta/objectmeta.go — add the export-mode annotation
  key + PLS annotation keys used by the member serviceexport controller
  (PATCH 08) and helper ExtractExportModeFromServiceExport.
- cmd/net-crd-installer/utils/util_test.go — expect AFD CRDs in the
  hub-mode CRD list (they are added by PATCH 03).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: a8fabc68-e113-49ef-b82d-d99a8502da11
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.

1 participant