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

Add assert_privileges() to download_migration_log() #1007

Merged
merged 1 commit into from
Jul 25, 2019
Merged

Add assert_privileges() to download_migration_log() #1007

merged 1 commit into from
Jul 25, 2019

Conversation

mzazrivec
Copy link
Contributor

@mzazrivec mzazrivec added the bz Issues filed by QE or having a BZ label Jul 24, 2019
@mzazrivec mzazrivec requested a review from martinpovolny July 24, 2019 15:06
@miq-bot
Copy link
Member

miq-bot commented Jul 24, 2019

Checked commit mzazrivec@98eeee6 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Rubocop - missing config files

@martinpovolny martinpovolny merged commit 1a69dd9 into ManageIQ:master Jul 25, 2019
@martinpovolny martinpovolny self-assigned this Jul 25, 2019
@martinpovolny martinpovolny added this to the Sprint 117 Ending Aug 5, 2019 milestone Jul 25, 2019
@mzazrivec mzazrivec deleted the add_assert_privileges_to_download_migration_log branch July 25, 2019 08:31
simaishi pushed a commit that referenced this pull request Jul 25, 2019
@martinpovolny @simaishi
Merge pull request #1007 from mzazrivec/add_assert_privileges_to_down…
01a0c2b 
…load_migration_log

Add assert_privileges() to download_migration_log()

(cherry picked from commit 1a69dd9)
@simaishi
Copy link
Contributor

Ivanchuk backport details:

$ git log -1
commit 01a0c2bb22a793054ea5b79100743217f14bf797
Author: Martin Povolny <[email protected]>
Date:   Thu Jul 25 09:58:49 2019 +0200

    Merge pull request #1007 from mzazrivec/add_assert_privileges_to_download_migration_log
    
    Add assert_privileges() to download_migration_log()
    
    (cherry picked from commit 1a69dd915eb7e816283c86ff466e07fda9fe48bf)

simaishi pushed a commit that referenced this pull request Jul 25, 2019
@martinpovolny @simaishi
Merge pull request #1007 from mzazrivec/add_assert_privileges_to_down…
6f294f5 
…load_migration_log

Add assert_privileges() to download_migration_log()

(cherry picked from commit 1a69dd9)

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

Hammer backport details:

$ git log -1
commit 6f294f5ee444da004152e6cf34499c7969e5f7fa
Author: Martin Povolny <[email protected]>
Date:   Thu Jul 25 09:58:49 2019 +0200

    Merge pull request #1007 from mzazrivec/add_assert_privileges_to_download_migration_log
    
    Add assert_privileges() to download_migration_log()
    
    (cherry picked from commit 1a69dd915eb7e816283c86ff466e07fda9fe48bf)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1718081

@simaishi simaishi mentioned this pull request Sep 10, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martinpovolny martinpovolny martinpovolny approved these changes

Assignees

@martinpovolny martinpovolny

Labels
bug bz Issues filed by QE or having a BZ hammer/backported ivanchuk/backported
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mzazrivec @miq-bot @simaishi @martinpovolny