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

Updating mgo dependency to bring in latest changes. #15510

Merged
merged 1 commit into from
Apr 21, 2023
Merged

Conversation

tlm
Copy link
Member

@tlm tlm commented Apr 20, 2023

Updates the mgo lib in 3.1 onwards. This to bring in the latest change that deals with some macos testing issues.

Checklist

  • Code style: imports ordered, good names, simple structure, etc
  • Comments saying why design decisions were made
  • Go unit tests, with comments saying what you're testing
  • Integration tests, with comments saying what you're testing
  • doc.go added or updated in changed packages

QA steps

Run juju tests

Documentation changes

N/A

Bug reference

N/A

- Specifically testing fix for macos
@tlm
Copy link
Member Author

tlm commented Apr 20, 2023

/merge

2 similar comments
@tlm
Copy link
Member Author

tlm commented Apr 20, 2023

/merge

@barrettj12
Copy link
Contributor

/merge

@jujubot jujubot merged commit 4e9c2e9 into juju:3.1 Apr 21, 2023
jujubot added a commit that referenced this pull request Apr 23, 2023
#15523

Merge 3.1 into 3.2

4d12a59 (upstream/3.1) Merge pull request #15520 from ycliuhw/merge-3.0-20230421
4e9c2e9 Merge pull request #15510 from tlm/mgo-update

There were some conflicts. Please @jack-w-shaw take a look at the firewall and spaces_ec2 to be sure it makes sense the changes I did.
```
Auto-merging apiserver/facades/client/application/application.go
Auto-merging apiserver/facades/client/application/application_unit_test.go
Auto-merging go.mod
CONFLICT (content): Merge conflict in go.mod
Auto-merging go.sum
CONFLICT (content): Merge conflict in go.sum
Auto-merging provider/openstack/provider.go
Auto-merging state/application.go
Auto-merging state/migration_import_input_mock_test.go
Auto-merging state/migration_import_tasks.go
Auto-merging state/migration_import_tasks_test.go
Auto-merging tests/main.sh
CONFLICT (content): Merge conflict in tests/main.sh
Auto-merging tests/suites/firewall/task.sh
CONFLICT (add/add): Merge conflict in tests/suites/firewall/task.sh
Auto-merging tests/suites/spaces_ec2/util.sh
CONFLICT (content): Merge conflict in tests/suites/spaces_ec2/util.sh
Automatic merge failed; fix conflicts and then commit the result.
```
@barrettj12 barrettj12 mentioned this pull request May 9, 2023
jujubot added a commit that referenced this pull request May 9, 2023
@barrettj12 barrettj12 mentioned this pull request May 10, 2023
jujubot added a commit that referenced this pull request May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants