Adds an OpenTelemetry profile for development purposes. #98
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lubosmj
reviewed
Mar 3, 2023
bmbouter
reviewed
Mar 3, 2023
profiles/opentelemetry_dev/s6-rc-modifications/pulpcore-api/run
Outdated
Show resolved
Hide resolved
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
from
a54f7c6
to
414222c
Compare
newswangerd
reviewed
Mar 3, 2023
|
I assume that we would be expected to use this via the using multiple environments instructions? https://github.com/pulp/oci_env#multiple-environments |
decko
changed the title
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
2 times, most recently
from
33b9a95
to
cf4b68e
Compare
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
2 times, most recently
from
15f9d77
to
dfc4bcf
Compare
dralley
reviewed
Apr 19, 2023
dralley
reviewed
Apr 19, 2023
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
from
dfc4bcf
to
91c5a08
Compare
dralley
approved these changes
Apr 19, 2023
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
6 times, most recently
from
32fdad2
to
39c0712
Compare
ggainey
requested changes
Apr 25, 2023
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
from
39c0712
to
b6d0c00
Compare
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
from
b6d0c00
to
4ece734
Compare
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
2 times, most recently
from
e4be476
to
7da6e09
Compare
ggainey
approved these changes
May 18, 2023
| ## Examples / Tips | ||
| 1. After calling `oci-env compose up` and waiting a moment for it to start up, you can access Grafana on | ||
| `http://localhost:3000`. If it is the first time you're accessing it, it will ask you to change the admin password. | ||
| After it, you can check on `Explore` section to find all the metrics collected by OpenTelemetry and sent to Prometheus. |
bmbouter
approved these changes
May 18, 2023
There was a problem hiding this comment.
I've also used this and it's worked well for me. I think if issues come up we can fix those too!
The CI failures are unrelated to this PR and intermittent. The same failures show on other PRs and the existing merged branches. To recap the convo at the working group here: Given they are unrelated to this PR, we'll rerun the CI until it passes and merge.
We need to tackle the intermittent CI failures as a separate effort.
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
4 times, most recently
from
adb4863
to
6a000ec
Compare
Signed-off-by: André "decko" de Brito <decko@redhat.com>
decko
force-pushed
the
add_opentelemetry_dev_profile
branch
from
6a000ec
to
fb75fa4
Compare
This was referenced May 19, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #45