Skip to content

[AMBARI-22806] Unable to delete files from HDFS using Ambari File View …#139

Merged
nitirajrathore merged 1 commit into
apache:trunkfrom
venkatasairamlanka:AMBARI-22806-trunk
Jan 22, 2018
Merged

[AMBARI-22806] Unable to delete files from HDFS using Ambari File View …#139
nitirajrathore merged 1 commit into
apache:trunkfrom
venkatasairamlanka:AMBARI-22806-trunk

Conversation

@venkatasairamlanka
Copy link
Copy Markdown

…when Ambari Views is accessed via Knox(Venkata Sairam)

What changes were proposed in this pull request?

The HTTP request is changed from delete to post as knox will discard http body for delete call.

How was this patch tested?

Manual testing done to verify and sanity tests are run for files view.

Please review Ambari Contributing Guide before opening a pull request.

@gauravn7 @pallavkul @nitirajrathore @padmapriyanitt @r-kamath @dipayanb

…when Ambari Views is accessed via Knox(Venkata Sairam)
@gauravn7
Copy link
Copy Markdown
Contributor

LGTM

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 18, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/175/
Test PASSed.

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 18, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/183/
Test FAILed.
Test FAILured.

@vivekratnavel vivekratnavel changed the title AMBARI-22806.Unable to delete files from HDFS using Ambari File View … [AMBARI-22806] Unable to delete files from HDFS using Ambari File View … Jan 18, 2018
@pallavkul-zz
Copy link
Copy Markdown

LGTM.

@rlevas
Copy link
Copy Markdown
Contributor

rlevas commented Jan 20, 2018

retest this please

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 20, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/218/
Test FAILed.
Test FAILured.

@Jetly-Jaimin
Copy link
Copy Markdown
Contributor

retest this please

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 20, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/220/
Test FAILed.
Test FAILured.

@venkatasairamlanka
Copy link
Copy Markdown
Author

[INFO] Ambari Metrics Assembly ............................ SUCCESS [01:44 min]
[INFO] Ambari Service Advisor ............................. SUCCESS [ 0.413 s]
[INFO] Ambari Server ...................................... SUCCESS [26:10 min]
[INFO] Ambari Functional Tests ............................ SUCCESS [ 0.715 s]
[INFO] Ambari Agent ....................................... FAILURE [ 52.498 s]
[INFO] Ambari Client ...................................... SUCCESS [ 0.039 s]
[INFO] Ambari Python Client ............................... SUCCESS [ 1.609 s]
[INFO] Ambari Groovy Client ............................... SUCCESS [ 6.524 s]
[INFO] Ambari Shell ....................................... SUCCESS [ 0.038 s]
[INFO] Ambari Python Shell ................................ SUCCESS [ 0.525 s]
[INFO] Ambari Groovy Shell ................................ SUCCESS [ 4.052 s]

This is a just a failure in Ambari-Agent which is unrelated to my changes.

@nitirajrathore nitirajrathore merged commit 980bf24 into apache:trunk Jan 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants