GithubHelp home page GithubHelp logo

Comments (7)

oliver-schoenherr avatar oliver-schoenherr commented on June 3, 2024 2

The AWS Java SDK includes a Backoff Strategy:
com.amazonaws.retry.PredefinedRetryPolicies.SDKDefaultBackoffStrategy

imho a retry parameter makes no sense from the User Perspective. Maybe this is a major drawback of the solution - because of, normal Plugin User don't understands the dependency between ThrottlingExceptions and Retry Counts.

You should start with something hardcoded like max 10 Retry.

envato/stack_master@12448d8#diff-1f8a98128798cdf6364b9ebc6358d44fR29

from aws-codebuild-jenkins-plugin.

uttamk avatar uttamk commented on June 3, 2024
Proposed solution

Add a pollingInterval build parameter to the CodeBuildStep. Keep the default to 5 seconds. Let the users decide if they want to increase it to suit their parallelism requirement.

I have a PR for this. See #32

from aws-codebuild-jenkins-plugin.

oliver-schoenherr avatar oliver-schoenherr commented on June 3, 2024

The Java AWS SDK includes some Retry Logic concerning ThrottlingExceptions, maybe you should review the default Client Configuration and increase the maxErrorRetry Value.

com.amazonaws.retry.RetryUtils#isThrottlingException

from aws-codebuild-jenkins-plugin.

uttamk avatar uttamk commented on June 3, 2024

@oliver-schoenherr .. nice suggestion. Do we retry with increasing sleep times after every retry? We would still need a limit for the number of retries. Do we hard code this in the code or let the plugin have a retries parameter ?

from aws-codebuild-jenkins-plugin.

uttamk avatar uttamk commented on June 3, 2024

sounds good @oliver-schoenherr. We can start off by hard coding the retry limit/sleep time and see if it works for most use cases. If users need a higher retry limit, we can think about parameterizing the retry limit.
cc. @leobaran-aws @jvusich any thoughts ?

from aws-codebuild-jenkins-plugin.

leoherran-aws avatar leoherran-aws commented on June 3, 2024

Sounds good to me! Implementing backoff for build status polling is a great idea.

from aws-codebuild-jenkins-plugin.

leoherran-aws avatar leoherran-aws commented on June 3, 2024

Merged #35.

from aws-codebuild-jenkins-plugin.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.