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

Remove update dependencies job throttling #3806

Merged
merged 1 commit into from
Oct 27, 2023

Conversation

marko-bekhta
Copy link
Member

Here's an ORM job triggering the Search dependency update job and waiting for the results: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-orm-personal-marko/detail/ci%2Fadd-hibernate-search-tests-to-build-test/15/pipeline/

I've tried git checkout as well as git clone -c 'core.sshCommand=ssh -v ... for cloning a repo using an ssh URL, but with no luck... it looks like git wasn't able to find the known_hosts that would contain the keys we had on CI... Hence I've just adapted the job to "convert" the URL instead 🙈 🙈 🙈
I'll backport to 6.2 once we merge the changes to main and then update the ORM PR so it can actually trigger the correct build 😃

@hibernate-github-bot
Copy link

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HSEARCH-\d+
    ↳ Offending commits: [7b06688]

› This message was automatically generated.

@yrodiere
Copy link
Member

LGTM, thanks

@yrodiere yrodiere merged commit f74a50f into hibernate:main Oct 27, 2023
3 of 5 checks passed
@yrodiere
Copy link
Member

I'll backport to 6.2 once we merge the changes to main and then update the ORM PR so it can actually trigger the correct build 😃

Please do :)

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

None yet

2 participants