r/ingress_class - new resource

[DrFaust92]

Description

Acceptance tests

• Have you added an acceptance test for the functionality being added?
• Have you run the acceptance tests on this branch?

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccKubernetesIngressClass_'
--- PASS: TestAccKubernetesIngressClass_basic (2.98s)
--- PASS: TestAccKubernetesIngressClass_parameters (4.78s)
--- PASS: TestAccKubernetesIngressClass_parameters_apiGroup (4.82s)

Release Note

Release note for CHANGELOG:

resource_kubernetes_ingress_class - new resource

References

Closes #1217
Closes #1218

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

[philicious]

any ETA when this might be ready ? watching the PR now for a while and waiting for a release :) 👍

[lkishalmi]

Waiting for this as well.

[philicious]

@DrFaust92 is there anything you know of that is blocking this PR to be merged ?

can we maybe get a reviewers comment too please ! @dak1n1 @alexsomesan

let me know if there is sth you could need help with to get this PR ready/merged !

[lkishalmi]

@alexsomesan is there anything keeping this PR from being merged?

[nickbruun]

Is there anything the community can do to drive this to be merged?

[philicious]

@dak1n1 @alexsomesan @jrhouston pinging you as you have approved previous PRs and seem to be maintainers. could you pls shed some light how the community can get this PR merged ?

[lkishalmi]

@alexsomesan @dak1n1 @jrhouston Is there someone to call to get this one merged?

[philicious]

@alexsomesan and @jrhouston I see you are active commiters so I yet again try to get your help on this PR: what would be missing to get this one merged ?
its open for so long and becoming a a real bummer to not IngressClass supported by k8s provider yet 👎

[jrhouston]

Apologies for the delay here @philicious @lkishalmi @DrFaust92 – I have made a few fixes (some validation, docs fixes, and the resource ID didn't need to be namespaced) to this branch, and will merge once the the checks pass.

[philicious]

thanks alot @jrhouston !!!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.