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

Automator: merge upstream changes to openshift-service-mesh/sail-operator@main #20

Conversation

openshift-service-mesh-bot

Generated by Automator - 2024-04-29T09:59:54+00:00

luksa and others added 20 commits April 19, 2024 05:35
…e is created (openshift-service-mesh#65)

Signed-off-by: Marko Lukša <mluksa@redhat.com>
* Wrap errors to provide context

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Add called function instead of the caller to error message

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Fix lint errors

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Fix test

Signed-off-by: Marko Lukša <mluksa@redhat.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>
This is required for openshift-service-mesh#14

Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
* Merge if statements

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Replace multiple ifs with switch

Signed-off-by: Marko Lukša <mluksa@redhat.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>
Signed-off-by: Marko Lukša <mluksa@redhat.com>
Signed-off-by: Marko Lukša <mluksa@redhat.com>
…sh#88)

Previously, when the namespace referenced in IstioCni or IstioRevision.spec.namespace was being deleted, the operator log would show this as an error (with a stacktrace). Because of the requeues, the error was also logged many times.

Now, if the namespace is being deleted, the operator logs this at INFO level. It also doesn't requeue the object in this case.

Signed-off-by: Marko Lukša <mluksa@redhat.com>
* upstream/main:
  Don't log error when namespace is being deleted (openshift-service-mesh#88)
  Automator: Update dependencies in istio-ecosystem/sail-operator@main (openshift-service-mesh#89)
  Introduce a central way to get the project's root directory (openshift-service-mesh#83)
  Remove duplicates of oneMinuteAgo() (openshift-service-mesh#82)
  Automator: Update dependencies in istio-ecosystem/sail-operator@main (openshift-service-mesh#81)
  Deploy CNI DaemonSet immediately when namespace is created (openshift-service-mesh#80)
  Remove unused fields from BaseConfig (openshift-service-mesh#79)
  Automator: Update dependencies in istio-ecosystem/sail-operator@main (openshift-service-mesh#77)
  Automator: Update dependencies in istio-ecosystem/sail-operator@main (openshift-service-mesh#67)
  Refactor (openshift-service-mesh#76)
  Make bundle generation work during merges (openshift-service-mesh#73)
  Wrap errors to provide context (openshift-service-mesh#68)
  Ensure update-istio.sh updates version field in versions.yaml (openshift-service-mesh#72)
  Make lint-watches ignore ValidatingAdmissionPolicy(Binding) (openshift-service-mesh#71)
  Ensure CI highlights failure in lint-watches.sh (openshift-service-mesh#70)
  Update SetDefaultEventuallyTimeout for e2e test (openshift-service-mesh#69)
  Automator: Update dependencies in istio-ecosystem/sail-operator@main (openshift-service-mesh#66)
  Ensure control plane is deployed immediately when the target namespace is created (openshift-service-mesh#65)
  Use INFO level to log missing namespace in IstioRevision/IstioCNI (openshift-service-mesh#64)

# Conflicts:
#	bundle/manifests/servicemeshoperator3.clusterserviceversion.yaml
@openshift-merge-bot openshift-merge-bot bot merged commit c3722ec into openshift-service-mesh:main Apr 29, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants