Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Make role attributes work, and add some to play around with in CB 2.0 [5/8] #70

Conversation

VictorLowther
Copy link
Contributor

  • Make Attrib.get smart. Instead of passing a blob of JSON, it now
    accepts the object it should try to work with. This centralizes the
    logic around what fields in an object are valid places to get
    attributes from, and will fail hard if you pass it something that
    should not work with the attribute system.

  • Add matching Attrib.set methods. This provides an abstract interface
    for setting attributes on objects that matches the use of
    Attrib.get.

  • Add some attributes to various nodes to lay the groundwork for
    adding logic to the annealer to make it consiter attribute visibility.

  • Numerous trivial cleanups to make things work with the above changes.

    crowbar.yml | 6 ++++++
    1 file changed, 6 insertions(+)

Crowbar-Pull-ID: cd953079fcc42015c6552d4b6d190cc54e898308

Crowbar-Release: development

@VictorLowther VictorLowther restored the pull-req-master-cd953079fcc42015c6552d4b6d190cc54e898308 branch December 5, 2013 20:08
@zehicle
Copy link
Member

zehicle commented Dec 6, 2013

+2

ravolt added a commit that referenced this pull request Dec 6, 2013
…42015c6552d4b6d190cc54e898308

#70

Merged by devtool for ravolt
@ravolt ravolt merged commit 5bdae3a into crowbar:master Dec 6, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
3 participants