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

Update dashboard #94

Closed
wants to merge 27 commits into from
Closed

Conversation

jwcastillo
Copy link
Contributor

PR Dashboard closing 2022

It includes:
All available k6 metrics - prometheus
Apdex

Does not include:
Custom Metrics

Copy link
Contributor

@codebien codebien left a comment

Choose a reason for hiding this comment

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

Hey @jwcastillo,
it seems the branch has some conflicts. Can you rebase, please?

Copy link
Contributor

@codebien codebien left a comment

Choose a reason for hiding this comment

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

Hey @jwcastillo,
the file with conflicts is grafana/dashboards/dashboard-results.json

You can pull the main branch's changes locally then move to your branch and try to rebase it, you should get the conflict and the specific lines.

codebien and others added 17 commits January 15, 2023 13:34
* Import k6's golangci-lint configuration
* Address linting issues repository-wide
There are no reasons to set it to true, the default server URL
is HTTP based and if an HTTPS endpoint is added then the
client validation is expected to happen with the default path.
Sample: add new sample
* Use instant query for counters
Labels with empty key or value are not allowed by the remote write spec,
so if any seen tag contains them then they are not mapped to the related
label.
Signed-off-by: Paul Balogh <javaducky@gmail.com>
Signed-off-by: Paul Balogh <javaducky@gmail.com>
@codebien
Copy link
Contributor

codebien commented Jan 16, 2023

Hey @jwcastillo,
the rebase didn't go well, you should drop the commits added after cc7387b.

If you can't recover then I suggest starting over in a clean branch where the parent is the main branch, then you should be able to apply the required changes by cherry-picking from this branch.

@jwcastillo
Copy link
Contributor Author

ok

@jwcastillo jwcastillo closed this Jan 16, 2023
@jwcastillo jwcastillo deleted the apdex_dashboard branch January 16, 2023 08:52
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.

None yet

4 participants