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

fix: Skip download of ready-made documents from apps #1674

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

arthurpitman
Copy link
Contributor

@arthurpitman arthurpitman commented Jan 21, 2025

With this PR only documents with no specified originAppId are downloaded, thus skipping the download of all "ready-made documents" for apps. This is important as these documents cannot be deployed.

@arthurpitman arthurpitman added the run-e2e-test Manually trigger the E2E tests for reviewed PRs label Jan 21, 2025
Copy link

github-actions bot commented Jan 21, 2025

Unit Test Results

1 972 tests  ±0   1 971 ✅ ±0   55s ⏱️ -1s
  135 suites ±0       1 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 43510ea. ± Comparison against base commit 7603f3b.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 21, 2025

E2E Test Results

    4 files   -   1    272 suites   - 135   26m 26s ⏱️ - 54m 39s
2 091 tests  -   2  2 085 ✅  -   2  2 💤 ±0  4 ❌ ±0 
2 243 runs   - 154  2 237 ✅  - 154  2 💤 ±0  4 ❌ ±0 

For more details on these failures, see this check.

Results for commit 1fbc33f. ± Comparison against base commit a928904.

This pull request removes 2 tests.
github.com/dynatrace/dynatrace-configuration-as-code/v2/cmd/monaco/integrationtest/v2 ‑ TestPaginationClassic
github.com/dynatrace/dynatrace-configuration-as-code/v2/cmd/monaco/integrationtest/v2 ‑ TestPaginationPlatform

♻️ This comment has been updated with latest results.

@arthurpitman arthurpitman added run-e2e-test Manually trigger the E2E tests for reviewed PRs and removed run-e2e-test Manually trigger the E2E tests for reviewed PRs labels Jan 21, 2025
arthurpitman added a commit to Dynatrace/dynatrace-configuration-as-code-core that referenced this pull request Jan 22, 2025
This PR adds `originAppId` to `documents.Metadata`, as needed for
Dynatrace/dynatrace-configuration-as-code#1674
@arthurpitman arthurpitman force-pushed the fix/only-dowload-documents-without-origin-app-id branch from 1fbc33f to 609fa4d Compare January 22, 2025 07:23
@arthurpitman arthurpitman changed the title fix: WIP skip download of ready-made documents fix: Skip download of ready-made documents from apps Jan 22, 2025
@arthurpitman arthurpitman force-pushed the fix/only-dowload-documents-without-origin-app-id branch from 849964d to 843436f Compare January 22, 2025 07:46
@arthurpitman arthurpitman force-pushed the fix/only-dowload-documents-without-origin-app-id branch from 843436f to 43510ea Compare January 22, 2025 07:48
@arthurpitman arthurpitman marked this pull request as ready for review January 22, 2025 13:34
@arthurpitman arthurpitman requested a review from a team as a code owner January 22, 2025 13:34
@arthurpitman arthurpitman merged commit 5beebaf into main Jan 22, 2025
12 checks passed
@arthurpitman arthurpitman deleted the fix/only-dowload-documents-without-origin-app-id branch January 22, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-e2e-test Manually trigger the E2E tests for reviewed PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants