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

[Site]: Test and deploy time increases and deployment failed after markdown link check action was modified #1682

Closed
yzeng25 opened this issue Sep 12, 2023 · 2 comments · Fixed by #1683

Comments

@yzeng25
Copy link
Contributor

yzeng25 commented Sep 12, 2023

Current State

Description of the current state/issue:

  1. feat : Markdown Link Check Action #1656 was merged on Sept 1st.
  2. Before Sept 1st, most pull requests' ci checks only take less than 5 minutes, see master branch commit history.
  3. After Sept 1st, blog: add release notes for 3.5.0 minor release #1672 's ci checks took 61 minute; fix: fix link in 3.5.0 release notes #1673 's checks took 70 minutes; doc: release APISIX 3.5.0 #1674 's took 85 minutes; #1677 : Changed Twitter Logo to X #1678 's checks took 75 minutes.
  4. docs: add APISIX-integrates-with-Coraza.md #1676 , #1677 : Changed Twitter Logo to X #1678, and site: Update twitter new x icon #1681 failed with similar error message, see their error logs: 1676, 1678, and 1681. The error messages are saying the sitemap is missing.

Impacts

  1. Modifications from docs: add APISIX-integrates-with-Coraza.md #1676 , #1677 : Changed Twitter Logo to X #1678, and site: Update twitter new x icon #1681 are not showing up in the production environment.
  2. It takes 60-90 minutes to wait for the ci check result, and the result is not a guaranteed pass.

Desired State

Description of the change you are proposing:

I would suggest revert #1656 . and use the previous ci check components.

Would you like to fix this issue?

No. Please assign another contributor

@yzeng25
Copy link
Contributor Author

yzeng25 commented Sep 12, 2023

@yzeng25
Copy link
Contributor Author

yzeng25 commented Sep 12, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant