Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Cleanup old ubiquity linux service, config files and few old deployment files #221

Merged
merged 1 commit into from
Jul 5, 2018

Conversation

shay-berman
Copy link
Contributor

@shay-berman shay-berman commented Jul 2, 2018

This PR is for removing old files that are no longer relevant in the project. Below the list:

  1. Remove old ubiqutiy linux service concept. From 1.0 we moved to deploy ubiqutiy as k8s deployment instead of regular linux service. Removing the following related files:
    scripts/setup
    scripts/ubiquity-server.env
    scripts/ubiquity.service
    ubiquity-server.conf

  2. Remove old k8s deployment files. Since the full installer is locate in https://github.com/IBM/ubiquity-k8s/tree/master/scripts/installer-for-ibm-storage-enabler-for-containers. Removing the following related files:
    deploy/postgres/master-service.json
    deploy/postgres/replica-service.json
    deploy/postgres/service.json
    deploy/postgres/service_account.json
    deploy/postgres/set.json
    deploy/ubiquity/scbe_ubiquity_deployment.yml
    deploy/ubiquity/ubiquity_deployment.yml
    deploy/ubiquity/ubiquity_namespace.yml
    deploy/ubiquity/ubiquity_service.yml

Same clean up was done on the ubiquity repo for ubiquity k8s -> IBM/ubiquity-k8s#201


This change is Reviewable

@shay-berman shay-berman self-assigned this Jul 2, 2018
@coveralls
Copy link

Coverage Status

Coverage remained the same at 51.479% when pulling 7ccf899 on cleanup_old_installation_method into da0d975 on dev.

@shay-berman shay-berman changed the title Remove old ubiquity linux service, config files and few old deployment files Cleanup old ubiquity linux service, config files and few old deployment files Jul 2, 2018
Copy link
Collaborator

@olgashtivelman olgashtivelman left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: 0 of 13 files reviewed, all discussions resolved (waiting on @hfeish and @olgashtivelman)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants