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

Add the "Internal Exception" Troubleshooting #9231

Closed
wants to merge 1 commit into from

Conversation

ersonjn
Copy link

@ersonjn ersonjn commented Sep 20, 2024

Summary

Pantheon YML Troubleshooting - This is to give the Pantheon customers and clients temporary official documentation for the workaround while this is still being addressed for better error handling/messaging.

This is to give the Pantheon customers and clients a temporary official documentation for the workaround while this is still being addressed for better error handling/messaging.
@ersonjn ersonjn requested a review from a team as a code owner September 20, 2024 03:30
```

To resolve this, create multidev first before pushing the `pantheon.yml` changes.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could not reproduce this issue on a d10 Integrated Composer performance small site and a vanilla WP sandbox site. In both cases, the push was successful without existing multidevs.

On both test sites, I checked out a new branch locally and made changes to pantheon.yml (upgraded PHP versions), committed, and pushed with the following result:

remote:
remote: PANTHEON NOTICE:
remote:
remote: Skipping code sync, no Multidev environments were found for branch "php8-3".
remote:

@ersonjn can you provide the full error message output for the support issue related to this docs PR? I expect something else caused this internal error and the surrounding output should help identify. Or if you have the ticket number?

@jazzsequence can you recommend an SME for this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rachelwhitton Not sure about SME but @greg-1-anderson might have some context...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure about the cause of the reported error. It happens intermittently, and rarely. I think it only happens if the name selected refers to a multidev that used to exist, but no longer does. I don't think that you can reproduce this simply by creating and deleting a multidev; there is something more complex at work.

Since this problem happens only rarely, and the symptoms might also be indicative of a different platform problem, I don't think that we should include this workaround in our public docs.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks all! I'll close this PR for now

@rachelwhitton rachelwhitton added Process: Needs SME Issue or PR that won't move forward without subject-matter expert contribution Topic: Code Structure Related to Upstreams and other code structure details Topic: Git Type: New Content Request for or PR containing new content to existing page labels Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Process: Needs SME Issue or PR that won't move forward without subject-matter expert contribution Topic: Code Structure Related to Upstreams and other code structure details Topic: Git Type: New Content Request for or PR containing new content to existing page
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants