Skip to content
This repository was archived by the owner on Mar 22, 2021. It is now read-only.

Update README.md #56

Closed
wants to merge 1 commit into from
Closed

Update README.md #56

wants to merge 1 commit into from

Conversation

cevich
Copy link
Contributor

@cevich cevich commented Jul 31, 2017

Add monitoring for CRI-O in an attempt to replace travis

Add monitoring for CRI-O in an attempt to replace travis
@rh-atomic-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@cevich
Copy link
Contributor Author

cevich commented Jul 31, 2017

quick question too: if I use multiple documents as in the example, are the results/output separated by "context" or just lumped all into one ourput file?

@jlebon
Copy link
Collaborator

jlebon commented Aug 3, 2017

bot, add author to whitelist

@jlebon
Copy link
Collaborator

jlebon commented Aug 3, 2017

Each new testsuite (separated by ---) will yield a different context, with different outputs. Check out PRs in e.g. ostree & rpm-ostree for example.

@cevich
Copy link
Contributor Author

cevich commented Aug 3, 2017

Excellent.

@cevich
Copy link
Contributor Author

cevich commented Aug 10, 2017

@jlebon FWIW, #57 isn't a blocker for this. The crio-o guys want to test on Fedora with this, and can limp along on travis for Ubuntu. Though having a unified PI-tester would be nice for re-usability. I don't think it's a hard-requirement.

@jlebon
Copy link
Collaborator

jlebon commented Aug 10, 2017

@cevich Sounds good. Just let me know when the proper access has been secured and we can move forward.

@cevich
Copy link
Contributor Author

cevich commented Aug 10, 2017

oh right, thx for reminding me. mrunal said he's still working on it.

@cevich cevich closed this Jan 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants