Skip to content

Add integration test to run Cortex with the getting started single binary config file #2143

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

Conversation

pracucci
Copy link
Contributor

What this PR does:
In this PR I've added a simple integration test to run Cortex with the getting started single binary config file and verify that the happy path works (push a metric, read it back).

Which issue(s) this PR fixes:
Fixes #2136

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@pracucci pracucci force-pushed the add-integration-test-for-single-process-config branch from b5d0e64 to 9567713 Compare February 17, 2020 16:41
@pracucci pracucci force-pushed the add-integration-test-for-single-process-config branch from 9567713 to 099e492 Compare February 17, 2020 17:57
Signed-off-by: Marco Pracucci <[email protected]>
Copy link
Contributor

@jtlisi jtlisi left a comment

Choose a reason for hiding this comment

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

LGTM

@pracucci pracucci merged commit 8808b90 into cortexproject:master Feb 17, 2020
@pracucci pracucci deleted the add-integration-test-for-single-process-config branch February 17, 2020 18:31
simonswine pushed a commit to grafana/e2e that referenced this pull request Jan 13, 2022
…nary config file (cortexproject/cortex#2143)

* Add integration test to run Cortex with the getting started single binary config file

Signed-off-by: Marco Pracucci <[email protected]>

* Fixed circleci config and lint issues

Signed-off-by: Marco Pracucci <[email protected]>
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.

Add integration test for the getting started guide
2 participants