Skip to content

Conversation

@schloerke
Copy link
Collaborator

@schloerke schloerke commented Aug 29, 2024

Configs:

  • (New) Deploy to connect.posit.co; pypi shiny and rsconnect
    • Determine if deployment can happen
  • (New) Deploy to dogfood; pypi shiny and rsconnect
    • Determine if deployment can happen to dogfood server
  • (New) Deploy to dogfood; pypi shiny and github rsconnect
    • Determine if deployment can happen with latest rsconnect
  • (New) Deploy to dogfood; github [email protected] and github rsconnect
    • Determine if deployment can happen with a github shiny package
  • (Previously existed) Deploy to dogfood; github shiny and rsconnect
    • Determine if deployment can happen with the latest github version of shiny package

When errors occur on deployment, we will now be able to quickly determine if the dogfood server, rsconnect package, or shiny is the reason.

@schloerke schloerke added chore A maintenance task that doesn't affect functionality testing Related to testing shiny apps labels Aug 29, 2024
@schloerke
Copy link
Collaborator Author

The GitHub installs now fail on dogfood. This should currently be working.

Relay from @karangattu

I can reprex that when installing packages from github, it fails on rsc.radixu.com

The goal of this PR is achieved. We can now determine where the install process fails on connect deployment testing.

Merging PR

@schloerke schloerke marked this pull request as ready for review August 29, 2024 20:31
@karangattu
Copy link
Collaborator

The GitHub installs now fail on dogfood. This should currently be working.

Relay from @karangattu

I can reprex that when installing packages from github, it fails on rsc.radixu.com

The goal of this PR is achieved. We can now determine where the install process fails on connect deployment testing.

Merging PR

Thank you!

@schloerke schloerke merged commit 24013f4 into main Aug 29, 2024
@schloerke schloerke deleted the deploy-add-vanilla-tests branch August 29, 2024 22:56
schloerke added a commit to machow/py-shiny that referenced this pull request Sep 5, 2024
* main:
  CI(deploy): Add more installation configs to surface failure cause (posit-dev#1658)
  `Chat.messages()` no longer trims messages by default (posit-dev#1657)
  tests(controllers): Split _controls.py into separate files (posit-dev#1652)
  Chat tweaks (posit-dev#1607)
  docs: Add modal_show/remove examples (posit-dev#1628)
  Delay sending of chat UI messages until reactive graph is flushed (posit-dev#1593)
  ci(remove future behavior warning): Set the `asyncio_default_fixture_loop_scope` to `fixture` (posit-dev#1655)
  bug: Verify mypy can run on CI (posit-dev#1650)
  Fix CI install failures on Windows (posit-dev#1651)
  Quartodoc 0.7.6 (posit-dev#1636)
  Allow `@chat.transform_assistant_response` function to return `None` (posit-dev#1641)
  feat: Support templates with `_template.json` metadata (posit-dev#1631)
  Fix KeyError when serving static files (posit-dev#1648)
  tests(navsets): Add navsets kitchensink tests (posit-dev#1602)
  Change default claude model to 3.5 sonnet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore A maintenance task that doesn't affect functionality testing Related to testing shiny apps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants