Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

fanny-jiang
Copy link
Contributor

What does this PR do?

Fix a syntax error in the datadog reports.

Motivation

Bugfix

Additional Notes

Describe your test plan

- 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
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
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
12 checks passed
@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