Skip to content
This repository has been archived by the owner on Mar 6, 2023. It is now read-only.

Some "false" were incorrectly substituted to "no" #225

Merged
merged 1 commit into from
Aug 21, 2019

Conversation

wzyboy
Copy link
Contributor

@wzyboy wzyboy commented Aug 21, 2019

This PR fixes them.

@paulfantom paulfantom merged commit ba84bdc into cloudalchemy:master Aug 21, 2019
@paulfantom
Copy link
Member

thanks! 👍

@dmke
Copy link

dmke commented Aug 21, 2019

Uhm... @paulfantom could you have a look at the diff again?

+  # run_once: true # <-- this canfalset be set due to multi-arch support
-  # run_once: true # <-- this cannot be set due to multi-arch support
+ # - name: change pam falsefile limits for prometheus
- # - name: change pam nofile limits for prometheus
+ #     limit_item: falsefile
- #     limit_item: nofile

@paulfantom
Copy link
Member

I think you did diff in the opposite way :) This PR is properly fixing incorrect substitution.

@dmke
Copy link

dmke commented Aug 21, 2019

Indeed. I need (more) coffee. 😀

Sorry for the noise.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants