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

Remove empty proxy tag from anlworkstation #1689

Merged
merged 1 commit into from
Jun 14, 2017

Conversation

dqwu
Copy link
Contributor

@dqwu dqwu commented Jun 14, 2017

This is a regression: the empty proxy tag was removed by PR #1590, but
it was added back by PR #1634 during the merge.

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status:

Fixes #1534

User interface changes?:

Code review:

@dqwu dqwu requested a review from jgfouca June 14, 2017 15:24
@jgfouca
Copy link
Contributor

jgfouca commented Jun 14, 2017

Sorry about that.

@jgfouca jgfouca merged commit 547f68b into master Jun 14, 2017
@jgfouca jgfouca deleted the dqwu/anlworkstation-remove-proxy-tag branch June 14, 2017 17:33
jgfouca pushed a commit that referenced this pull request Oct 17, 2017
Update ALCF Theta machine config

[BFB]
jgfouca added a commit that referenced this pull request Oct 24, 2017
Remove empty proxy tag from anlworkstation again

This is a regression: the empty proxy tag for anlworkstation was removed
by PR #1689, but it was added back by PR #1970 during the merge.

Test suite: scripts_regression_tests
Test baseline:
Test namelist changes:
Test status:

Fixes #1534

User interface changes?: N

Update gh-pages html (Y/N)?: N

Code review: @jgfouca
jgfouca pushed a commit that referenced this pull request Feb 23, 2018
Update ALCF Theta machine config

[BFB]
jgfouca pushed a commit that referenced this pull request Mar 13, 2018
Update ALCF Theta machine config

[BFB]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants