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

Spec for configuration manager providers sort by url #1086

Merged
merged 1 commit into from Apr 21, 2017
Merged

Spec for configuration manager providers sort by url #1086

merged 1 commit into from Apr 21, 2017

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Apr 19, 2017 •
edited

Spec for configuration manager providers sort by url

Depends on ManageIQ/manageiq#14810
https://bugzilla.redhat.com/show_bug.cgi?id=1384330

@lgalis lgalis mentioned this pull request Apr 19, 2017
Merged
@miq-bot miq-bot added the wip label Apr 19, 2017
@miq-bot
Copy link
Member

miq-bot commented Apr 19, 2017

Checked commit lgalis@07ca7d7 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 🍪

@lgalis
Copy link
Contributor Author

lgalis commented Apr 19, 2017

@miq-bot add_label bug, test, fine/yes, euwe/yes

@lgalis lgalis closed this Apr 20, 2017
@lgalis lgalis reopened this Apr 20, 2017
@lgalis lgalis changed the title [WIP] Spec for configuration manager providers sort by url Spec for configuration manager providers sort by url Apr 20, 2017
@miq-bot miq-bot removed the wip label Apr 20, 2017
@h-kataria h-kataria added this to the Sprint 59 Ending Apr 24, 2017 milestone Apr 21, 2017
@h-kataria h-kataria merged commit 7769b5c into ManageIQ:master Apr 21, 2017
@lgalis lgalis deleted the foreman_url_sort_spec branch April 22, 2017 03:43
simaishi pushed a commit that referenced this pull request Apr 24, 2017
@h-kataria @simaishi
Merge pull request #1086 from lgalis/foreman_url_sort_spec
537fe10 
Spec for configuration manager providers sort by url
(cherry picked from commit 7769b5c)

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

Fine backport details:

$ git log -1
commit 537fe10c64b8286630ecd0c12e65c057ded94040
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Fri Apr 21 17:30:39 2017 -0400

    Merge pull request #1086 from lgalis/foreman_url_sort_spec
    
    Spec for configuration manager providers sort by url
    (cherry picked from commit 7769b5cad7cb8623009de2e6e5573cf68a92f590)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1444180

@simaishi
Copy link
Contributor

simaishi commented May 4, 2017

Euwe backport (to manageiq repo) details:

$ git log -1
commit 4fc6b137b5bc3dfc7e206271a100fe1b9e003ef1
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Fri Apr 21 17:30:39 2017 -0400

    Merge pull request #1086 from lgalis/foreman_url_sort_spec
    
    Spec for configuration manager providers sort by url
    (cherry picked from commit 7769b5cad7cb8623009de2e6e5573cf68a92f590)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1444182

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@h-kataria h-kataria

Labels
bug euwe/backported fine/backported test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@lgalis @miq-bot @simaishi @h-kataria