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

Add statefulset and daemonset samples #41

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nirs
Copy link
Member

@nirs nirs commented Nov 13, 2023

  • Add statefulset application
  • Add daemonset application

Not tested yet, need samples for discovered apps.

@nirs nirs force-pushed the statefulset branch 2 times, most recently from 0c8b7df to 69c7a85 Compare November 13, 2023 19:27
@nirs

This comment was marked as outdated.

@nirs

This comment was marked as outdated.

@nirs

This comment was marked as outdated.

@nirs nirs force-pushed the statefulset branch 2 times, most recently from b0b2dd2 to f40c24f Compare November 14, 2023 15:55
@nirs nirs changed the title Add statefulset sample Add statefulset and daemonset samples Nov 14, 2023
@nirs nirs force-pushed the statefulset branch 2 times, most recently from b748df9 to 0cd51d1 Compare November 15, 2023 21:28
@nirs nirs marked this pull request as ready for review November 15, 2023 21:30
@nirs

This comment was marked as outdated.

@nirs nirs marked this pull request as draft November 26, 2023 16:36
@nirs

This comment was marked as outdated.

@nirs nirs marked this pull request as ready for review November 26, 2023 23:58
@nirs nirs marked this pull request as draft November 27, 2023 22:47
@nirs nirs marked this pull request as ready for review November 29, 2023 23:27
@nirs nirs mentioned this pull request Nov 29, 2023
nirs added a commit to nirs/ocm-kubevirt-samples that referenced this pull request Nov 30, 2023
Similar to ocm-ramen-sample PR[1] move the channel into the subscription
kustomization.

This is better for users, removing setup and teardown from the
deployment steps.  Now it is not possible to forget to deploy or remove
the channel after testing.

This is better for developers, since now we can test multiple VMs
concurrently, without dealing with setup and cleanup of shared
resources, since VMs do not share the channel.

[1] RamenDR/ocm-ramen-samples#41
nirs added a commit to nirs/ocm-kubevirt-samples that referenced this pull request Nov 30, 2023
Similar to ocm-ramen-sample PR[1] move the channel into the subscription
kustomization.

This is better for users, removing setup and teardown from the
deployment steps.  Now it is not possible to forget to deploy or remove
the channel after testing.

This is better for developers, since now we can test multiple VMs
concurrently, without dealing with setup and cleanup of shared
resources, since VMs do not share the channel.

[1] RamenDR/ocm-ramen-samples#41
nirs added a commit to nirs/ocm-kubevirt-samples that referenced this pull request Jan 15, 2024
Similar to ocm-ramen-sample PR[1] move the channel into the subscription
kustomization.

This is better for users, removing setup and teardown from the
deployment steps.  Now it is not possible to forget to deploy or remove
the channel after testing.

This is better for developers, since now we can test multiple VMs
concurrently, without dealing with setup and cleanup of shared
resources, since VMs do not share the channel.

[1] RamenDR/ocm-ramen-samples#41
@nirs nirs marked this pull request as draft March 7, 2024 22:29
@nirs nirs mentioned this pull request Mar 7, 2024
2 tasks
Each sample has a:
- workload that can be used later for testing discovered apps
- subscription kustomization that can be used manual testing or with ramen basic-test
- dr kustomization that can be used manually (ramen basic-test generated
  its own drpc)

Unfinished:
- Not test yet
- Need to add dr samples for discovered apps

Signed-off-by: Nir Soffer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant