Skip to content

Latest commit

 

History

History
112 lines (69 loc) · 4.15 KB

CONTRIBUTING.md

File metadata and controls

112 lines (69 loc) · 4.15 KB
Raw

Hacking on source-to-image

Local development

S2I comes with a Makefile which defines following targets:

  • build - is the default target responsible for building S2I binary, under the covers it calls hack/build-go.sh. The resulting binary will be placed in _output/local/go/bin/.
  • all - is synonym for build.
  • test - is responsible for testing S2I, under the covers it calls hack/test-go.sh. Additionally you can pass WHAT or TEST variable specifying directory names to test, eg. make test WHAT=pkg/build
  • check - is synonym for test.
  • clean - cleans environment by removing _output

Generating Bash completion

If you are modifying or adding sub-command or command flag, make sure you update the generated Bash completion and include it in your PR. To update Bash completion, you can run the following command:

$ hack/update-generated-completions.sh

This will regenerate the ./contrib/bash/s2i file.

Test Suites

S2I uses two levels of testing - unit tests and integration tests, both of them are run on each pull request, so make sure to run those before submitting one.

Unit tests

Unit tests follow standard Go conventions and are intended to test the behavior and output of a single package in isolation. All code is expected to be easily testable with mock interfaces and stubs, and when they are not it usually means that there's a missing interface or abstraction in the code. A unit test should focus on testing that branches and error conditions are properly returned and that the interface and code flows work as described. Unit tests can depend on other packages but should not depend on other components.

The unit tests for an entire package should not take more than 0.5s to run, and if they do, are probably not really unit tests or need to be rewritten to avoid sleeps or pauses. Coverage on a unit test should be above 70% unless the units are a special case.

Run the unit tests with:

$ hack/test-go.sh

or an individual package unit test with:

$ hack/test-go.sh pkg/build/strategies/sti

To run only a certain regex of tests in a package, use:

$ hack/test-go.sh pkg/build/strategies/sti -test.run=TestLayeredBuild

To get verbose output add -v to the end:

$ hack/test-go.sh pkg/build/strategies/sti -test.run=TestLayeredBuild -v

To run all tests with verbose output:

$ hack/test-go.sh "" -v

To run tests without the Go race detector, which is on by default, use:

$ S2I_RACE="" hack/test-go.sh

A line coverage report is printed by default. To turn it off, use:

$ S2I_COVER="" hack/test-go.sh

To create an HTML coverage report for all packages:

$ OUTPUT_COVERAGE=/tmp/s2i-cover hack/test-go.sh

Integration tests

The second category are integration tests which verify the whole S2I flow. The integration tests require a couple of images for testing, these can be built with hack/build-test-images.sh, if integration tests don't find them it'll print appropriate information regarding running this command.

Run the integration tests with:

$ hack/test-integration.sh

Installing Glide

S2I uses Glide for dependency management. Glide allows versions of dependent packages to be locked at a specific commit by vendoring them (checking a copy of them into vendor). This means that everything you need for S2I is checked into this repository. To install glide locally run:

$ go get github.com/Masterminds/glide

If you are not updating packages you should not need glide installed.

Building a Release

To build a S2I release you run make release on a system with Docker, which will create a build environment image and then execute a cross platform Go build within it. The build output will be copied to _output/releases as a set of tars containing each version.

  1. Create a new git tag git tag vX.X.X -a -m "vX.X.X" HEAD
  2. Push the tag to GitHub git push origin --tags where origin is github.com/openshift/source-to-image.git
  3. Run make release
  4. Upload the binary artifacts generated by that build to GitHub release page.
  5. Send an email to the dev list, including the important changes prior to the release.