Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

puppetboard/app.py: Simplifying the Inventory Code #289

Merged
merged 2 commits into from Sep 3, 2016
Merged

puppetboard/app.py: Simplifying the Inventory Code #289

merged 2 commits into from Sep 3, 2016

Conversation

corey-hammerton
Copy link
Contributor

This resolves #275

This update eliminates one iteration over the resulting inventory facts
that generates a multidimensional dictionary keyed by the node's certname
to another dictionary of key-value pairs of the fact name and fact value.

… Inventory

This resolves #275

This update eliminates one iteration over the resulting inventory facts
that generates a multidimensional dictionary keyed by the node's certname
to another dictionary of key-value pairs of the fact name and fact value.
@corey-hammerton corey-hammerton added this to the 0.2.1 milestone Aug 10, 2016
@coveralls
Copy link

coveralls commented Aug 10, 2016

Coverage Status

Coverage increased (+0.3%) to 23.328% when pulling 48cfbcc on corey-hammerton:issue-275 into feac444 on voxpupuli:master.

@tux-o-matic
Copy link
Contributor

@corey-hammerton The template for the inventory page still doesn't link to the node page as presented #280. Could you consider this improvemt fro your PR?

…ks to the Node page

This comes as a request from #280
@coveralls
Copy link

coveralls commented Aug 19, 2016

Coverage Status

Coverage decreased (-1.04%) to 21.971% when pulling fb74853 on corey-hammerton:issue-275 into feac444 on voxpupuli:master.

@corey-hammerton
Copy link
Contributor Author

@tux-o-matic I've updated this PR, does this fit with your request?

@tux-o-matic
Copy link
Contributor

@corey-hammerton thanks, that should do it.

@corey-hammerton corey-hammerton merged commit 5048662 into voxpupuli:master Sep 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Simplify Inventory Code
3 participants