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

cirrus: Schedule one task with paid credits for faster CI feedback #20615

Merged
merged 1 commit into from
Dec 15, 2020

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Dec 10, 2020

During times of high activity in the repo, the scheduling of Cirrus CI tasks might put them a few hours in the future. This is fine when all the tasks eventually pass. Though for failing tasks, a failure should ideally be shown to the author and reviewer as soon as possible.

Compute credits can be used to schedule immediately: https://cirrus-ci.org/pricing/#compute-credits. Running all tasks with compute credits will probably be more expensive than our previous CI invoice. However, they are also more flexible.

As a start we could enable only a single task and revisit/re-evaluate the next steps in a month.

@fanquake fanquake added the Tests label Dec 10, 2020
@laanwj
Copy link
Member

laanwj commented Dec 11, 2020

Concept ACK faf2c6e

@fanquake
Copy link
Member

ACK faf2c6e

@luke-jr
Copy link
Member

luke-jr commented Dec 14, 2020

Who is funding the credits? They should ACK specifically.

@practicalswift
Copy link
Contributor

practicalswift commented Dec 14, 2020

Who is funding the credits? They should ACK specifically.

… and receive a giant thank you:

Thank you kind person! :) ❤️

cr ACK faf2c6e: patch looks correct

@laanwj laanwj merged commit c434e2c into bitcoin:master Dec 15, 2020
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 15, 2020
…faster CI feedback

faf2c6e cirrus: Schedule one task with paid credits for faster CI feedback (MarcoFalke)

Pull request description:

  During times of high activity in the repo, the scheduling of Cirrus CI tasks might put them a few hours in the future. This is fine when all the tasks eventually pass. Though for failing tasks, a failure should ideally be shown to the author and reviewer as soon as possible.

  Compute credits can be used to schedule immediately: https://cirrus-ci.org/pricing/#compute-credits. Running all tasks with compute credits will probably be more expensive than our previous CI invoice. However, they are also more flexible.

  As a start we could enable only a single task and revisit/re-evaluate the next steps in a month.

ACKs for top commit:
  laanwj:
    Concept ACK faf2c6e
  fanquake:
    ACK faf2c6e
  practicalswift:
    cr ACK faf2c6e: patch looks correct

Tree-SHA512: df599e1c4cc0394f7f03413ad29954ddc87b163b02640d8bfc0497a5dc8d237b8c963c1dd9d01ac83c4a044f575300763097dac2ea6d1a4a163a1cece342b743
adamjonas pushed a commit to adamjonas/bitcoin that referenced this pull request Dec 16, 2020
…faster CI feedback

faf2c6e cirrus: Schedule one task with paid credits for faster CI feedback (MarcoFalke)

Pull request description:

  During times of high activity in the repo, the scheduling of Cirrus CI tasks might put them a few hours in the future. This is fine when all the tasks eventually pass. Though for failing tasks, a failure should ideally be shown to the author and reviewer as soon as possible.

  Compute credits can be used to schedule immediately: https://cirrus-ci.org/pricing/#compute-credits. Running all tasks with compute credits will probably be more expensive than our previous CI invoice. However, they are also more flexible.

  As a start we could enable only a single task and revisit/re-evaluate the next steps in a month.

ACKs for top commit:
  laanwj:
    Concept ACK faf2c6e
  fanquake:
    ACK faf2c6e
  practicalswift:
    cr ACK faf2c6e: patch looks correct

Tree-SHA512: df599e1c4cc0394f7f03413ad29954ddc87b163b02640d8bfc0497a5dc8d237b8c963c1dd9d01ac83c4a044f575300763097dac2ea6d1a4a163a1cece342b743
@maflcko maflcko deleted the 2012-ciFasterFeedback branch December 16, 2020 06:16
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants