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

RSE-979: Fix: Remove dependency from rd-api-client #8852

Merged
merged 5 commits into from Feb 7, 2024

Conversation

Jesus-Osuna-M
Copy link
Contributor

@Jesus-Osuna-M Jesus-Osuna-M commented Jan 28, 2024

Related to
https://pagerduty.atlassian.net/browse/RSE-401

Problem: A dependency on the rd-api-client external repository was introduced on this PR, #8504

Solution: Revert changes that removed the RundeckApi and create a new RundeckClient to allow to export projects to other instances via API without depending on the rd-api-client

@Jesus-Osuna-M Jesus-Osuna-M added this to the 5.2.0 milestone Jan 29, 2024
@Jesus-Osuna-M Jesus-Osuna-M marked this pull request as ready for review January 29, 2024 16:42
@Jesus-Osuna-M Jesus-Osuna-M force-pushed the rse-979-revert-dependency-on-rd-api-client branch 2 times, most recently from 97133e9 to 810d2e0 Compare January 30, 2024 13:26
@Jesus-Osuna-M Jesus-Osuna-M force-pushed the rse-979-revert-dependency-on-rd-api-client branch 2 times, most recently from a2634b4 to bbf63db Compare February 1, 2024 16:12
@Jesus-Osuna-M Jesus-Osuna-M force-pushed the rse-979-revert-dependency-on-rd-api-client branch from bbf63db to e9b20d6 Compare February 7, 2024 13:13
@Jesus-Osuna-M Jesus-Osuna-M merged commit 0e9bd5a into main Feb 7, 2024
3 checks passed
@Jesus-Osuna-M Jesus-Osuna-M deleted the rse-979-revert-dependency-on-rd-api-client branch February 7, 2024 14:54
@fdevans fdevans changed the title RSE-979 Fix: Remove dependency from rd-api-client RSE-979: Fix: Remove dependency from rd-api-client Mar 7, 2024
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.

None yet

2 participants