Fix: Site BaseURLs

[Ajay-singh1]

Description

Related Issue:-#16559

This PR fixes incorrect baseURL values used during site generation, which were causing invalid or broken URLs in sitemap.xml.

Testing:-

1.Locally tested this with the command CONTEXT=production make netlify this generates the correct sitemaps.

Reviewers

• Ambient
• Docs
• Installation
• Networking
• Performance and Scalability
• Extensions and Telemetry
• Security
• Test and Release
• User Experience
• Developer Infrastructure
• Localization/Translation

[istio-testing]

Hi @Ajay-singh1. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[craigbox]

Can you ping me on the Istio Slack please Ajay?

[craigbox]

/ok-to-test

(I think you can join as a member now btw)

[Ajay-singh1]

@craigbox I have got a working solution now.We can check the sitemap for preliminary deployment and it is generating those sitemaps correctly.See :- Sitemap for Preliminary deployment.

[Ajay-singh1]

We have to cherry-pick release to see it in production.

[craigbox]

we could do that, assuming we have a roll back plan.

One thing I was wondering is if we should set up a custom page which just shows the variables, push that through all the sites first, and verify the values are as expected?

[Ajay-singh1]

I’ve reviewed the changes and I’m confident they won’t break anything. We have a rollback plan — we can revert the commit or use Netlify’s rollback option. I think we’re safe to proceed without the custom debug page.

[craigbox]

This isn't working as expected.

Look at https://deploy-preview-16568--preliminary-istio.netlify.app/ for example - all the links are now to preliminary.istio.io.