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

[stable-2.8] Revert "Disable hcloud tests until issues are resolved." #55337

Merged
merged 1 commit into from Apr 16, 2019

Conversation

Projects
None yet
4 participants
@mattclay
Copy link
Member

commented Apr 15, 2019

SUMMARY

This reverts commit 216cd86.

Backport of #55333

(cherry picked from commit a8955b0)

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

hcloud integration tests

[stable-2.8] Revert "Disable hcloud tests until issues are resolved."
This reverts commit 216cd86.
(cherry picked from commit a8955b0)

Co-authored-by: Matt Clay <matt@mystile.com>
@ansibot

This comment has been minimized.

@mattclay mattclay removed the needs_triage label Apr 15, 2019

@LKaemmerling

This comment has been minimized.

Copy link
Contributor

commented Apr 15, 2019

shipit

@ansibot ansibot added shipit and removed community_review labels Apr 15, 2019

@mattclay

This comment has been minimized.

Copy link
Member Author

commented Apr 15, 2019

Temporarily closing this, as it seems the 403 Forbidden errors have returned now that the tests have been enabled again.

@mattclay mattclay closed this Apr 15, 2019

@LKaemmerling

This comment has been minimized.

Copy link
Contributor

commented Apr 15, 2019

I can not find the error on our end. (In the CI it is still working?https://app.shippable.com/github/ansible/ansible/runs/119327/summary)

@mattclay mattclay reopened this Apr 15, 2019

@mattclay

This comment has been minimized.

Copy link
Member Author

commented Apr 15, 2019

This issue should hopefully be resolved now.

@abadger abadger merged commit fc37aed into ansible:stable-2.8 Apr 16, 2019

1 check passed

Shippable Run 119351 status is SUCCESS.
Details
@abadger

This comment has been minimized.

Copy link
Member

commented Apr 16, 2019

Merged for the 2.8.0rc1 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.