Project

General

Profile

Bug #16164

Testing connection gives false positives for multiple compute resources

Added by Ondřej Pražák almost 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Compute resources
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1331550
Description of problem:

Under "Infrastructure" -> "Compute resources" -> "New compute resource",
before I add any details about my compute resource, if I click 'test connection'
I get a green box in the top right hand corner saying "Success: Test connection was successful".
Version-Release number of selected component (if applicable):
Sat 6.2.0 Beta -
Steps to Reproduce:
1.Log into Sat 6
2. Select "Infrastructure" -> "Compute resources" -> "New compute resource"
3. click 'Test connection' for Libvirt or Rackspace, 'Load Datacenters' for VMware
4. Observe success message.
Actual results:
False success message on screen.
Expected results:
Error message, or nothing at all.

Associated revisions

Revision f68a467f (diff)
Added by Tomáš Strachota almost 4 years ago

Fixes #16164 - False positives on testing CR connection

Prevents from showing success message when there's a validation
error in the returned form and adds error handling for
the test connection ajax call.

History

#1 Updated by Ondřej Pražák almost 4 years ago

  • Subject changed from Testing connection gives false positives for multiple compute resources to Testing connection gives false positives for multiple compute resources
  • Category changed from VM management to Compute resources
  • Target version set to 115

#2 Updated by Marek Hulán almost 4 years ago

  • Target version changed from 115 to 1.6.0

#3 Updated by Tomáš Strachota almost 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Tomáš Strachota

#4 Updated by The Foreman Bot almost 4 years ago

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

#5 Updated by Marek Hulán almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 160

#6 Updated by Anonymous almost 4 years ago

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

Also available in: Atom PDF