Skip to content

Add Quick Start steps#25

Merged
jtaleric merged 1 commit intoquay:masterfrom
kurtismullins:add-quick-start-docs
Aug 21, 2020
Merged

Add Quick Start steps#25
jtaleric merged 1 commit intoquay:masterfrom
kurtismullins:add-quick-start-docs

Conversation

@kurtismullins
Copy link
Contributor

This is the first time I have walked through the steps, individually, under a fresh environment. It is also the first time using the tests with the Quay Operator TNG.

I decided to document the steps that I needed to take to run the tests using Centos 8. This includes installing several dependencies which the test scripts require but weren't mentioned previously.

- `bc`
- SSH into the VM.
- Clone this repository: `git clone https://github.com/quay/quay-performance-scripts.git`
- Install the test tools: `cd quay-performance-scripts; ./install.sh`
Copy link
Contributor

Choose a reason for hiding this comment

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

can we document setting the labels? the quay:app label for the quay pods and the quay:db for the database?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We would have to work with @alecmerdler to see if that's feasible and determine the steps. That's not a bad idea though.

Copy link
Contributor

@jtaleric jtaleric left a comment

Choose a reason for hiding this comment

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

One nit, not worth blocking, LGTM

@kurtismullins
Copy link
Contributor Author

One nit, not worth blocking, LGTM

Thanks @jtaleric! It looks like I don't have write access so I can't merge the PR. Want to do it?

I'll ask @thomasmckay about getting access in the interim :)

Copy link
Contributor

@jtaleric jtaleric left a comment

Choose a reason for hiding this comment

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

LGTM

@jtaleric jtaleric merged commit 58e4e88 into quay:master Aug 21, 2020
@kurtismullins kurtismullins deleted the add-quick-start-docs branch August 24, 2020 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants