Infrastructure for creating a binary authentication attestor

[mike-ensor]

This is a module that can be used to create an Attestor for the Binary Authorization workflow. Outputs include the full path/name to the attestor and the full path/name to the associated key

[bharathkkb]

Thanks for the PR!

• Looks like we are missing a Readme similar to this

• Additionally it would be great, if you can add an example with this module

[bharathkkb]

Looks like the CI failed to run on latest commit due to a GitHub outage.

Left a few comments, particularly some nits the linter may have caught if it ran. You can also run it locally using the command make docker_test_lint. More details here.

Additionally most of our modules have an example, which we use to showcase how the GKE module works alongside other add-on modules. As this can exist without a referencing a GKE cluster, I will defer to @morgante on how to proceed.

[bharathkkb]

LGTM overall.
Tests failed due to missing variable project_id.

[bharathkkb]

@mike-ensor looks like just these two needs to be fixed for linting to pass #530 (comment) and #530 (comment). You can also run it locally using the command make docker_test_lint. More details here.

[mike-ensor]

@mike-ensor looks like just these two needs to be fixed for linting to pass #530 (comment) ...

I've updated the outputs file, run both the "generate docs" and "lint" tools without errors.

[mike-ensor]

@bharathkkb I'm not sure why the build is failing in GitHub, but it might be related to the failure of the ./test/setup folder not validating fully.

[bharathkkb]

@mike-ensor looks like make docker_generate_docs was not re run after you added project_id variable. It's complaining the documentation is out of date. If you rerun that and commit changes, it should be good.

[mike-ensor]

@bharathkkb yes, I didn't run the create docs after :-(

the build is running now