Branch: refs/heads/feature/fix-dependency-injection
Home: https://github.com/jenkinsci/scm-sqs-plugin
Commit: a223f887014f44d71cbeb5ebf74c7ff5ec67a8f6
https://github.com/jenkinsci/scm-sqs-plugin/commit/a223f887014f44d71cbeb5ebf74c7ff5ec67a8f6
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M
src/main/java/io/relution/jenkins/scmsqs/model/CodeCommitMessageParser.java
Log Message:
-----------
Fix queue monitor stopping on invalid message
This adds a try-catch-block to message parsing to prevent invalid messages from
stopping the queue monitor. Otherwise monitoring stops when a message in the
queue cannot be parsed. Log a warning and continue with polling/processing
other messages.
Commit: 605f2191c57069210bea9eb36e1ffdd3c9a3f8a1
https://github.com/jenkinsci/scm-sqs-plugin/commit/605f2191c57069210bea9eb36e1ffdd3c9a3f8a1
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
Log Message:
-----------
Fix crash bug on Jenkins 2.x.
Commit: fab057772df5f743ab00dbec23dc28d4d4deb46d
https://github.com/jenkinsci/scm-sqs-plugin/commit/fab057772df5f743ab00dbec23dc28d4d4deb46d
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
M
src/main/java/io/relution/jenkins/scmsqs/interfaces/EventTriggerMatcher.java
M
src/main/java/io/relution/jenkins/scmsqs/model/EventTriggerMatcherImpl.java
Log Message:
-----------
Change trigger to match events to jobs
This changes the trigger interface to accept a job (AbstractProject) rather
then an SCM. This allows triggers to use any property of a job to match an
event, not just a job’s SCM. The default implementation continues to use the
job’s SCM internally.
Commit: 10fea720c655a8a3dd747cea590aa9f3135dfa6c
https://github.com/jenkinsci/scm-sqs-plugin/commit/10fea720c655a8a3dd747cea590aa9f3135dfa6c
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/Context.java
Log Message:
-----------
Fix race condition when creating the injector.
Commit: bc3efb1702d3e4abc30c43fddd1b474e19f7ba18
https://github.com/jenkinsci/scm-sqs-plugin/commit/bc3efb1702d3e4abc30c43fddd1b474e19f7ba18
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImpl.java
Log Message:
-----------
Synchronize access to monitor registration
Commit: 58b6b8776b80e01e3a4176847f86cf00370432f2
https://github.com/jenkinsci/scm-sqs-plugin/commit/58b6b8776b80e01e3a4176847f86cf00370432f2
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
A src/main/java/io/relution/jenkins/scmsqs/model/constants/ErrorCode.java
M src/main/java/io/relution/jenkins/scmsqs/net/SQSChannelImpl.java
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorImpl.java
A src/main/java/io/relution/jenkins/scmsqs/util/ErrorType.java
Log Message:
-----------
Re-throw exception if credentials are invalid
This changes the channel implementation to re-throw the AmazonServiceException
if the provided credentials are invalid. This allows the monitor to catch the
exception and shut down. This fixes the issue of running into an endless loop
when credentials become invalid.
Commit: 20322603ac090ace2c7a129d21cd7aa3709cf5b6
https://github.com/jenkinsci/scm-sqs-plugin/commit/20322603ac090ace2c7a129d21cd7aa3709cf5b6
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/Context.java
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
A src/main/java/io/relution/jenkins/scmsqs/model/SQSQueueProviderImpl.java
Log Message:
-----------
Resolve circular dependency for SQS queue provider
SQSTrigger.DescriptorImpl depends on SQSQueueMonitorScheduler.
SQSQueueMonitorScheduler depends on SQSQueueProvider, which is provided by
SQSTrigger.DescriptorImpl.
Add a new SQS queue provider implementation, which uses the static Trigger
class provided by Jenkins to access the trigger’s descriptor.
Commit: 7ba99a23908696a437548bf14cbaebfdebf08dd9
https://github.com/jenkinsci/scm-sqs-plugin/commit/7ba99a23908696a437548bf14cbaebfdebf08dd9
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
Log Message:
-----------
Explicitly specify descriptor’s outer class
This seems to fix the “this class does not have a descriptor” error.
Commit: 1d01cb03bf5e7d4a735f38c9ad96a78789d8e7ba
https://github.com/jenkinsci/scm-sqs-plugin/commit/1d01cb03bf5e7d4a735f38c9ad96a78789d8e7ba
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
Log Message:
-----------
Break up single line
Commit: 0d50efe097e8c1faaf9d197f205b0d94a243501d
https://github.com/jenkinsci/scm-sqs-plugin/commit/0d50efe097e8c1faaf9d197f205b0d94a243501d
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
Log Message:
-----------
Use sentinel value for lazy loading.
Commit: 33f4d0a676d22924b02b8a4051921c0fd01e2967
https://github.com/jenkinsci/scm-sqs-plugin/commit/33f4d0a676d22924b02b8a4051921c0fd01e2967
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/Context.java
Log Message:
-----------
No longer auto-load the Context class
If the Context is automatically instantiated this causes two injectors to be
created, which in turn creates two instances of classes that should only have a
single instance. E.g. the SQS trigger’s descriptor will receive one instance of
the SQS queue monitor scheduler, while actual monitor instances will be
registered with a second instance of the scheduler. This in turn causes
reconfiguration of the scheduler to “fail” because the wrong (i.e. empty)
scheduler is reconfigured.
Commit: 9a7ec47406af2cdc734d99717290c9720b0fa1cc
https://github.com/jenkinsci/scm-sqs-plugin/commit/9a7ec47406af2cdc734d99717290c9720b0fa1cc
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
Log Message:
-----------
Asynchronously start and stop queue monitors
This fixes issues where the monitors tried to access the descriptor instance
before it was fully constructed.
Commit: 30a80a171bfcccec36821f8b57f25a35df8676fe
https://github.com/jenkinsci/scm-sqs-plugin/commit/30a80a171bfcccec36821f8b57f25a35df8676fe
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/SQSTrigger.java
M
src/main/java/io/relution/jenkins/scmsqs/interfaces/SQSQueueMonitorScheduler.java
A
src/main/java/io/relution/jenkins/scmsqs/model/events/ConfigurationChangedEvent.java
A src/main/java/io/relution/jenkins/scmsqs/model/events/EventBroker.java
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImpl.java
M
src/test/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImplTest.java
Log Message:
-----------
Decouple trigger descriptor and scheduler
This uses the Guava event bus to decouple the SQS trigger’s descriptor and the
SQS queue monitor scheduler.
Cannot inject the scheduler into the descriptor, since DI is not set up at this
time. If Context is turned into an @Extension, DI will be set up, but that
results in two instances of the injector which causes other issues.
To resolve this conflict, add a singleton instance of the event bus to
communicate configuration changes to the scheduler.
Commit: 9036f9198a64afc64e35819baf4c8fd098cfcba2
https://github.com/jenkinsci/scm-sqs-plugin/commit/9036f9198a64afc64e35819baf4c8fd098cfcba2
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/net/RequestFactoryImpl.java
Log Message:
-----------
Remove deprecated code.
Commit: 667c3ec6f99e74b1f4229e6a22d2258c3779922a
https://github.com/jenkinsci/scm-sqs-plugin/commit/667c3ec6f99e74b1f4229e6a22d2258c3779922a
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/factories/SQSFactoryImpl.java
M src/main/java/io/relution/jenkins/scmsqs/interfaces/SQSQueueMonitor.java
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorImpl.java
Log Message:
-----------
Add queue to the SQS queue monitor instance.
This adds the queue to the monitor instance to make it possible to determine
which queue a monitor is associated with in addition to the queue’s channel.
Commit: 43bd5694dbe8c0585322be544d9d306e837665be
https://github.com/jenkinsci/scm-sqs-plugin/commit/43bd5694dbe8c0585322be544d9d306e837665be
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M
src/main/java/io/relution/jenkins/scmsqs/interfaces/SQSQueueMonitorScheduler.java
Log Message:
-----------
Add missing parameter documentation.
Commit: 2f1f5422b58b778559c2d202b8a3d4b5b4000172
https://github.com/jenkinsci/scm-sqs-plugin/commit/2f1f5422b58b778559c2d202b8a3d4b5b4000172
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M src/main/java/io/relution/jenkins/scmsqs/factories/SQSFactoryImpl.java
M src/main/java/io/relution/jenkins/scmsqs/interfaces/SQSFactory.java
M src/main/java/io/relution/jenkins/scmsqs/interfaces/SQSQueueMonitor.java
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorImpl.java
Log Message:
-----------
Add methods to clone a SQS queue monitor
Commit: 9a15edbf775c7b40ada0bb83cdc9619d59430c11
https://github.com/jenkinsci/scm-sqs-plugin/commit/9a15edbf775c7b40ada0bb83cdc9619d59430c11
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImpl.java
M
src/test/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorImplTest.java
M
src/test/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImplTest.java
Log Message:
-----------
Create new SQS queue monitor if queue has changed
This changes the SQS queue monitor scheduler to replace a queue’s monitor in
case the queue’s configuration has changed.
Commit: c63158bd3017157bbca96a68229b18e179011432
https://github.com/jenkinsci/scm-sqs-plugin/commit/c63158bd3017157bbca96a68229b18e179011432
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImpl.java
Log Message:
-----------
Handle exception when comparing queues
Comparing queues may fail, e.g. if a queue’s access key or secret is invalid.
In this case a new monitor must always be started for the new configuration.
Commit: 96a06ddab54016b5389798a84cbdaea2ebfc7823
https://github.com/jenkinsci/scm-sqs-plugin/commit/96a06ddab54016b5389798a84cbdaea2ebfc7823
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorImpl.java
Log Message:
-----------
No longer create new monitor in stopped state.
Commit: 879377e7edce289c2b8b68a1a4d81e6797299161
https://github.com/jenkinsci/scm-sqs-plugin/commit/879377e7edce289c2b8b68a1a4d81e6797299161
Author: Markus Pfeiffer <[email protected]>
Date: 2016-06-16 (Thu, 16 Jun 2016)
Changed paths:
M
src/main/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImpl.java
M
src/test/java/io/relution/jenkins/scmsqs/threading/SQSQueueMonitorSchedulerImplTest.java
Log Message:
-----------
Create now monitor if old monitor is shut down
Monitors cannot be restarted, create a new instance instead.
Compare:
https://github.com/jenkinsci/scm-sqs-plugin/compare/a223f887014f^...879377e7edce
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
