Project

General

Custom queries

Profile

Actions
Copy link

Bug #14797

closed

Medium validation wrong with Katello and foreman_bootdisk

Added by Timo Goebel about 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Lukas Zapletal
Difficulty:
Triaged:
Yes
Bugzilla link:
1659042
Pull request:
https://github.com/theforeman/foreman_bootdisk/pull/73
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Both foreman_bootdisk and Katello overwrite validate_media? in the Host Model and the wrong order is applied.

What you can try as a workaround is the following:
Remove or comment

alias_method_chain :validate_media?, :bootdisk
from app/models/concerns/foreman_bootdisk/host_ext.rb

See:
https://github.com/Katello/katello/blob/b92a833e38fa3b39deff26839b219c0b7da53bbb/app/models/katello/concerns/host_managed_extensions.rb#L34-L36
https://github.com/theforeman/foreman_bootdisk/blob/master/app/models/concerns/foreman_bootdisk/host_ext.rb#L36-L38

Related issues 5 (1 open — 4 closed)

Related to Foreman Remote Execution - Refactor #21205: Replace alias_method_chain with Module prepend ClosedIvan Necas10/05/2017Actions
Related to Katello - Bug #21243: Rails 5 - Replace alias_method_chain with Module prependClosedJustin Sherrill10/09/2017Actions
Related to Foreman - Refactor #20957: Replace alias_method_chain with Module prepend Closed09/16/2017Actions
Related to Foreman - Refactor #22219: Refactor validate_media? to be hookableNew01/10/2018Actions
Related to Foreman - Refactor #25680: Allow plugins to override validate_media?ClosedLukas ZapletalActions
#1

Updated by Lukas Zapletal over 7 years ago

  • Related to Refactor #21205: Replace alias_method_chain with Module prepend added
#3

Updated by Lukas Zapletal over 7 years ago

  • Bugzilla link set to 1513327
#4

Updated by Lukas Zapletal over 7 years ago

  • Related to Bug #21243: Rails 5 - Replace alias_method_chain with Module prepend added
#5

Updated by Lukas Zapletal over 7 years ago

  • Related to Refactor #20957: Replace alias_method_chain with Module prepend added
#6

Updated by Lukas Zapletal over 7 years ago

  • Status changed from New to Need more information
#7

Updated by The Foreman Bot over 7 years ago

  • Status changed from Need more information to Ready For Testing
  • Assignee set to Lukas Zapletal
  • Pull request https://github.com/theforeman/foreman_bootdisk/pull/55 added
#8

Updated by Lukas Zapletal over 7 years ago

  • Related to Refactor #22219: Refactor validate_media? to be hookable added
#9

Updated by Lukas Zapletal over 6 years ago

  • Status changed from Ready For Testing to Need more information
  • Triaged changed from No to Yes
#11

Updated by Lukas Zapletal over 6 years ago

  • Related to Refactor #25680: Allow plugins to override validate_media? added
#12

Updated by The Foreman Bot over 6 years ago

  • Status changed from Need more information to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_bootdisk/pull/73 added
#13

Updated by Lukas Zapletal over 6 years ago

  • Bugzilla link changed from 1513327 to 1659042
  • Pull request deleted (https://github.com/theforeman/foreman_bootdisk/pull/55)
#14

Updated by Anonymous over 6 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link

Also available in: Atom PDF