Skip to content

Conversation

Jooho
Copy link
Contributor

@Jooho Jooho commented Aug 12, 2025

Issue: #148

This commit adds GitHub Actions workflows for Prow CI/CD pipeline, including the following files:

  • Prow GitHub workflow
  • PR auto-merge workflow
  • PR LGTM removal workflow
  • Re-run action workflow
  • Prow labels configuration
  • OWNERS file

@@ -0,0 +1,15 @@
approvers:
Copy link
Collaborator

Choose a reason for hiding this comment

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

out of curiosity, will this work with OWNER_ALIASES file also?
e.g., like this one:
https://github.com/kubernetes-sigs/gateway-api-inference-extension/blob/main/OWNERS_ALIASES

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yea I think we can use it. I added the file

@elevran
Copy link
Collaborator

elevran commented Aug 13, 2025

@Jooho - thanks!
While you're making changes, can you please also ensure that files have a trailing newline?
GH seems unhappy with the current end-of-files.

nirrozenbaum
nirrozenbaum previously approved these changes Aug 14, 2025
Copy link
Collaborator

@nirrozenbaum nirrozenbaum left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

Thanks @Jooho! 🙏

@nirrozenbaum
Copy link
Collaborator

@Jooho merging is blocked due to commit signature not verified.
can you please handle this so we can continue with merging your PR?

@Jooho Jooho force-pushed the prow_ci branch 2 times, most recently from f084766 to fd6a04c Compare August 14, 2025 13:26
@Jooho
Copy link
Contributor Author

Jooho commented Aug 14, 2025

@nirrozenbaum I rebased and amened :)

@nirrozenbaum
Copy link
Collaborator

@Jooho hate to nag, we required verified signatures. your commit is signed but your signature is not verified.
can you please make sure you have verified your signature in github settings?
see for example the other merged PRs, all with verified signature.
image

@Jooho
Copy link
Contributor Author

Jooho commented Aug 15, 2025

@nirrozenbaum oh.. right my gpg key was expired before. I extended the date so now it should show a verified mark.

@Jooho
Copy link
Contributor Author

Jooho commented Aug 15, 2025

interesting.. it was gone again after a few mins.. let me check it again.

@Jooho
Copy link
Contributor Author

Jooho commented Aug 15, 2025

I hope that this time it works
image

Signed-off-by: Jooho Lee <[email protected]>

follw up comments

Signed-off-by: Jooho Lee <[email protected]>
@Jooho
Copy link
Contributor Author

Jooho commented Aug 15, 2025

Ok, I figured out what caused the issue. When I rebased via the GitHub workflow button, the Verified badge disappeared, so I did a rebase first and then amended the commit again.

@Jooho
Copy link
Contributor Author

Jooho commented Aug 15, 2025

By the way, I don’t understand why the workflow isn’t triggered, even though this isn’t the first PR.

Copy link
Collaborator

@nirrozenbaum nirrozenbaum left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

Thanks!

@nirrozenbaum nirrozenbaum merged commit bbf4047 into llm-d:main Aug 16, 2025
5 checks passed
@Jooho Jooho deleted the prow_ci branch August 16, 2025 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants