Skip to content
This repository has been archived by the owner on Jul 13, 2021. It is now read-only.

Parameter -c changes all-xccdf.xml #247

Closed
MajoDurco opened this issue Feb 27, 2017 · 4 comments
Closed

Parameter -c changes all-xccdf.xml #247

MajoDurco opened this issue Feb 27, 2017 · 4 comments

Comments

@MajoDurco
Copy link
Collaborator

Main problem is that if PA run with --contents parameter which takes path to all-xccdf.xml, then this file is changed even though it should not be. When you run PA for a second time it throws the traceback.

Traceback (most recent call last):
  File "/usr/bin/preupg", line 35, in <module>
    sys.exit(main())
  File "/usr/bin/preupg", line 21, in main
    ret = app.run()
  File "/usr/lib/python2.6/site-packages/preupg/application.py", line 748, in run
    retval = self.scan_system()
  File "/usr/lib/python2.6/site-packages/preupg/application.py", line 507, in scan_system
    self.run_scan_process()
  File "/usr/lib/python2.6/site-packages/preupg/application.py", line 260, in run_scan_process
    self.run_scan(function=self.scanning_progress.show_progress)
  File "/usr/lib/python2.6/site-packages/preupg/application.py", line 276, in run_scan
    return ProcessHelper.run_subprocess(cmd, print_output=False, function=function)
  File "/usr/lib/python2.6/site-packages/preupg/utils.py", line 374, in run_subprocess
    function(stdout_data.decode(settings.defenc))
  File "/usr/lib/python2.6/site-packages/preupg/scanning.py", line 101, in show_progress
    self.output_data.append(u'{0}:{1}'.format(self.names[xccdf_rule], stdout_data.strip()))
@MajoDurco MajoDurco added the bug label Feb 27, 2017
@AloisMahdal
Copy link
Collaborator

BTW, this was reported in august downstream as rhbz#1368823.

@bocekm
Copy link
Collaborator

bocekm commented May 20, 2017

I suspect this line

XccdfHelper.update_platform(self.content)

@AloisMahdal
Copy link
Collaborator

qa_ack: downstream suite needs to work around this, so removing it would be nice, and would actually constitute the verification test itself.

@MajoDurco
Copy link
Collaborator Author

Solved #281.

@bocekm bocekm added this to the RHEL 6.9 -> RHEL 7.4 batch 1 milestone Oct 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants