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

🌱 Setup CI workflow to use same container as Dockerfile #1804

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

sjd78
Copy link
Member

@sjd78 sjd78 commented Mar 29, 2024

Lookup the "FROM .* as builder" image from the project's Dockerfile, and use that image to run the CI unit-test job.

This will ensure the unit-test job is using the same container image that the image builds will be using. A problem with the container should show up quickly in the unit-testing.

@sjd78 sjd78 requested review from ibolton336 and rszwajko March 29, 2024 01:04
@sjd78 sjd78 force-pushed the github-ci-on-ubi branch from 6fdc0b2 to ef49f60 Compare March 29, 2024 01:05
@sjd78 sjd78 added the cherry-pick/release-0.3 This PR should be cherry-picked to release-0.3 branch. label Apr 3, 2024
@sjd78 sjd78 force-pushed the github-ci-on-ubi branch from ef49f60 to cfddb4e Compare April 3, 2024 20:37
Copy link
Collaborator

@rszwajko rszwajko left a comment

Choose a reason for hiding this comment

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

Looks good. One question below.

.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
Lookup the "FROM .* as builder" image from the project's
Dockerfile, and use that image to run the CI unit-test job.

This will ensure the unit-test job is using the same container
image that the image builds will be using.  A problem with the
container should show up quickly in the unit-testing.

Signed-off-by: Scott J Dickerson <[email protected]>
@sjd78 sjd78 force-pushed the github-ci-on-ubi branch from cfddb4e to 8e1cb9b Compare April 5, 2024 14:29
@sjd78 sjd78 requested a review from rszwajko April 5, 2024 14:29
@sjd78
Copy link
Member Author

sjd78 commented Apr 5, 2024

Kicked the codecov resolution to #1820

@sjd78 sjd78 merged commit a5d3ec4 into konveyor:main Apr 5, 2024
7 checks passed
@sjd78 sjd78 deleted the github-ci-on-ubi branch April 5, 2024 19:14
github-actions bot pushed a commit that referenced this pull request Apr 5, 2024
Lookup the "FROM .* as builder" image from the project's Dockerfile, and
use that image to run the CI unit-test job.

This will ensure the unit-test job is using the same container image
that the image builds will be using. A problem with the container should
show up quickly in the unit-testing.

Signed-off-by: Scott J Dickerson <[email protected]>
Signed-off-by: Cherry Picker <[email protected]>
sjd78 pushed a commit that referenced this pull request Apr 5, 2024
Lookup the "FROM .* as builder" image from the project's Dockerfile, and
use that image to run the CI unit-test job.

This will ensure the unit-test job is using the same container image
that the image builds will be using. A problem with the container should
show up quickly in the unit-testing.

Signed-off-by: Scott J Dickerson <[email protected]>
Signed-off-by: Cherry Picker <[email protected]>
sjd78 added a commit to sjd78/tackle2-ui that referenced this pull request Apr 19, 2024
Backport repo level CI updates following changes in konveyor#1804
and others.  This will allow the "required" PR check of
"unit-test" to be run on the release-0.2 branch.

Signed-off-by: Scott J Dickerson <[email protected]>
sjd78 added a commit to sjd78/tackle2-ui that referenced this pull request Apr 19, 2024
Backport repo level CI updates following changes in konveyor#1804
and others.  This will allow the "required" PR check of
"unit-test" to be run on the release-0.2 branch.

Specific changes:
  - Workflow `ci-actions.yml` replaced by `ci-repo.yml`
  - Add `./scripts/verify_lock.mjs`
  - Disable codecov uploads since we don't need it on
    an old release branch

Signed-off-by: Scott J Dickerson <[email protected]>
sjd78 added a commit that referenced this pull request Apr 19, 2024
Backport repo level CI updates following changes in #1804 and others.
This will allow the "required" PR check of "unit-test" to be run on the
release-0.2 branch.

Specific changes:
  - Workflow `ci-actions.yml` replaced by `ci-repo.yml`
  - Add `./scripts/verify_lock.mjs` updated for the branch's package.json
    project structure
  - Disable the lint step since linting does not exist on release-0.2
  - Disable codecov uploads since we don't need it on an old release
    branch

---------

Signed-off-by: Scott J Dickerson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick/release-0.3 This PR should be cherry-picked to release-0.3 branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants