Skip to content

AMBARI-22413 fix tar_archive.archive_directory_dereference (benyoka)#196

Merged
benyoka merged 2 commits into
apache:branch-2.6from
benyoka:AMBARI-22413-branch-2.6
Jan 26, 2018
Merged

AMBARI-22413 fix tar_archive.archive_directory_dereference (benyoka)#196
benyoka merged 2 commits into
apache:branch-2.6from
benyoka:AMBARI-22413-branch-2.6

Conversation

@benyoka
Copy link
Copy Markdown
Contributor

@benyoka benyoka commented Jan 25, 2018

Same as #195 but for branch-2.6

What changes were proposed in this pull request?

Fix archive_directory_dereference in tar_archive.py that caused a customer issue with upgrading Falcon.

Patch was once reviewed and approved on review board: https://reviews.apache.org/r/63725/

How was this patch tested?

  • modified python tests
  • all python tests succeeded
  • manually tested Falcon data backup/restore on service stop/start.

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 25, 2018

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

@benyoka
Copy link
Copy Markdown
Contributor Author

benyoka commented Jan 26, 2018

retest this please

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 26, 2018

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

@benyoka
Copy link
Copy Markdown
Contributor Author

benyoka commented Jan 26, 2018

retest this please

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 26, 2018

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

@benyoka benyoka merged commit e08d056 into apache:branch-2.6 Jan 26, 2018
@benyoka benyoka deleted the AMBARI-22413-branch-2.6 branch March 28, 2018 14:30
mpapirkovskyy pushed a commit to mpapirkovskyy/ambari that referenced this pull request Apr 12, 2019
…pache#196)

(cherry picked from commit e08d056)

Change-Id: I66e9aadb2718bec4f3623395c025fc68777c1155
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants