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

Functional tests passing in new studios #243

Merged
merged 1 commit into from
Feb 25, 2016
Merged

Conversation

adamhjk
Copy link
Contributor

@adamhjk adamhjk commented Feb 25, 2016

gif-keyboard-4830865314462957489

  • Adds chef/rngd and chef/gcc-libs to the bldr plan
  • Adds iptables plan
  • Fix rngd plan to build correctly
  • Run tests in the studio
  • Fix the internal functional test fixture plans
  • Fix error reporting for gpg
  • Add a CommandArgs struct for internal test commands
  • Use the chef/iptables for partitioning
  • Ignore tests that are flapping

@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

* Adds chef/rngd and chef/gcc-libs to the bldr plan
* Adds iptables plan
* Fix rngd plan to build correctly
* Run tests in the studio
* Fix the internal functional test fixture plans
* Fix error reporting for gpg
* Add a CommandArgs struct for internal test commands
* Use the chef/iptables for partitioning
* Ignore tests that are flapping
pkg_service_run="sbin/rngd -f -r /dev/urandom"
pkg_service_user=root

do_build() {
./autogen.sh && ./configure --prefix=$pkg_prefix && make
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this because we're grabbing it from sourceforge now?

@reset
Copy link
Collaborator

reset commented Feb 25, 2016

gif-keyboard-9087419986622362833

Gotta rebase again but this looks good!

@adamhjk adamhjk force-pushed the fix-functional-tests branch from 3836474 to fa5c7e2 Compare February 25, 2016 20:59
@adamhjk
Copy link
Contributor Author

adamhjk commented Feb 25, 2016

Rebase is done.

@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@@ -45,7 +45,11 @@ clean:

image:
if [ -n "${force}" -o -z "`$(docker_cmd) images -q $(dimage)`" ]; then \
$(docker_cmd) build $(build_args) -t $(dimage) .; \
if [ -n "${force}" ]; then \
$(docker_cmd) build --no-cache $(build_args) -t $(dimage) .; \
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ugh, were you getting old cached image layers?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sometimes I just want an image that has a newer bldr, bpm, or studio. No way to do that if the line remains the same, other than forcing a rebuild from scratch.

@fnichol
Copy link
Collaborator

fnichol commented Feb 25, 2016

@delivery approve

chef-delivery added a commit that referenced this pull request Feb 25, 2016
Merged change 42de7c8b-c159-40cd-af8c-99efb5eb8b37

From review branch fix-functional-tests into master

Signed-off-by: fnichol <[email protected]>
@chef-delivery chef-delivery merged commit 0def8fe into master Feb 25, 2016
@chef-delivery chef-delivery deleted the fix-functional-tests branch February 25, 2016 22:11
@fnichol
Copy link
Collaborator

fnichol commented Feb 25, 2016

gif-keyboard-6476607918485203747

@chef-delivery
Copy link
Contributor

This PR has passed 'Acceptance' and is ready to be delivered!
Use: '@delivery deliver' when validated in acceptance.

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.

4 participants