Feature #10064

Reports should return the environment of the client in the report

Added by James Turnbull 7 months ago. Updated 3 months ago.

Status:Closed Start date:10/13/2011
Priority:Normal Due date:
Assignee:Daniel Pittman % Done:

0%
Category:reports
Target version:2.7.12
Affected Puppet version:2.6.6 Branch:https://github.com/puppetlabs/puppet/pull/169
Keywords:
Votes: 3

Description

The report format should add an attribute to Puppet::Transaction::Report for environment.

History

#1
Updated by Nigel Kersten 7 months ago

  • Status changed from Needs Decision to Accepted
  • Assignee deleted (Nigel Kersten)
  • Target version set to 3.X

I actually thought we already did, but it’s because I used a fact to set the environment.

#2
Updated by James Turnbull 7 months ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Target version deleted (3.X)
  • Branch set to https://github.com/puppetlabs/puppet/pull/169

Added code for this and added a pull request.

#3
Updated by James Turnbull 7 months ago

  • Target version set to 3.X

#4
Updated by Patrick Carlisle 6 months ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

Merged in commit:3406081b3d91bb796f8fa666d22a05054fda42f4

(#10064) Add the Puppet environment to reports

Previously the Puppet environment was not included in
the report output. Using the model of how configuration_version
was added to the reports the environment will now also be reported.

Puppet::Transaction::Report now takes three arguments:

* Kind of report
* Configuration version
* Environment

Tests for this have been added and existing tests updated.

#5
Updated by James Turnbull 4 months ago

  • Assignee set to Daniel Pittman

Daniel – the customer who wanted this needs it in 2.7.x also. Can that be done please.

#6
Updated by Nick Lewis 4 months ago

  • Status changed from Merged - Pending Release to Code Insufficient

Pull request 419 addresses this against 2.7.x.

#7
Updated by Daniel Pittman 4 months ago

  • Status changed from Code Insufficient to Merged - Pending Release
  • Target version changed from 3.X to 2.7.11

#8
Updated by Matthaus Litteken 3 months ago

  • Status changed from Merged - Pending Release to Closed
  • Target version changed from 2.7.11 to 2.7.12

Released in 2.7.12rc1

Also available in: Atom PDF