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

🌱 Update install script with debug output #252

Merged
merged 4 commits into from
Jul 26, 2023

Conversation

aufi
Copy link
Member

@aufi aufi commented Jul 24, 2023

The Tackle/Konveyor install script has few waiting loops checking the installation progress/completeness. Based on last few errors, adding an output with tackle details when waiting loop timeouted to help with debugging CI errors directly in github workflow execution.

Example with recent installation error:

...
error: timed out waiting for the condition on tackles/tackle
...
status:
  conditions:
  - lastTransitionTime: "2023-07-24T08:51:07Z"
    message: ""
    reason: ""
    status: "False"
    type: Successful
  - ansibleResult:
      changed: 0
      completion: 2023-07-24T09:05:50.668218
      failures: 1
      ok: 14
      skipped: 48
    lastTransitionTime: "2023-07-24T09:05:51Z"
    message: |
      The task includes an option with an undefined variable. The error was: 'hub_seed_configmap_name' is undefined

      The error appears to be in '/opt/ansible/roles/tackle/tasks/main.yml': line 406, column 3, but may
      be elsewhere in the file depending on the exact syntax problem.

      The offending line appears to be:


      - name: "Setup Hub API Deployment"
        ^ here
    reason: Failed
    status: "False"
    type: Failure
...

Related to #133

Signed-off-by: Marek Aufart <maufart@redhat.com>
@aufi aufi changed the title Update install script with debug output 🌱 Update install script with debug output Jul 24, 2023
Signed-off-by: Marek Aufart <maufart@redhat.com>
@aufi aufi marked this pull request as ready for review July 25, 2023 06:55
@aufi aufi requested a review from djzager July 25, 2023 06:56
@djzager
Copy link
Member

djzager commented Jul 25, 2023

This looks like it covers at least a portion of #133; do you think you could check to see if you could cover whatever remains of that issue?

aufi added 2 commits July 26, 2023 08:39
Signed-off-by: Marek Aufart <maufart@redhat.com>
Signed-off-by: Marek Aufart <maufart@redhat.com>
@djzager
Copy link
Member

djzager commented Jul 26, 2023

Thank you

@aufi aufi merged commit 0ae765e into konveyor:main Jul 26, 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 this pull request may close these issues.

2 participants