Skip to content

Remove command-line flags page from query UI#1216

Merged
bwplotka merged 3 commits intothanos-io:masterfrom
alexissellier:remove-command-line-flags-ui
Jun 4, 2019
Merged

Remove command-line flags page from query UI#1216
bwplotka merged 3 commits intothanos-io:masterfrom
alexissellier:remove-command-line-flags-ui

Conversation

@alexissellier
Copy link
Contributor

This PR closes #248

Changes

I removed the command line flags panel from the query UI. Since there is now only one section to the Status drop-down, I removed it to only keep the Runtime & Build Information panel in the global menu

@bwplotka
Copy link
Member

bwplotka commented Jun 4, 2019

Awesome, thanks for this! I think it makes sense but I would vote for not changing the Status page more.
The reason is that we copy the UI from Prometheus and it easier if we do as small number of our changes as possible.

Can we just remove command line flags? (:

@alexissellier
Copy link
Contributor Author

Ok i will leave the status page as it is in master

@alexissellier
Copy link
Contributor Author

@bwplotka status dropdown is back

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks (: That will save some confusion.

@bwplotka bwplotka merged commit bab8886 into thanos-io:master Jun 4, 2019
FUSAKLA pushed a commit to FUSAKLA/thanos that referenced this pull request Jun 8, 2019
* Remove command-line flags page from query UI

* Dummy commit for relaunch ci job

* Put back status dropdown
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.

Command-line flags page for query instances is not populated

2 participants