Skip to content

[BEAM-1112] Improve Python E2E Test Framework#1711

Closed
markflyhigh wants to merge 2 commits intoapache:python-sdkfrom
markflyhigh:skip-it
Closed

[BEAM-1112] Improve Python E2E Test Framework#1711
markflyhigh wants to merge 2 commits intoapache:python-sdkfrom
markflyhigh:skip-it

Conversation

@markflyhigh
Copy link
Copy Markdown
Contributor

@markflyhigh markflyhigh commented Dec 28, 2016

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify. (Even better, enable
    Travis-CI on your fork and ensure the whole test matrix passes).
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

IT tests should not failed python setup.py test.

  • Skip IT(integration test) if --test-pipeline-options is not specified in command line arguments.

@markflyhigh
Copy link
Copy Markdown
Contributor Author

+R: @aaltay

@markflyhigh
Copy link
Copy Markdown
Contributor Author

markflyhigh commented Dec 28, 2016

@aaltay

@asfbot
Copy link
Copy Markdown

asfbot commented Dec 28, 2016

@aaltay
Copy link
Copy Markdown
Member

aaltay commented Dec 28, 2016

LGTM (please wait for tests to finish).

@asfbot
Copy link
Copy Markdown

asfbot commented Dec 28, 2016

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/6308/
--none--

@markflyhigh
Copy link
Copy Markdown
Contributor Author

@dhalperi please merge if looks good to you.

@dhalperi
Copy link
Copy Markdown
Contributor

LGTM, merged, thanks.

asfgit pushed a commit that referenced this pull request Dec 28, 2016
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