Release radix-operator, dns aliases #1017
Merged
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
* Add TektonImage to RadixJobSpec * Add Tests --------- Co-authored-by: Richard Hagen <[email protected]>
* 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 * Registry moved from radix to radixv1 * Cleanup * Cleanup * Cleanup * Cleanup * Cleanup * Changed status column title * Cleanup * Controllers fail with panic, when creating informers with error * Fixed of review commented issues * Fixed of review commented issues
nilsgstrabo
approved these changes
Dec 20, 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.