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

Add bud regression tests #2422

Merged
merged 1 commit into from
Jul 1, 2020

Conversation

TomSweeneyRedHat
Copy link
Member

We recently made a couple of commits to the imagebuilder
project to address #2345 and #2192. Tests were created
in that project, but I thought it best to add some
regression tests here in Buildah too.

FYI: @petr-motejlek and @tachoknight

Signed-off-by: TomSweeneyRedHat [email protected]

What type of PR is this?

/kind api-change
/kind bug
/kind cleanup
/kind deprecation
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake
/kind other

What this PR does / why we need it:

How to verify it

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?


@rhatdan
Copy link
Member

rhatdan commented Jun 24, 2020

Tests are not happy.

@TomSweeneyRedHat
Copy link
Member Author

This is failing even before it gets into messing with any Buildah code, up in the spot where the registry is started. I'm going to restart and hope.

@TomSweeneyRedHat
Copy link
Member Author

bors retry

@cevich
Copy link
Member

cevich commented Jun 26, 2020

Opened #2427

@rhatdan
Copy link
Member

rhatdan commented Jun 29, 2020

LGTM

We recently made a couple of commits to the imagebuilder
project to address containers#2345 and containers#2192.  Tests were created
in that project, but I thought it best to add some
regression tests here in Buildah too.

FYI: @petr-motejlek and @tachoknight

Signed-off-by: TomSweeneyRedHat <[email protected]>
@TomSweeneyRedHat TomSweeneyRedHat force-pushed the dev/tsweeney/budtestmr branch from feb1833 to d29655b Compare June 30, 2020 23:34
@TomSweeneyRedHat
Copy link
Member Author

Rebased on top of #2427 hopefully this will pass now.

@rhatdan
Copy link
Member

rhatdan commented Jul 1, 2020

bors r+

bors bot added a commit that referenced this pull request Jul 1, 2020
2418: buildah, bud: support --jobs=N for parallel execution r=rhatdan a=giuseppe

<!--
Please label this pull request according to what type of issue you are
addressing, especially if this is a release targeted pull request.

Uncomment only one `/kind <>` line, hit enter to put that in a new line, and
remove leading whitespace from that line:
-->

/kind feature

#### What this PR does / why we need it:

it enables running multi stages Containerfiles in parallel

#### How to verify it

buildah bud --jobs=$(nproc) .....

#### Which issue(s) this PR fixes:

None


#### Special notes for your reviewer:

#### Does this PR introduce a user-facing change?

```release-note
add support for running multi stages Containerfiles in parallel
```



2422: Add bud regression tests r=rhatdan a=TomSweeneyRedHat

We recently made a couple of commits to the imagebuilder
project to address #2345 and #2192.  Tests were created
in that project, but I thought it best to add some
regression tests here in Buildah too.

FYI: @petr-motejlek and @tachoknight

Signed-off-by: TomSweeneyRedHat <[email protected]>

<!--
Thanks for sending a pull request!

Please make sure you've read and understood our contributing guidelines
(https://github.com/containers/buildah/blob/master/CONTRIBUTING.md) as well as ensuring
that all your commits are signed with `git commit -s`.
-->

#### What type of PR is this?

<!--
Please label this pull request according to what type of issue you are
addressing, especially if this is a release targeted pull request.

Uncomment only one `/kind <>` line, hit enter to put that in a new line, and
remove leading whitespace from that line:
-->

> /kind api-change
> /kind bug
> /kind cleanup
> /kind deprecation
> /kind design
> /kind documentation
> /kind failing-test 
> /kind feature
> /kind flake
> /kind other

#### What this PR does / why we need it:

#### How to verify it

#### Which issue(s) this PR fixes:

<!--
Automatically closes linked issue when PR is merged.
Uncomment the following comment block and include the issue
number or None on one line.
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`, or `None`.
-->

<!--
Fixes #
or
None
-->

#### Special notes for your reviewer:

#### Does this PR introduce a user-facing change?

<!--
If no, just write `None` in the release-note block below. If yes, a release note
is required: Enter your extended release note in the block below. If the PR
requires additional action from users switching to the new release, include the
string "action required".

For more information on release notes please follow the kubernetes model:
https://git.k8s.io/community/contributors/guide/release-notes.md
-->

```release-note

```



Co-authored-by: Giuseppe Scrivano <[email protected]>
Co-authored-by: TomSweeneyRedHat <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jul 1, 2020

Build failed (retrying...):

  • cirrus-ci/success

@bors
Copy link
Contributor

bors bot commented Jul 1, 2020

Build succeeded:

  • cirrus-ci/success

@bors bors bot merged commit a0500c2 into containers:master Jul 1, 2020
@TomSweeneyRedHat TomSweeneyRedHat deleted the dev/tsweeney/budtestmr branch August 14, 2020 20:38
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants