Page MenuHomePhabricator

Update AQS yaml format to match new convention {melc}
Closed, ResolvedPublic3 Estimated Story Points

Description

With https://github.com/wikimedia/restbase/commit/8a7b4362638f48ce03f8def0440edbb803993c18 the services team is now using {{...}} instead of {$....} or {}. This broke some of our tests.

In the future, @Pchelolo, we would appreciate if you would ping us about this. Or publish a "Breaking Changes" notice somewhere.

Related Objects

Event Timeline

Change 271535 had a related patch set uploaded (by Milimetric):
Use {{...}} in configs

https://gerrit.wikimedia.org/r/271535

Change 271535 merged by Ppchelko:
Use {{...}} in configs

https://gerrit.wikimedia.org/r/271535

@Milimetric We're trying to follow the server semantics with HyperSwitch and all the dependencies, so that braking changes shouldn't be automatically picked up, but looks like this time we've messed it up.

We've made one more breaking change, in HyperSwitch 0.2.0, and I will update AQS shortly after T126294 is done.

Thanks, @Pchelolo, but you don't need to update AQS, I'm ok with doing that as long as I know to do it.

Milimetric renamed this task from Update AQS yaml format to match new convention {melc} to Update AQS yaml format to match new convention {melc} [3 pts].Feb 22 2016, 5:44 PM
Milimetric renamed this task from Update AQS yaml format to match new convention {melc} [3 pts] to Update AQS yaml format to match new convention {melc}.