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

Job spec upload #14747

Merged
merged 8 commits into from
Nov 2, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .changelog/14747.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:improvement
ui: allow users to upload files by click or drag in the web ui
```
11 changes: 10 additions & 1 deletion ui/app/components/job-editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ export default class JobEditor extends Component {
planOutput = null;

@localStorageProperty('nomadMessageJobPlan', true) showPlanMessage;
@localStorageProperty('nomadMessageJobEditor', true) showEditorMessage;

@computed('planOutput')
get stage() {
Expand Down Expand Up @@ -117,4 +116,14 @@ export default class JobEditor extends Component {
window.scrollTo(0, 0);
}
}

@action uploadJobSpec(event) {
const reader = new FileReader();
reader.onload = () => {
this.updateCode(reader.result);
};

const [file] = event.target.files;
reader.readAsText(file);
}
}
22 changes: 22 additions & 0 deletions ui/app/styles/components/codemirror.scss
Original file line number Diff line number Diff line change
Expand Up @@ -129,3 +129,25 @@ $dark-bright: lighten($dark, 15%);
background-color: $dark-2;
}
}

header.run-job-header {
display: grid;
grid-template-columns: 1fr auto;
margin-bottom: 2rem;
gap: 0 1rem;
& > h1 {
grid-column: -1 / 1;
}

.job-spec-upload {
.button {
cursor: pointer;
}
input {
width: 100%;
height: 100%;
position: absolute;
display: none;
}
}
}
31 changes: 11 additions & 20 deletions ui/app/templates/components/job-editor.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -18,26 +18,16 @@
{{/if}}

{{#if (eq this.stage "editor")}}
{{#if (and this.showEditorMessage (eq this.context "new"))}}
<div class="notification is-info">
<div class="columns">
<div class="column">
<h3 class="title is-4" data-test-editor-help-title>Run a Job</h3>
<p data-test-editor-help-message>Paste or author HCL or JSON to submit
to your cluster. A plan will be requested before the job is
submitted.</p>
</div>
<div class="column is-centered is-minimum">
<button
class="button is-info"
onclick={{toggle-action "showEditorMessage" this}}
data-test-editor-help-dismiss
type="button"
>Okay</button>
</div>
</div>
</div>
{{/if}}

<header class="run-job-header">
<h1 class="title is-3">Run a job</h1>
<p>Paste or author HCL or JSON to submit to your cluster. A plan will be requested before the job is submitted. You can also attach a job spec by uploading a job file or dragging &amp; dropping a file to the editor.</p>
<label class="job-spec-upload">
<input type="file" onchange={{action this.uploadJobSpec}} accept=".hcl,.json,.nomad" />
<span class="button">Upload a job spec file</span>
</label>
</header>

<div class="boxed-section">
<div class="boxed-section-head">
Job Definition
Expand All @@ -63,6 +53,7 @@
/>
</div>
</div>

<div class="content is-associative">
<button
class="button is-primary {{if this.plan.isRunning 'is-loading'}}"
Expand Down
38 changes: 4 additions & 34 deletions ui/tests/integration/components/job-editor-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -105,59 +105,29 @@ module('Integration | Component | job-editor', function (hooks) {
};

const planJob = async (spec) => {
await Editor.editor.fillIn(spec);
const cm = getCodeMirrorInstance(['data-test-editor']);
cm.setValue(spec);
await Editor.plan();
};

test('the default state is an editor with an explanation popup', async function (assert) {
assert.expect(3);
assert.expect(2);

const job = await this.store.createRecord('job');

await renderNewJob(this, job);
assert.ok(
Editor.editorHelp.isPresent,
'Editor explanation popup is present'
);
assert.ok(Editor.editor.isPresent, 'Editor is present');

await componentA11yAudit(this.element, assert);
});

test('the explanation popup can be dismissed', async function (assert) {
const job = await this.store.createRecord('job');

await renderNewJob(this, job);
await Editor.editorHelp.dismiss();
assert.notOk(
Editor.editorHelp.isPresent,
'Editor explanation popup is gone'
);
assert.equal(
window.localStorage.nomadMessageJobEditor,
'false',
'Dismissal is persisted in localStorage'
);
});

test('the explanation popup is not shown once the dismissal state is set in localStorage', async function (assert) {
window.localStorage.nomadMessageJobEditor = 'false';

const job = await this.store.createRecord('job');

await renderNewJob(this, job);
assert.notOk(
Editor.editorHelp.isPresent,
'Editor explanation popup is gone'
);
});

test('submitting a json job skips the parse endpoint', async function (assert) {
const spec = jsonJob();
const job = await this.store.createRecord('job');

await renderNewJob(this, job);
await planJob(spec);
console.log('wait');
const requests = this.server.pretender.handledRequests.mapBy('url');
assert.notOk(
requests.includes('/v1/jobs/parse'),
Expand Down
7 changes: 0 additions & 7 deletions ui/tests/pages/components/job-editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,6 @@ export default () => ({
dismiss: clickable('[data-test-plan-help-dismiss]'),
},

editorHelp: {
isPresent: isPresent('[data-test-editor-help-title]'),
title: text('[data-test-editor-help-title]'),
message: text('[data-test-editor-help-message]'),
dismiss: clickable('[data-test-editor-help-dismiss]'),
},

editor: {
isPresent: isPresent('[data-test-editor]'),
contents: code('[data-test-editor]'),
Expand Down