Skip to content
This repository has been archived by the owner on Oct 10, 2020. It is now read-only.

Atomic/util.py: Add logic to install lookup for shortnames #1010

Closed
wants to merge 1 commit into from

Conversation

baude
Copy link
Member

@baude baude commented May 22, 2017

If a shortname is used to run an image, we need to transform the short
name into the fq-name when doing the lookup in the installed images
data.

Reported in BZ #1454292

Description

Related Bugzillas

  • 1454292

Related Issue Numbers

Pull Request Checklist:

If your Pull request contains new features or functions, tests are required. If the PR is a bug fix and no tests exist, please consider adding some to prevent regressions.

  • Unittests
  • Integration Tests

@baude baude force-pushed the install_shortname branch 2 times, most recently from de005e5 to d86a346 Compare May 22, 2017 19:38
If a shortname is used to run an image, we need to transform the short
name into the fq-name when doing the lookup in the installed images
data.

Reported in BZ #1454292
@baude
Copy link
Member Author

baude commented May 23, 2017

merging based on feedback from @miabbott and others ...

@baude
Copy link
Member Author

baude commented May 23, 2017

@rh-atomic-bot r+ f5e4b38

@rh-atomic-bot
Copy link

⌛ Testing commit f5e4b38 with merge dbc34c7...

@rh-atomic-bot
Copy link

☀️ Test successful - status-redhatci
Approved by: baude
Pushing dbc34c7 to master...

@lslebodn
Copy link

Is there a plan to write a unit/integration test for this regression?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants