Skip to content

Commit

Permalink
github-ci: follow-up; add support for python 3.8+, remove xattrs inst…
Browse files Browse the repository at this point in the history
…allation

Signed-off-by: Abhishek Gaikwad <[email protected]>
  • Loading branch information
gaikwadabhishek committed Sep 10, 2024
1 parent 6a9c9fc commit 34e3b60
Show file tree
Hide file tree
Showing 7 changed files with 4 additions and 37 deletions.
6 changes: 0 additions & 6 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,6 @@ jobs:
- name: Checkout code
uses: actions/checkout@v4

- name: Setup additional system libraries
if: startsWith(matrix.os, 'ubuntu')
run: |
sudo apt update
sudo apt install -y xattr attr
- name: Build AIStore on ${{ matrix.os }}
run: |
export GOPATH="$(go env GOPATH)"
Expand Down
8 changes: 1 addition & 7 deletions .github/workflows/test-python-authn.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
matrix:
go-version: ['1.22.x']
os: [ubuntu-latest, macos-latest]
python-version: ['3.11']
python-version: ['3.8','3.9','3.10','3.11']
runs-on: ${{ matrix.os }}
steps:
- name: Install Go
Expand All @@ -32,12 +32,6 @@ jobs:
sudo rm -rf /opt/ghc
sudo rm -rf /usr/local/share/boost
- name: Setup additional system libraries
if: startsWith(matrix.os, 'ubuntu')
run: |
sudo apt update
sudo apt install -y xattr attr
- name: Run AIStore
env:
GOPATH: ${{ secrets.GOPATH }}
Expand Down
8 changes: 1 addition & 7 deletions .github/workflows/test-python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
matrix:
go-version: ['1.22.x']
os: [ubuntu-latest, macos-latest]
python-version: ['3.11']
python-version: ['3.8','3.9','3.10','3.11']
runs-on: ${{ matrix.os }}
steps:
- name: Install Go
Expand All @@ -23,12 +23,6 @@ jobs:
- name: Checkout code
uses: actions/checkout@v4

- name: Setup additional system libraries
if: startsWith(matrix.os, 'ubuntu')
run: |
sudo apt update
sudo apt install -y xattr attr
- name: Remove unnecessary directories to free up space
run: |
sudo rm -rf /usr/local/.ghcup
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/test-short.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,12 +24,6 @@ jobs:
- name: Checkout code
uses: actions/checkout@v4

- name: Setup additional system libraries
if: startsWith(matrix.os, 'ubuntu')
run: |
sudo apt update
sudo apt install -y xattr attr
- name: Run and test AIStore
run: |
export GOPATH="$(go env GOPATH)"
Expand Down
8 changes: 1 addition & 7 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,13 +24,7 @@ runs:
uses: actions/setup-go@v3
with:
go-version: 1.22.x
- name: Setup additional system libraries
shell: bash
run: |
if [ "$RUNNER_OS" == "Linux" ]; then
sudo apt update
sudo apt install -y xattr attr
fi

- name: Build and deploy AIStore locally
shell: bash
run: |
Expand Down
3 changes: 0 additions & 3 deletions docs/getting_started.md
Original file line number Diff line number Diff line change
Expand Up @@ -494,9 +494,6 @@ os.environ['PATH'] += ':/usr/local/go/bin:/content/go/bin'
After installing Go, you can proceed to install and run AIStore:

```bash
# Update package lists and install dependencies
!sudo apt update
!sudo apt install -y xattr attr

# Clone the AIStore repository
!git clone https://github.com/NVIDIA/aistore.git
Expand Down
2 changes: 1 addition & 1 deletion python/aistore/sdk/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ The project is, essentially, a Python port of the [AIS Go APIs](https://aistore.

The SDK also includes the `authn` sub-package for managing authentication, users, roles, clusters, and tokens. For more details, refer to the [AuthN sub-package README](https://github.com/NVIDIA/aistore/blob/main/python/aistore/sdk/authn/README.md).

> Only Python 3.x (version 3.6 or later) is currently supported.
> Support is currently limited to Python 3.x, with a minimum requirement of version 3.8 or later.
---

Expand Down

0 comments on commit 34e3b60

Please sign in to comment.