Project

General

Profile

Bug #27157

[UI] External IP value is not being saved for GCE Compute Profile

Added by Kavita Gaikwad 4 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Category:
Compute resources - GCE
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1721871

Description of problem:
The External IP check is not being saved for GCE Compute Profile.

Version-Release number of selected component (if applicable):
Satellite 6.6 snap 7

How reproducible:
Always

Steps to Reproduce:
1. Create GCE Compute Resource, load data from GCE and test connection successfully.
2. Create a new compute profile for GCE.
3. While editing compute profile, select External IP checkbox and save the compute profile.
4. Reopen the GCE compute profile and verify External IP checkbox is checked.

Actual results:
1. The External IP check is not saved(checked in this case) in GCE Compute Profile.

Expected results:
The External IP check should be saved for GCE Compute Profile.

Additional info:

Associated revisions

Revision 862f09f2 (diff)
Added by Kavita Gaikwad 3 months ago

Fixes #27157 - persist associate_external_ip value in GCE (#6873)

Revision 2237a3ed (diff)
Added by Ewoud Kohl van Wijngaarden 3 months ago

Refs #27157 - Fix rubocop

f2ee562ece5d5084f39160524ac57b23b7bde251 flipped the rubocop style but
862f09f24878b0e1c66beca094fc37bb291911c0 wasn't retested after and broke
on merge.

History

#1 Updated by Kavita Gaikwad 4 months ago

  • Assignee set to Kavita Gaikwad

#2 Updated by The Foreman Bot 4 months ago

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

#3 Updated by Ondřej Ezr 3 months ago

  • Triaged changed from No to Yes
  • Fixed in Releases 1.23.0 added

#4 Updated by Kavita Gaikwad 3 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot 3 months ago

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

#6 Updated by The Foreman Bot 3 months ago

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

#7 Updated by Michael Moll 3 months ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/6898)

Also available in: Atom PDF