Skip to content

tests: add kfp-ui test case in repo level integration tests#253

Open
DnPlas wants to merge 11 commits into
mainfrom
KF-3609-frontend-test-case
Open

tests: add kfp-ui test case in repo level integration tests#253
DnPlas wants to merge 11 commits into
mainfrom
KF-3609-frontend-test-case

Conversation

@DnPlas

@DnPlas DnPlas commented Jul 14, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Add a test case for reaching kfp-ui and getting back a 200 response.

NOTE: merge after #261

@DnPlas DnPlas requested a review from a team as a code owner July 14, 2023 11:12
@github-actions github-actions Bot added the Libraries: Out of sync The charm libs used are out-of-sync label Jul 14, 2023
@github-actions github-actions Bot added Libraries: OK The charm libs used are OK and in-sync and removed Libraries: Out of sync The charm libs used are out-of-sync labels Jul 18, 2023
@i-chvets

i-chvets commented Jul 18, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Looks good. Bundle tests are failing in build charm.

i-chvets
i-chvets previously approved these changes Jul 18, 2023
View reviewed changes

@i-chvets i-chvets left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved.

@DnPlas

DnPlas commented Jul 19, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

The bundle integration test seems to be failing because of #243, will re-trigger the CI once again.

@github-actions github-actions Bot added Libraries: Out of sync The charm libs used are out-of-sync and removed Libraries: OK The charm libs used are OK and in-sync labels Jul 19, 2023
@github-actions github-actions Bot added Libraries: OK The charm libs used are OK and in-sync and removed Libraries: Out of sync The charm libs used are out-of-sync labels Jul 19, 2023
@github-actions github-actions Bot added Libraries: Out of sync The charm libs used are out-of-sync Libraries: OK The charm libs used are OK and in-sync and removed Libraries: OK The charm libs used are OK and in-sync Libraries: Out of sync The charm libs used are out-of-sync labels Jul 19, 2023
@github-actions github-actions Bot added Libraries: OK The charm libs used are OK and in-sync and removed Libraries: Out of sync The charm libs used are out-of-sync labels Jul 20, 2023
@github-actions github-actions Bot added Libraries: Out of sync The charm libs used are out-of-sync Libraries: OK The charm libs used are OK and in-sync and removed Libraries: OK The charm libs used are OK and in-sync Libraries: Out of sync The charm libs used are out-of-sync labels Jul 21, 2023
@github-actions github-actions Bot added Libraries: Out of sync The charm libs used are out-of-sync Libraries: OK The charm libs used are OK and in-sync and removed Libraries: OK The charm libs used are OK and in-sync Libraries: Out of sync The charm libs used are out-of-sync labels Jul 21, 2023
i-chvets
i-chvets approved these changes Aug 22, 2023
View reviewed changes

@i-chvets i-chvets left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved.

@github-actions github-actions Bot added Libraries: Out of sync The charm libs used are out-of-sync Libraries: OK The charm libs used are OK and in-sync and removed Libraries: OK The charm libs used are OK and in-sync Libraries: Out of sync The charm libs used are out-of-sync labels Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@i-chvets i-chvets i-chvets approved these changes
Reviewers whose approvals may not affect merge requirements

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

Libraries: OK The charm libs used are OK and in-sync

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DnPlas @i-chvets