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

glusterfs: update for new API #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Meoop
Copy link

@Meoop Meoop commented Sep 21, 2020

This PR does the following two parts of work:

  • Adapt to the newer interfaces to compile against release-6 apis.
    In order to work with old versions of GlusterFS, it is required to pass build-tags (glusterfs_legacy_api) to the go command line. For example, Compile with < gluster6:

    go build -tags glusterfs_legacy_api ....
    go test -tags glusterfs_legacy_api ....
    
  • Update travis testing, complete the test in the container to achieve the purpose of testing both the legacy APIs and the enhanced APIs.

Reference PR:

Fix issue: #37

New versions of Glusterfs have an enhanced set of APIs that returns
additional 'struct stat' structures to enable advanced caching of
attributes.  Applications using gfapi would need to adapt to the
newer interfaces to compile against release-6 APIs.

This should fix compilation with v6.0 of the API.
@Meoop
Copy link
Author

Meoop commented Sep 22, 2020

cc @kshlm

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

Successfully merging this pull request may close these issues.

1 participant