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

Issue 5230 #470

Closed
wants to merge 15 commits into from
Closed

Issue 5230 #470

wants to merge 15 commits into from

Conversation

be-a-bee
Copy link
Contributor

@be-a-bee be-a-bee commented Oct 19, 2023

Add trace_id and span_id to decision logs

there were few conflicts in PR 462 ( #462 ). So creating this to avoid confusion.

be-a-bee and others added 14 commits October 20, 2023 19:27
added traceid and spanid to decision log.

test for checking traceid in decision log is not passing

Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 2 to 3.
- [Release notes](https://github.com/docker/setup-buildx-action/releases)
- [Commits](docker/setup-buildx-action@v2...v3)

---
updated-dependencies:
- dependency-name: docker/setup-buildx-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.58.0 to 1.58.1.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.0...v1.58.1)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.58.1 to 1.58.2.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.1...v1.58.2)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.13.0 to 0.14.0.
- [Release notes](https://github.com/golang/tools/releases)
- [Commits](golang/tools@v0.13.0...v0.14.0)

---
updated-dependencies:
- dependency-name: golang.org/x/tools
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.58.2 to 1.58.3.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.2...v1.58.3)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.5.0 to 0.17.0.
- [Commits](golang/net@v0.5.0...v0.17.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.16.0 to 0.17.0.
- [Commits](golang/net@v0.16.0...v0.17.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [github.com/open-policy-agent/opa](https://github.com/open-policy-agent/opa) from 0.57.0 to 0.57.1.
- [Release notes](https://github.com/open-policy-agent/opa/releases)
- [Changelog](https://github.com/open-policy-agent/opa/blob/v0.57.1/CHANGELOG.md)
- [Commits](open-policy-agent/opa@v0.57.0...v0.57.1)

---
updated-dependencies:
- dependency-name: github.com/open-policy-agent/opa
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.58.3 to 1.59.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.3...v1.59.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
The `hub` tool is deprecated in favor of the GitHub CLI and
is removed from GitHub's action runner images.

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
This version includes security fixes which addresses
vulnerability GO-2023-2102.

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Signed-off-by: talip <47842333+LionOnTheChase@users.noreply.github.com>
Comment on lines +52 to +53
"traceparent" : "00-0000000000000000f7bea0490c3cb268-b9543a567b0289f8-01",
"tracestate" : "rojo=00f067aa0ba902b7",
Copy link
Member

Choose a reason for hiding this comment

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

Where is this used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I intended to use it in a unit test but didn't need it. Can we keep this because it is innocuous ?

@ashutosh-narkar
Copy link
Member

@be-a-bee please squash your commits and we can then get this in. Thanks!

@be-a-bee
Copy link
Contributor Author

@ashutosh-narkar , I squashed the commits and had to create a new PR
#473 .

thank you for your patience.

@ashutosh-narkar
Copy link
Member

Closing in favor of #473.

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.

2 participants