Skip to content

Conversation

mattn
Copy link
Contributor

@mattn mattn commented Dec 15, 2021

OCI server accept milliseconds not nanoseconds for any datetime formats. I noticed this issue because oracle/oci-dotnet-sdk does not get errors for loggingingestion.PutLogs but Go get errors. Python SDK might have same issue.

Related issue:
oracle/oci-python-sdk#333

OCI server accept millisecond not nanosecond
@jodoglevy
Copy link
Contributor

@mattn thanks for filing this, however the changes you've made would change the entire SDK to not have nanosecond precision. I think rather than changing the SDK, the service Logging Ingestion's PutLogs operation should accept nanosecond precision. I will follow up with the service team on that

@mattn
Copy link
Contributor Author

mattn commented Dec 16, 2021

Yes, this is POC to let you known this issue. Feel free to close this after fixing your service. Thanks.

@mattn
Copy link
Contributor Author

mattn commented Dec 27, 2021

Well, do you have plan to fix this issue in a few days/weeks or months?

@jodoglevy
Copy link
Contributor

Hi @mattn - we are reviewing the issue and will get back with an estimate

@davidkyesOCI
Copy link

@mattn We have reviewed and will try to have deployed by end of month.

@davidkyesOCI
Copy link

Quick update, we are working on this change but its not ready for release yet. Will send out an update once we have clear dates for release.

@davidkyesOCI
Copy link

The fix has been made and tested. It is being deployed to production regions this week. Thank you for your patience.

@mattn
Copy link
Contributor Author

mattn commented Feb 9, 2022

Thank you. Closing.

@mattn mattn closed this Feb 9, 2022
@mattn mattn reopened this Feb 9, 2022
@mattn
Copy link
Contributor Author

mattn commented Feb 9, 2022

Ah, sorry. Yet not deployed on my region. I'll close this if confirmed on my region.

@davidkyesOCI
Copy link

Deployment to OC1 Realm is completed now.

@jodoglevy
Copy link
Contributor

@mattn this should be all set for you now to use. Can you try again and let us know if the problem is resolved?

@mattn
Copy link
Contributor Author

mattn commented Feb 17, 2022

I confirmed OCI works without my changes. Thank you.

@mattn mattn closed this Feb 17, 2022
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