Skip to content

Containerfile: copy the entire "." dir into /build#588

Merged
achilleas-k merged 2 commits intoosbuild:mainfrom
mvo5:fix-tmt
Aug 8, 2024
Merged

Containerfile: copy the entire "." dir into /build#588
achilleas-k merged 2 commits intoosbuild:mainfrom
mvo5:fix-tmt

Conversation

@mvo5
Copy link
Contributor

@mvo5 mvo5 commented Aug 5, 2024

When tmt runs it will not include the .git dir when it prepares
the source. This was breaking the tmt tests. This commit just makes
the containerfile copy the entire current working dir so that we
get .git if it's there or it is skipped otherwise.

Closes: #583

achilleas-k
achilleas-k previously approved these changes Aug 5, 2024
Copy link
Member

@achilleas-k achilleas-k left a comment

Choose a reason for hiding this comment

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

LGTM. Tiny git history comment, but good to go.

@mvo5 mvo5 enabled auto-merge August 7, 2024 07:46
@mvo5 mvo5 added this pull request to the merge queue Aug 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 7, 2024
@mvo5 mvo5 added this pull request to the merge queue Aug 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 7, 2024
@mvo5
Copy link
Contributor Author

mvo5 commented Aug 7, 2024

Hrm, this will need #596 first

mvo5 added 2 commits August 8, 2024 14:10
When tmt runs it will create a build from a non-git directory.
This was breaking the tests and code `test_bib_version`. This
commit makes it more robust by ensuring that `unkown` is printed
when `bib` was build from a non-git directory.
When `tmt` runs it will not include the `.git` dir when it prepares
the source. This was breaking the tmt tests. This commit just makes
the containerfile copy the entire current working dir so that we
get `.git` if it's there or it is skipped otherwise.

Closes: osbuild#583
@achilleas-k achilleas-k enabled auto-merge August 8, 2024 12:11
@achilleas-k achilleas-k added this pull request to the merge queue Aug 8, 2024
Merged via the queue into osbuild:main with commit 1baf9f3 Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants