Skip to content

Commit

Permalink
Automator: merge upstream changes to openshift-service-mesh/sail-oper…
Browse files Browse the repository at this point in the history
…ator@main (openshift-service-mesh#17)

* Use INFO level to log missing namespace in IstioRevision/IstioCNI (openshift-service-mesh#64)

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

* Ensure control plane is deployed immediately when the target namespace is created (openshift-service-mesh#65)

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

* Automator: Update dependencies in istio-ecosystem/sail-operator@main (openshift-service-mesh#66)

Signed-off-by: istio-testing <istio-testing-bot@google.com>

* Update SetDefaultEventuallyTimeout for e2e test (openshift-service-mesh#69)

Signed-off-by: frherrer <frherrer@redhat.com>

* Ensure CI highlights failure in lint-watches.sh (openshift-service-mesh#70)

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

* Make lint-watches ignore ValidatingAdmissionPolicy(Binding) (openshift-service-mesh#71)

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

* Ensure update-istio.sh updates version field in versions.yaml (openshift-service-mesh#72)

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

* Wrap errors to provide context (openshift-service-mesh#68)

* 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>

* Make bundle generation work during merges (openshift-service-mesh#73)

This is required for openshift-service-mesh#14

Signed-off-by: Daniel Grimm <dgrimm@redhat.com>

* Refactor (openshift-service-mesh#76)

* 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>

* Automator: Update dependencies in istio-ecosystem/sail-operator@main (openshift-service-mesh#67)

Signed-off-by: istio-testing <istio-testing-bot@google.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>
Signed-off-by: istio-testing <istio-testing-bot@google.com>
Signed-off-by: frherrer <frherrer@redhat.com>
Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
Co-authored-by: Marko Lukša <mluksa@redhat.com>
Co-authored-by: Istio Automation <istio-testing-bot@google.com>
Co-authored-by: Francisco Herrera <frherrer@redhat.com>
Co-authored-by: Daniel Grimm <dgrimm@redhat.com>
Co-authored-by: openshift-service-mesh-bot <null>
  • Loading branch information
5 people authored Apr 24, 2024
1 parent 5fbdbe3 commit f747b18
Show file tree
Hide file tree
Showing 73 changed files with 1,643 additions and 184 deletions.
2 changes: 1 addition & 1 deletion .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "istio build-tools",
"image": "gcr.io/istio-testing/build-tools:master-f24be7b713480aab44d862ac839ead0b5324d593",
"image": "gcr.io/istio-testing/build-tools:master-8fb9ce88f6ad4cdd35c1660cd0ad0ab67eff4c6c",
"privileged": true,
"remoteEnv": {
"USE_GKE_GCLOUD_AUTH_PLUGIN": "True",
Expand Down
4 changes: 2 additions & 2 deletions Makefile.core.mk
Original file line number Diff line number Diff line change
Expand Up @@ -403,7 +403,7 @@ OPM ?= $(LOCALBIN)/opm
## Tool Versions
OPERATOR_SDK_VERSION ?= v1.34.1
HELM_VERSION ?= v3.14.4
CONTROLLER_TOOLS_VERSION ?= v0.14.0
CONTROLLER_TOOLS_VERSION ?= v0.15.0
OPM_VERSION ?= v1.39.0
GITLEAKS_VERSION ?= v8.18.2

Expand Down Expand Up @@ -461,7 +461,7 @@ bundle: gen helm operator-sdk ## Generate bundle manifests and metadata, then va
if (git ls-files --error-unmatch "$$csvPath" &>/dev/null); then \
if ! (git diff "$$csvPath" | grep '^[+-][^+-][^+-]' | grep -v "createdAt:" >/dev/null); then \
echo "reverting timestamp change in $$csvPath"; \
git checkout "$$csvPath"; \
git checkout "$$csvPath" || echo "failed to revert timestamp change. assuming we're in the middle of a merge"; \
fi \
fi
$(OPERATOR_SDK) bundle validate ./bundle
Expand Down
4 changes: 2 additions & 2 deletions api/v1alpha1/istio_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,10 +51,10 @@ type IstioSpec struct {
// +sail:profile
// The built-in installation configuration profile to use.
// The 'default' profile is always applied. On OpenShift, the 'openshift' profile is also applied on top of 'default'.
// Must be one of: ambient, default, demo, empty, external, minimal, openshift-ambient, openshift, preview, remote.
// Must be one of: ambient, default, demo, empty, external, minimal, openshift-ambient, openshift, preview, remote, stable.
// +++PROFILES-DROPDOWN-HIDDEN-UNTIL-WE-FULLY-IMPLEMENT-THEM+++operator-sdk:csv:customresourcedefinitions:type=spec,displayName="Profile",xDescriptors={"urn:alm:descriptor:com.tectonic.ui:fieldGroup:General", "urn:alm:descriptor:com.tectonic.ui:select:ambient", "urn:alm:descriptor:com.tectonic.ui:select:default", "urn:alm:descriptor:com.tectonic.ui:select:demo", "urn:alm:descriptor:com.tectonic.ui:select:empty", "urn:alm:descriptor:com.tectonic.ui:select:external", "urn:alm:descriptor:com.tectonic.ui:select:minimal", "urn:alm:descriptor:com.tectonic.ui:select:preview", "urn:alm:descriptor:com.tectonic.ui:select:remote"}
// +operator-sdk:csv:customresourcedefinitions:type=spec,xDescriptors={"urn:alm:descriptor:com.tectonic.ui:hidden"}
// +kubebuilder:validation:Enum=ambient;default;demo;empty;external;minimal;openshift-ambient;openshift;preview;remote
// +kubebuilder:validation:Enum=ambient;default;demo;empty;external;minimal;openshift-ambient;openshift;preview;remote;stable
Profile string `json:"profile,omitempty"`

// Namespace to which the Istio components should be installed.
Expand Down
4 changes: 2 additions & 2 deletions api/v1alpha1/istiocni_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,10 @@ type IstioCNISpec struct {
// +sail:profile
// The built-in installation configuration profile to use.
// The 'default' profile is always applied. On OpenShift, the 'openshift' profile is also applied on top of 'default'.
// Must be one of: ambient, default, demo, empty, external, minimal, openshift-ambient, openshift, preview, remote.
// Must be one of: ambient, default, demo, empty, external, minimal, openshift-ambient, openshift, preview, remote, stable.
// +++PROFILES-DROPDOWN-HIDDEN-UNTIL-WE-FULLY-IMPLEMENT-THEM+++operator-sdk:csv:customresourcedefinitions:type=spec,displayName="Profile",xDescriptors={"urn:alm:descriptor:com.tectonic.ui:fieldGroup:General", "urn:alm:descriptor:com.tectonic.ui:select:ambient", "urn:alm:descriptor:com.tectonic.ui:select:default", "urn:alm:descriptor:com.tectonic.ui:select:demo", "urn:alm:descriptor:com.tectonic.ui:select:empty", "urn:alm:descriptor:com.tectonic.ui:select:external", "urn:alm:descriptor:com.tectonic.ui:select:minimal", "urn:alm:descriptor:com.tectonic.ui:select:preview", "urn:alm:descriptor:com.tectonic.ui:select:remote"}
// +operator-sdk:csv:customresourcedefinitions:type=spec,xDescriptors={"urn:alm:descriptor:com.tectonic.ui:hidden"}
// +kubebuilder:validation:Enum=ambient;default;demo;empty;external;minimal;openshift-ambient;openshift;preview;remote
// +kubebuilder:validation:Enum=ambient;default;demo;empty;external;minimal;openshift-ambient;openshift;preview;remote;stable
Profile string `json:"profile,omitempty"`

// Namespace to which the Istio CNI component should be installed.
Expand Down
1 change: 1 addition & 0 deletions bundle/manifests/extensions.istio.io_wasmplugins.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,7 @@ spec:
type: string
priority:
description: Determines ordering of `WasmPlugins` in the same `phase`.
format: int32
nullable: true
type: integer
selector:
Expand Down
Loading

0 comments on commit f747b18

Please sign in to comment.