New split repo - manageiq-ui-classic

ui

#1

We have just completed another HUGE repo split. This time the UI team has extracted the ManageIQ classic UI code into the https://github.com/ManageIQ/manageiq-ui-classic repo. Coming along for the split are all of the UI specific specs as well as the javascript test suite.

If you would like to contribute to the UI code, now it should be much easier for you to create Pull Requests in this isolated repo. @himdel has written up some instructions on how to get started on developing in this new repo. The instructions are at https://github.com/ManageIQ/guides/blob/master/developer_setup/classic_ui_split.md and he will be updating as things progress.

If you have any questions, please ask them here.


#2

Created a PR adding instructions on how to move pull requests over to manageiq-ui-classic: https://github.com/ManageIQ/guides/pull/176

EDIT: the PR is merged, so… https://github.com/ManageIQ/guides/blob/master/developer_setup/classic_ui_split.md


#3

[RESOLVED] Heads up: any relative :path in manageiq/Gemfile will confuse bundler run from in the manageiq-ui-classic/ dir, with unhelpful messages, e.g.:

~/m/manageiq-ui-classic (master)> bundle install
The path `/home/bpaskinc/miq/manageiq-ui-classic/manageiq-gems-pending` does not exist.

Sure, it doesn’t, but why did it expect manageiq-gems-pending under the ui dir?!
The error actually comes from me having this line in manageiq/Gemfile:

gem "manageiq-gems-pending", ">0", :require => 'manageiq-gems-pending', :path => '../../manageiq-gems-pending'

=> Either only use absolute :path, or apply https://github.com/ManageIQ/manageiq-ui-classic/pull/32 [MERGED]


#4

For anyone who followed original instructions and is now getting bundle error:

[!] There was an error parsing `Gemfile.dev.rb`: You cannot specify the same gem twice coming from different sources.
You specified that manageiq-ui-classic (>= 0) should come from source at `.` and source at `/home/bpaskinc/miq/manageiq-ui-classic`
. Bundler cannot continue.

[OUTDATED AGAIN] Solution: Update manageiq/Gemfile.dev.rb to do gem "manageiq-ui-classic" conditionally, as shown in current classic_ui_split.md.


#5

Ok, Gemfile.dev.rb is in place, then because I had to refresh my test environment (openstack reset) I ran bundle exec rake db:drop. While running bin/setup, it fails.

== 20150224192447 AddProviderIdToEms: migrated (0.0006s) ======================

rails aborted!
GoodMigrations::LoadError: Rails attempted to auto-load:

/home/gildub/manageiq/app/models/mixins/uuid_mixin.rb

Is the approach wrong?


#6

Hi, we have a bugfix https://github.com/ManageIQ/manageiq/pull/13450 , but according to the comments it will be reworked.

Anyway, the easiest workaround you can use $ GOOD_MIGRATIONS=skip any_rake_task (ie. in your case rake db:drop)

Hopefully, it helps you :wink:


#7

Thanks @jzigmund that’s what I was missing.

Just to clarify, I suppose until that’s been addressed it doesn’t really make sense to bin/update when coming from an existing running environment/test set since the migration is not executed. That’s okay just have to run bin/setup directly from a clean slate, if I’m correct.