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

breaking changes: Woodpecker CI updates and deprecations #2335

Conversation

uncomfyhalomacro
Copy link
Contributor

@uncomfyhalomacro uncomfyhalomacro commented Nov 8, 2023

  • Woodpecker CI is no longer backwards compatible with Drone due to lots of deprecated syntax and env vars

https://woodpecker-ci.org/docs/next/migrations#next-200

- Woodpecker CI is no longer backwards compatible with Drone
  due to lots of deprecated syntax and env vars

Signed-off-by: Soc Virnyl Estela <[email protected]>
Signed-off-by: Soc Virnyl Estela <[email protected]>
@uncomfyhalomacro
Copy link
Contributor Author

uncomfyhalomacro commented Nov 8, 2023

I will fix this over the weekend. :)

EDIT: i found the fix earlier lol

Signed-off-by: Soc Virnyl Estela <[email protected]>
- FORGE_URL is removed. CI_FORGE_URL is favored.
- Removed CI_REPO_LINK. We already have CI_FORGE_URL alongside CI_REPO as needed info
- authenticated_repo_url is updated. the new auth url structure is used
- updated documentation for Woodpecker struct

Signed-off-by: Soc Virnyl Estela <[email protected]>
@mortenpi
Copy link
Member

mortenpi commented Nov 9, 2023

So if I understand correctly, this will now only work in pipelines running on Woodpecker 2? And it would start failing on any Woodpecker 1 installations?

@uncomfyhalomacro
Copy link
Contributor Author

uncomfyhalomacro commented Nov 9, 2023

So if I understand correctly, this will now only work in pipelines running on Woodpecker 2? And it would start failing on any Woodpecker 1 installations?

Yes and No. The previous PR i made before was pre-1.x, i think specifically 0.15.x (?). The latest stable release version 1.0.0 starts to deprecate but not remove some old environmental variables e.g. CI_BUILD* becomes CI_PIPELINE*. In the 2.0.0 next branch which is not released yet, those deprecated becomes "removed" so those env vars won't exist anymore.

@mortenpi
Copy link
Member

mortenpi commented Nov 9, 2023

Ah, right-right. But this would fail for anyone using Woodpecker 0.15? And this is theoretically possible, since it can be self-hosted.. actually.. I am a little bit confused -- does Woodpecker even offer a central service? Or is it always self-hosted?

I feel that to do this properly, we would need to version guard things, if that's something that can be done. I.e. use different sets of environment variables as needed. Can we detect 0.15 vs 1? Otherwise, I feel that this would constitute dropping 0.15 support, which would be a breaking change for Documenter.

@uncomfyhalomacro
Copy link
Contributor Author

Ah, right-right. But this would fail for anyone using Woodpecker 0.15? And this is theoretically possible, since it can be self-hosted.. actually.. I am a little bit confused -- does Woodpecker even offer a central service? Or is it always self-hosted?

I feel that to do this properly, we would need to version guard things, if that's something that can be done. I.e. use different sets of environment variables as needed. Can we detect 0.15 vs 1? Otherwise, I feel that this would constitute dropping 0.15 support, which would be a breaking change for Documenter.

Sure. I can help on that. I will check out the docs later. :D

@uncomfyhalomacro
Copy link
Contributor Author

Ah, right-right. But this would fail for anyone using Woodpecker 0.15? And this is theoretically possible, since it can be self-hosted.. actually.. I am a little bit confused -- does Woodpecker even offer a central service? Or is it always self-hosted?

I feel that to do this properly, we would need to version guard things, if that's something that can be done. I.e. use different sets of environment variables as needed. Can we detect 0.15 vs 1? Otherwise, I feel that this would constitute dropping 0.15 support, which would be a breaking change for Documenter.

Seems we can detect the version with CI_SYSTEM_VERSION I will see how to work with it once I am done with things. Possibly I will ping again for a code review next week

@mortenpi
Copy link
Member

Re #2339, this one we can close, right?

@uncomfyhalomacro
Copy link
Contributor Author

Yes. Closed now. As for the superseding PR, 1.0.x and 2.0.x are compatible except some removed support for other environmental variables pre-1.0.x

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 this pull request may close these issues.

2 participants