Merge v2.8.0 #735
Merged
Merge v2.8.0 #735
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
Update stable to v2.7.0
This reverts commit df5c004.
Setting the same image that rhoai dashboard is currently using Signed-off-by: Jorge Garcia Oncins <[email protected]>
Signed-off-by: Jorge Garcia Oncins <[email protected]>
…y-image fix: update oauth-proxy-image and MLMD Envoy images
Signed-off-by: Humair Khan <[email protected]>
when a dspa is marked for deletion, there is no state logic to handle dspa statuses, so the update can at times try to update a dspa that may have already been deleted, this change should prevent (or reduce) the chances of this happening, in the future proper/graceful dspa termination handling should be added Signed-off-by: Humair Khan <[email protected]>
…_err prevent dspa status update on a terminating dspa
This change adds a new apiversion "v1", which will remove all the dsp "v1" related fields, and also defaults to dsp "v2". The new apiversion is set as the stored version, however k8s will continue to serve v1alpha1, however it is deprecated. All tests are updated acordingly. Signed-off-by: Humair Khan <[email protected]>
This change makes it so that new reconciles on dspa owned resources are labelled with a dsp-version=<dspa-version> label. This is to allow for the dspa selectively watch (for manual WatchesRawSources) on resources that are descendents of DSPAs with .spec.dspVersion set to supported versions. Signed-off-by: Humair Khan <[email protected]>
Chore: Remove v1 fields and reconcile logic from dspo
this will match the config path to what we are specifying for env vars for the pa image foudn in config/base/params.env i.e. IMAGES_PERSISTENCEAGENT should match Images.PersistenceAgent Signed-off-by: Humair Khan <[email protected]>
Signed-off-by: Humair Khan <[email protected]>
correct pa config path
Signed-off-by: Humair Khan <[email protected]>
update compatibility doc for 2.8
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
A new image has been built to help with testing out this PR: To use this image run the following: cd $(mktemp -d)
git clone [email protected]:opendatahub-io/data-science-pipelines-operator.git
cd data-science-pipelines-operator/
git fetch origin pull/735/head
git checkout -b pullrequest 57db0b4d0839776a1c1382ba207f0a987e4dd23d
oc new-project opendatahub
make deploy IMG="quay.io/opendatahub/data-science-pipelines-operator:pr-735"More instructions here on how to deploy and test a Data Science Pipelines Application. |
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.
Merge v2.8.0