-
Notifications
You must be signed in to change notification settings - Fork 380
[TEST] Bartek's Lab #629
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
Draft
Ickerday
wants to merge
45
commits into
develop
Choose a base branch
from
dev/BJ/pyproject-toml
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
[TEST] Bartek's Lab #629
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
f2c297a
Try converting project to pyproject.toml and uv
Ickerday 81a4e95
Update project URLs
Ickerday 1ebcdc6
Remove unused packages and add flake8 found in tox.ini
Ickerday 4502606
Update name, remove setup.py
Ickerday dae08b0
Extract deps into dev deps
Ickerday 69bb731
Move versioning to pyproject.toml, adjust version extracting to use i…
Ickerday 0c2b2a5
Try fix CI/CD
Ickerday 7b320c4
Try fix build #2
Ickerday 343713e
Reformat README
Ickerday b59c407
Remove Commands.conf.spec.xlsx
Ickerday 338170e
Remove .env
Ickerday 01774fa
Add missing deps
Ickerday 7bd53fc
Fiddle around with CI/CD
Ickerday db6b3ee
Add miscellaneous changes
Ickerday 2c386a9
Fiddle around with CI/CD #2
Ickerday f154e90
Fiddle around with CI/CD #3
Ickerday 9fed603
Fiddle around with CI/CD #4
Ickerday 0cfbaae
Fiddle around with CI/CD #5
Ickerday e51e42f
Fiddle around with CI/CD #6
Ickerday a865779
Fiddle around with CI/CD #7
Ickerday c8ee057
Change default Splunk version in .env.template to v10
Ickerday a0498aa
Fiddle around with CI/CD #8
Ickerday 79d2dd5
Move tox config to pyproject.toml, return to old version accessing st…
Ickerday fa7fb74
Assorted small changes
Ickerday 4cef846
Try caching virtualenvs in GitHub Actions
Ickerday 1bf86e6
Fix not using venv after restoring cache
Ickerday 9ce725d
Add caching .tox as well
Ickerday 18b97df
Test cache
Ickerday c114d78
Fix build breaking if the cache is invalid
Ickerday 5770ac3
Back out of caching for now
Ickerday bbff0d6
Add `build` package as release won't work without it
Ickerday 5297914
Pin GH Action versions to specific commits
Ickerday caeba73
Format GH templates
Ickerday bb5f45c
Try another approach to splitting dependencies in pyproject.toml
Ickerday 0e0f2a5
Finally figure out how dependency groups are managed in pip?
Ickerday 8855231
Add matrix to release.yml for easier configuration in the future, add…
Ickerday 4a8bb2d
Try using an older Splunk version to determine cause of CI failures
Ickerday 89ca01b
Change `where` calls to `type -a`
Ickerday 5114df6
Fix wrong Splunk version in .env.template
Ickerday 728c67a
Reformat .env.template, add new Splunk 10 env variable to docker-compose
Ickerday 3af9355
Change .env template name, add Docker health check ot test.yml
Ickerday 5f74aab
Add quotes to port mappings
Ickerday ec11167
Enable only Python 3.7, 3.9 and splunk 8.2 and 9.4
Ickerday c09e8de
Set release.yml to use Python 3.9
Ickerday 13fbd86
Readd Python 3.7 to tox
Ickerday File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,16 +1,19 @@ | ||
# Access scheme (default: https) | ||
scheme=https | ||
# Splunk host (default: localhost) | ||
host=localhost | ||
# Splunk admin port (default: 8089) | ||
port=8089 | ||
|
||
# Splunk username | ||
username=admin | ||
# Splunk password | ||
password=changed! | ||
# Access scheme (default: https) | ||
scheme=https | ||
# Your version of Splunk (default: 6.2) | ||
version=9.0 | ||
|
||
# Your version of Splunk (default: latest) | ||
version=latest | ||
|
||
# Bearer token for authentication | ||
#splunkToken="<Bearer-token>" | ||
# splunkToken="<Bearer-token>" | ||
# Session key for authentication | ||
#token="<Session-Key>" | ||
# token="<Session-Key>" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,7 @@ | ||
--- | ||
name: Custom issue template | ||
about: Describe this issue template's purpose here. | ||
title: '' | ||
labels: '' | ||
assignees: '' | ||
|
||
title: "" | ||
labels: "" | ||
assignees: "" | ||
--- | ||
|
||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,43 +1,45 @@ | ||
name: Release | ||
name: Publish release | ||
on: | ||
release: | ||
types: [published] | ||
workflow_dispatch: | ||
inputs: | ||
dry-run: | ||
type: boolean | ||
required: true | ||
default: true | ||
|
||
jobs: | ||
publish: | ||
name: Deploy Release to PyPI | ||
runs-on: ubuntu-latest | ||
release: | ||
runs-on: ${{ matrix.os }} | ||
strategy: | ||
matrix: | ||
os: [ubuntu-latest] | ||
python-version: [3.9] | ||
steps: | ||
- name: Checkout source | ||
uses: actions/checkout@v3 | ||
- name: Set up Python | ||
uses: actions/setup-python@v4 | ||
- name: Checkout repo | ||
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 | ||
- name: Set up Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 | ||
with: | ||
python-version: 3.7 | ||
python-version: ${{ matrix.python-version }} | ||
- name: Install dependencies | ||
run: pip install twine | ||
run: pip install . --group build | ||
- name: Build package | ||
run: python setup.py sdist | ||
run: python -m build | ||
- name: Publish package to PyPI | ||
uses: pypa/[email protected] | ||
if: ${{ inputs.dry-run == 'false' }} | ||
uses: pypa/gh-action-pypi-publish@76f52bc884231f62b9a034ebfe128415bbaabdfc | ||
with: | ||
user: __token__ | ||
password: ${{ secrets.pypi_password }} | ||
- name: Install tox | ||
run: pip install tox | ||
- name: Generate API docs | ||
password: ${{ secrets.PYPI_PASSWORD }} | ||
- name: Generate API reference docs | ||
run: | | ||
rm -rf ./docs/_build | ||
tox -e docs | ||
- name : Docs Upload | ||
uses: actions/upload-artifact@v3 | ||
python -m tox -e docs | ||
- name: Upload API reference docs | ||
if: ${{ inputs.dry-run == 'false' }} | ||
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 | ||
with: | ||
name: python_sdk_docs | ||
path: docs/_build/html | ||
# Test upload | ||
# - name: Publish package to TestPyPI | ||
# uses: pypa/gh-action-pypi-publish@master | ||
# with: | ||
# user: __token__ | ||
# password: ${{ secrets.test_pypi_password }} | ||
# repository_url: https://test.pypi.org/legacy/ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,51 +1,48 @@ | ||
name: Python CI | ||
name: Run test suite | ||
|
||
on: | ||
[ push, workflow_dispatch ] | ||
on: [push, workflow_dispatch] | ||
|
||
jobs: | ||
build: | ||
|
||
test: | ||
runs-on: ${{ matrix.os }} | ||
strategy: | ||
matrix: | ||
os: | ||
- ubuntu-latest | ||
python: [ 3.9, 3.13 ] | ||
os: [ubuntu-latest] | ||
python-version: | ||
- 3.7 | ||
- 3.9 | ||
# - 3.13 | ||
splunk-version: | ||
- "8.1" | ||
- "8.2" | ||
- "latest" | ||
include: | ||
- os: ubuntu-22.04 | ||
python: 3.7 | ||
splunk-version: "8.1" | ||
- os: ubuntu-22.04 | ||
python: 3.7 | ||
splunk-version: "8.2" | ||
- os: ubuntu-22.04 | ||
python: 3.7 | ||
splunk-version: "latest" | ||
|
||
- "9.3" | ||
# - "9.4" | ||
# - "latest" | ||
fail-fast: false | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v3 | ||
|
||
- name: Run docker compose | ||
run: SPLUNK_VERSION=${{matrix.splunk-version}} docker compose up -d | ||
|
||
- name: Setup Python | ||
uses: actions/setup-python@v4 | ||
- name: Checkout repo | ||
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 | ||
- name: Set up environment variables | ||
run: cp ./.env.test ./.env | ||
- name: Launch Splunk ${{ matrix.splunk-version }} | ||
run: SPLUNK_VERSION=${{ matrix.splunk-version }} docker compose up -d | ||
- name: Set up Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 | ||
with: | ||
python-version: ${{ matrix.python }} | ||
|
||
- name: Install tox | ||
run: pip install tox | ||
|
||
- name: Test Execution | ||
run: tox -e py | ||
fossa-scan: | ||
uses: splunk/oss-scanning-public/.github/workflows/oss-scan.yml@main | ||
secrets: inherit | ||
python-version: ${{ matrix.python-version }} | ||
- name: Install dependencies | ||
run: type -a python && python -m pip install tox | ||
# - name: Run Docker health check | ||
# # This probably solves nothing | ||
# run: | | ||
# timeout 30s make wait_up | ||
# if [ $? -eq 124 ]; then | ||
# echo "Splunk failed to start within 30 seconds." | ||
# docker compose logs | ||
# exit 1 | ||
# fi | ||
- name: Run test suite | ||
run: type -a python && python -m tox -e py | ||
# [BJ] I'll uncomment this step after I finish fiddling | ||
# fossa-scan: | ||
# uses: splunk/oss-scanning-public/.github/workflows/oss-scan.yml@main | ||
# secrets: inherit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
3.9.23 | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.