Skip to content

Conversation

@klakin-pivotal
Copy link
Contributor

Description

This commit ensures that users will not encounter the following error while deploying system-metrics on Windows stemcells:

Task 14 | 03:07:40 | L starting jobs: no_op/fb4da847-c87e-4532-9cce-9ae174d9520d (0) (canary) (00:00:30)
                   L Error: Action Failed start: Configuring jobs: Configuring job loggr-system-metrics-agent-windows: Adding monit configuration: json: cannot unmarshal bool into Go value of type string
Task 14 | 03:07:41 | Error: Action Failed start: Configuring jobs: Configuring job loggr-system-metrics-agent-windows: Adding monit configuration: json: cannot unmarshal bool into Go value of type string

This was brought to our attention via a trouble report in an internal Slack channel.

Signed-off-by: Ben Fuller [email protected]

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Testing performed?

  • Unit tests
  • Integration tests
  • Acceptance tests

Checklist:

  • This PR is being made against the main branch, or relevant version branch
  • I have made corresponding changes to the documentation
  • I have added testing for my changes

This commit ensures that users will not encounter the following error
while deploying `system-metrics` on Windows stemcells:

```
Task 14 | 03:07:40 | L starting jobs: no_op/fb4da847-c87e-4532-9cce-9ae174d9520d (0) (canary) (00:00:30)
                   L Error: Action Failed start: Configuring jobs: Configuring job loggr-system-metrics-agent-windows: Adding monit configuration: json: cannot unmarshal bool into Go value of type string
Task 14 | 03:07:41 | Error: Action Failed start: Configuring jobs: Configuring job loggr-system-metrics-agent-windows: Adding monit configuration: json: cannot unmarshal bool into Go value of type string
```

This was brought to our attention via a trouble report in an internal
Slack channel.

Signed-off-by: Ben Fuller <[email protected]>
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

Copy link
Member

@Benjamintf1 Benjamintf1 left a comment

Choose a reason for hiding this comment

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

lgtm

@Benjamintf1 Benjamintf1 merged commit 7425361 into main May 24, 2022
@Benjamintf1 Benjamintf1 deleted the fix-limited-metrics-boolean-interpretation branch May 24, 2022 16:55
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.

3 participants