Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add groups for PhysicalServer#show page #1294

Merged
merged 1 commit into from
May 9, 2017

Conversation

gabrielsvinha
Copy link
Contributor

@gabrielsvinha gabrielsvinha commented May 5, 2017

This Pull Request add the groups in physical server controller to show in Physical Server summary page.

Adds the following groups:

  • properties;
  • relationships;

Depends on: #1295 #1281

@gabrielsvinha
Copy link
Contributor Author

@miq-bot add_labels graphics, textual summaries

@miq-bot
Copy link
Member

miq-bot commented May 5, 2017

Checked commit gabrielsvinha@4055129 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@gabrielsvinha
Copy link
Contributor Author

@miq-bot add_label compute/physical infrastructure

@martinpovolny
Copy link
Member

The travis failure seems related.

@gabrielsvinha
Copy link
Contributor Author

gabrielsvinha commented May 5, 2017

@martinpovolny I believe it's failing due to the lack of changes made in #1295

It shows
undefined method textual_group_properties' which is fixed in the PR above.

@AparnaKarve
Copy link
Contributor

AparnaKarve commented May 8, 2017

@gabrielsvinha Ideally #1295 should have included what you currently have in this PR.
Try closing and opening the PR which should re-run Travis and take care of the failed spec.

@gabrielsvinha
Copy link
Contributor Author

Closing to run travis tests.

@AparnaKarve
Copy link
Contributor

@dclarizio Spec failures are unrelated, so we can merge this.

@dclarizio dclarizio merged commit e3bad4d into ManageIQ:master May 9, 2017
@dclarizio dclarizio added this to the Sprint 61 Ending May 22, 2017 milestone May 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants