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

NO-ISSUE: Trigger jitexecutor-native Github workflow as part of the Kogito Apps weekly job #2052

Merged
merged 1 commit into from
May 21, 2024

Conversation

rodrigonull
Copy link
Contributor

This PR is part of the effort for the Apache 10 release and it adds a new trigger for the jitexecutor-native workflow to run as part of the Kogito Apps weekly job.

Depends on: #2049

@tiagobento tiagobento changed the title Trigger jitexecutor-native Github workflow as part of the Kogito Apps weekly job NO-ISSUE: Trigger jitexecutor-native Github workflow as part of the Kogito Apps weekly job May 20, 2024
Copy link
Contributor

@jstastny-cz jstastny-cz left a comment

Choose a reason for hiding this comment

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

LGTM, I suppose the gh workflow run command requires our "push" credentials.

@rodrigonull rodrigonull force-pushed the trigger-jit-executor-workflow branch from 93a3d09 to 5ed5da4 Compare May 21, 2024 14:34
@tiagobento
Copy link
Contributor

getGitAuthor and getRepoName will always return apache and incubator-kie-tools in this case?

@tiagobento
Copy link
Contributor

tiagobento commented May 21, 2024

Or it returns whatever is configured in the Jenkins job "Configuration"?

@rodrigonull
Copy link
Contributor Author

It returns what is in the job configuration, which is apache and incubator-kie-kogito-apps.

@tiagobento tiagobento merged commit e0701e8 into apache:main May 21, 2024
1 of 2 checks passed
@rodrigonull rodrigonull deleted the trigger-jit-executor-workflow branch August 27, 2024 18:16
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