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.

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

Do not nullify the record when saving/cancelling log collection #1258

Merged
merged 1 commit into from May 4, 2017

Conversation

skateman
Copy link
Member

@skateman skateman commented May 4, 2017

@skateman skateman changed the title Do not nullify the @record when saving/cancelling log collection Do not nullify the record when saving/cancelling log collection May 4, 2017
@miq-bot
Copy link
Member

miq-bot commented May 4, 2017

Checked commit skateman@9652a5a with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@martinpovolny martinpovolny merged commit a23d052 into ManageIQ:master May 4, 2017
@martinpovolny martinpovolny self-assigned this May 4, 2017
@martinpovolny martinpovolny added this to the Sprint 60 Ending May 8, 2017 milestone May 4, 2017
@skateman skateman deleted the keep-record-logdepot branch May 4, 2017 11:35
simaishi pushed a commit that referenced this pull request Jun 5, 2017
Do not nullify the record when saving/cancelling log collection
(cherry picked from commit a23d052)

https://bugzilla.redhat.com/show_bug.cgi?id=1444505
@simaishi
Copy link
Contributor

simaishi commented Jun 5, 2017

Fine backport details:

$ git log -1
commit 24948a43599ec4e4a758b00b89a98df0dceeb002
Author: Martin Povolny <mpovolny@redhat.com>
Date:   Thu May 4 13:23:05 2017 +0200

    Merge pull request #1258 from skateman/keep-record-logdepot
    
    Do not nullify the record when saving/cancelling log collection
    (cherry picked from commit a23d052e8a80cb937bc4d427e07ce2d5c17b7ded)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1444505

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.

None yet

4 participants