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.

Sign up for GitHub

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

Jump to bottom

tendrl-storage-node: install tendrl-gluster-integration [blocked] #138

Closed
wants to merge 1 commit into from
Closed

tendrl-storage-node: install tendrl-gluster-integration [blocked] #138

wants to merge 1 commit into from

Conversation

mbukatov
Copy link
Collaborator

@mbukatov mbukatov commented Jan 7, 2019 •
edited
Loading

Install tendrl-gluster-integration on all storage nodes. This package
was previously installed during import cluster operation and installing
it here should work no matter if the package is installed during import
or not.

Fixing #126

Blocked by Tendrl/commons#1073

GowthamShanmugam
GowthamShanmugam previously approved these changes Jan 7, 2019
View reviewed changes
Copy link
Contributor

@GowthamShanmugam GowthamShanmugam left a comment

Choose a reason for hiding this comment

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

LGTM

dahorak
dahorak previously approved these changes Jan 7, 2019
View reviewed changes
Copy link
Contributor

@dahorak dahorak left a comment

Choose a reason for hiding this comment

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

LGTM

shtripat
shtripat reviewed Jan 8, 2019
View reviewed changes
Copy link
Member

@shtripat shtripat left a comment

Choose a reason for hiding this comment

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

LGTM

@mbukatov
tendrl-storage-node: install tendrl-gluster-integration
970033f 
Install tendrl-gluster-integration on all storage nodes. This package
was previously installed during import cluster operation and installing
it here should work no matter if the package is installed during import
or not.

Fixing Tendrl#126
@mbukatov mbukatov dismissed stale reviews from dahorak and GowthamShanmugam via 970033f January 8, 2019 15:30
@mbukatov
Copy link
Collaborator Author

mbukatov commented Jan 8, 2019 •
edited
Loading

After running tendrl-ansible on fresh CentOS 7 cluster, I see that gluster integration is installed, but not running nor enabled:

  • Loaded: loaded (...; disabled; vendor preset: disabled)
  • Active: inactive (dead)

So far so good.

Then the import task went through Installing tendrl-gluster-integration on mbukatov-usm1-gl2 tasks, which is good. But in the end, it failed for not immediately clear reason: Failed atom: tendrl.objects.Cluster.atoms.ImportCluster on flow: Import existing Gluster Cluster

@mbukatov
Copy link
Collaborator Author

mbukatov commented Jan 8, 2019

Import without this change passed, so I guess there will be some problem after all.

This means we need to wait for changes to be done in tendrl Tendrl/commons#1073

@shtripat
Copy link
Member

shtripat commented Jan 9, 2019

Import without this change passed, so I guess there will be some problem after all.

This means we need to wait for changes to be done in tendrl Tendrl/commons#1073

@mbukatov ack. Lets test this change with some changes which are required in tendrl code base.

@mbukatov mbukatov changed the title tendrl-storage-node: install tendrl-gluster-integration tendrl-storage-node: install tendrl-gluster-integration [blocked] Mar 15, 2019
@mbukatov mbukatov added the blocked depends on issue in other component or something outside of tendrl-ansible itself label Mar 22, 2019
@mbukatov mbukatov closed this Jun 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shtripat shtripat shtripat left review comments

@GowthamShanmugam GowthamShanmugam GowthamShanmugam left review comments

@dahorak dahorak dahorak left review comments

Assignees
No one assigned
Labels
blocked depends on issue in other component or something outside of tendrl-ansible itself
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mbukatov @shtripat @dahorak @GowthamShanmugam