Project

General

Profile

Refactor #20033

Avoid redefining SEEDED_TEMPLATES and SEEDED_PARTITION_TABLES constants

Added by Marek Hulán over 1 year ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
DB migrations
Target version:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Since db migration loads these constants from seeds.d and seed task does not use require but rather load, both constants can be redefined resulting in Ruby printing warning. The issue was introduced by #18639


Related issues

Related to Foreman - Bug #18639: All templates that are shipped with Foreman should be locked by defaultClosed2017-02-23

Associated revisions

Revision bbb1b90c (diff)
Added by Marek Hulán over 1 year ago

Fixes #20033 - load seed data just once

Revision 597501b4 (diff)
Added by Lukas Zapletal over 1 year ago

Fixes #20143 - changed kind for Preseed default PXEGrub2 (#4630)

  • Fixes #20143 - changed kind for Preseed default PXEGrub2
  • Refs #20033 - renamed contents of templates seeds

Revision dc77e8f6 (diff)
Added by Lukas Zapletal over 1 year ago

Fixes #20143 - changed kind for Preseed default PXEGrub2 (#4630)

  • Fixes #20143 - changed kind for Preseed default PXEGrub2
  • Refs #20033 - renamed contents of templates seeds

(cherry picked from commit 597501b460f4d1131870f6d3e7f4fbae5d6db830)

History

#1 Updated by Marek Hulán over 1 year ago

  • Related to Bug #18639: All templates that are shipped with Foreman should be locked by default added

#2 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4596 added

#3 Updated by Anonymous over 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Marek Hulán over 1 year ago

  • Legacy Backlogs Release (now unused) set to 240

#5 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/4630 added

Also available in: Atom PDF