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.

Sign up for GitHub

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

Jump to bottom

Watch routes and ingress, ensure we're only using one #1169

Merged
merged 3 commits into from
Sep 20, 2024
Merged

Watch routes and ingress, ensure we're only using one #1169

merged 3 commits into from
Sep 20, 2024

Commits on Sep 19, 2024

  1. Watch Ingresses and Routes if available 

    Allow changes to our Ingress and Route to trigger a reconcile.
Routes are not available outside of OpenShift and the controller setup
will fail if we try to watch resources that are not registered to the
cluster.  So, we first need to see if we get an error if we try to list
routes.

CP4AIOPS-6311
    @bdunne
    bdunne committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    f2b13af View commit details
    Browse the repository at this point in the history

  2. If we are using Routes, ensure that the ingress is removed 

    Sometimes when we are initially created we go down the ingress path rather
than the routes.  Then certificates are created and we switch to routes
instead.  Unfortunately nothing was cleaning up the ingress (and potentially
associated route httpd-xxxxx if we are running in OpenShift.)  This ensures
that if we are using routes, we clean up any ingress that we may have created
in the past.

CP4AIOPS-6311
    @bdunne
    bdunne committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    6eb42fb View commit details
    Browse the repository at this point in the history

  3. Ensure that the Certificate and Key are removed from the route 

    In the past we had set certificates on routes, since ManageIQ#1161 we want to use
the cluster default certificates.  So, if the route existed before that
was merged, we need to remove the certificate and key fields.

CP4AIOPS-6311
    @bdunne
    bdunne committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    e550de3 View commit details
    Browse the repository at this point in the history