updating the version of Jackson to 2.7 #49
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(cherry picked from commit d6c8398) (cherry picked from commit 4a47d5491e9ad682a8ee2769f262f014ebcd1cb5)
Added list_dfs_services to endpoints/cluster.py's ApiCluster, to match API addition in OPSAPS-31495.
Testing:
- new unit test
- verified with manual test against CM via a simple test script:
---
from cm_api.api_client import ApiResource
api=ApiResource("nightly-1.vpc.cloudera.com", username="foo", password="bar")
cluster=api.get_cluster("Cluster 1")
apiList=cluster.list_dfs_services(view="export")
print ("%s" %(apiList))
---
(cherry picked from commit adef14d)
(cherry picked from commit e40c6259f119cc9067db9d03b092ad0e72ac5f14)
in the exported template Added cardinality attribute to host template in cluster template (cherry picked from commit 0c8595d) (cherry picked from commit 5fd588f77b0300c2bc25e6ebf508f66e338d1f22)
(cherry picked from commit e6bc828) (cherry picked from commit 44c479034092ff51c6f206f4101a156b086f7757)
- Fixed the api (cherry picked from commit 2151df0) (cherry picked from commit 47531a5d9c4670aac1c8089d4c76774ec5c5e73a)
Nothing special. It's been MIA for a while.
>>> from cm_api.api_client import ApiResource
>>> api=ApiResource("nightly-1.gce.cloudera.com", username="admin", password="admin")
>>> cluster=api.get_cluster("Cluster 1")
>>> cmd = cluster.pools_refresh()
(cherry picked from commit c385539)
(cherry picked from commit 3b6cb12bb0ff730d14bf847d3b4c2f96a4c39d17)
Since it's actually been around for a while. (cherry picked from commit ff09be8) (cherry picked from commit 8846594073fe12823afb3ef17757dd8cc823debd)
|
this pull request handles issue #35 , |
matthew-dailey
added a commit
to matthew-dailey/cm_api
that referenced
this pull request
Aug 14, 2017
* One-line fix * Similar to Issue cloudera#35 and PR cloudera#49, but without the changes to the Jackson dependency itself
matthew-dailey
added a commit
to matthew-dailey/cm_api
that referenced
this pull request
Aug 14, 2017
* Similar to Issue cloudera#35 and PR cloudera#49, but without the changes to the Jackson dependency itself
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Jackson 2.1 is a pretty old version. Also, it appears that the CM is dependent upon a deprecated class which is not a great practice. These changes work and all tests pass.