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

Support annotations on Policy objects #471

Merged

Conversation

caseydavenport
Copy link
Member

@caseydavenport caseydavenport commented Jul 6, 2017

Description

Enhance Policy objects so that they support annotations. This will be useful for clients who want to track extra metadata about a given resource, or e.g. keep track of resources they specifically have created.

Once this is approved, I intend to do the same for Profiles.

Todos

Release Note

Policy objects now support arbitrary key/value annotations.

@caseydavenport caseydavenport self-assigned this Jul 6, 2017
@caseydavenport
Copy link
Member Author

Still want to do a manual test of this, but @robbrockbank might want to take a first look.

Copy link
Contributor

@robbrockbank robbrockbank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - should we be adding this to all of our resources?

@caseydavenport caseydavenport changed the title [WIP] Support annotations on Policy objects Support annotations on Policy objects Jul 10, 2017
@caseydavenport
Copy link
Member Author

@robbrockbank I have a particular use-case in mind for policy/profiles, but I would like to see it added to other things down the road.

I'd really like to see us update to use a common metadata struct, but that's a longer discussion :)

@caseydavenport caseydavenport merged commit 3f64efc into projectcalico:master Jul 10, 2017
@caseydavenport caseydavenport deleted the policy-annotations branch July 10, 2017 20:55
@caseydavenport caseydavenport added this to the Calico v2.4.0 milestone Jul 11, 2017
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.

2 participants