Opened on 03/12/2014 at 04:13:10 PM

Closed on 03/17/2015 at 03:41:02 PM

Last modified on 03/18/2015 at 08:09:16 PM

#112 closed change (fixed)

Use an external node classifier to map nodes to classes

Reported by: fhd Assignee: matze
Priority: P4 Milestone:
Module: Infrastructure Keywords:
Cc: trev Blocked By: #2168
Blocking: #123, #1568 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

Description (last modified by fhd)


Right now, nodes are defined in the manifests imported by site.pp. We should use an external node classifier to not expose our exact infrastructure, and provide a stub ENC that configures nodes for all classes.

What to change

Use an external node classifier to map nodes to classes. The biggest deal here is probably to make the monitoring configuration generic.

Attachments (0)

Change History (20)

comment:1 Changed on 03/12/2014 at 10:42:58 PM by fhd

  • Description modified (diff)
  • Priority changed from Unknown to P4
  • Reporter changed from philll to fhd
  • Status changed from new to accepted

comment:2 Changed on 03/21/2014 at 09:11:02 AM by philll

  • Status changed from accepted to new

comment:3 Changed on 04/19/2014 at 07:18:01 AM by poz2k4444

I've been reading about this one, and maybe it could be a better idea to use Hiera instead of ENC, you could check the pros and cons here:

comment:4 Changed on 04/29/2014 at 03:36:48 PM by trev

  • Cc trev added
  • Ready set

comment:5 Changed on 04/29/2014 at 09:03:26 PM by trev

I think the main issue with Hiera was: ruby-hiera-puppet package isn't available for Ubuntu 12.04. fhd, do you remember more than me here?

comment:6 Changed on 04/30/2014 at 02:40:58 AM by fhd

No, I never looked into Hiera personally. Looking through this a bit, I think I'm still leaning towards an ENC. Simpler, highly flexible and it's built-in (Hiera not yet). The cons listed in that comparison seem a bit dubious to me.

What would the actual benefits of using Hiera be for us?

comment:7 Changed on 04/30/2014 at 02:14:04 PM by trev

The main benefit would be having actual data files that we can access from the Puppet manifests, without the additional complexity of a script. However, I tried to install Hiera on Ubuntu 12.04 and this seems complicated enough that it isn't worth doing. After all, we can just use the same data files but have our ENC script read them.

For reference: Hiera is built into Puppet 3 but Ubuntu 12.04 uses Puppet 2.7. There is no install package so you need to install the Ruby gem. And that gem is installed to a location where Puppet doesn't pick it up. In theory, symlinking from /etc/puppet/modules to the gem install directory should do - but that didn't work for me, I still couldn't use the hiera() function. There is no obvious way to debug this issue and no helpful documentation.

comment:8 Changed on 05/02/2014 at 06:27:44 AM by poz2k4444

  • Owner set to poz2k4444

comment:9 Changed on 08/26/2014 at 11:00:40 AM by matze

  • Platform set to Unknown

Can you please create a pull request for that?

I know @trev suggested in #IRC that we do not need one in general, but it's way more comfortable for reviewing the current state, especially since recent history and any intended changes are easily accessible (AND linked in the "Review URLs" section above), and because it allows to easily place and review comments as well.
Furthermore, since GitHub will auto-update any pull request when new commits are introduced in either source or target branch, this is probably the easiest way to go.

comment:10 Changed on 08/26/2014 at 11:37:15 AM by trev

FYI: The current state is visible under The problem is, at the moment there are many unnecessary changes that will make the pull request unnecessary hard to review. In particular, the subrepositories have been checked in. That's why I suggested cleaning up first and only create a pull request later. But it probably makes sense to do this and to clean up whatever prevents useful reviews there.

comment:11 Changed on 09/19/2014 at 06:18:19 PM by matze

  • Review URL(s) modified (diff)

comment:12 Changed on 09/22/2014 at 02:20:34 PM by matze

  • Review URL(s) modified (diff)

comment:13 Changed on 09/22/2014 at 02:22:42 PM by matze

A general status update for reference and documentation: We've decided to follow the Hiera approach. @aalvz, @poz2k4444 and @matze are currently working on addressing the last issues from the code-review (see the linked review-URLs).

comment:14 Changed on 10/06/2014 at 11:25:49 AM by matze

  • Blocking 123 added

comment:15 Changed on 10/07/2014 at 10:15:24 PM by matze

  • Review URL(s) modified (diff)

comment:16 Changed on 11/13/2014 at 05:54:52 PM by matze

  • Blocking 1568 added

comment:17 Changed on 02/23/2015 at 11:11:28 AM by poz2k4444

  • Owner changed from poz2k4444 to matze

comment:18 Changed on 03/17/2015 at 08:37:08 AM by matze

  • Status changed from new to reviewing

comment:19 Changed on 03/17/2015 at 03:41:02 PM by matze

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:20 Changed on 03/18/2015 at 08:09:16 PM by trev

  • Blocked By 2168 added

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from matze.
Note: See TracTickets for help on using tickets.