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

Move scripts to help manage items in crontab on the teuthology box #332

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

Conversation

yuriw
Copy link
Contributor

@yuriw yuriw commented Sep 12, 2017

Replaces ceph/teuthology#1108

We have these scripts in ceph tree (ceph/qa/nightlies/ and ceph/qa/machine_types/) and used to keep check out directory of ceph-qa-suites on the teuthology box in order to use with nightlies. I think it'd be easier if we move them, and keep pulling latest for teuthology.

Signed-off-by: Yuri Weinstein [email protected]

@yuriw
Copy link
Contributor Author

yuriw commented Sep 12, 2017

@zmc @djgalloway for some reason I can't assign Reviewers, pls tale a look

@@ -0,0 +1,14 @@
overrides:
Copy link
Member

Choose a reason for hiding this comment

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

Remind me why this file can't be in ceph.git/qa/suites/ ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zmc because the way we use it I believe per @liewegas ' initial suggestion is we add vps.yaml on every line in crontab that runs on vps nodes

Copy link
Member

Choose a reason for hiding this comment

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

That doesn't really answer the question :-/

@djgalloway djgalloway changed the base branch from master to main June 1, 2022 17:02
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