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
Issue #12081: Add Github action 'Copy site to sourceforge' #12420
Issue #12081: Add Github action 'Copy site to sourceforge' #12420
Conversation
0ad5c9d
to
a1114a0
Compare
@romani Let me know if indeed there is some internal folder structure that I need to set up manually so the script can run successfully. https://github.com/stoyanK7/checkstyle/actions/runs/3488545560/jobs/5837520525#step:4:6769
|
|
htdocs-version is for links at https://checkstyle.org/#Previous_Version_Documentation htdocs-archive is simply archive, just in case. |
@romani Much better now except for the
https://github.com/stoyanK7/checkstyle/actions/runs/3505485642/jobs/5871823126#step:4:6926
|
|
a1114a0
to
45d93a5
Compare
.ci/copy-site-to-sourceforge.sh
Outdated
$SF_USER,checkstyle@shell.sourceforge.net:/home/project-web/checkstyle/ | ||
echo "Uploading to sourceforge" | ||
scp -i ~/.ssh/private_sourceforge_key checkstyle.github.io.tar.gz \ | ||
"$SF_USER"@shell.sourceforge.net:/home/project-web/stoyank-checkstyle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will undo this when approved before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As you confirm testing, I will merge it.
I will test all action in release this Sunday.
thanks a lot for your work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Final test before reverting back from stoyank-checkstyle
: https://github.com/stoyanK7/checkstyle/actions/runs/3541990560/jobs/5946940410
Website: https://stoyank-checkstyle.sourceforge.io/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. Should be good to merge once CI passes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not forget to add the SSH_KEY
secret variable with the private ssh key for Sourceforge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Items
5492457
to
6ef3697
Compare
6ef3697
to
992e13e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok to merge
Resolves #12081
The job: https://github.com/stoyanK7/checkstyle/actions/runs/3488545560/jobs/5837520525
Even though it failed, the files got uploaded and are accessible at https://stoyank-checkstyle.sourceforge.io.
The job failed on the archiving part since (I suspect) there is some internal folder structure that I need to set up manually once before running the script for the first time.
htdocs-version
,htdocs/version
folders and so on..We introduce two new secrets:
SF_USER
- SourceForge username for loginSSH_KEY
(Probably better to name itSF_SSH_KEY
) - Private SSH key to access SourceForge. Public one has to be placed here https://sourceforge.net/auth/shell_services