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

PROXY not set on anlworkstation #1534

Closed
dqwu opened this issue May 15, 2017 · 3 comments
Closed

PROXY not set on anlworkstation #1534

dqwu opened this issue May 15, 2017 · 3 comments
Assignees

Comments

@dqwu
Copy link
Contributor

dqwu commented May 15, 2017

On anlworkstation, there is a error during phase CREATE_NEWCASE: No variable PROXY found in case

In config/acme/machines/config_machines.xml, some machines define a PROXY element (melvin, sandia-srn-sems, anlworkstation, skybridge, redsky). For anlworkstation, this element is empty:
<PROXY></PROXY>

Should we set PROXY for anlworkstation or it can be deleted?

@jedwards4b
Copy link
Contributor

I believe that this problem was introduced in PR #1517 - can you confirm that removing the PROXY element from config_machines.xml for your system solves the problem? Thanks,

@dqwu
Copy link
Contributor Author

dqwu commented May 15, 2017

@jedwards4b
Yes, it works if we remove this element for anlworkstation.

@rljacob rljacob added the ready label May 17, 2017
@sarich sarich added in progress and removed ready labels May 22, 2017
jgfouca added a commit that referenced this issue May 22, 2017
…y-tag

Sarich/anlworkstation remove proxy tag

Remove empty <Proxy> tag from anlworkstation

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

Fixes #1534

User interface changes?:

Code review:
@ghost ghost removed the in progress label May 22, 2017
@dqwu
Copy link
Contributor Author

dqwu commented Jun 14, 2017

@jgfouca
This issue is reproducible again. The empty proxy tag was removed in PR #1590, but it was added back by PR #1634 during the merge.

@dqwu dqwu reopened this Jun 14, 2017
@dqwu dqwu assigned dqwu and unassigned jgfouca Jun 14, 2017
jgfouca added a commit that referenced this issue Jun 14, 2017
Remove empty proxy tag from anlworkstation

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:
@ghost ghost removed the in progress label Jun 14, 2017
jgfouca pushed a commit that referenced this issue Oct 17, 2017
Add the jobmin=0 and jobmax=299008 to the Titan's default batch queue in
config_batch.xml. The number of 299008 is total physical cores in Titan.

This addition will let all jobs run in the default batch queue and avoid
the failure caused by putting more than one job into the debug queue
simultaneously as only 1 debug job is allowed in Titan.

This fix is a better solution than that in the reverted PR #1534

[BFB]
jgfouca pushed a commit that referenced this issue Oct 17, 2017
Add jobmax and jobmin to the Titan's batch queue

Add the jobmin=0 and jobmax=299008 to the Titan's default batch queue in
config_batch.xml. The number of 299008 is total physical cores in Titan.

This addition will let all jobs run in the default batch queue and avoid
the failure caused by putting more than one job into the debug queue
simultaneously as only 1 debug job is allowed in Titan.

This fix is a better solution than that in the reverted PR #1534

[BFB]
jgfouca added a commit that referenced this issue 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 issue Feb 23, 2018
Add the jobmin=0 and jobmax=299008 to the Titan's default batch queue in
config_batch.xml. The number of 299008 is total physical cores in Titan.

This addition will let all jobs run in the default batch queue and avoid
the failure caused by putting more than one job into the debug queue
simultaneously as only 1 debug job is allowed in Titan.

This fix is a better solution than that in the reverted PR #1534

[BFB]
jgfouca pushed a commit that referenced this issue Feb 23, 2018
Add jobmax and jobmin to the Titan's batch queue

Add the jobmin=0 and jobmax=299008 to the Titan's default batch queue in
config_batch.xml. The number of 299008 is total physical cores in Titan.

This addition will let all jobs run in the default batch queue and avoid
the failure caused by putting more than one job into the debug queue
simultaneously as only 1 debug job is allowed in Titan.

This fix is a better solution than that in the reverted PR #1534

[BFB]
jgfouca pushed a commit that referenced this issue Mar 13, 2018
Add the jobmin=0 and jobmax=299008 to the Titan's default batch queue in
config_batch.xml. The number of 299008 is total physical cores in Titan.

This addition will let all jobs run in the default batch queue and avoid
the failure caused by putting more than one job into the debug queue
simultaneously as only 1 debug job is allowed in Titan.

This fix is a better solution than that in the reverted PR #1534

[BFB]
jgfouca pushed a commit that referenced this issue Mar 13, 2018
Add jobmax and jobmin to the Titan's batch queue

Add the jobmin=0 and jobmax=299008 to the Titan's default batch queue in
config_batch.xml. The number of 299008 is total physical cores in Titan.

This addition will let all jobs run in the default batch queue and avoid
the failure caused by putting more than one job into the debug queue
simultaneously as only 1 debug job is allowed in Titan.

This fix is a better solution than that in the reverted PR #1534

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

No branches or pull requests

5 participants