e2e: add subscription workload #1357
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
Signed-off-by: jacklu <jilu@redhat.com>
|
I have verified this PR works in my test env: |
added 2 commits
May 6, 2024 18:34
Signed-off-by: jacklu <jilu@redhat.com>
Signed-off-by: jacklu <jilu@redhat.com>
|
@par97 for this PR to work in the e2e environment, we need to create a Channel. As the script sets up the RDR env, and deploys and configures Ramen but expects the next tool to create the Channel. (see failure here: #1373 ) In the case of e2e we would need another option, or just take in a .yaml file with the options for the git link to the Channel sources and create a channel from the code to begin with. Example YAML can look like: https://github.com/RamenDR/ramen/blob/main/test/configs/deployment-k8s-regional-rbd.yaml |
|
@ShyamsundarR The channel is a prereq to run the subscription + deployment workload. As documented in https://github.com/RamenDR/ramen/blob/main/docs/testing.md, we need first add a channel: I have plan to add check for the channel in the Validate Suite. To add such channel, we could have a few options:
|
I am suggesting (3) because in local setups or when experimenting with non-standard channels (forms of ocm-ramen-sample) it is easier to pass in the git source and create a channel for e2e to use. |
added 2 commits
May 7, 2024 15:23
Signed-off-by: jacklu <jilu@redhat.com>
Signed-off-by: jacklu <jilu@redhat.com>
par97
force-pushed
the
add-subscription-workload
branch
from
dac65c9
to
8ae1d11
Compare
|
I added this issue to track some review comments that we can fix subsequently, so that we can make progress on using what is available (most will potentially be resolved once we get to the Sub+CephFS and/or AppSet+Workload as we generalize the framework and utils functions) |
added 3 commits
May 15, 2024 19:03
Signed-off-by: jacklu <jilu@redhat.com>
Signed-off-by: jacklu <jilu@redhat.com>
Signed-off-by: jacklu <jilu@redhat.com>
Signed-off-by: jacklu <jilu@redhat.com>
ShyamsundarR
approved these changes
May 17, 2024
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.
This PR is to enable subscription application with busybox deployment for e2e test.