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

feat: specify prom server start addr #1111

Merged
merged 2 commits into from
Mar 6, 2023
Merged

feat: specify prom server start addr #1111

merged 2 commits into from
Mar 6, 2023

Conversation

yuanbohan
Copy link
Contributor

@yuanbohan yuanbohan commented Mar 1, 2023

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

start prometheus API server with specified addr, so the /query and /query_range can be accessible outside of container by running the standalone mode: cargo run -- standalone start --prom-addr=0.0.0.0:4004

@yuanbohan yuanbohan requested review from evenyag and waynexia March 1, 2023 16:02
@codecov
Copy link

codecov bot commented Mar 1, 2023

Codecov Report

Merging #1111 (50c261e) into develop (9865989) will increase coverage by 0.07%.
The diff coverage is 64.00%.

@@             Coverage Diff             @@
##           develop    #1111      +/-   ##
===========================================
+ Coverage    85.09%   85.16%   +0.07%     
===========================================
  Files          473      474       +1     
  Lines        68605    68897     +292     
===========================================
+ Hits         58380    58679     +299     
+ Misses       10225    10218       -7     

@waynexia
Copy link
Member

waynexia commented Mar 2, 2023

promql-server will change to prometheus (discuss). So for this PR we'd better name the config prometheus-server

@yuanbohan yuanbohan changed the title feat: specify promql server start addr feat: specify prom server start addr Mar 2, 2023
Copy link
Member

@waynexia waynexia left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@evenyag evenyag left a comment

Choose a reason for hiding this comment

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

LGTM. Now we have two similar configurations prometheus_options and prom_options...... Could you also update the configuration reference in our docs repo?

@evenyag evenyag added the docs-required This change requires docs update. label Mar 6, 2023
@evenyag
Copy link
Contributor

evenyag commented Mar 6, 2023

promql-server will change to prometheus (discuss).

Shall we create an issue for this? I'm not sure whether we already have an issue tracking this.

@evenyag evenyag merged commit d4e0dc3 into develop Mar 6, 2023
@evenyag evenyag deleted the start-promql-cmd branch March 6, 2023 03:07
@waynexia
Copy link
Member

waynexia commented Mar 6, 2023

promql-server will change to prometheus (discuss).

Shall we create an issue for this? I'm not sure whether we already have an issue tracking this.

Sure: #1127

paomian pushed a commit to paomian/greptimedb that referenced this pull request Oct 19, 2023
* feat: specify promql server start addr

* refactor: rename promql to prom in Prometheus API server scenario
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-required This change requires docs update.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants