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

chore: Add pre-release PR template to run and update nightly demos #64

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Techassi
Copy link
Member

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There probably needs to be another list for things to check during the demos.

  • Check that the correct images are being used
    # Ensure correct product versions are used as well as 0.0.0-dev
    kubectl get pods -n default -o yaml | yq -y '.items[] | .status.containerStatuses[] | .image'
    # Ensure the 0.0.0-dev operators are used.
    kubectl get pods -n stackable-operators -o yaml | yq -y '.items[] | .status.containerStatuses[] | .image'
  • Check the demo (duh)

Copy link
Member

@NickLarsenNZ NickLarsenNZ Jul 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the release upgrade testing, we would modify it to ensure that the correct product versions are used, and the SDP X.XX version is used (not 0.0.0-dev).

On top of that, I think the demo should be run (according to the docs), and then the upgrade should be done. This mimics a real upgrade where there was data.

Comment on lines +19 to +26
> 1. Create a `.env` file in your local copy of this repository.
> 2. Create local stack and demo definition files, like `stacks-v2.local.yaml`
> and `demos-v2.local.yaml`. They should use the nightly version instead of
> stable. They also might need to point to local manifests instead of remote
> ones.
> 3. Add the `STACKABLE_STACK_FILES` and `STACKABLE_DEMO_FILES` env vars to the
> file. They should point to the absolute path of your local stack and demo
> definition files.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just adding a different perspective to consider...

Tbh, I prefer using the checked out stack files explicitly on the command line.
The same changes get made, as you described, but if there are actual fixes needed, it is a matter of editing the same files and only checking in those changes (and reverting the other ones that get it working with nightly).

Not at all suggesting my way is better, but for me, doing the above, I would forget to unset the variable, or I would run a new shell that doesn't have it set, and invalidate the testing). I use the shell history heavily.

@NickLarsenNZ
Copy link
Member

NickLarsenNZ commented Sep 10, 2024

This template is a partial duplicate of the one added in stackabletech/issues#609 and modified in stackabletech/issues#602

We will consolidate them later.

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.

2 participants