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

[ci] Fix Azure Pipeline Agents #6407

Merged
merged 8 commits into from
Apr 10, 2024
Merged

[ci] Fix Azure Pipeline Agents #6407

merged 8 commits into from
Apr 10, 2024

Conversation

shiyu1994
Copy link
Collaborator

@shiyu1994 shiyu1994 commented Apr 5, 2024

Azure pipeline agents are failing. See #6316 (comment) for the details.

@shiyu1994 shiyu1994 changed the title Update .vsts-ci.yml for Azure Pipelines Fix Azure Pipeline Agents Apr 5, 2024
@shiyu1994 shiyu1994 self-assigned this Apr 5, 2024
@shiyu1994 shiyu1994 changed the title Fix Azure Pipeline Agents [ci] Fix Azure Pipeline Agents Apr 5, 2024
@mayer79
Copy link
Contributor

mayer79 commented Apr 9, 2024

Out of curiosity: why is the failing check necessary? It seems to create a heavy dependence.

@jameslamb
Copy link
Collaborator

jameslamb commented Apr 9, 2024

why is the failing check necessary

What you've referred to as "the" failing check is actually 18 different CI jobs, testing a wide range of Python versions, compilers, operating systems, and interfaces to LightGBM (Python package, Java interface via SWIG, CLI, and more).

image

Microsoft pays for a pool of machines on Azure DevOps to help run all these jobs, which is great because it allows us to achieve better CI coverage than we could with only free CI resources.

But because it's paid for and managed by Microsoft, only certain Microsoft employees are able to work on fixing it when it breaks in certain ways. At the moment, that's only @shiyu1994 , and LightGBM is not his only priority. Hopefully he will be able to fix it soon or get some other folks from Microsoft to help out, and we can continue with development on this project.

Thanks for your patience and sorry this is disrupting your momentum on the things you want to work on here.

@mayer79
Copy link
Contributor

mayer79 commented Apr 10, 2024

Pipeline is green :-). @jameslamb thanks a lot for the explanations. Makes perfect sense.

@jameslamb
Copy link
Collaborator

jameslamb commented Apr 10, 2024

pipeline is green

Heyyyy excellent! Thanks so much for the help @shiyu1994 . I'll merge this and work on merging/re-building the other already-approved PRs today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants