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

pre-workflow-hook step fails due to a websocket error #2993

Closed
mcrivar opened this issue Jan 17, 2023 · 5 comments
Closed

pre-workflow-hook step fails due to a websocket error #2993

mcrivar opened this issue Jan 17, 2023 · 5 comments
Labels
bug Something isn't working hook

Comments

@mcrivar
Copy link

mcrivar commented Jan 17, 2023

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request. Searching for pre-existing feature requests helps us consolidate datapoints for identical requirements into a single place, thank you!
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Overview of the Issue

image

  • I'm not sure if its output should be present using details as stated in the docs: "Pre workflow hooks are run outside of Atlantis commands. Which means they do not surface their output back to the PR as a comment."
  • Also if the failure of this step is due to WS connection error or the actual hook failure.
    this is the only error I receive from UI on details access for this step
WebSocket connection to 'wss://<URL>/jobs/d1c71b04-7810-40bb-8532-7763b065673b/ws' failed

I will check if I find any errors with the hook which might be an internal issue although I don't experience any impact on tf execution commands, just having it present now is troublesome and fails our CI flow.

If this step could be committed from the outputs and steps - that would be great.

@mcrivar mcrivar added the bug Something isn't working label Jan 17, 2023
@mcrivar
Copy link
Author

mcrivar commented Jan 17, 2023

so apparently the failure of this step was due to an issue with the hook script, as it was fixed - the failure of this step was gone.

remains just the WS connection issue but again, I guess it was intended to be like that.

@Fabianoshz
Copy link
Contributor

Hi @mcrivar, the websocket error is not expected, do the plan and apply details link work?

@nitrocode nitrocode changed the title pre-workflow-hook step in pr after upgrade from v0.18.2 to v0.22.2 pre-workflow-hook step fails due to a websocket error Jan 17, 2023
@mcrivar
Copy link
Author

mcrivar commented Jan 17, 2023

@Fabianoshz yes they work just fine.

@mcrivar
Copy link
Author

mcrivar commented Jan 18, 2023

@Fabianoshz just to clarify, the step is not failing anymore as the reason for the failure was the actual hook script that was running and had some issues with it. ( the changed name of the issue misrepresents its a bit )

It is just the -details- option that is available now upon this step addition, which does not present any logging of this hook script and has the WS error.

@nitrocode
Copy link
Member

To make this easier for others to reproduce, could you share your custom workflow config, output of the working run, and any other details that might help?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hook
Projects
None yet
Development

No branches or pull requests

3 participants