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

Develop->Master #360

Open
wants to merge 72 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
72 commits
Select commit Hold shift + click to select a range
965cc0c
make generated python code pep8 compliant
eapearson Jan 23, 2020
e3684ed
revert test file change, not necessary and can't be tested.
eapearson Jan 23, 2020
4d7bdb7
Allow KBASE_TEST_TOKEN env var to be used as well as hardcoding the t…
ialarmedalien Jan 29, 2020
fa01d62
Adding KBASE_TEST_TOKEN to entrypoint
ialarmedalien Jan 31, 2020
ef10268
Removing stray "D" from env var
ialarmedalien Feb 7, 2020
2ea5555
Merge pull request #346 from ialarmedalien/kbase_test_token_env_var
sychan Feb 27, 2020
b1c51ba
Merge remote-tracking branch 'origin/develop' into make-python-pep8-c…
eapearson Mar 31, 2020
35e7479
Whitespace changes only: removing trailing spaces, replacing tabs wit…
ialarmedalien Jun 11, 2020
8d0e981
Typo in error message
ialarmedalien Jun 11, 2020
2a7474a
Adding build date/time to kb-sdk. Failing earlier in kb-sdk test.
ialarmedalien Jun 11, 2020
7e58f55
Adding actions file
ialarmedalien Jun 11, 2020
835b9cc
Adding in Java example generate/test to github actions; DRYing out re…
ialarmedalien Jun 12, 2020
9ac1aba
Updating actions to create a build/test matrix. Report version when s…
ialarmedalien Jun 16, 2020
2384e3b
Updating release notes with more specific details
ialarmedalien Jun 17, 2020
48f8e29
Reworded release notes message.
ialarmedalien Jun 17, 2020
c3e81fb
Die if the git.properties file cannot be found
ialarmedalien Jun 18, 2020
ad5e402
Merge pull request #350 from ialarmedalien/java_edits_for_kb_sdk_1.3
MrCreosote Jun 19, 2020
2813c80
Update dockerfile with updated dependencies
ialarmedalien Jun 22, 2020
965b840
Bump version to 1.2.3
ialarmedalien Aug 20, 2020
8fc4c60
Merge pull request #352 from ialarmedalien/update_Dockerfile
sychan Sep 2, 2020
76e3728
try removing sdkbase dir
ialarmedalien Jun 16, 2020
5be329c
jsf-nose-mod
jsfillman Sep 11, 2020
782e415
Merge pull request #355 from kbase/jsfillman-patch-1
ialarmedalien Sep 11, 2020
b7b7d16
Adding kb-sdk image build and upload to Dockerhub to GitHub actions
ialarmedalien Sep 10, 2020
ebfcf73
Merge pull request #354 from kbase/autobuild_docker_images
ialarmedalien Sep 12, 2020
c03ad16
Merge branch 'develop' into remove_sdkbase_dir
ialarmedalien Sep 14, 2020
7df70d8
Merge pull request #353 from kbase/remove_sdkbase_dir
sychan Sep 14, 2020
55a2d35
Remove return statement that stops kb-sdk test after adding the test.…
ialarmedalien Sep 21, 2020
0325651
Test both env var and test.cfg-based auth
ialarmedalien Sep 22, 2020
492c5f7
Merge branch 'develop' into make-python-pep8-compliant
sychan Sep 23, 2020
23401e5
Use both auth methods for existing repo tests
ialarmedalien Sep 23, 2020
e7ad917
Merge pull request #357 from kbase/remove_no_token_return
sychan Sep 23, 2020
6d0d36f
Perl tidy -- mostly whitespace changes
ialarmedalien Jun 3, 2020
ad6b089
Perl changes:
ialarmedalien Sep 18, 2020
51a73c7
Add maintainer as a LABEL to the dockerfile
ialarmedalien Sep 21, 2020
9a37a53
Merge branch 'develop' into make-python-pep8-compliant
ialarmedalien Sep 23, 2020
861d335
Merge pull request #356 from ialarmedalien/better_perl
ialarmedalien Sep 25, 2020
54ac374
Merge branch 'develop' into make-python-pep8-compliant
ialarmedalien Sep 25, 2020
b2f6651
Merge pull request #344 from eapearson/make-python-pep8-compliant
ialarmedalien Sep 25, 2020
1c53e97
Add cpanfile for perl dependencies
ialarmedalien Sep 28, 2020
3ef8ad2
Merge pull request #358 from kbase/perl_add_cpanfile_compile_tests
ialarmedalien Oct 7, 2020
71fc4d7
Update entrypoint
bio-boris Oct 9, 2020
b9a313a
Update RELEASE_NOTES.txt
bio-boris Oct 14, 2020
60d9939
Update ModuleBuilder.java
bio-boris Oct 14, 2020
0f955f3
Merge pull request #359 from kbase/force_root_user
ialarmedalien Oct 14, 2020
cf29601
Merge branch 'master' into develop
bio-boris Oct 15, 2020
2eacd99
Removing "-t" flag to allow github actions to run kb-sdk
ialarmedalien Oct 22, 2020
07a0a3c
Merge pull request #362 from kbase/remove_-t_flag
ialarmedalien Oct 22, 2020
401b3d7
Update module_test_python_client.vm.properties
bio-boris Jun 22, 2022
29d1ee5
Update python_server.vm.properties
bio-boris Jun 22, 2022
6eecbf5
Create manual-build.yml
bio-boris Feb 16, 2023
e9dd950
Create pr_build.yml
bio-boris Feb 16, 2023
03c6a09
Create release-main.yml
bio-boris Feb 16, 2023
8b2d1e6
Update kb_sdk-actions.yml
bio-boris Feb 16, 2023
f22afe7
Merge pull request #369 from kbase/add_gha_build
bio-boris Feb 16, 2023
f7f7f26
Merge branch 'develop' into issue_347_with_updated_kb-sdk
Feb 16, 2023
dad228a
Fixing kb_sdk tests, slimming down dockerfile
ialarmedalien Mar 7, 2023
830e4af
removing travis file, updating test workflow to remove push to github
ialarmedalien Mar 7, 2023
c1b938c
adding missing distro tag to java action
ialarmedalien Mar 7, 2023
13caa04
fix java action again
ialarmedalien Mar 7, 2023
3acd45d
update to using two most recent ubuntus
ialarmedalien Mar 10, 2023
28f15f5
disable generating the example module as the software dep is no longe…
ialarmedalien Mar 10, 2023
85bc764
Merge pull request #367 from kbase/issue_347_with_updated_kb-sdk
ialarmedalien Mar 14, 2023
edd9146
Disable testing of WSLargeDataIO as it relies on a Java repository th…
ialarmedalien Mar 15, 2023
8f14d80
disable WsLargeDataIO tests
ialarmedalien Mar 15, 2023
1a1e51e
Renaming workflow file, adding comments about what tests are doing
ialarmedalien Mar 15, 2023
89e620e
Update python_server.vm.properties
bio-boris Mar 15, 2023
2501e08
Merge pull request #370 from kbase/fix_actions
ialarmedalien Mar 15, 2023
ad8134e
Merge branch 'develop' into adjust_repr_use
Mar 15, 2023
0ab2a97
Updating docker install to enable `docker buildx`
jsfillman May 27, 2022
3a7e1b2
Merge pull request #366 from kbase/DEVOPS-765-add_buildx
ialarmedalien Mar 15, 2023
cedcfc2
Merge pull request #371 from kbase/adjust_repr_use
ialarmedalien Apr 23, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
174 changes: 174 additions & 0 deletions .github/workflows/basic_sdk_functionality_tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,174 @@
name: Basic kb-sdk functionality tests

on:
pull_request:
branches:
- develop
- main
- master
types:
- opened
- reopened
- synchronize
- closed

jobs:
test_kb-sdk_builds:
if: "!contains(github.event.head_commit.message, 'skip ci')"
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-20.04, ubuntu-latest]
language: [perl, python, java]
test_type: [base] #, example]
auth: [token, env_var]

steps:
- name: Checkout
uses: actions/checkout@v3

# build kb-sdk from source and test basic functions
- name: Set up JDK 1.8
uses: actions/setup-java@v3
with:
distribution: temurin
java-version: 8.0.362+9

- name: Build with Ant
run: |
make

- name: Add bin to $PATH
run: |
echo "$GITHUB_WORKSPACE/bin" >> $GITHUB_PATH

- name: checking kb-sdk functions
env:
KBASE_TEST_TOKEN: ${{ secrets.KBASE_TEST_TOKEN }}
LANGUAGE_TOKEN: ${{ matrix.language }}
run: |
env
kb-sdk help
kb-sdk version

# the following steps create a KBase app using the SDK
# and runs some checks to ensure the app generated
# is functional and that the SDK can test it.
- name: init base repo
if: matrix.test_type == 'base'
env:
LANGUAGE_TOKEN: ${{ matrix.language }}
run: |
kb-sdk init -l $LANGUAGE_TOKEN -u user SampleApp

- name: init example repo
if: matrix.test_type == 'example'
env:
LANGUAGE_TOKEN: ${{ matrix.language }}
run: |
kb-sdk init -l $LANGUAGE_TOKEN -u user --example SampleApp

- name: test ${{ matrix.test_type }} repo, using test.cfg for auth
if: matrix.auth == 'token'
run: |
cd SampleApp
kb-sdk test || true

- name: test ${{ matrix.test_type }} repo, using test.cfg for auth, add env var to test.cfg
if: matrix.auth == 'token'
env:
KBASE_TEST_TOKEN: ${{ secrets.KBASE_TEST_TOKEN }}
run: |
cd SampleApp
sed -i "s/test_token=/test_token=$KBASE_TEST_TOKEN/" test_local/test.cfg
kb-sdk test

- name: test ${{ matrix.test_type }} repo, using env var for auth
if: matrix.auth == 'env_var'
env:
KBASE_TEST_TOKEN: ${{ secrets.KBASE_TEST_TOKEN }}
run: |
cd SampleApp
kb-sdk test

- name: make resulting app available as artefact in case of failure
if: ${{ failure() }}
uses: actions/upload-artifact@v3
with:
name: kbaseapp-${{ matrix.language }}-${{ matrix.test_type }}-${{ matrix.os }}
path: SampleApp


test_existing_repos:
if: "!contains(github.event.head_commit.message, 'skip ci')"
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-20.04, ubuntu-latest]
# TODO: add a Java app. WsLargeDataIO does not currently build as it relies on a Java
# repo that no longer exists.
app: [KBaseReport] # , WsLargeDataIO]
auth: [token, env_var]

steps:
- name: Checkout
uses: actions/checkout@v3

- name: Set up JDK 1.8
uses: actions/setup-java@v3
with:
distribution: temurin
java-version: 8.0.362+9

- name: Build with Ant
run: |
make

- name: Add bin to $PATH
run: |
echo "$GITHUB_WORKSPACE/bin" >> $GITHUB_PATH

- name: checking basic kb-sdk functions
run: |
env
kb-sdk help
kb-sdk version

# the following steps check out and run the tests of an existing KBase app
- name: Checkout existing kbase module ${{ matrix.app }}
uses: actions/checkout@v3
with:
repository: kbaseapps/${{ matrix.app }}
path: kbase_app

- name: run repo tests, using test.cfg for auth
if: matrix.auth == 'token'
run: |
cd kbase_app
kb-sdk test || true

- name: run repo tests, using test.cfg for auth
if: matrix.auth == 'token'
env:
KBASE_TEST_TOKEN: ${{ secrets.KBASE_TEST_TOKEN }}
run: |
cd kbase_app
sed -i "s/test_token=/test_token=$KBASE_TEST_TOKEN/" test_local/test.cfg
kb-sdk test

- name: run repo tests, using env var for auth
if: matrix.auth == 'env_var'
env:
KBASE_TEST_TOKEN: ${{ secrets.KBASE_TEST_TOKEN }}
run: |
cd kbase_app
kb-sdk test

- name: make resulting app available as artefact in case of failure
if: ${{ failure() }}
uses: actions/upload-artifact@v3
with:
name: kbaseapp-${{ matrix.app }}-${{ matrix.os }}
path: kbase_app
11 changes: 11 additions & 0 deletions .github/workflows/manual-build.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
---
name: Manual Build & Push
on:
workflow_dispatch:
jobs:
build-push:
uses: kbase/.github/.github/workflows/reusable_build-push.yml@main
with:
name: '${{ github.event.repository.name }}-develop'
tags: br-${{ github.ref_name }}
secrets: inherit
43 changes: 43 additions & 0 deletions .github/workflows/pr_build.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
---
name: Pull Request Build, Tag, & Push
on:
pull_request:
branches:
- develop
- main
- master
types:
- opened
- reopened
- synchronize
- closed
jobs:
build-develop-open:
if: github.base_ref == 'develop' && github.event.pull_request.merged == false
uses: kbase/.github/.github/workflows/reusable_build.yml@main
secrets: inherit
build-develop-merge:
if: github.base_ref == 'develop' && github.event.pull_request.merged == true
uses: kbase/.github/.github/workflows/reusable_build-push.yml@main
with:
name: '${{ github.event.repository.name }}-develop'
tags: pr-${{ github.event.number }},latest
secrets: inherit
build-main-open:
if: (github.base_ref == 'main' || github.base_ref == 'master') && github.event.pull_request.merged == false
uses: kbase/.github/.github/workflows/reusable_build-push.yml@main
with:
name: '${{ github.event.repository.name }}'
tags: pr-${{ github.event.number }}
secrets: inherit
build-main-merge:
if: (github.base_ref == 'main' || github.base_ref == 'master') && github.event.pull_request.merged == true
uses: kbase/.github/.github/workflows/reusable_build-push.yml@main
with:
name: '${{ github.event.repository.name }}'
tags: pr-${{ github.event.number }},latest-rc
secrets: inherit
trivy-scans:
if: (github.base_ref == 'develop' || github.base_ref == 'main' || github.base_ref == 'master' ) && github.event.pull_request.merged == false
uses: kbase/.github/.github/workflows/reusable_trivy-scans.yml@main
secrets: inherit
25 changes: 25 additions & 0 deletions .github/workflows/release-main.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
---
name: Release - Build & Push Image
on:
release:
branches:
- main
- master
types: [ published ]
jobs:
check-source-branch:
uses: kbase/.github/.github/workflows/reusable_validate-branch.yml@main
with:
build_branch: '${{ github.event.release.target_commitish }}'
validate-release-tag:
needs: check-source-branch
uses: kbase/.github/.github/workflows/reusable_validate-release-tag.yml@main
with:
release_tag: '${{ github.event.release.tag_name }}'
build-push:
needs: validate-release-tag
uses: kbase/.github/.github/workflows/reusable_build-push.yml@main
with:
name: '${{ github.event.repository.name }}'
tags: '${{ github.event.release.tag_name }},latest'
secrets: inherit
3 changes: 1 addition & 2 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@
/temp
/dist
/eclipse-classes/
/sdkbase/ssl
/sdkbase/cluster.ini
/classes/
/temp_*
/ASimpleModule*
Expand All @@ -13,3 +11,4 @@
/test_scripts/test.cfg
*.pyc
env-*/
submodules/auth
45 changes: 0 additions & 45 deletions .travis.yml

This file was deleted.

31 changes: 17 additions & 14 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,24 +1,27 @@
FROM ubuntu:14.04
MAINTAINER Shane Canon <[email protected]>
FROM ubuntu:20.04
LABEL MAINTAINER KBase developers

# Update apt and install jdk and docker engine to get docker clients
# Docker installation instructions from https://docs.docker.com/engine/install/ubuntu/
RUN apt-get -y update && \
apt-get -y install openjdk-7-jdk make git ant && \
apt-get -y install apt-transport-https ca-certificates && \
apt-key adv \
--keyserver hkp://ha.pool.sks-keyservers.net:80 \
--recv-keys 58118E89F3A912897C070ADBF76221572C52609D && \
echo "deb https://apt.dockerproject.org/repo ubuntu-trusty main" > /etc/apt/sources.list.d/docker.list && \
apt-get -y update && apt-get -y install docker-engine=1.11.2-0~trusty
DEBIAN_FRONTEND=noninteractive apt-get -y install tzdata openjdk-8-jdk make git ant curl gnupg-agent apt-transport-https ca-certificates software-properties-common && \
update-java-alternatives -s java-1.8.0-openjdk-amd64 && \
curl -fsSL https://download.docker.com/linux/ubuntu/gpg | apt-key add - && \
add-apt-repository \
"deb [arch=amd64] https://download.docker.com/linux/ubuntu \
$(lsb_release -cs) \
stable" && \
apt-get -y update && \
apt-get -y install docker-ce docker-ce-cli containerd.io docker-compose-plugin

# Add kb-sdk src and fix CallbackServer interface
ADD . /src

# Add kb_sdk src and fix CallbackServer interface
RUN \
cd /src && \
sed -i 's/en0/eth0/' src/java/us/kbase/common/executionengine/CallbackServer.java && \
make && \
/src/entrypoint prune && rm -rf /src/.git
cd /src && \
sed -i 's/en0/eth0/' src/java/us/kbase/common/executionengine/CallbackServer.java && \
make && \
/src/entrypoint prune && rm -rf /src/.git

ENV PATH=$PATH:/src/bin

Expand Down
9 changes: 2 additions & 7 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ KBASE_COMMON_JAR = kbase/common/kbase-common-0.0.23.jar
QUOTE = '\''

# make sure our make test works
.PHONY : test test-python sdkbase
.PHONY : test test-python


default: compile
Expand Down Expand Up @@ -59,7 +59,7 @@ bin: jars-submodule-init
echo '#!/bin/bash' > bin/kb-sdk
echo 'DIR=$(DIR)' >> bin/kb-sdk
echo 'KBASE_JARS_DIR=$$DIR/submodules/jars/lib/jars' >> bin/kb-sdk
@# Next command processes links in JAR_DEPS_BIN file and has 5 parts (one on each line):
@# Next command processes links in JAR_DEPS_BIN file and has 5 parts (one on each line):
@# (1) removing comments
@# (2) trimming each line (picking first word actually)
@# (3) skipping empty lines
Expand Down Expand Up @@ -103,11 +103,6 @@ deploy-scripts:
fi;
$(ANT) deploy_bin -DBIN_TARGET=$(TARGET)/bin -DBIN_LIB_TARGET=$(TARGET)/lib -DKBASE_COMMON_JAR=$(KBASE_COMMON_JAR)

sdkbase:
# docker rmi -f kbase/deplbase:latest
cd sdkbase && ./makeconfig
docker build --no-cache -t kbase/kbase:sdkbase2.latest sdkbase

test: submodule-init
@echo "Running unit tests"
make test-python
Expand Down
Loading
Loading