Skip to content

Conversation

@tuftedocelot
Copy link
Contributor

What this PR does / why we need it:
Allows for updating to the latest Loki version on OpenBSD after a series of updates in https://github.com/golang/x/sys were committed (and after #6833)

Fixes running on OpenBSD and tested on arm64 and amd64
@tuftedocelot tuftedocelot requested a review from a team as a code owner September 1, 2022 01:14
@CLAassistant
Copy link

CLAassistant commented Sep 1, 2022

CLA assistant check
All committers have signed the CLA.

@grafanabot
Copy link
Contributor

./tools/diff_coverage.sh ../loki-main/test_results.txt test_results.txt ingester,distributor,querier,querier/queryrange,iter,storage,chunkenc,logql,loki

Change in test coverage per package. Green indicates 0 or positive change, red indicates that test coverage for a package fell.

+           ingester	0%
+        distributor	0%
+            querier	0%
+ querier/queryrange	0%
+               iter	0%
+            storage	0%
+           chunkenc	0%
+              logql	0%
+               loki	0%

@MasslessParticle MasslessParticle merged commit a13754a into grafana:main Sep 7, 2022
@tuftedocelot tuftedocelot deleted the bump-x_sys branch September 7, 2022 20:24
lxwzy pushed a commit to lxwzy/loki that referenced this pull request Nov 7, 2022
…7021)

<!--  Thanks for sending a pull request!  Before submitting:

1. Read our CONTRIBUTING.md guide
2. Name your PR as `<Feature Area>: Describe your change`.
a. Do not end the title with punctuation. It will be added in the
changelog.
b. Start with an imperative verb. Example: Fix the latency between
System A and System B.
  c. Use sentence case, not title case.
d. Use a complete phrase or sentence. The PR title will appear in a
changelog, so help other people understand what your change will be.
3. Rebase your PR if it gets out of sync with main
-->

**What this PR does / why we need it**:
Allows for updating to the latest Loki version on OpenBSD after a series
of updates in https://github.com/golang/x/sys were committed (and after
grafana#6833)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants