Skip to content

Some improvements and new/missing features#4

Open
tbe wants to merge 5 commits intonhuff:masterfrom
tbe:improvements
Open

Some improvements and new/missing features#4
tbe wants to merge 5 commits intonhuff:masterfrom
tbe:improvements

Conversation

@tbe
Copy link
Copy Markdown

@tbe tbe commented Apr 18, 2019

Improvements:

  • moved to Go Modules
  • replaces the custom chef API code with go-chef/chef
  • moved testing only code to the unit test

Features:

  • implemented role policies
  • implemented tag policies
  • implemented environment policies
  • added support for metadata mapping

@tbe tbe force-pushed the improvements branch from 19a76d1 to 96b8dc3 Compare May 3, 2019 10:24
@tbe tbe force-pushed the improvements branch from 96b8dc3 to 481504f Compare May 3, 2019 12:45
@nhuff
Copy link
Copy Markdown
Owner

nhuff commented Jun 17, 2019

  • implemented role policies
  • implemented tag policies
  • implemented environment policies

I had implemented these originally, but after discussion with some Chef folks it turns out you can't really use these securely because the chef client can change them arbitrarily.

I will take a look at the module conversion and metadata mapping.

@tbe
Copy link
Copy Markdown
Author

tbe commented Jun 17, 2019

I had implemented these originally, but after discussion with some Chef folks it turns out you can't really use these securely because the chef client can change them arbitrarily.

In theory, this is correct. I'm using a lightwight proxy, written in go, in front of the chef server API to disallow changes to the run_list at the moment.

I think there should be a big, fat, warning in the docs about this.

If i find time, i will make the proxy releasable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants