Skip to content

e2e test should be executed from repository root directory#860

Merged
olblak merged 7 commits into
updatecli:mainfrom
olblak:fixE2e
Sep 11, 2022
Merged

e2e test should be executed from repository root directory#860
olblak merged 7 commits into
updatecli:mainfrom
olblak:fixE2e

Conversation

@olblak

@olblak olblak commented Sep 11, 2022

Copy link
Copy Markdown
Member

e2e tests were executed from e2e directory instead of the git root directory.
This messed up manifest with relative path.

Test

To test this pull request, you can run the following commands:

export GITHUB_ACTOR=xxx
export GITHUB_TOKEN=yyy
make build test-e2e

Additional Information

Tradeoff

Potential improvement

olblak and others added 7 commits September 9, 2022 09:06
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
@olblak olblak enabled auto-merge (squash) September 11, 2022 15:35
@olblak olblak requested a review from lemeurherve September 11, 2022 15:36

@lemeurherve lemeurherve left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good one on the "ERROR" :)

@olblak olblak merged commit 105bb1f into updatecli:main Sep 11, 2022
@olblak olblak added bug Something isn't working skip-changelog Used by release-drafter to ignore pullrequest in changelog labels Sep 11, 2022
@olblak olblak deleted the fixE2e branch June 12, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working skip-changelog Used by release-drafter to ignore pullrequest in changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants