Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
162 commits
Select commit Hold shift + click to select a range
a444bcc
add support for cloning Git repository via SSH rather than HTTPS
boegel Feb 6, 2025
6dcdd80
fix use of config.BUILDENV_SETTING_CLONE_GIT_REPO_VIA
boegel Feb 6, 2025
3f2cfcb
also use clone_git_repo_via in app.cfg.example
boegel Feb 6, 2025
37c06af
mark clone_git_repo_via as optional + add logging
boegel Feb 6, 2025
4b046f6
fix get_build_env_cfg to get clone_git_repo_via value from buildenv i…
boegel Feb 6, 2025
45bd556
remove curl command to obtain PR diff + fix _ERROR_GIT_CLONE constant
boegel Feb 6, 2025
b19fe23
fix return value for 'git diff' command in download_pr function
boegel Feb 6, 2025
5c51665
make sure that download_comment is defined in comment_download_pr
boegel Feb 6, 2025
a1c2635
fix 'git diff' command in download_repo
boegel Feb 6, 2025
02f3125
fix branch name used by 'git diff' command
boegel Feb 6, 2025
47e2076
Merge branch 'main' into git_clone_via_ssh
boegel Feb 6, 2025
c7a98f2
comment out clone_git_repo_via setting in REQUIRED_CONFIG to avoid ma…
boegel Feb 21, 2025
7da317d
fix order in REQUIRED_CONFIG (recommended settings before optional ones)
boegel Feb 21, 2025
293ce3c
add more info on use case of clone_git_repo_via set to 'ssh'
boegel Feb 21, 2025
668915c
also add more info on use case of clone_git_repo_via set to 'ssh' to …
boegel Feb 21, 2025
882354c
Merge branch 'git_clone_via_ssh' of github.com:boegel/eessi-bot-softw…
boegel Feb 21, 2025
a99ae60
only use 'git diff' to obtain PR diff when 'ssh' is used to clone rep…
boegel Feb 21, 2025
9b582ab
git diff from merge-base instead of from HEAD
smoors Apr 4, 2025
4415afc
Make a first attempt to change the expected arch_target_map. Make sur…
Apr 17, 2025
d3240e4
Remove old code that was replaced
Apr 17, 2025
1ea33d9
Merge pull request #1 from smoors/pr300
boegel Apr 25, 2025
df1a258
Merge branch 'develop' into git_clone_via_ssh
boegel Apr 25, 2025
46d14a9
Merge branch 'git_clone_via_ssh' of github.com:boegel/eessi-bot-softw…
boegel Apr 25, 2025
f4c0940
update README
smoors May 22, 2025
4021c20
Merge pull request #2 from smoors/pr300
boegel May 28, 2025
4a3dd94
Merge branch 'develop' into git_clone_via_ssh
boegel May 28, 2025
e30cd5b
Merge branch 'git_clone_via_ssh' of github.com:boegel/eessi-bot-softw…
boegel May 28, 2025
bc5c154
workflow for building a smee-client container image
bedroge Jun 12, 2025
57fb5ed
dockerfile for smee-client container
bedroge Jun 12, 2025
7ac0dd1
also trigger for PRs
bedroge Jun 12, 2025
a3e5c31
only for pushes
bedroge Jun 12, 2025
d12c2dc
also for PRs
bedroge Jun 12, 2025
9dc32ca
use correct dockerfile filename
bedroge Jun 12, 2025
7811db6
Merge pull request #321 from bedroge/smee_client_container_image
trz42 Jun 12, 2025
6c599f4
Add package write permission to container build jon
bedroge Jun 12, 2025
d14a08e
Merge pull request #322 from EESSI/bedroge-patch-1
boegel Jun 12, 2025
7f29d4d
make space before bot command optional
boegel Jun 19, 2025
a985803
Merge branch 'main' into git_clone_via_ssh
boegel Jun 19, 2025
66005da
fix regex for bot build command + add log messages
boegel Jun 19, 2025
02c2465
fix logging with regex pattern in get_bot_command
boegel Jun 19, 2025
8eec372
mention function name in log message in get_bot_command
boegel Jun 19, 2025
4b7c50c
Merge pull request #324 from EESSI/bot-command-optional-space
trz42 Jun 20, 2025
1381689
Merge branch 'develop' into git_clone_via_ssh
boegel Jun 20, 2025
750b8a4
add pr_diff_failure and pr_diff_tip to required configuration settings
boegel Jun 20, 2025
50c21d7
Merge pull request #300 from boegel/git_clone_via_ssh
trz42 Jun 23, 2025
fd31cd1
update prerequisites and section 1
truib Jun 23, 2025
1a8211c
revise sections 2 and 3
truib Jun 23, 2025
85cb96f
reformatting note
truib Jun 25, 2025
9d422b2
add CI for markdown linting and fix all issues in README.md
truib Jun 25, 2025
a24ac74
layout tweaking
truib Jun 25, 2025
47dd306
refine section 4
truib Jun 25, 2025
29de868
improve sections 5.1 to 5.3
truib Jun 25, 2025
a015086
half way through revising app.cfg settings
truib Jun 25, 2025
673c411
improved app.cfg section and cfg example file
truib Jun 26, 2025
05786f3
apply suggestions from code review
trz42 Jun 27, 2025
e3041c7
addressing more suggested changes
truib Jun 27, 2025
baa16fe
Remove trailing white spaces
Neves-P Jun 28, 2025
5065c16
Merge pull request #325 from trz42/update_readme_app_cfg_example
Neves-P Jun 28, 2025
dfbcec3
Merge branch 'EESSI:develop' into adapt_arch_target_map
casparvl Jun 30, 2025
b54fdf4
Fix xome small issues with non-existing keys
Jun 30, 2025
1e6c3d9
Avoid doing string += None for the arch_dir if accelerator = None. Al…
Jun 30, 2025
e000c91
Make sure that if the context (i.e. app.cfg) defines AN accelerator, …
Jun 30, 2025
40ceefe
Some cleanup
Jun 30, 2025
fad4f47
Remove repo_target_map from config, and all occurences that import it…
Jul 1, 2025
b4032a6
Fix quotation of keys
Jul 1, 2025
3898df7
Fix flake8 issue
Jul 1, 2025
ef0c430
Unpack the actual arch_target_map by accessing it with a key to get t…
Jul 1, 2025
e3df690
Fix mistake in build path
Jul 8, 2025
424a001
Parse on: and for: options, and pass the correct values on to the com…
Jul 9, 2025
3f00e51
Make sure that the for: arguments are used as build parameters
Jul 10, 2025
2625b30
Change path for job dir so that it represents the 'for' architectures
Jul 10, 2025
ffa2303
More extensive reporting by the bot on what to build for/on
Jul 14, 2025
c98e7e8
This is no longer needed, as it is done with the codecs (decode) now
Jul 14, 2025
5904f11
Print real arch_target_map keys when doing show_config
Jul 14, 2025
7c869f0
Reduce number of possible accelerators per node type to one. Nodes wi…
Jul 14, 2025
179ab0a
Fix app.cfg for the fact that partition_info['accel'] is now a string…
Jul 14, 2025
a9a2585
Make sure that we don't access a dict item that doesn't exist
Jul 14, 2025
51a9c74
Make sure a context match fails if the context doesn't provide e.g. a…
Jul 14, 2025
b12c911
Make old config items invalid, rename to node_type and note_type_map,…
Jul 15, 2025
697dc6e
Update the status command to account for the new on:... for:... syntax
Jul 16, 2025
c0fe051
Remove debugging print statements
Jul 16, 2025
f179b66
Warn about the removal of the repo_target_map
Jul 16, 2025
aad663e
Fix typo
Jul 16, 2025
be8c7d0
Fix hound issues
Jul 16, 2025
7f766f4
Format releveant output of show_config as code
Jul 16, 2025
d205598
Rephrase to make things more clear
Jul 16, 2025
ebcc7fd
Forgot to add this new file...
Jul 16, 2025
0a8bc9b
Fix hound issues
Jul 16, 2025
81257db
Update build params call signature
Jul 16, 2025
f974463
Fix example argument, and argument used to create build parameters in…
Jul 16, 2025
4104796
Forgot to actually git add this file again... anyway, updated the syn…
Jul 16, 2025
0b82386
Update the app.cfg used for the unit tests to account for the changes…
Jul 16, 2025
372a7fe
Update tests for new requirement that all filters have to be present …
Jul 16, 2025
d2be02a
Update tests to accomodate for new behaviour of filter checking that …
Jul 16, 2025
6b3a118
Fix hound issues
Jul 16, 2025
3b310f5
Fix flake8 issues
Jul 16, 2025
624109e
update the scontrol command to work on system with multiple clusters
laraPPr Jul 17, 2025
b14043e
clean up
laraPPr Jul 17, 2025
de0bd1c
Removed some comments that were only there for development, no longer…
Jul 21, 2025
d4ecc7b
Apply suggestions from code review
casparvl Jul 24, 2025
af731e1
Re-comment the awaits_release, as this was done in develop as well. T…
Jul 28, 2025
d48b355
Replace Partition with Node type in show_config output. Also, update …
Jul 28, 2025
6017433
Processed various smaller review comments for tasks/build.py. Elabora…
Jul 28, 2025
279e08f
Apply suggestions from code review
casparvl Jul 29, 2025
80f5f1d
Fix indentation issue
Jul 29, 2025
2f3c0ae
Update tasks/build.py
casparvl Jul 31, 2025
7a92892
Merge pull request #312 from casparvl/adapt_arch_target_map
bedroge Aug 4, 2025
03e720d
add exception and simply using placeholder in scontrol command
laraPPr Aug 4, 2025
54e502a
resolve merge conflict with upstream branch
laraPPr Aug 4, 2025
53b032f
make stylecheck happy
laraPPr Aug 4, 2025
519428f
some more error logging for the scontrol command
laraPPr Aug 4, 2025
e42b34e
Update readme for new node_type_map functionality
Aug 4, 2025
0b4e22f
Add examples of bot build commands
Aug 4, 2025
be7efe4
Process review comments
Aug 5, 2025
ae691e6
Fix linting errors
Aug 5, 2025
2ad48ec
Update README.md
casparvl Aug 5, 2025
769150d
Update README.md
casparvl Aug 5, 2025
b4d86e0
Update README.md
casparvl Aug 5, 2025
334fe8f
Merge pull request #329 from casparvl/update_readme_node_type_map
bedroge Aug 5, 2025
9fb883b
Merge branch 'develop' into test_submit_to_all
laraPPr Aug 5, 2025
72c39a4
simplify using placeholder for scontrol command
laraPPr Aug 6, 2025
19c28f5
remove uneccesary change
laraPPr Aug 6, 2025
9433bea
use rstrip in a list comprehension
laraPPr Aug 6, 2025
09a4842
Update eessi_bot_job_manager.py
laraPPr Aug 6, 2025
a2e52dd
Update README
laraPPr Aug 7, 2025
63fa9f3
Set a default self.build_params = None on constructiong the EESSIBotC…
Aug 7, 2025
4360e3d
Fix typo
Aug 7, 2025
1ad4640
WIP: trying to get a status command that only prints the last status …
Aug 18, 2025
7a55d30
Update README.md
laraPPr Aug 19, 2025
e349257
Update README.md
laraPPr Aug 19, 2025
9a5bf4d
Add support for passing general arguments to a command, so that we ca…
Aug 19, 2025
583cacc
Fix typo
Aug 19, 2025
7c85854
Merge pull request #331 from casparvl/print_nice_error
boegel Aug 19, 2025
16a4e0f
add description to app.cfg.example
laraPPr Aug 19, 2025
6bfb246
Merge pull request #327 from laraPPr/test_submit_to_all
casparvl Aug 19, 2025
25dd76d
Make sure that commands still function if no action filters were defi…
Aug 19, 2025
296942f
Fix some bugs. Also, make sure to actually use the sorted table when …
Aug 19, 2025
d335c3d
Fix too long lines
Aug 19, 2025
3fc6a6e
Don't define the lambda's separately
Aug 19, 2025
7b284ba
Merge branch 'develop' into print_last_status_only
casparvl Aug 19, 2025
fc3c23d
don't replace self.scontrol_command in process_new_job, to keep place…
boegel Aug 20, 2025
0fdfed4
fix comment
boegel Aug 20, 2025
5656a69
Merge pull request #335 from boegel/fix_templated_scontrol_command
laraPPr Aug 20, 2025
2c5569b
Merge pull request #334 from casparvl/print_last_status_only
trz42 Aug 21, 2025
4b6d62a
release notes for v0.9.9
truib Aug 21, 2025
e0800eb
add missing information
truib Aug 21, 2025
42d95ae
replacing more arch_target_map with node_type_map
truib Aug 21, 2025
1e2697a
Merge pull request #337 from trz42/minor_tweaks_readme
boegel Aug 21, 2025
45f0c10
Merge branch 'develop' of github.com:EESSI/eessi-bot-software-layer i…
truib Aug 21, 2025
fdc3b3a
add PR with additional fixes for README.md
truib Aug 21, 2025
c7e61aa
improvements/fixes taking review into account
truib Aug 21, 2025
405fb2f
rephrase info about breaking changes
trz42 Aug 22, 2025
447a05c
date bump
truib Aug 22, 2025
d4d649c
remove duplicate information
truib Aug 22, 2025
0e21338
Merge pull request #336 from trz42/release_notes_v0_9_0
boegel Aug 22, 2025
1e70f3e
update upload-artifact version to 4.6.2
truib Aug 22, 2025
05ead1d
also run scorecards when pushing/pull_requesting to develop branch
truib Aug 22, 2025
c6220fe
change action version 4.3.3
truib Aug 22, 2025
7e197ae
also mention v0.9.0 release notes that initial_comment was replaced i…
boegel Aug 22, 2025
fc4d204
update version of codeql-action/upload-sarif to v3.25.6
truib Aug 22, 2025
fb2e1a0
Merge pull request #341 from boegel/rip_initial_comment
trz42 Aug 22, 2025
764c5c9
Merge pull request #340 from trz42/update_scorecards_CI
boegel Aug 22, 2025
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
64 changes: 64 additions & 0 deletions .github/workflows/build_smee_client_container_image.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,64 @@
name: Build and publish a Smee client container image

on:
push:
paths:
- containers/Dockerfile.smee-client
- .github/workflows/build_smee_client_container_image.yaml
pull_request:
paths:
- containers/Dockerfile.smee-client
- .github/workflows/build_smee_client_container_image.yaml

# Declare default permissions as read only.
permissions: read-all

jobs:
docker_build_smee_client:
runs-on: ubuntu-latest
permissions:
packages: write
steps:
- name: Check out the repo
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2

- name: Login to GitHub Container Registry
if: github.event_name != 'pull_request'
uses: docker/login-action@74a5d142397b4f367a81961eba4e8cd7edddf772 # v3.4.0
with:
registry: ghcr.io
username: ${{ github.repository_owner }}
password: ${{ secrets.GITHUB_TOKEN }}

- name: Convert and store repository owner in lowercase, replace colon in tag names by hyphen
run: |
echo REPOSITORY_OWNER=$(echo ${{ github.repository_owner }} | tr '[:upper:]' '[:lower:]') >> $GITHUB_ENV

- name: Set up QEMU
uses: docker/setup-qemu-action@29109295f81e9208d7d86ff1c6c12d2833863392 # v3.6.0

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@b5ca514318bd6ebac0fb2aedd5d36ec1b5c232a2 # v3.10.0

- name: Cache Docker layers
uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3
with:
path: ${{ runner.temp }}/.buildx-cache
key: ${{ runner.os }}-buildx-${{ github.sha }}
restore-keys: |
${{ runner.os }}-buildx-

- name: Build and push
uses: docker/build-push-action@263435318d21b8e681c14492fe198d362a7d2c83 # v6.18.0
with:
tags: ghcr.io/${{ env.REPOSITORY_OWNER }}/smee-client
platforms: linux/amd64,linux/arm64
push: ${{ github.event_name != 'pull_request' }} # don't publish if this is part of an open PR
file: containers/Dockerfile.smee-client
cache-from: type=local,src=${{ runner.temp }}/.buildx-cache
cache-to: type=local,dest=${{ runner.temp }}/.buildx-cache-new,mode=max

- name: Move cache
run: |
rm -rf ${{ runner.temp }}/.buildx-cache
mv ${{ runner.temp }}/.buildx-cache-new ${{ runner.temp }}/.buildx-cache
33 changes: 33 additions & 0 deletions .github/workflows/markdown-lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
# This file is part of the EESSI build-and-deploy bot,
# see https://github.com/EESSI/eessi-bot-software-layer
#
# The bot helps with requests to add software installations to the
# EESSI software layer, see https://github.com/EESSI/software-layer
#
# author: Thomas Roeblitz (@trz42)
#
# license: GPLv2
#

name: Markdown Lint
on: [push, pull_request]
# Declare default permissions as read only.
permissions: read-all

jobs:
markdown-lint:
runs-on: ubuntu-24.04
steps:
- name: Checkout
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # v3.1.0

- name: Setup Node.js
uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.0
with:
node-version: '18'

- name: Install markdownlint-cli
run: npm install -g markdownlint-cli

Check warning

Code scanning / Scorecard

Pinned-Dependencies Medium

score is 7: npmCommand not pinned by hash
Click Remediation section below to solve this issue
- name: Run markdownlint
run: markdownlint "**/*.md" --ignore .git
9 changes: 4 additions & 5 deletions .github/workflows/scorecards.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,9 @@ on:
schedule:
- cron: '25 15 * * 3'
push:
branches: [ "main" ]
branches: [ "main", "develop" ]
pull_request:
branches:
- main
branches: [ "main", "develop" ]

# Declare default permissions as read only.
permissions: read-all
Expand Down Expand Up @@ -67,14 +66,14 @@ jobs:
# Upload the results as artifacts (optional). Commenting out will disable uploads of run results in SARIF
# format to the repository Actions tab.
- name: "Upload artifact"
uses: actions/upload-artifact@3cea5372237819ed00197afe530f5a7ea3e805c8 # v3.1.0
uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4.3.3
with:
name: SARIF file
path: results.sarif
retention-days: 5

# Upload the results to GitHub's code scanning dashboard.
- name: "Upload to code-scanning"
uses: github/codeql-action/upload-sarif@807578363a7869ca324a79039e6db9c843e0e100 # v2.1.27
uses: github/codeql-action/upload-sarif@9fdb3e49720b44c48891d036bb502feb25684276 # v3.25.6
with:
sarif_file: results.sarif
10 changes: 10 additions & 0 deletions .markdownlint.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
"default": true,
"MD013": false,
"MD033": false,
"MD041": false,
"MD024": false,
"MD026": {
"punctuation": ".,;:!"
}
}
Loading