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

Make fedora tests pass #179

Merged
merged 1 commit into from Nov 24, 2017

Conversation

Projects
None yet
4 participants
@shoekstra
Contributor

shoekstra commented Nov 9, 2017

Fedora tests should be fixed once dev-sec/linux-baseline#84 is merged.

CC @artem-sidorenko

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 9, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 6301fa3 on shoekstra:fix_fedora_tests into 0476764 on dev-sec:master.

coveralls commented Nov 9, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 6301fa3 on shoekstra:fix_fedora_tests into 0476764 on dev-sec:master.

@shoekstra shoekstra referenced this pull request Nov 16, 2017

Merged

Update Fedora controls #84

@shoekstra

This comment has been minimized.

Show comment
Hide comment
@shoekstra

shoekstra Nov 20, 2017

Contributor

@artem-sidorenko @chris-rock could you restart the tests for this PR please?

Contributor

shoekstra commented Nov 20, 2017

@artem-sidorenko @chris-rock could you restart the tests for this PR please?

@artem-sidorenko

This comment has been minimized.

Show comment
Hide comment
@artem-sidorenko

artem-sidorenko Nov 20, 2017

Member

I repushed this PR to my fork, CI tests are running here: https://travis-ci.org/artem-forks/chef-os-hardening/builds/304789718

Member

artem-sidorenko commented Nov 20, 2017

I repushed this PR to my fork, CI tests are running here: https://travis-ci.org/artem-forks/chef-os-hardening/builds/304789718

@artem-sidorenko

This comment has been minimized.

Show comment
Hide comment
@artem-sidorenko

artem-sidorenko Nov 23, 2017

Member

@shoekstra audit failures are fixed in master. Can I ask you to rebase this PR on latest master please?

Member

artem-sidorenko commented Nov 23, 2017

@shoekstra audit failures are fixed in master. Can I ask you to rebase this PR on latest master please?

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 24, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 30b3df9 on shoekstra:fix_fedora_tests into bb6a35f on dev-sec:master.

coveralls commented Nov 24, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 30b3df9 on shoekstra:fix_fedora_tests into bb6a35f on dev-sec:master.

@shoekstra

This comment has been minimized.

Show comment
Hide comment
@shoekstra

shoekstra Nov 24, 2017

Contributor

@artem-sidorenko done 👍

Contributor

shoekstra commented Nov 24, 2017

@artem-sidorenko done 👍

@chris-rock

Thank you @shoekstra

@artem-sidorenko

This comment has been minimized.

Show comment
Hide comment
Member

artem-sidorenko commented Nov 24, 2017

@artem-sidorenko

@shoekstra big thanks for making the fedora tests finally green!:-)

@artem-sidorenko artem-sidorenko merged commit 4062a22 into dev-sec:master Nov 24, 2017

2 checks passed

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

@shoekstra shoekstra deleted the shoekstra:fix_fedora_tests branch Nov 24, 2017

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