Build Spring Cloud Data Flow

Build: #1412 was successful Code changes detected

Build result summary


11 minutes
d26944952303bf90adced43122472a919457ea84 d26944952303bf90adced43122472a919457ea84
Total tests
First to pass since
#1411 (Code changes detected – )


Code commits

Author Commit Message Commit date
Ilayaperumal Gopinathan Ilayaperumal Gopinathan d26944952303bf90adced43122472a919457ea84 m d26944952303bf90adced43122472a919457ea84 Support undeploy/destroy operation via StreamService
 - Have the StreamDeploymentController delegate the undeploy operation via StreamService
 - Add SPI contract `undeployStream` at StreamService and StreamDeployer
 - Implement `undeployStream` in SkipperStreamDeployer to invoke `delete` release REST endpoint after constructing the release name
 - Move AppDeployerStreamDeployer specific code
 - Add tests

Resolves #1682
Resolves #1683

Polished tests per code review on merge
Ilayaperumal Gopinathan Ilayaperumal Gopinathan 4146e97c613bf65df5890a9551b38d0120b0102b m 4146e97c613bf65df5890a9551b38d0120b0102b Record stream deployment
 - Add StreamDeployment repository that keeps track stream deployments
   - Define necessary operations in StreamDeploymentRepository interface
   - Implement both RDBMS and in-memory (for test only)
   - Define DDL for supported DB schemas
 - Add StreamDeployment domain object
 - Verify StreamDeployment entry when upgrading stream
 - Verify StreamDeployment when calculating state
 - Remove usage of USE_SKIPPER as the system property
 - Add tests to verify both the upgrade and state in DefaultStreamService

Resolves #1681

Store stream deployment for appdeployer

Remove unwanted spring-data-rest dependency

Fix StreamDeploymentRepository deletion

 - Make sure the delete the Stream Deployment entry when the stream is undeployed

Address review comments
Soby Chacko Soby Chacko 82499b2059837b3c0a28d1a5aeab02e0744e230d m 82499b2059837b3c0a28d1a5aeab02e0744e230d Upgrade spring-cloud-build to 1.3.5.RELEASE
* Fix checkstyle errors
* Remove guava dependency in spring-cloud-starter-dataflow-server-local
* Add new test utility method n spring-cloud-starter-dataflow-server-local
  to create immutable map from <String, String> pairs
* Fix tests in spring-cloud-starter-dataflow-server-local because of guava removal
* Remove redundant groupId in root pom.xml

Fixes #1663

Fixed Code Review Comments on Merge
Mark Pollack Mark Pollack e116e82dd63a324d621373f34c0df76e1791b3e4 m e116e82dd63a324d621373f34c0df76e1791b3e4 Add "stream update" command in shell and server
Test by starting up SCDF server/shell and Skipper server/shell - no args/options

dataflow:>app import --uri

dataflow:>stream create --name ticktock --definition "time | log" --deploy --useSkipper
Created new stream 'ticktock'
Deployment request has been sent
dataflow:>stream list
║Stream Name│Stream Definition│                 Status                 ║
║ticktock   │time | log       │All apps have been successfully deployed║

dataflow:>stream update --name ticktock --yaml log.version=1.1.1.RELEASE
Update request has been sent for stream 'ticktock'
dataflow:>stream list
║Stream Name│Stream Definition│                 Status                 ║
║ticktock   │time | log       │All apps have been successfully deployed║

Simultaneous commands in skipper

║   Name   │Version│      Last updated      │ Status │  Package  │   Package   │  Platform  │  Platform   ║
║          │       │                        │        │   Name    │   Version   │    Name    │   Status    ║
║myticktock│1      │Wed Oct 11 02:02:35 EDT │DEPLOYED│ticktock   │1.0.0        │default     │             ║
║          │       │2017                    │        │           │             │            │             ║

skipper:>status --release-name myticktock
║Last Deployed  │Wed Oct 11 02:02:35 EDT 2017                  ║
║Status         │DEPLOYED                                      ║
║Platform Status│The applications are being deployed.          ║
║               │ticktock.log-v1[ticktock.log-v1-0=deploying]  ║
║               │ticktock.time-v1[ticktock.time-v1-0=deploying]║

skipper:>status --release-name myticktock
║Last Deployed  │Wed Oct 11 02:03:17 EDT 2017                     ║
║Status         │DEPLOYED                                         ║
║Platform Status│All applications have been successfully deployed.║
║               │ticktock.log-v2[ticktock.log-v2-0=deployed]      ║
║               │ticktock.time-v1[ticktock.time-v1-0=deployed]    ║

skipper:>get manifest --release-name myticktock --release-version 2

apiVersion: skipper/v1
kind: SpringBootApp
  count: 1
  name: log
  resource: maven://
  resourceMetadata: maven://
    spring.metrics.export.triggers.application.includes: integration** log ticktock.log.${} ticktock,spring.application.index,*,* ticktock sink ticktock.time
  deploymentProperties: true ticktock

apiVersion: skipper/v1
kind: SpringBootApp
  count: 1
  name: time
  resource: maven://
  resourceMetadata: maven://
    spring.metrics.export.triggers.application.includes: integration** time ticktock.time.${} ticktock,spring.application.index,*,* ticktock.time ticktock source
  deploymentProperties: ticktock


Fixes #154
Ilayaperumal Gopinathan Ilayaperumal Gopinathan 6e72785fc15cf1ce43071011735cba36ea5c77fc m 6e72785fc15cf1ce43071011735cba36ea5c77fc Handle error conditions and null values
 - In SkipperStreamDeployer handle exception thrown when status() is invoked
 - In StreamDefinitionController check for null condition on DeploymentState
 - Fix test to mock StreamService

Jira issues

Unknown Issue TypeGH-50Could not obtain issue details from Jira
Unknown Issue TypeGH-1495Could not obtain issue details from Jira
Unknown Issue TypeGH-1568Could not obtain issue details from Jira