agent icon indicating copy to clipboard operation
agent copied to clipboard

chore(faro): prefix measurement values when parsing faro measurements

Open codecapitano opened this issue 1 year ago • 4 comments

PR Description

This PR aligns the hot to translate Faro measuerements with the latest Faro cloud receiver updates.

For reference have a look at the https://github.com/grafana/app-o11y-kwl-endpoint/pull/397.

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

  • [x] CHANGELOG.md updated
  • [ ] Documentation added
  • [ ] Tests updated
  • [ ] Config converters updated

codecapitano avatar Apr 02 '24 12:04 codecapitano

Hi @wildum 👋 I have no permission to merge the branch. How's the process here, just ping the agent team or will automatically be merged at some time?

codecapitano avatar Apr 10 '24 08:04 codecapitano

Hey sorry for dragging this up, we just made the Alloy repo public and we need to redirect the PRs there. We will only be merging bugfixes to the agent from now on. We are just figuring out a process to make the transition smooth. You don't need to do anything, the PR will be merged to Alloy soon

wildum avatar Apr 10 '24 09:04 wildum

Hey sorry for dragging this up, we just made the Alloy repo public and we need to redirect the PRs there. We will only be merging bugfixes to the agent from now on. We are just figuring out a process to make the transition smooth. You don't need to do anything, the PR will be merged to Alloy soon

Great, thank you so much 🙏

codecapitano avatar Apr 10 '24 09:04 codecapitano

This PR has not had any activity in the past 30 days, so the needs-attention label has been added to it. If you do not have enough time to follow up on this PR or you think it's no longer relevant, consider closing it. The needs-attention label signals to maintainers that something has fallen through the cracks. No action is needed by you; your PR will be kept open and you do not have to respond to this comment. The label will be removed the next time this job runs if there is new activity. Thank you for your contributions!

github-actions[bot] avatar May 16 '24 00:05 github-actions[bot]

Hi @wildum any updates on this? We have customers running into this issue.

codecapitano avatar Jun 06 '24 06:06 codecapitano

Hey, super sorry we actually moved all the issues but not the PRs and I forgot about it. Actually this seems to be relevant for the agent as it looks more like a bugfix than an enhancement. I can merge this PR and create a new one in Alloy and get it merged. Is that ok?

wildum avatar Jun 06 '24 08:06 wildum

here is the alloy PR: https://github.com/grafana/alloy/pull/991. I extended a unit test to check that the float values had the prefix

wildum avatar Jun 06 '24 08:06 wildum

merged in Alloy, I will also merge it in a bit to the agent

wildum avatar Jun 06 '24 09:06 wildum

merged in Alloy, I will also merge it in a bit to the agent

Great thank you so much @wildum 🙏

codecapitano avatar Jun 06 '24 09:06 codecapitano

All done, sorry again for the unnecessary delay!

wildum avatar Jun 06 '24 09:06 wildum

Thank you so much and no worries. I think your team had a ton of work around Alloy release time. Ans I could've asked earlier as well.

codecapitano avatar Jun 06 '24 11:06 codecapitano