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

Fix include domain prefix behavior for catalog entrypoint selection #4166

Merged
merged 1 commit into from
Jun 19, 2018

Conversation

skateman
Copy link
Member

This feature has been accidentally removed, fixing it...

@miq-bot add_reviewer @h-kataria
@miq-bot add_label bug, gaprindashvili/yes

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1592581

@miq-bot
Copy link
Member

miq-bot commented Jun 19, 2018

Checked commit skateman@0f00b7d with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@chrispy1
Copy link

@miq-bot add_label blocker

@dclarizio dclarizio self-assigned this Jun 19, 2018
@dclarizio dclarizio merged commit a57956f into ManageIQ:master Jun 19, 2018
@dclarizio dclarizio added this to the Sprint 89 Ending Jul 2, 2018 milestone Jun 19, 2018
@dclarizio
Copy link

@skateman Can you please follow this up with an upstream test that would have failed when this happened.

@skateman skateman deleted the include-domain-prefix branch June 19, 2018 18:09
simaishi pushed a commit that referenced this pull request Jun 19, 2018
Fix include domain prefix behavior for catalog entrypoint selection
(cherry picked from commit a57956f)

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

Gaprindashvili backport details:

$ git log -1
commit af27e8195c2cf6e9697ba2504b7e86f176134d08
Author: Dan Clarizio <[email protected]>
Date:   Tue Jun 19 11:07:18 2018 -0700

    Merge pull request #4166 from skateman/include-domain-prefix
    
    Fix include domain prefix behavior for catalog entrypoint selection
    (cherry picked from commit a57956f513ed62d21845daf9a3086e99e0c10520)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1592973

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.

6 participants