Project

General

Profile

Actions

Bug #25613

open

Template snapshot tests should use multiple OS families

Added by Tomer Brisker over 5 years ago. Updated about 4 years ago.

Status:
New
Priority:
Normal
Category:
-
Target version:
-
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Currently, all snaps are generated using a RH host, so they don't correctly reflect real use case. https://github.com/theforeman/foreman/blob/develop/test/unit/foreman/renderer/template_snapshot_service.rb#L25


Related issues 1 (0 open1 closed)

Related to Foreman - Bug #25567: Wrong provision method signature for RancherOSClosedTomer BriskerActions
Actions #1

Updated by Lukas Zapletal over 5 years ago

  • Triaged changed from No to Yes

So what is exactly wrong with this? RH is the primary architectury for foreman, at least most devs use RH systems. Do you mean we should run them twice - once for Debian and once for RH system?

Actions #2

Updated by Tomer Brisker over 5 years ago

  • Related to Bug #25567: Wrong provision method signature for RancherOS added
Actions #3

Updated by Tomer Brisker over 5 years ago

That we don't validate the templates work for the correct OS family - this was raised because #25567 was hit because the snapshots missed to catch the fact that rancherOS couldn't render templates at all, since the template snapshot for the rancheros templates was rendered using a rh host.

Actions #4

Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Lukas Zapletal
  • Pull request https://github.com/theforeman/foreman/pull/7140 added
Actions #5

Updated by Lukas Zapletal over 4 years ago

  • Subject changed from template snapshots should use correct os family to Template snapshot tests should use multiple OS families
  • Category deleted (Tests)
  • Assignee deleted (Lukas Zapletal)

And PXE loader as well.

Actions #6

Updated by The Foreman Bot over 4 years ago

  • Assignee set to Lukas Zapletal
Actions #7

Updated by The Foreman Bot about 4 years ago

  • Status changed from Ready For Testing to New
  • Pull request deleted (https://github.com/theforeman/foreman/pull/7140)
Actions

Also available in: Atom PDF