quattor / CCM

Configuration Cache Manager
www.quattor.org
Other
4 stars 13 forks source link

Configuration: add name #137

Closed stdweird closed 8 years ago

stdweird commented 8 years ago

Completes work in #122 by adding more configuration/profile information Related to https://github.com/quattor/template-library-core/pull/127 Fixes #138 as a matter of course.

stdweird commented 8 years ago

@ned21 following messages are generated with this PR

[root@snorlax ~]# ccm /metadata
+-/metadata
  +-template
    +-branch
      $ author : 'stdweird'
      $ id : 'ab974fbf'
      $ name : 'stdweird-cobalion_fixes'
      $ timestamp : 1476081455
      $ type : 'test'

[root@snorlax ~]# ccm-fetch
[INFO] Fetched profile was not new, latest CID 71 name stdweird-cobalion_fixes-test-stdweird-ab974fbf-1476081455
[root@snorlax ~]# ccm-fetch --force
[INFO] Profile updated, new latest CID 72 name stdweird-cobalion_fixes-test-stdweird-ab974fbf-1476081455

as discussed in https://github.com/quattor/template-library-core/pull/127, the id and timestamp are optional

jrha commented 8 years ago

@ned21 @stdweird can we reach an agreement on the discussion around this?

ned21 commented 8 years ago

I was waiting to see if anyone else had any strong opinions. :) I was going to grab the rpm and try it out on our systems just to see what it looked like but I probably won't get much time to do that next week. Since the message is configurable, I think we could merge this and then if the string needs tweaking to work better with Aquilon then we can do that later.

jrha commented 8 years ago

Sounds good to me.

jrha commented 7 years ago

Just wanted to note that this is working nicely in 16.10.0-rc2, thanks!

2016-11-18T13:33:05.609597+00:00 vm197 ccm-fetch[2620]: Profile updated, new latest CID 8 name test_16.10.0-rc2-sandbox-tkk76468

It's quite nice to be able to just grep CID /var/log/messages too!