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

Issues/1345 - EB/EFS on/offhours tagging workaround #1421

Merged
merged 6 commits into from
Jul 22, 2017

Conversation

jantman
Copy link
Contributor

@jantman jantman commented Jul 17, 2017

This is a workaround for #1345, where we can't use on/offhours with resources in Elastic Beanstalk or in CF stacks that include EFS Volumes, as those services have tag value constraints that don't allow empty tag values, and don't allow many of the characters used in the tag-based scheduling.

  • The on/offhours filter actually accepts any string that doesn't match as a schedule, as the default (i.e. on or foobar has the same operation as an empty tag value).
  • Add tests to ensure that "on" operates identically to "" as a tag value.
  • Document "on" as an alternative to the empty tag value.
  • Add documentation of multi-policy workaround for EB apps and EFS

stalk/latest/dg/using-features.tagging.html>`_ and `EFS <http://docs.aws.amazon.
com/efs/latest/ug/API_Tag.html>`_. In particular, these services do not allow
parenthesis, square brackets, commas, or semicolons, or empty tag values. This
proves to be problematic with the tag-based schedule configuration describe
Copy link
Collaborator

Choose a reason for hiding this comment

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

s/describe/described

Copy link
Collaborator

@kapilt kapilt left a comment

Choose a reason for hiding this comment

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

thanks, lgtm

@jantman
Copy link
Contributor Author

jantman commented Jul 20, 2017

Typo fixed. Sorry about that, but good catch. Thanks so much!

@kapilt kapilt merged commit 8423d9a into cloud-custodian:master Jul 22, 2017
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