[1/2] mesos git commit: Allowed empty resource provider selector in `UriDiskProfileAdaptor`.
Repository: mesos Updated Branches: refs/heads/1.5.x d0b303e41 -> 537c54c5b Allowed empty resource provider selector in `UriDiskProfileAdaptor`. Review: https://reviews.apache.org/r/65718/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/b350b369 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/b350b369 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/b350b369 Branch: refs/heads/1.5.x Commit: b350b36948f366254299326f7fe0f4f199c37734 Parents: d0b303e Author: Chun-Hung Hsiao Authored: Thu Feb 22 16:32:00 2018 -0800 Committer: Jie Yu Committed: Thu Feb 22 16:47:09 2018 -0800 -- src/resource_provider/storage/disk_profile_utils.cpp | 6 -- src/tests/disk_profile_tests.cpp | 11 --- 2 files changed, 17 deletions(-) -- http://git-wip-us.apache.org/repos/asf/mesos/blob/b350b369/src/resource_provider/storage/disk_profile_utils.cpp -- diff --git a/src/resource_provider/storage/disk_profile_utils.cpp b/src/resource_provider/storage/disk_profile_utils.cpp index a243bd6..3e8b0c5 100644 --- a/src/resource_provider/storage/disk_profile_utils.cpp +++ b/src/resource_provider/storage/disk_profile_utils.cpp @@ -107,12 +107,6 @@ static Option validateSelector( const auto& selector = profileManifest.resource_provider_selector(); - if (selector.resource_providers_size() == 0) { -return Error( -"Expecting one or more 'resource_providers' for " -"ResourceProviderSelector"); - } - foreach (const auto& resourceProvider, selector.resource_providers()) { if (resourceProvider.type().empty()) { return Error( http://git-wip-us.apache.org/repos/asf/mesos/blob/b350b369/src/tests/disk_profile_tests.cpp -- diff --git a/src/tests/disk_profile_tests.cpp b/src/tests/disk_profile_tests.cpp index 538208e..a82c37d 100644 --- a/src/tests/disk_profile_tests.cpp +++ b/src/tests/disk_profile_tests.cpp @@ -205,17 +205,6 @@ TEST_F(UriDiskProfileTest, ParseInvalids) } })~", -// Empty 'resource_providers' in 'resource_provider_selector'. -R"~({ -"profile_matrix" : { - "profile" : { -"resource_provider_selector" : { - "resource_providers" : [] -} - } -} - })~", - R"~({ "profile_matrix" : { "profile" : {
mesos git commit: Allowed empty resource provider selector in `UriDiskProfileAdaptor`.
Repository: mesos Updated Branches: refs/heads/master 9635d4a2d -> 2ddc3c6a1 Allowed empty resource provider selector in `UriDiskProfileAdaptor`. Review: https://reviews.apache.org/r/65718/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/2ddc3c6a Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/2ddc3c6a Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/2ddc3c6a Branch: refs/heads/master Commit: 2ddc3c6a18de01eb51448779de1dff311d5dbe51 Parents: 9635d4a Author: Chun-Hung Hsiao Authored: Thu Feb 22 16:32:00 2018 -0800 Committer: Jie Yu Committed: Thu Feb 22 16:32:00 2018 -0800 -- src/resource_provider/storage/disk_profile_utils.cpp | 6 -- src/tests/disk_profile_tests.cpp | 11 --- 2 files changed, 17 deletions(-) -- http://git-wip-us.apache.org/repos/asf/mesos/blob/2ddc3c6a/src/resource_provider/storage/disk_profile_utils.cpp -- diff --git a/src/resource_provider/storage/disk_profile_utils.cpp b/src/resource_provider/storage/disk_profile_utils.cpp index a243bd6..3e8b0c5 100644 --- a/src/resource_provider/storage/disk_profile_utils.cpp +++ b/src/resource_provider/storage/disk_profile_utils.cpp @@ -107,12 +107,6 @@ static Option validateSelector( const auto& selector = profileManifest.resource_provider_selector(); - if (selector.resource_providers_size() == 0) { -return Error( -"Expecting one or more 'resource_providers' for " -"ResourceProviderSelector"); - } - foreach (const auto& resourceProvider, selector.resource_providers()) { if (resourceProvider.type().empty()) { return Error( http://git-wip-us.apache.org/repos/asf/mesos/blob/2ddc3c6a/src/tests/disk_profile_tests.cpp -- diff --git a/src/tests/disk_profile_tests.cpp b/src/tests/disk_profile_tests.cpp index 1bbbde0..577ccfe 100644 --- a/src/tests/disk_profile_tests.cpp +++ b/src/tests/disk_profile_tests.cpp @@ -205,17 +205,6 @@ TEST_F(UriDiskProfileTest, ParseInvalids) } })~", -// Empty 'resource_providers' in 'resource_provider_selector'. -R"~({ -"profile_matrix" : { - "profile" : { -"resource_provider_selector" : { - "resource_providers" : [] -} - } -} - })~", - R"~({ "profile_matrix" : { "profile" : {