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.

Sign up for GitHub

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

Jump to bottom

Update several dependencies to latest versions #3022

Merged
merged 1 commit into from
Oct 14, 2022
Merged

Update several dependencies to latest versions #3022

merged 1 commit into from
Oct 14, 2022

Conversation

cshannon
Copy link
Contributor

No description provided.

@dlmarion
Copy link
Contributor

Updating OpenTelemetry and Micrometer here requires an update in the Terraform code in accumulo-testing. You can look at the prior commits in accumulo-testing to see what needs to change.

@EdColeman
Copy link
Contributor

@dlmarion - would it be feasible to have the teraform configuration inherit package versions from the main accumulo.pom - that would eliminate the need to manually sync configuration changes.

@dlmarion
Copy link
Contributor

It's not just the code, it's the README as well. We could remove that from the README I guess. I'll create a ticket to investigate.

@dlmarion
Copy link
Contributor

Created apache/accumulo-testing#246

@cshannon cshannon mentioned this pull request Oct 14, 2022
Merged
@cshannon
Copy link
Contributor Author

Updating OpenTelemetry and Micrometer here requires an update in the Terraform code in accumulo-testing. You can look at the prior commits in accumulo-testing to see what needs to change.

Done: apache/accumulo-testing#247

@cshannon cshannon self-assigned this Oct 14, 2022
dlmarion
dlmarion approved these changes Oct 14, 2022
View reviewed changes
Copy link
Contributor

@dlmarion dlmarion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cshannon
Copy link
Contributor Author

I'll merge once the checks finish

@DomGarguilo
Copy link
Member

I may have found a few more that could be included here. I'll make a PR against this branch soon.

@cshannon
Copy link
Contributor Author

I may have found a few more that could be included here. I'll make a PR against this branch soon.

I will just merge this one and if you find some more you can open another PR against main

@cshannon cshannon merged commit 4df8ae2 into apache:main Oct 14, 2022
@cshannon cshannon deleted the dependencyUpdates branch October 14, 2022 15:44
@ctubbsii ctubbsii added this to the 2.1.0 milestone Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlmarion dlmarion dlmarion approved these changes

Assignees

@cshannon cshannon

Labels
None yet
Projects
None yet
Milestone
2.1.0
Development

Successfully merging this pull request may close these issues.
5 participants
@cshannon @dlmarion @EdColeman @DomGarguilo @ctubbsii