Skip to content

AMBARI-25397 Upgrading ambari-logsearch-logfeeder to 2.7.4 rpm gives warnings#3103

Merged
Akhilsnaik merged 1 commit intoapache:branch-2.7from
sziszo:AMBARI-25397-branch-2.7
Oct 18, 2019
Merged

AMBARI-25397 Upgrading ambari-logsearch-logfeeder to 2.7.4 rpm gives warnings#3103
Akhilsnaik merged 1 commit intoapache:branch-2.7from
sziszo:AMBARI-25397-branch-2.7

Conversation

@sziszo
Copy link
Copy Markdown
Contributor

@sziszo sziszo commented Oct 18, 2019

What changes were proposed in this pull request?

Customer is upgrading Ambari Logsearch to 2.7.4, while upgrading the rpm it gives below errors/warnings

ln: failed to create symbolic link '/usr/bin/logfeeder': File exists
rm: cannot remove '/etc/ambari-logsearch-logfeeder': Is a directory
warning: %postun(ambari-logsearch-logfeeder-2.7.4.*noarch) scriptlet failed, exit status 1
The issue can be reproduced in local environment
Also the symlink for file "/usr/bin/logfeeder" does not get created since Ambari 2.7.4
In Ambari 2.7.3 the symlink is created

How was this patch tested?

The patch was tested manually.

Created a higher version RPM, then upgraded and removed the logfeeder packages. After removing the logfeeder the error message did not appear on the output and all related symlinks and directories were removed.

Created 2 new higher RPMs and then upgrading the logfeeder 2x. After the second update, errors / warning messages did not appear on the output.

Copy link
Copy Markdown

@Akhilsnaik Akhilsnaik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@kasakrisz kasakrisz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@asf-ci
Copy link
Copy Markdown

asf-ci commented Oct 18, 2019

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

@Akhilsnaik Akhilsnaik merged commit 470bbe9 into apache:branch-2.7 Oct 18, 2019
@sziszo sziszo deleted the AMBARI-25397-branch-2.7 branch October 21, 2019 09:48
payert pushed a commit to payert/ambari that referenced this pull request Apr 16, 2020
…warnings (santal) (apache#3103)

(cherry picked from commit 470bbe9)

Change-Id: Iba3c9a0b9044e4aeaa9574a8a45db8028b829347
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