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

State stores: expose TTL as a feature #2987

Merged
merged 7 commits into from
Aug 4, 2023

Conversation

ItalyPaleAle
Copy link
Contributor

@ItalyPaleAle ItalyPaleAle commented Jul 12, 2023

State stores that support TTLs now expose the TTL feature. This can be returned by the runtime when retrieving the metadata.

Updated conformance tests to ensure the feature is present when the "ttl" test is enabled, or missing otherwise (with the exception of CF Workers KV, for which TTLs are not tested in the conformance tests).

@ItalyPaleAle ItalyPaleAle requested review from a team as code owners July 12, 2023 20:11
@ItalyPaleAle ItalyPaleAle added this to the v1.12 milestone Jul 12, 2023
@dapr-bot

This comment was marked as outdated.

@dapr-bot

This comment was marked as outdated.

@dapr-bot

This comment was marked as outdated.

@dapr-bot

This comment was marked as resolved.

@dapr-bot

This comment was marked as resolved.

@dapr-bot

This comment was marked as resolved.

@ItalyPaleAle

This comment was marked as resolved.

@ItalyPaleAle ItalyPaleAle marked this pull request as draft July 13, 2023 01:31
@ItalyPaleAle
Copy link
Contributor Author

/ok-to-test

@dapr-bot
Copy link
Collaborator

dapr-bot commented Aug 2, 2023

Complete Build Matrix

The build status is currently not updated here. Please visit the action run below directly.

🔗 Link to Action run

Commit ref: 912baa2

@dapr-bot
Copy link
Collaborator

dapr-bot commented Aug 2, 2023

Components conformance test

🔗 Link to Action run

Commit ref: 912baa2

✅ All conformance tests passed

All tests have reported a successful status

@dapr-bot
Copy link
Collaborator

dapr-bot commented Aug 2, 2023

Components certification test

🔗 Link to Action run

Commit ref: 912baa2

❌ Some certification tests failed

These tests failed:

  • pubsub.pulsar

@ItalyPaleAle ItalyPaleAle marked this pull request as ready for review August 2, 2023 22:20
@ItalyPaleAle ItalyPaleAle merged commit b10ce96 into dapr:master Aug 4, 2023
82 of 83 checks passed
@ItalyPaleAle ItalyPaleAle deleted the ttl-feature-statestores branch August 4, 2023 16:38
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.

3 participants