Project

General

Profile

Actions

Bug #28124

closed

Compute Resource create does not validates the zone name and creates the Compute Resource.

Added by Jitendra Yejare about 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
High
Category:
Compute resources - GCE
Target version:
-
Fixed in Releases:

Description

Description:
-------------------
Compute Resource create does not validate the zone name for GCE CR and creates the Compute Resource. If I provide any non-existence zone name while creating the CR, it doesn't validate if the zone is present or not and creates the CR.

How Reproducible:
-------------------
Always

Steps:
-------------------
1. Create a Compute Resource from a hammer with the zone(that doesn't exist in GCE).

Actual Behavior:
-------------------
1. The Compute Resource is created
2. No sign of error or warning for non-existence of zone.

Expected Behavior:
--------------------
1. Either the Compute Resource should not be created or should show a warning and create CR with zone.

Actions #1

Updated by Oleh Fedorenko about 5 years ago

Hi Jitendra,

Is it possible to create a CR the same way but with direct API calls?

Actions #2

Updated by Shira Maximov about 5 years ago

  • Project changed from Hammer CLI to Foreman
  • Category changed from Compute resources to API

As we talked at the hammer grooming meeting, the validation should be in foreman core.

Actions #3

Updated by Kavita Gaikwad about 5 years ago

  • Category changed from API to Compute resources - GCE
  • Assignee set to Kavita Gaikwad
Actions #4

Updated by Kavita Gaikwad about 5 years ago

  • Subject changed from [Hammer] Compute Resource create does not validates the zone name and creates the Compute Resource. to Compute Resource create does not validates the zone name and creates the Compute Resource.
Actions #5

Updated by Kavita Gaikwad about 5 years ago

  • Bugzilla link set to 1785284
Actions #6

Updated by The Foreman Bot about 5 years ago

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

Updated by Kavita Gaikwad about 5 years ago

  • Status changed from Ready For Testing to Closed
Actions #8

Updated by The Foreman Bot about 5 years ago

  • Pull request https://github.com/theforeman/foreman/pull/7333 added
Actions #9

Updated by Tomer Brisker about 5 years ago

  • Fixed in Releases 1.24.1, 2.0.0 added
Actions

Also available in: Atom PDF