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

fix(openshift): update Quickstart text/formatting #688

Merged
merged 4 commits into from
Nov 24, 2023

Conversation

ebaron
Copy link
Member

@ebaron ebaron commented Nov 23, 2023

Welcome to Cryostat! 👋

Before contributing, make sure you have:

  • Read the contributing guidelines
  • Linked a relevant issue which this PR resolves
  • Linked any other relevant issues, PR's, or documentation, if any
  • Resolved all conflicts, if any
  • Rebased your branch PR on top of the latest upstream main branch
  • Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
  • Signed all commits: git commit -S -m "YOUR_COMMIT_MESSAGE"

Related to: #668

Description of the change:

Fixes indentation and fields. Removes explicit summary of tasks since this is already generated when working through the Quickstart. Removed placeholder text and summary fields. Generally followed the example of our existing quickstart: openshift/console-operator#770

Motivation for the change:

When installing the bundle with the Quickstart included, I see the following errors:

unknown field "spec.tasks[0].review.summary", unknown field "spec.tasks[1].review.summary", unknown field "spec.tasks[2].conclusion", unknown field "spec.title"

How to manually test:

  1. oc create -f config/openshift/cryostat-quickstart.yaml
  2. Access the Quickstart from the Help menu in the OpenShift Console.

andrewazores
andrewazores previously approved these changes Nov 24, 2023
Copy link

@cmah88 cmah88 left a comment

Choose a reason for hiding this comment

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

LGTM

@ebaron ebaron merged commit e8e0bef into cryostatio:main Nov 24, 2023
5 checks passed
mergify bot pushed a commit that referenced this pull request Nov 24, 2023
* fix(openshift): update Quickstart text/formatting

* Update final task description

* Add additional port verification step

(cherry picked from commit e8e0bef)
ebaron added a commit that referenced this pull request Nov 24, 2023
* fix(openshift): update Quickstart text/formatting

* Update final task description

* Add additional port verification step

(cherry picked from commit e8e0bef)

Co-authored-by: Elliott Baron <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants