Skip to content
This repository has been archived by the owner on Mar 26, 2020. It is now read-only.

Bump dependencies version #1123

Merged
merged 3 commits into from
Aug 3, 2018

Conversation

prashanthpai
Copy link
Contributor

  • Bump up golang version required to 1.9.x
    Centos 7 is 1.9.4, latest stable release is 1.9.7
  • etcd: Bump up etcd version required to 3.3.9
    We specifically need fixes related to auto compaction of etcd keyspace.
    These fixes are in an etcd version >= 3.3.3. Centos7 is still on 3.2.22 though.
  • Bump up version of dep as we were using a really old version

@prashanthpai prashanthpai requested a review from kshlm August 3, 2018 05:52
@@ -39,5 +39,27 @@ install_gometalinter() {
rm -f "$GOBINDIR/$LINTER_TARBALL"
}

install_etcd() {
Copy link
Member

Choose a reason for hiding this comment

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

We'd removed this! When?

Also, @sseshasa will be bumping up etcd version in Gopkg.toml in #1068.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I"m not sure if it was present as elastic etcd was the default.

@kshlm
Copy link
Member

kshlm commented Aug 3, 2018

retest this please

1 similar comment
@prashanthpai
Copy link
Contributor Author

retest this please

...as we were using a really old version

Signed-off-by: Prashanth Pai <ppai@redhat.com>
We specifically need fixes related to auto compaction of etcd keyspace.
These fixes are in an etcd version > v3.3.3

More here:
https://github.com/coreos/etcd/blob/master/Documentation/op-guide/maintenance.md

Signed-off-by: Prashanth Pai <ppai@redhat.com>
The latest golang version is 1.10.3. We are bumping golang version
to the previous stable version that is present as a package in base
repo on centos 7.

Signed-off-by: Prashanth Pai <ppai@redhat.com>
@kshlm kshlm merged commit 52747ea into gluster:master Aug 3, 2018
rishubhjain added a commit to rishubhjain/glusterd2 that referenced this pull request Aug 8, 2018
With respect to gluster#1123. Using golang 1.9 or higher instead of 1.8. @prashanthpai Please check if we need to make changes in any other documentation.
kshlm pushed a commit that referenced this pull request Aug 8, 2018
With respect to #1123. Using golang 1.9 or higher instead of 1.8. @prashanthpai Please check if we need to make changes in any other documentation.
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.

2 participants