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

kubesecondarydns, presubmits: Add build and unit-tests for s390x #3797

Merged
merged 2 commits into from
Dec 3, 2024

Conversation

jschintag
Copy link
Contributor

What this PR does / why we need it:
Add testing for building and unit-tests on s390x to kubesecondarydns
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

Release note:

NONE

@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Dec 3, 2024
@oshoval
Copy link
Contributor

oshoval commented Dec 3, 2024

Thanks

lgtm basically,

@brianmcarey can you take a look as well please?
it just runs build and unit tests as far as i can tell

@brianmcarey
Copy link
Member

/rehearse

@kubevirt-bot
Copy link
Contributor

Rehearsal jobs created for this PR:

rehearsal-pull-kubesecondarydns-unit-test-s390x
rehearsal-pull-kubesecondarydns-build-s390x

You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all
on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

@oshoval
Copy link
Contributor

oshoval commented Dec 3, 2024

interesting
Thanks Brian
https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_project-infra/3797/rehearsal-pull-kubesecondarydns-unit-test-s390x/1863963052123099136

please take a look @jschintag

@jschintag jschintag force-pushed the kubesecondary-s390x branch from 7189384 to 256a58b Compare December 3, 2024 15:49
@jschintag
Copy link
Contributor Author

Should be fixed, for some reason it wanted to use CGO, despite not needing it, and failed to find the gcc binary.

@oshoval
Copy link
Contributor

oshoval commented Dec 3, 2024

/rehearse

thanks
shouldnt it be on the makefile ?

@kubevirt-bot
Copy link
Contributor

Rehearsal jobs created for this PR:

rehearsal-pull-kubesecondarydns-unit-test-s390x
rehearsal-pull-kubesecondarydns-build-s390x

You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all
on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

@jschintag
Copy link
Contributor Author

Yes i can add it to the makefile as well.

@oshoval
Copy link
Contributor

oshoval commented Dec 3, 2024

Yes i can add it to the makefile as well.

better please, lets add it and then remove it from here ?

@jschintag
Copy link
Contributor Author

Done in kubevirt/kubesecondarydns#77

@oshoval
Copy link
Contributor

oshoval commented Dec 3, 2024

/rehearse

@kubevirt-bot
Copy link
Contributor

Rehearsal jobs created for this PR:

rehearsal-pull-kubesecondarydns-unit-test-s390x
rehearsal-pull-kubesecondarydns-build-s390x

You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all
on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

@oshoval
Copy link
Contributor

oshoval commented Dec 3, 2024

/lgtm

@brianmcarey can you ptal ?
should be fine now

@kubevirt-bot kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Dec 3, 2024
Copy link
Member

@brianmcarey brianmcarey left a comment

Choose a reason for hiding this comment

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

/approve

Looks good to me.

@kubevirt-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brianmcarey

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubevirt-bot kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 3, 2024
@kubevirt-bot kubevirt-bot merged commit c68aea4 into kubevirt:main Dec 3, 2024
7 checks passed
@kubevirt-bot
Copy link
Contributor

@jschintag: Updated the job-config configmap in namespace kubevirt-prow at cluster default using the following files:

  • key kubesecondarydns-presubmits.yaml using file github/ci/prow-deploy/files/jobs/kubevirt/kubesecondarydns/kubesecondarydns-presubmits.yaml

In response to this:

What this PR does / why we need it:
Add testing for building and unit-tests on s390x to kubesecondarydns
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

Release note:

NONE

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@jschintag jschintag deleted the kubesecondary-s390x branch December 4, 2024 07:15
@oshoval
Copy link
Contributor

oshoval commented Dec 5, 2024

@ashokpariya0
might good to consider please doing the same for CNAO ?
maybe even not always run but optional

@ashokpariya0
Copy link
Contributor

ashokpariya0 commented Dec 5, 2024

@ashokpariya0 might good to consider please doing the same for CNAO ? maybe even not always run but optional

Okay, Noted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants