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

[AMBARI-22995] Remove deprecated Upgrade Packs #365

Closed
wants to merge 27 commits into from
Closed

[AMBARI-22995] Remove deprecated Upgrade Packs #365

wants to merge 27 commits into from

Conversation

ncole
Copy link
Contributor

@ncole ncole commented Feb 14, 2018

What changes were proposed in this pull request?

Remove upgrade packs and config packs that all adhere to the old upgrade packs. This is to make way for upgrade pack changes to support the Mpack world.

How was this patch tested?

Unit tests that don't break any more than what was already there in this feature branch. Many tests were removed which ended up being an improvement of test results.

[ERROR] Tests run: 5027, Failures: 25, Errors: 19, Skipped: 37
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 27:55 min
[INFO] Finished at: 2018-02-14T16:52:58-05:00
[INFO] Final Memory: 100M/1902M
[INFO] ------------------------------------------------------------------------

Jetly-Jaimin and others added 27 commits October 12, 2017 10:58
…Download screen. (Jason Golieb via jonathanhurley)
….e. local) repo support (Jason Golieb via jonathanhurley)
# Conflicts:
#	ambari-web/app/controllers/installer.js
#	ambari-web/app/mappers/stack_mapper.js
#	ambari-web/app/messages.js
#	ambari-web/app/models/repository.js
#	ambari-web/app/styles/application.less
#	ambari-web/app/styles/wizard.less
#	ambari-web/app/templates/installer.hbs
…ple mpacks (#199)

* AMBARI-22802 Styling for install wizard

* Install services from multiple mpacks. Service groups are created with names hard-coded to be <mpackId>-<mpackVersion>.

* Fixed unit tests.

* Moved style changes out of bootstrap-ambari.css

* Additional style tweaks.

* Additional style tweaks.

* Restore line removed by wrong conflict resolution.

* More style tweaks.

* Moved tooltip activation to individual components.

* Merge branch 'branch-feature-AMBARI-14714-ui' into AMBARI-22804_multi-mpack-deployment
#232)

* AMBARI-22802 Styling for install wizard

* Install services from multiple mpacks. Service groups are created with names hard-coded to be <mpackId>-<mpackVersion>.

* Fixed unit tests.

* Moved style changes out of bootstrap-ambari.css

* Additional style tweaks.

* Additional style tweaks.

* Restore line removed by wrong conflict resolution.

* More style tweaks.

* Moved tooltip activation to individual components.

* Implemented mpack and service filtering.

* Unit tests

* Merge branch 'branch-feature-AMBARI-14714-ui' into AMBARI-22804_multi-mpack-deployment

# Conflicts:
#	ambari-web/app/controllers/wizard/step7_controller.js
#	ambari-web/app/styles/application.less
#	ambari-web/app/styles/bootstrap_overrides.less
#	ambari-web/app/styles/wizard.less
#	ambari-web/app/templates/wizard/downloadMpacks.hbs
#	ambari-web/app/templates/wizard/selectMpacks.hbs
#	ambari-web/app/templates/wizard/verifyProducts.hbs

* Unit tests.

* Unit tests and minor tweaks.

* Unit tests.
* [AMBARI-22872] Register/Confirm Host for modular deployment.

* Moved removeselectedhost function to the view and added a few tests

* Minor tweaks
* Implemented "view log" feature for mpack downloads.

* Removed .only

* Fix typo.
Copy link
Member

@jonathan-hurley jonathan-hurley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 to removing the old upgrade packs (especially since we're not shipping them anymore.

@ncole
Copy link
Contributor Author

ncole commented Feb 15, 2018

There's something wrong with this pull request. Working on a different one.

@ncole ncole closed this Feb 15, 2018
@ncole ncole deleted the AMBARI-22995 branch February 15, 2018 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants