-
Notifications
You must be signed in to change notification settings - Fork 104
ENT-10962: Fixed mission portal restoring changes from /var/cfengine/share/GUI #2796
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
craigcomstock
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sweet.
|
@cf-bottom jenkins, please. |
|
Sure, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/10154/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-10154/ |
7ea901a to
dd91713
Compare
|
@cf-bottom jenkins, please. |
|
Sure, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/10184/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-10184/ |
|
sequential-test failed due to missing cf-support not being able to process core dumps on ubuntu 20 apparently. I will fix in ENT-11106. |
|
acceptance-test problem here was a publishing error for release-data which was fixed last week so I have re-run that job here. |
|
The no-noise failure in sequential-tests maybe is a flake with aws ubuntu repos not being available? Logged a ticket: ENT-11107 So I think your PR is good to go. |
dd91713 to
d03e93c
Compare
|
@cf-bottom jenkins, please. |
|
Alright, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/10255/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-10255/ |
d03e93c to
615fe78
Compare
|
@cf-bottom jenkins, please. |
|
Alright, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/10264/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-10264/ |
|
Sure, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/10268/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-10268/ |
|
made a ticket for your sequential-tests failure, maybe it is a flake? https://northerntech.atlassian.net/browse/ENT-11182 |
Like the existing no_backup_cp this copy_from body is used to copy files locally without making backups but with the additional ability to specify the comparison used. Ticket: ENT-10962 Changelog: commit
…hen modified Prior to this change if Mission Portal application code was modified it would not be returned to the packaged state because the deployed code would have a newer timestamp than the packaged source. This change switches to using binary comparison of Mission Portal to it's packaged source to determine if the deployed application should be reverted. Ticket: ENT-10962 Changelog: Title
615fe78 to
d37dedd
Compare
|
@cf-bottom jenkins, please. |
|
Alright, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/10321/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-10321/ |
|
@craigcomstock this is failing FR distributed cleanup and bootstrap tests, should we go ahead and merged (I know you already approved but I don't want to make new CI problems. Bootstrap failed in relation to log permission change: And FR maybe had issues with SSH? |
seems you could go ahead and merge 👍 |
No description provided.