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

🚮 Removed dynatrace, as it is no longer supporting amp #40057

Merged
merged 2 commits into from
Aug 13, 2024

Conversation

kamtschatka
Copy link
Contributor

Dynatrace is removing support for AMP
The removed section contained a link to the dynatrace documentation, which no longer works and redirects to another page: https://www.dynatrace.com/technologies/web/amp-monitoring/

We have also announced the removal of AMP already on our webpage for some time: https://docs.dynatrace.com/docs/platform-modules/digital-experience/web-applications/initial-setup/set-up-amp-monitoring. It shows that starting with version 272, it will no longer be possible to create new AMP applications. For reference, right now we are in the process of rolling out version ~293 (depending on the component, but you get the picture. Versions are increased every 2 weeks, so it is about 1 year since the announcement)

This is the PR where I added the feature in 2017: 096efcd

@CLAassistant
Copy link

CLAassistant commented Jun 17, 2024

CLA assistant check
All committers have signed the CLA.

@powerivq
Copy link
Contributor

powerivq commented Aug 8, 2024

@kamtschatka Sorry this slipped through because it is unassigned. I see that your announcement seems to mean that older version of Dynatrace still supports AMP. Do you really want to remove your code here? It will have the impact of immediately cutting off all AMP publishers from your platform, no matter what version they are using.

Starting with Dynatrace version 1.272, it isn't possible to create an AMP application, and we will discontinue support for AMP application monitoring sometime in the future.

@powerivq powerivq self-assigned this Aug 8, 2024
@powerivq powerivq self-requested a review August 8, 2024 17:46
@kamtschatka
Copy link
Contributor Author

Hi, thanks for doublechecking with me.
We ran an analysis on our customers' AMP usage and AMP apps are basically unused. Additionally these versions mentioned above are no longer supported (we support 1 year back), so that is totally fine.

@powerivq
Copy link
Contributor

@kamtschatka Please rebase against latest main branch to have the build pass.

@powerivq powerivq merged commit 32540e3 into ampproject:main Aug 13, 2024
52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants