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

Update sysctl.rb #81

Merged
merged 1 commit into from Jun 25, 2015

Conversation

Projects
None yet
4 participants
@Rockstar04
Member

Rockstar04 commented Jun 25, 2015

Lowercase file. Relates to #80

Update sysctl.rb
Lowercase file
@Rockstar04

This comment has been minimized.

Show comment
Hide comment
@Rockstar04

Rockstar04 Jun 25, 2015

Member

I'll to wait until I get to work tomorrow to run the rest of the tests for this and see if it is as simple as that. (8AM CST or 1PM UTC)

Member

Rockstar04 commented Jun 25, 2015

I'll to wait until I get to work tomorrow to run the rest of the tests for this and see if it is as simple as that. (8AM CST or 1PM UTC)

@Rockstar04

This comment has been minimized.

Show comment
Hide comment
@Rockstar04

Rockstar04 Jun 25, 2015

Member

Yes, I can confirm that this will solve the issue in #80 (for at least Ubuntu 14.04, I am letting all of the kitchen tests runs now).

Member

Rockstar04 commented Jun 25, 2015

Yes, I can confirm that this will solve the issue in #80 (for at least Ubuntu 14.04, I am letting all of the kitchen tests runs now).

chris-rock added a commit that referenced this pull request Jun 25, 2015

@chris-rock chris-rock merged commit a045ac6 into dev-sec:master Jun 25, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details
@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock
Member

chris-rock commented Jun 25, 2015

@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock Jun 25, 2015

Member

I'll do some extra tests over the weekend and bump the version afterwards. In the meantime, please go ahead and use the master

Member

chris-rock commented Jun 25, 2015

I'll do some extra tests over the weekend and bump the version afterwards. In the meantime, please go ahead and use the master

@lhriley-tm

This comment has been minimized.

Show comment
Hide comment
@lhriley-tm

lhriley-tm Jun 25, 2015

I just completed a kitchen build successfully against your master branch. I will consider this issue resolved once I see the version bump.

Thanks for the quick fix!

lhriley-tm commented Jun 25, 2015

I just completed a kitchen build successfully against your master branch. I will consider this issue resolved once I see the version bump.

Thanks for the quick fix!

@dupuy

This comment has been minimized.

Show comment
Hide comment
@dupuy

dupuy Jun 29, 2015

Contributor

@chris-rock version bump on this?

Contributor

dupuy commented Jun 29, 2015

@chris-rock version bump on this?

@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock Jun 29, 2015

Member

@dupuy latest master is pushed to supermarket as version 1.3.0

Member

chris-rock commented Jun 29, 2015

@dupuy latest master is pushed to supermarket as version 1.3.0

rollbrettler pushed a commit to rollbrettler/chef-os-hardening that referenced this pull request Sep 16, 2016

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