Skip to content
This repository has been archived by the owner on Aug 18, 2020. It is now read-only.

[DEVOPS-1172] faucet: Use OptimizeForHighThroughput grouping policy #3934

Merged
merged 2 commits into from
Dec 6, 2018

Conversation

rvl
Copy link
Contributor

@rvl rvl commented Dec 6, 2018

Description

Fix faucet withdrawals which occasionally fail due to unavailable UTxO.

Linked issue

https://iohk.myjetbrains.com/youtrack/issue/DEVOPS-1172

Type of change

  • Bug fix in faucet -- i.e. testnet only.

Testing

This is done with a local build with a faucet connected to the local Daedalus wallet.

  • Successful withdrawals with a local faucet and wallet.
  • Withdrawal transaction does not result in grouping of UTxO. See transaction in explorer

@rvl rvl requested review from commandodev and a team December 6, 2018 07:49
Copy link
Contributor

@commandodev commandodev left a comment

Choose a reason for hiding this comment

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

LGTM!

@rvl
Copy link
Contributor Author

rvl commented Dec 6, 2018

Thanks

bors r+

iohk-bors bot added a commit that referenced this pull request Dec 6, 2018
3934: [DEVOPS-1172] faucet: Use OptimizeForHighThroughput grouping policy r=rvl a=rvl

## Description

Fix faucet withdrawals which occasionally fail due to unavailable UTxO.

## Linked issue

https://iohk.myjetbrains.com/youtrack/issue/DEVOPS-1172



Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Dec 6, 2018

@iohk-bors iohk-bors bot merged commit 46dc932 into release/2.0.0 Dec 6, 2018
@iohk-bors iohk-bors bot deleted the devops-1172-faucet-transactions branch December 6, 2018 13:52
iohk-bors bot added a commit that referenced this pull request Dec 21, 2018
3972: Cherry-picks from release/2.0.0 branch to develop r=rvl a=rvl

These are cherry-picks of the PRs: #3811 #3816 #3817 #3831 #3846 #3867 #3879 #3883 #3919 #3934 #3936 #3946 #3962.

Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
Co-authored-by: Karl Knutsson <karl.knutsson@iohk.io>
Co-authored-by: Michael Hueschen <michael.hueschen@iohk.io>
Co-authored-by: Alexander Diemand <codieplusplus@apax.net>
Co-authored-by: KtorZ <matthias.benkort@gmail.com>
Co-authored-by: Rupert Horlick <rupert.horlick@iohk.io>
Co-authored-by: Luke Nadur <luke.nadur@iohk.io>
Co-authored-by: Jordan Millar <jordan.millar@iohk.io>
Co-authored-by: Marcin Szamotulski <marcin.szamotulski@iohk.io>
Co-authored-by: Samuel Leathers <samuel.leathers@iohk.io>
Co-authored-by: Jean-Christophe Mincke <jeanchristophe.mincke@gmail.com>
@KtorZ KtorZ mentioned this pull request Jan 4, 2019
12 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants