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.

Sign up for GitHub

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

Jump to bottom

CI: Move openshift-ci from kata-containers/tests here #8653

Closed
ldoktor opened this issue Dec 13, 2023 · 0 comments · Fixed by #8654 or #9169
Closed

CI: Move openshift-ci from kata-containers/tests here #8653

ldoktor opened this issue Dec 13, 2023 · 0 comments · Fixed by #8654 or #9169
Labels
feature New functionality needs-review Needs to be assessed by the team.
Projects
Issue backlog

Comments

@ldoktor
Copy link
Contributor

ldoktor commented Dec 13, 2023

Is your feature request related to a problem? Please describe.

The kata-containers/tests repo repo should become read-only by the end of this year (according to @fidencio), let's move the scripts and helpers from that repo here to preserve the possibility to modify the configuration.

Describe the solution you'd like

Move and modify the kata-containers/tests/.ci/openshift-ci and related utils into this repo, once ready modify the openshift/release repo to point to the new location.

Describe alternatives you've considered

We can use a different repo, but that wouldn't really make sense
@ldoktor ldoktor added feature New functionality needs-review Needs to be assessed by the team. labels Dec 13, 2023
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Dec 13, 2023
@ldoktor
ci/openshift-ci: Correct the lib location
7453e79 
correct the lib file locations after the move from
tests->kata-containers repo and add a minimized version of the
".ci/lib.sh" library into the "ci/openshift-ci" as we don't really
utilize all of the features.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
@ldoktor ldoktor mentioned this issue Dec 13, 2023
Merged
@katacontainersbot katacontainersbot moved this from To do to In progress in Issue backlog Dec 13, 2023
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Dec 13, 2023
@ldoktor
ci/openshift-ci: Correct the lib location
8b78ac6 
correct the lib file locations after the move from
tests->kata-containers repo and add a minimized version of the
".ci/lib.sh" library into the "ci/openshift-ci" as we don't really
utilize all of the features.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Dec 19, 2023
@ldoktor
ci/openshift-ci: Correct the lib location
7cdc30c 
correct the lib file locations after the move from
tests->kata-containers repo and add a minimized version of the
".ci/lib.sh" library into the "ci/openshift-ci" as we don't really
utilize all of the features.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Jan 25, 2024
@ldoktor
ci/openshift-ci: Correct the lib location
f93d8c5 
correct the lib file locations after the move from
tests->kata-containers repo and add a minimized version of the
".ci/lib.sh" library into the "ci/openshift-ci" as we don't really
utilize all of the features.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Jan 30, 2024
@ldoktor
ci/openshift-ci: Correct the lib location
b0b7748 
correct the lib file locations after the move from
tests->kata-containers repo and add a minimized version of the
".ci/lib.sh" library into the "ci/openshift-ci" as we don't really
utilize all of the features.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Feb 5, 2024
@ldoktor
tools.kata-webhook: Fix lib path
3b0049b 
When moving the webhook we skipped the common.bash as (close-enough)
version is already in `/tests` but we forgot to update the source path,
fixing it here.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
ChengyuZhu6 pushed a commit to ChengyuZhu6/kata-containers that referenced this issue Feb 21, 2024
@ldoktor @ChengyuZhu6
tools.kata-webhook: Fix lib path
e83dba4 
When moving the webhook we skipped the common.bash as (close-enough)
version is already in `/tests` but we forgot to update the source path,
fixing it here.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
c3d pushed a commit to c3d/kata-containers that referenced this issue Feb 23, 2024
@ldoktor @c3d
ci/openshift-ci: Correct the lib location
ab863cf 
correct the lib file locations after the move from
tests->kata-containers repo and add a minimized version of the
".ci/lib.sh" library into the "ci/openshift-ci" as we don't really
utilize all of the features.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
c3d pushed a commit to c3d/kata-containers that referenced this issue Feb 23, 2024
@ldoktor @c3d
tools.kata-webhook: Fix lib path
4b76b3f 
When moving the webhook we skipped the common.bash as (close-enough)
version is already in `/tests` but we forgot to update the source path,
fixing it here.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Feb 27, 2024
@ldoktor
ci.ocp: Backport service-up detection fixes
4f33648 
This backports the:

9060e930caf2d20f413df07778d3ab497493161c

    ci.ocp: Add debug output on HTTP service failure

    these logs are vital to analyze a setup failure.

a10a1e2c9cbc21afc1e80f22b0fb8634d27cbd8d

    ci.ocp: Improve the service-up detection

    waiting for the first response is not sufficient as OCP returns html
    page without error even when the route is not yet established describing
    the issue (why it doesn't reply with 500?). Waiting for the correct
    output should do better.

commits from the kata-containers/tests repo.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
@ldoktor ldoktor mentioned this issue Feb 27, 2024
Merged
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Feb 29, 2024
@ldoktor
ci.ocp: Backport service-up detection fixes
c81cbf0 
This backports the:

9060e930caf2d20f413df07778d3ab497493161c

    ci.ocp: Add debug output on HTTP service failure

    these logs are vital to analyze a setup failure.

a10a1e2c9cbc21afc1e80f22b0fb8634d27cbd8d

    ci.ocp: Improve the service-up detection

    waiting for the first response is not sufficient as OCP returns html
    page without error even when the route is not yet established describing
    the issue (why it doesn't reply with 500?). Waiting for the correct
    output should do better.

commits from the kata-containers/tests repo.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
ldoktor added a commit to ldoktor/kata-containers that referenced this issue Mar 1, 2024
@ldoktor
ci.ocp: Backport service-up detection fixes
6fffbaa 
This backports the:

9060e930caf2d20f413df07778d3ab497493161c

    ci.ocp: Add debug output on HTTP service failure

    these logs are vital to analyze a setup failure.

a10a1e2c9cbc21afc1e80f22b0fb8634d27cbd8d

    ci.ocp: Improve the service-up detection

    waiting for the first response is not sufficient as OCP returns html
    page without error even when the route is not yet established describing
    the issue (why it doesn't reply with 500?). Waiting for the correct
    output should do better.

commits from the kata-containers/tests repo.

Fixes: kata-containers#8653

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New functionality needs-review Needs to be assessed by the team.
Projects
Issue backlog
  
In progress
Milestone
No milestone
1 participant
@ldoktor