Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Microsoft.KubernetesConfiguration/extensions #3528

Merged
merged 19 commits into from
Nov 14, 2023

Conversation

super-harsh
Copy link
Collaborator

@super-harsh super-harsh commented Nov 7, 2023

Closes #2934

What this PR does / why we need it:

This PR adds support for Microsoft.KubernetesConfiguration/extensions.

Also includes a generator fix to identify the extension types

Special notes for your reviewer:
Removing the ConfigurationProtectedSettings configuration from the resource for now as it's dependent on #3435 being resolved. Once resolved, we can revert this field to be on the resource.

@super-harsh
Copy link
Collaborator Author

TODO: Need to add sample and record sample test

@super-harsh super-harsh force-pushed the feature/k8s-extension branch from 71bd0c4 to c0c4203 Compare November 8, 2023 08:01
@super-harsh super-harsh self-assigned this Nov 9, 2023
v2/api/kubernetesconfiguration/v1api20221101/structure.txt Outdated Show resolved Hide resolved
v2/api/kubernetesconfiguration/v1api20221101/structure.txt Outdated Show resolved Hide resolved
v2/azure-arm.yaml Outdated Show resolved Hide resolved
v2/internal/controllers/samples_test.go Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Nov 12, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c3f0aa6) 54.02% compared to head (120191a) 52.84%.
Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3528       +/-   ##
===========================================
- Coverage   54.02%   52.84%    -1.18%     
===========================================
  Files        1578     1285      -293     
  Lines      657622   401929   -255693     
===========================================
- Hits       355267   212404   -142863     
+ Misses     244736   158918    -85818     
+ Partials    57619    30607    -27012     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@matthchr matthchr added this to the v2.4.0 milestone Nov 13, 2023
@super-harsh super-harsh force-pushed the feature/k8s-extension branch from 4724666 to e44fc1e Compare November 13, 2023 23:45
Copy link
Member

@matthchr matthchr left a comment

Choose a reason for hiding this comment

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

LGTM!

@super-harsh super-harsh force-pushed the feature/k8s-extension branch from 365e033 to df81002 Compare November 14, 2023 01:47
@super-harsh super-harsh added this pull request to the merge queue Nov 14, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 14, 2023
@super-harsh super-harsh added this pull request to the merge queue Nov 14, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 14, 2023
@super-harsh super-harsh added this pull request to the merge queue Nov 14, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 14, 2023
@super-harsh super-harsh added this pull request to the merge queue Nov 14, 2023
Merged via the queue into main with commit 5949407 Nov 14, 2023
9 checks passed
@super-harsh super-harsh deleted the feature/k8s-extension branch November 14, 2023 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Feature: Support for Azure container marketplace offerings (Microsoft.KubernetesConfiguration/extensions)
3 participants