release radix-operator, fix with aux ingresses, dns alias roles #1024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Added dns aliases types and crd * Added dns aliases controller and handler bases * Extended dns aliases controller and handler, added appName to the RadixDNSAlias * Added ingresses stubs to test * Creating, updating RadixDNSAliases * Extended applicationconfig and RadixDNSAliases controllers * Extended RadixDNSAliases controllers and handler, added ClusterConfig and Radix constants * Extended RadixDNSAliases syncer * Extended RadixDNSAliases syncer and handler, unit-tests, added cluster config * Added config to syncers, added ingresses * Fixed unit-test * Added unit-test * Added unit-test, updating of an ingress port * Added unit-test for owner ref * Added unit-test for header * Updated unit-test for controller * Updated unit-test for app controller * Updated unit-test for app controller * Handled changed, new a dn deleted aliases app-config * Added unit-tests for dns aliases in app-config * Added port to the Radix DNS Alias * Update unit-tests * Update unit-tests * Added DNS alias validation * Added DNS alias validation tests. Removed dot from validation of resources * Extended DNS alias validation * Added validation on existing RadixDNSAliases on applying RadixApplication * Added whitelisted DNS aliases to the chart * Added reserved DNS aliases to the chart and config * Added unit-tests * Added dnsalias reserved list check to common validation * Fixed method invocation * Added validation on applying ra * Added pipeline args for reserved dns aliases * Merged with master * Fixed unit-test * Added validation for reserved aliases * Config moved to pkg * Renamed CRD plural name for radixdnsalias * ClusterConfig renamed to DNSConfig * DNSConfig moved to dnsalias package * Merged * Merged * Full config used as an argument of pipelinejob handler * Added reserved dns aliases validation to pipeline jobs * Fixed test * Adding RDA role and binding to tekton * Added RDA role and binding to tekton * Extracted a method for tekton use * Renamed property * Merged * Added reserved DNS alias arguments to pipeline and tekton jobs * Added reserved DNS alias roles * Removed reconcile from RDA * Domain in DNSAlias renamed to Alias. DNS aliases added to RadixDeployComponent * Ingress interfaces moved to pkg * Fixed linter failure * Extracted kubeUtil for all controllers * Extracted ingress logic from deployment package * Added args ingress and oauthconfig to dnsalias * Replaced registration of ingress by RadixDNSAlias. Needs to fix unit-tests * Do not sync dnsaliases whyle there is no deployment existing * Update refs * Added deletion of dns aliases on deleting radix environments, create/update/swap/delete aliases. Needs to fox deleting radix environments on deleting application * Fixing sync of oauth ingresses * Fixed sync of oauth ingress path * Fixed some alias tests * Added rd informer * Merged * Restored tests * Adding garbage collect for DNS alias * Removed unnecessary ingress garbage collect DNS alias ingresses logic * Added update RD informer * Added finalizer to RadixDNSAlias * Added status to RadixDNSAlias * Replaced standard ingress webhook error message to hide application name * Removed yaml attr from radixdnsalias * Fixed unit-tests * Set version * Fixed linter reported issues * Fixed linter reported issues * Added app alias as reserved * Removed receiver from UpdatePrivateImageHubsSecretsPassword * Removed receiver from GetPendingPrivateImageHubSecrets * Corrected error message * Fixed reported review issues * Fixed reported review issues. * Fixed reported review issues. * Fixed reported review issues. * Fixed reported review issues. * Fixed reported review issues. * Fixed reported review issues. Replaced commonerrors.Concat with standard lib * Fixed reported review issues. Replaced k8s attributes * Merge * Moved methods, used only in radix-api to radix-api * Cleanup * Reworked method syncDNSAliases. Added unit-tests * Removed port from RadixDNSAlias.Take it from RadixDeployment. RDA AppName and environment are unmutable * Replaced panic for test errs with required * DNS alias specific ingresses are processed independently from other ingresses * Fixed tests * Fixed deleted dnsalias logic, method moved to radix-api * Removed GetIngresses methods, Changed log-level method * Corrected logic for DNSAlias deletion * Cleaned linter reported issues * Cleaned linter reported issues * Added defaultAlias label to default ingresses * Corrected deletion of radix environment. Added tests * Added unit-tests for ingress labels * Fixed unit-tests * Added cluster roles and bindings for dns-aliases * Registry moved from radix to radixv1 * Cleanup * Cleanup * Cleanup * Cleanup * Cleanup * Cleanup * Added dns alias cluster role and binding for api * Changed status column title * Added rback for dns alias * Corrected rbac ofnerreferences for dns alias * Adding unit-tests * Cleanup * Unit tests * Merged with master * Unit-test * Added unit-tests * Delete not needed role bindings * Added unit-tests. Corrected deleting role and binding * Corrected deleting role and binding * Merged existing ownerreferences for role and binding * Corrected dns alias rbac sync * Corrected dns alias rbac sync * Corrected dns alias rbac sync * Fixed linter reported issues * Fixed reported suggestions, set new chart version
* Upgrade dependencies * run golang lint on everything and remove staticcheck * bump chart version --------- Co-authored-by: Richard Hagen <[email protected]>
* Do not allow to change env for Radix DNS alias * Validate DNS alias appName. Cleanup * Updated chart version
* Fixed missing ingresses for oauth * handle app alias ingress in oauth aux * Added unit-tests * Added unit-tests * Fixed label --------- Co-authored-by: Nils Gustav Stråbø <[email protected]>
nilsgstrabo
approved these changes
Dec 29, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.