Merge lp:~tcuthbert/wordpress/openstack-objectstorage-k8s into lp:~canonical-sysadmins/wordpress/openstack-objectstorage-k8s

Proposed by Thomas Cuthbert
Status: Merged
Approved by: Thomas Cuthbert
Approved revision: 7
Merged at revision: 7
Proposed branch: lp:~tcuthbert/wordpress/openstack-objectstorage-k8s
Merge into: lp:~canonical-sysadmins/wordpress/openstack-objectstorage-k8s
Diff against target: 29 lines (+3/-3)
1 file modified
src/OpenStack/ObjectStore/v1/Resource/Container.php (+3/-3)
To merge this branch: bzr merge lp:~tcuthbert/wordpress/openstack-objectstorage-k8s
Reviewer Review Type Date Requested Status
Stephen Muss (community) Approve
Joel Sing (community) +1 Approve
Review via email: mp+386794@code.launchpad.net

Commit message

Rename missing 'Object' references to new 'SwiftObject' name

To post a comment you must log in.
Revision history for this message
Joel Sing (jsing) wrote :

LGTM

review: Approve (+1)
Revision history for this message
Stephen Muss (stephenmuss) wrote :

lgtm

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change must be approved by a member of one of the following teams:
Setting status to needs review.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change must be approved by a member of one of the following teams:
Setting status to needs review.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 7

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/OpenStack/ObjectStore/v1/Resource/Container.php'
--- src/OpenStack/ObjectStore/v1/Resource/Container.php 2015-02-11 15:35:31 +0000
+++ src/OpenStack/ObjectStore/v1/Resource/Container.php 2020-07-03 05:41:30 +0000
@@ -451,7 +451,7 @@
451 * @throws \OpenStack\Common\Exception when an unexpected (usually451 * @throws \OpenStack\Common\Exception when an unexpected (usually
452 * network-related) error condition arises.452 * network-related) error condition arises.
453 */453 */
454 public function save(Object $obj, $file = null)454 public function save(SwiftObject $obj, $file = null)
455 {455 {
456 if (empty($this->token)) {456 if (empty($this->token)) {
457 throw new Exception('Container does not have an auth token.');457 throw new Exception('Container does not have an auth token.');
@@ -555,7 +555,7 @@
555 * @throws \OpenStack\Common\Transport\Exception\ResourceNotFoundException if the object does not already555 * @throws \OpenStack\Common\Transport\Exception\ResourceNotFoundException if the object does not already
556 * exist on the object storage.556 * exist on the object storage.
557 */557 */
558 public function updateMetadata(Object $obj)558 public function updateMetadata(SwiftObject $obj)
559 {559 {
560 $url = self::objectUrl($this->url, $obj->name());560 $url = self::objectUrl($this->url, $obj->name());
561 $headers = ['X-Auth-Token' => $this->token];561 $headers = ['X-Auth-Token' => $this->token];
@@ -607,7 +607,7 @@
607 * set, the object will be saved into this container. If this is not sent,607 * set, the object will be saved into this container. If this is not sent,
608 * the copy will be performed inside of the original container.608 * the copy will be performed inside of the original container.
609 */609 */
610 public function copy(Object $obj, $newName, $container = null)610 public function copy(SwiftObject $obj, $newName, $container = null)
611 {611 {
612 $sourceUrl = self::objectUrl($this->url, $obj->name());612 $sourceUrl = self::objectUrl($this->url, $obj->name());
613613

Subscribers

People subscribed via source and target branches