Skip to content

ENT-5986 LMDB files are now created with correct permissions#4200

Merged
nickanderson merged 1 commit intocfengine:masterfrom
craigcomstock:ENT-5986-v2/master
Jun 4, 2020
Merged

ENT-5986 LMDB files are now created with correct permissions#4200
nickanderson merged 1 commit intocfengine:masterfrom
craigcomstock:ENT-5986-v2/master

Conversation

@craigcomstock
Copy link
Copy Markdown
Contributor

@craigcomstock craigcomstock commented May 28, 2020 •
edited
Loading

Permissions will be 0600 as expected by MPF
cfe_internal/enterprise/CFE_knowledge.cf
Ticket: ENT-5986
Changelog: Title

merge with:
cfengine/masterfiles#1768

@craigcomstock
LMDB files are now created with correct permissions
44130ec 
Permissions will be 0600 as expected by MPF
cfe_internal/enterprise/CFE_knowledge.cf
Ticket: ENT-5986
Changelog: Title
@craigcomstock craigcomstock added the WIP Work in Progress label May 28, 2020
@craigcomstock craigcomstock changed the title LMDB files are now created with correct permissions ENT-5986 LMDB files are now created with correct permissions May 28, 2020
@craigcomstock craigcomstock mentioned this pull request May 29, 2020
Merged
@craigcomstock craigcomstock removed the WIP Work in Progress label May 29, 2020
@craigcomstock craigcomstock requested a review from vpodzime May 29, 2020 18:12
@craigcomstock
Copy link
Copy Markdown
Contributor Author

craigcomstock commented Jun 1, 2020

sparc64_solaris_11 ssh timed out so will retry:

15:00:10 ./04_examples/outputs/check_outputs.cf Skipped (No platform support)
15:45:10 Build timed out (after 45 minutes). Marking the build as failed.

vpodzime
vpodzime approved these changes Jun 2, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@vpodzime vpodzime left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@craigcomstock
Copy link
Copy Markdown
Contributor Author

craigcomstock commented Jun 4, 2020

CI failures are:

  • ppc_64_aix_71 ssh instability

  • upgrade test failed due to javascript file linking error

+ grep error: .27661.new_cfe_agent_runs.log
34.246.201.112	   error: Cannot follow symlink "/var/cfengine/httpd/htdocs/public/scripts/./node_modules/inflight/inflight.js"; it is not owned by root or the user running this process, and the target owner and/or group differs from that of the symlink itself.
34.246.201.112	   error: chmod failed on "/var/cfengine/httpd/htdocs/public/scripts/./node_modules/inflight/inflight.js". (chmod: Link has been severed)
+ fail_test 1

This strangely does not fail in master as-is. Retrying sequential-tests: https://ci.cfengine.com/job/sequential-tests/379/

@nickanderson nickanderson merged commit 0193468 into cfengine:master Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vpodzime vpodzime vpodzime approved these changes

@olehermanse olehermanse olehermanse approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@craigcomstock @vpodzime @olehermanse @nickanderson