[ARVADOS] updated: 1.1.4-565-ge2f50c08c

[Git user]

Summary of changes:
 apps/workbench/lib/app_version.rb                             |  3 ++-
 services/api/app/controllers/arvados/v1/schema_controller.rb  |  3 ++-
 services/api/lib/app_version.rb                               |  3 ++-
 .../api/test/functional/arvados/v1/schema_controller_test.rb  | 11 +++++++----
 4 files changed, 13 insertions(+), 7 deletions(-)

       via  e2f50c08c4fc6da7065be222c2620264b521897f (commit)
      from  6dd60d7c70a63ac884515387cd8c92bb97433e1b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit e2f50c08c4fc6da7065be222c2620264b521897f
Author: Lucas Di Pentima <ldipentima at veritasgenetics.com>
Date:   Tue Jul 10 14:47:52 2018 -0300

    13668: Replaced package_version with packageVersion on discovery doc.
    
    Also, enhanced package version caching on api server & workbench.
    
    Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima at veritasgenetics.com>

diff --git a/apps/workbench/lib/app_version.rb b/apps/workbench/lib/app_version.rb
index 303d853ec..9db76e257 100644
--- a/apps/workbench/lib/app_version.rb
+++ b/apps/workbench/lib/app_version.rb
@@ -64,8 +64,9 @@ class AppVersion
     begin
       @package_version = IO.read(Rails.root.join("package-build.version")).strip
     rescue Errno::ENOENT
+      @package_version = "unknown"
     end
 
-    @package_version || "unknown"
+    @package_version
   end
 end
diff --git a/services/api/app/controllers/arvados/v1/schema_controller.rb b/services/api/app/controllers/arvados/v1/schema_controller.rb
index 8f6e2be72..49fc398e1 100644
--- a/services/api/app/controllers/arvados/v1/schema_controller.rb
+++ b/services/api/app/controllers/arvados/v1/schema_controller.rb
@@ -33,7 +33,8 @@ class Arvados::V1::SchemaController < ApplicationController
         version: "v1",
         revision: "20131114",
         source_version: AppVersion.hash,
-        package_version: AppVersion.package_version,
+        sourceVersion: AppVersion.hash, # source_version should be deprecated in the future
+        packageVersion: AppVersion.package_version,
         generatedAt: db_current_time.iso8601,
         title: "Arvados API",
         description: "The API to interact with Arvados.",
diff --git a/services/api/lib/app_version.rb b/services/api/lib/app_version.rb
index 6dcdafb50..335608b2b 100644
--- a/services/api/lib/app_version.rb
+++ b/services/api/lib/app_version.rb
@@ -63,8 +63,9 @@ class AppVersion
     begin
       @package_version = IO.read(Rails.root.join("package-build.version")).strip
     rescue Errno::ENOENT
+      @package_version = "unknown"
     end
 
-    @package_version || "unknown"
+    @package_version
   end
 end
diff --git a/services/api/test/functional/arvados/v1/schema_controller_test.rb b/services/api/test/functional/arvados/v1/schema_controller_test.rb
index 067e7b2bb..40868c87b 100644
--- a/services/api/test/functional/arvados/v1/schema_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/schema_controller_test.rb
@@ -31,26 +31,29 @@ class Arvados::V1::SchemaControllerTest < ActionController::TestCase
     assert_includes discovery_doc, 'defaultTrashLifetime'
     assert_equal discovery_doc['defaultTrashLifetime'], Rails.application.config.default_trash_lifetime
     assert_match(/^[0-9a-f]+(-modified)?$/, discovery_doc['source_version'])
-    assert_match(/^unknown$/, discovery_doc['package_version'])
+    assert_match(/^[0-9a-f]+(-modified)?$/, discovery_doc['sourceVersion'])
+    assert_match(/^unknown$/, discovery_doc['packageVersion'])
     assert_equal discovery_doc['websocketUrl'], Rails.application.config.websocket_address
     assert_equal discovery_doc['workbenchUrl'], Rails.application.config.workbench_address
     assert_equal('zzzzz', discovery_doc['uuidPrefix'])
   end
 
-  test "discovery document overrides source_version with config" do
+  test "discovery document overrides source_version & sourceVersion with config" do
     Rails.configuration.source_version = 'aaa888fff'
     get :index
     assert_response :success
     discovery_doc = JSON.parse(@response.body)
+    # Key source_version will be replaced with sourceVersion
     assert_equal 'aaa888fff', discovery_doc['source_version']
+    assert_equal 'aaa888fff', discovery_doc['sourceVersion']
   end
 
-  test "discovery document overrides package_version with config" do
+  test "discovery document overrides packageVersion with config" do
     Rails.configuration.package_version = '1.0.0-stable'
     get :index
     assert_response :success
     discovery_doc = JSON.parse(@response.body)
-    assert_equal '1.0.0-stable', discovery_doc['package_version']
+    assert_equal '1.0.0-stable', discovery_doc['packageVersion']
   end
 
   test "empty disable_api_methods" do

-----------------------------------------------------------------------


hooks/post-receive
--