Skip to content
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.

Sign up for GitHub

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

Jump to bottom

have graph operations populate event log #5171

Merged
merged 4 commits into from
Mar 14, 2017
Merged

have graph operations populate event log #5171

merged 4 commits into from
Mar 14, 2017

Conversation

mtbc
Copy link
Member

@mtbc mtbc commented Mar 10, 2017 •
edited
Loading

What this PR does

Causes GraphModify operations Chgrp2, Chmod2, Chown2, Delete2 add UPDATE, DELETE entries to the eventlog table in the database.

Testing this PR

Perform some graphs operations and watch the eventlog table to ensure that corresponding entries are being added.

Related reading

https://trello.com/c/lOFHl8Na/314-eventlog-delete-entries

@mtbc mtbc closed this Mar 10, 2017
@mtbc mtbc reopened this Mar 10, 2017
@jburel jburel added the develop label Mar 10, 2017
@jburel
Copy link
Member

jburel commented Mar 13, 2017 •
edited
Loading

Tested various graph commands: Chgrp2, Chown2, Delete2
either some UPDATE or DELETE entries are inserted

Will do Chmod2 while reviewing again #5170

@jburel
Copy link
Member

jburel commented Mar 14, 2017

Tested chmod2 today
everything works as expected, entries are added to the eventlog

@jburel jburel merged commit 8949f58 into ome:develop Mar 14, 2017
@mtbc mtbc deleted the populate-event-log branch March 14, 2017 10:54
@jburel jburel added this to the 5.3.0 milestone Mar 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
develop
Projects
None yet
Milestone
5.3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@mtbc @jburel