Skip to content

News: Vespa cloud instead of local. HF dataset fix - #4816

Merged
BrageHK merged 3 commits into
masterfrom
bragehk/news-tutorial-update
Aug 4, 2026
Merged

News: Vespa cloud instead of local. HF dataset fix#4816
BrageHK merged 3 commits into
masterfrom
bragehk/news-tutorial-update

Conversation

@BrageHK

@BrageHK BrageHK commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

News tutorial now use Vespa Cloud instead of local vespa instance. The HuggingFace dataset also got some updated instructions as it is now needed to accept terms and conditions. Relevant PR

<button class="d-icon d-duplicate pre-copy-button" onclick="copyPreContent(this)"></button>
<pre>
$ vespa config set target local
$ vespa config set default_config_scope global

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Dont remember if global is already default? If it is then maybe we can instead switch to this section saying
vespa config set target cloud. For the sake of the tutorial it is kinda agnostic if they use global or local config scope, but target must be cloud

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I see now further down we say set target cloud, so it might be worth considering just dropping the config scope section?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oh ok now it makes sense, since the tutorial follow multiple steps we set the config scope to global so we dont have to set it over and over again. Maybe add a short sentence about this already when we set it to global

Comment thread en/learn/tutorials/news-1-deploy-an-application.md Outdated
BrageHK and others added 2 commits August 4, 2026 15:05
Co-authored-by: Sebastian August Berg <95385037+sebastiannberg@users.noreply.github.com>
@BrageHK
BrageHK merged commit fbd1794 into master Aug 4, 2026
2 checks passed
@BrageHK
BrageHK deleted the bragehk/news-tutorial-update branch August 4, 2026 13:24
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.

2 participants