Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

Neofs testcases workflow. Fixes #254 #279

Merged
merged 1 commit into from
Sep 28, 2023
Merged

Conversation

MaxGelbakhiani
Copy link

Workflow for neofs-testcases.

Before merging the PR we'll need to add TEST_RESULTS_CID variable to GH and the secrets.

@codecov
Copy link

codecov bot commented Sep 22, 2023

Codecov Report

Merging #279 (11797ce) into master (8cd4fff) will not change coverage.
Report is 3 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #279   +/-   ##
=======================================
  Coverage   34.24%   34.24%           
=======================================
  Files          10       10           
  Lines        1384     1384           
=======================================
  Hits          474      474           
  Misses        868      868           
  Partials       42       42           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

.github/workflows/system-tests.yml Outdated Show resolved Hide resolved
Copy link
Collaborator

@vvarg229 vvarg229 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MaxGelbakhiani
Copy link
Author

https://github.com/nspcc-dev/neofs-http-gw/actions/runs/6277027149/job/17047962671?pr=279
image

Fixed. Shifted the string:
echo "$(pwd)" >> $GITHUB_PATH
a few steps earlier

@MaxGelbakhiani MaxGelbakhiani force-pushed the testcases_workflow branch 3 times, most recently from 2ade056 to 391b99b Compare September 26, 2023 03:47
Signed-off-by: Maksim Gelbakhiani <[email protected]>
env:
TEST_RESULTS_PASSWORD: ${{ secrets.TEST_RESULTS_PASSWORD }}
TEST_RESULTS_NEOFS_NETWORK_DOMAIN: ${{ vars.TEST_RESULTS_NEOFS_NETWORK_DOMAIN }}
TEST_RESULTS_CID: ${{ vars.TEST_RESULTS_CID }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Cmd: NEOFS_CLI_PASSWORD=*** neofs-cli --rpc-endpoint st1.storage.fs.neo.org:8080 --wallet wallet.json object put --cid _ --timeout 600s --expire-at 18511 --file /home/runner/work/neofs-http-gw/neofs-http-gw/neofs-testcases/combine/complete.html --attributes RunNumber=7-1695717445,FilePath=7-1695717445/index.html

--cid _
Instead of the "_" character there should be a valid CID, for example "HXSaMJXk2g8C14ht8HSi7BBaiYZ1HeWh2xnWPGQCg4H6".
This must be changed in the project settings

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a task for @roman-khimov to add a valid container and wallet for MainNet before merging the PR.

@roman-khimov roman-khimov merged commit 4fca940 into master Sep 28, 2023
12 of 13 checks passed
@roman-khimov roman-khimov deleted the testcases_workflow branch September 28, 2023 15:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants