Skip to content

Fix Datadog Reports#848

Merged
fanny-jiang merged 5 commits intomainfrom
fanny/reports
Mar 24, 2025
Merged

Fix Datadog Reports#848
fanny-jiang merged 5 commits intomainfrom
fanny/reports

Conversation

@fanny-jiang
Copy link
Copy Markdown
Contributor

What does this PR do?

Fix a syntax error in the datadog reports.

Motivation

Bugfix

Additional Notes

Describe your test plan

Comment thread kitchen.yml
- rpm -Uvh https://yum.puppet.com/puppet8-release-el-9.noarch.rpm #installs the puppet-agent repo
- yum install -y puppet-agent-8.10.0 rubygems ruby-devel procps-ng
- dnf group install -y "Development Tools"
- dnf group install -y "Development Tools" --nobest
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

--nobest is to allow installing a lower version that meets package version requirements

@fanny-jiang fanny-jiang marked this pull request as ready for review March 13, 2025 21:01
@fanny-jiang fanny-jiang requested review from a team as code owners March 13, 2025 21:01
@dog.batch_metrics do
metrics.each do |metric, data|
data.each_value do |val|
data.values.each do |val| # rubocop:disable Style/HashEachMethods
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

error that requires this:
2025-03-14T15:12:49.135Z ERROR [qtp2067368747-71] [puppetserver] Puppet Report processor failed: undefined method each_value' for #<Puppet::Util::Metric:0x7317bca6>

@fanny-jiang fanny-jiang merged commit 56a50aa into main Mar 24, 2025
@fanny-jiang fanny-jiang deleted the fanny/reports branch March 24, 2025 10:30
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