-
Notifications
You must be signed in to change notification settings - Fork 27
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
[CI/CD] Chances the online website cannot be accessed after deploy #65
Comments
But can be accessed by https://open-leaderboard.x-lab.info/static/js/ ...folded content: Do you think it might be caused by a wrongly set cache strategy? @zhicheng-ning |
@gymgym1212 Do you know why |
I config the DNS since the OpenLeaderboard is hosted on CDN service but not simply OSS. The CNAME is the CDN address for corresponding OSS. So I don't think it is something wrong about the DNS, maybe caused by CDN cache strategy. @zhicheng-ning |
ok |
CNAME is used to map another domain, and usually could be found in DNS. And I think the reason is the Nginx config. Plz delete the zip function in |
Not sure what you mean but we build and publish the website directly to OSS and add a DNS service on it, so we don't host any Nginx service on ourselves. @bifenglin |
I'm not sure about the deployment architecture. And I don't know this method to deploy, I will ask to @tyn1998 face-to-face. And thanks for reply @frank-zsy .
|
Here is the publish action: https://github.com/X-lab2017/open-leaderboard/blob/main/.github/workflows/publish.yml |
I have read:
Still cannot have a conclusion, but I suspect that delete-oss before update-oss might be the cause: open-leaderboard/.github/workflows/publish.yml Lines 28 to 44 in 1af79f3
I don't think a delete operation is necessary as the old files should be covered by new ones. |
#66 removed code of delete-oss, but it didn't bring any change of built files so the files on the oss remained the same(i.e. main.xxxx.js is still main.xxxx.js). Following merges into main should be observed to test if the problem is resolved. |
@tyn1998 It could be the problem. Delete the files first because for the generated files, I am not sure but perhaps the numbers are random number by building timestamp. So if we do not clean the files in time, there will be lots of legacy files left on OSS. |
Hi @frank-zsy, do you have access to the oss console? Would you mind checking if old files were simply covered? Since
|
Upload a folder to the OSS will cover the old files with same name but do not remove old files unless explicitly remove them like in the action. Actually I think we can set some flag to remove old files on upload but not in 2 steps, although this may cause file inconsistent for a short time. |
Several days ago we encountered it. Now twice:
Network tab in DevTools:
...folded content:
The text was updated successfully, but these errors were encountered: